Skip to content

Container registry - general description, link to official docu, context warning#35

Merged
UndeadFairy merged 2 commits intomainfrom
container_registry
Mar 16, 2026
Merged

Container registry - general description, link to official docu, context warning#35
UndeadFairy merged 2 commits intomainfrom
container_registry

Conversation

@ziselsberger
Copy link
Contributor

@santilland : I've added the container registry to the service mapping (dd0881f), so the Context Warning will appear on the top of the page. Is this correct, or do I need to add/adapt something else?

@UndeadFairy : If you have an example or a better description how to include the image in argo, please tell me, because this was written by Gemini 😅

@github-actions
Copy link

github-actions bot commented Mar 9, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-03-16 13:18 UTC

@santilland
Copy link
Contributor

This is great! Thank you for working on keeping the documentation up to date.
The config you added should do the trick, looks good.

As to the argo workflow in principle maybe we could expand to say something like:

container:
        name: example
        image: registry.[workspace-name].[cluster].eox.at/my-image:latest

i am a bit unsure about the cluster thing though.... users do not really have this information/understanding.

I was having a bit the crazy idea, if we already modify content of the docs, we might actually also try to modify the content with the script and replace things like [workspace-name] and [cluster] with the actual values of the workspace one is looking the documentation from....

@UndeadFairy
Copy link
Collaborator

I would merge this as it is now and lets do chages afterwords. I would like to add more tuotrials on that as it should also cover examples in argo. We can re-open the problem if this is not sufficient

@UndeadFairy UndeadFairy merged commit ecd5ef4 into main Mar 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants