-
Notifications
You must be signed in to change notification settings - Fork 115
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow overriding container images used in devfiles #110
Conversation
Adds the ability to override the registry url, organization, and tag of images used in devfiles via environment variables CHE_DEVFILE_IMAGE_REGISTRY_URL CHE_DEVFILE_IMAGE_REGISTRY_ORGANIZATION CHE_DEVFILE_IMAGE_REGISTRY_TAG Signed-off-by: Angel Misevski <amisevsk@redhat.com>
Signed-off-by: Angel Misevski <amisevsk@redhat.com>
Signed-off-by: Angel Misevski <amisevsk@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggest embedding the logic to start httpd inside the entrypoint script, like we do for the plugin registry, because consistency ;)
You just approved a commit that moves the start httpd logic into a CMD ;) |
Signed-off-by: Angel Misevski <amisevsk@redhat.com>
Signed-off-by: Angel Misevski <amisevsk@redhat.com>
@nickboldt Updated PR, please take another look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
questions posed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 all my complaints were refused or fixed
* Add ability to override images used in devfiles at runtime Adds the ability to override the registry url, organization, and tag of images used in devfiles via environment variables CHE_DEVFILE_IMAGE_REGISTRY_URL CHE_DEVFILE_IMAGE_REGISTRY_ORGANIZATION CHE_DEVFILE_IMAGE_REGISTRY_TAG Signed-off-by: Angel Misevski <amisevsk@redhat.com>
* Add ability to override images used in devfiles at runtime Adds the ability to override the registry url, organization, and tag of images used in devfiles via environment variables CHE_DEVFILE_IMAGE_REGISTRY_URL CHE_DEVFILE_IMAGE_REGISTRY_ORGANIZATION CHE_DEVFILE_IMAGE_REGISTRY_TAG Signed-off-by: Angel Misevski <amisevsk@redhat.com>
What does this PR do?
Allow overriding image registry, organization, and tag in devfiles at runtime using the env vars
Also updates helm charts and OpenShift template.
What issues does this PR fix or reference?
eclipse-che/che#14693