Skip to content
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 sidecar images at runtime using environment variables #245

Merged
merged 1 commit into from
Oct 9, 2019

Conversation

amisevsk
Copy link
Contributor

@amisevsk amisevsk commented Oct 6, 2019

What does this PR do?

Allows users to override the registry, organization, and tag of sidecar images for all plugins at runtime, using the environment variables

CHE_SIDECAR_CONTAINERS_REGISTRY_URL
CHE_SIDECAR_CONTAINERS_REGISTRY_ORGANIZATION
CHE_SIDECAR_CONTAINERS_REGISTRY_TAG

This PR consists of 45 commits, to make reviewing easier:

  1. Reorganize repo to but build files in one place (move dockerfiles to ./build/dockerfiles and scripts to ./build/scripts).
  2. Improve the build.sh script to support options, making it easier to build various branches of the plugin registry
  3. Add an entrypoint.sh script to the docker images, which consumes the environment variables above
  4. Add support for --latest-only in the build.sh script.
  5. Update helm chart and OpenShift template to accommodate these changes

Related Issue

eclipse-che/che#14693

@amisevsk
Copy link
Contributor Author

amisevsk commented Oct 8, 2019

Rebased onto current master

Copy link
Contributor

@nickboldt nickboldt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1

Add entrypoint script to registry docker images that allows overriding
images used in plugin sidecars with a user-defined registry, organization,
and tag, according to env vars

     CHE_SIDECAR_CONTAINERS_REGISTRY_URL
     CHE_SIDECAR_CONTAINERS_REGISTRY_ORGANIZATION
     CHE_SIDECAR_CONTAINERS_REGISTRY_TAG

Additionally:

- Reorganize repo so that build artifacts are in one place
- Update build.sh script to take basic options.

Signed-off-by: Angel Misevski <amisevsk@redhat.com>
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.

None yet

3 participants