8000 install/helm: Add Image Override Option to All Images by nathanjsweet · Pull Request #18849 · cilium/cilium · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

install/helm: Add Image Override Option to All Images #18849

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

Merged
merged 1 commit into from
Feb 25, 2022

Conversation

nathanjsweet
Copy link
Member

In order to enable offline deployment for certain
platforms (like OpenShift) we need to be able to have
a universal override for all images so that
the OpenShift certified operator can list its
"related images"1 2.

Signed-off-by: Nate Sweet nathanjsweet@pm.me

helm: Enable offline deployments for OpenShift clusters

@nathanjsweet nathanjsweet added release-note/misc This PR makes changes that have no direct user impact. area/helm Impacts helm charts and user deployment experience labels Feb 17, 2022
@nathanjsweet nathanjsweet requested a review from a team February 17, 2022 19:20
@nathanjsweet nathanjsweet requested a review from a team as a code owner February 17, 2022 19:20
@nathanjsweet nathanjsweet force-pushed the pr/nathanjsweet/image-override-all-helm-values branch from 1271f94 to aba9f12 Compare February 17, 2022 19:25
In order to enable offline deployment for certain
platforms (like OpenShift) we need to be able to have
a universal override for all images so that
the OpenShift certified operator can list its
"related images"[1][2].

[1]https://docs.openshift.com/container-platform/4.9/operators/operator_sdk/osdk-generating-csvs.html#olm-enabling-operator-for-restricted-network_osdk-generating-csvs
[2]https://redhat-connect.gitbook.io/certified-operator-guide/appendix/offline-enabled-operators

Signed-off-by: Nate Sweet <nathanjsweet@pm.me>
@nathanjsweet nathanjsweet force-pushed the pr/nathanjsweet/image-override-all-helm-values branch from aba9f12 to 0285646 Compare February 17, 2022 23:41
ctmach
ctmach approved these changes Feb 18, 2022
Copy link
Member
@sayboras sayboras left a comment

Choose a reason for hiding this comment

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

LGTM 💯

@joestringer
Copy link
Member

/test

@joestringer
Copy link
Member
joestringer commented Feb 24, 2022

This seems good to merge and low-risk, just wanted to know that the helm NULL usage reacts the way we expect. Even the first jenkins-based test run would make this clear, then we can merge.

@kaworu kaworu merged commit 9b1d3a3 into master Feb 25, 2022
@kaworu kaworu deleted the pr/nathanjsweet/image-override-all-helm-values branch February 25, 2022 16:06
@pchaigno
Copy link
Member

@kaworu @nathanjsweet The Travis CI failure was ignored here and broke master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/helm Impacts helm charts and user deployment experience backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants
0