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
Implement per-provider operator deployments in Helm #12029
Implement per-provider operator deployments in Helm #12029
Conversation
Please set the appropriate release note label. |
1 similar comment
Please set the appropriate release note label. |
test-focus K8sUpdates |
88e7f5b
to
6136c92
Compare
test-focus K8sUpdates |
test-me-please |
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.
The following needs to be changed as well:
docker tag "cilium/cilium:$2" "$1/cilium/cilium:$2"
docker tag "cilium/cilium:$2" "$1/cilium/cilium-dev:$2"
docker tag "cilium/operator:$2" "$1/cilium/operator:$2"
docker tag "cilium/hubble-relay:$2" "$1/cilium/hubble-relay:$2"
6136c92
to
3cf8464
Compare
test-focus K8sUpdates |
Update the helm charts for for generic, aws and azure operator images. Fixes: cilium#11800 Signed-off-by: Sean Winn <sean@isovalent.com> Signed-off-by: Joe Stringer <joe@cilium.io>
3cf8464
to
41ac24b
Compare
test-focus K8sUpdates |
docker-image: clean docker-image-no-clean docker-plugin-image docker-hubble-relay-image | ||
$(MAKE) docker-operator-image | ||
$(MAKE) docker-operator-aws-image | ||
$(MAKE) docker-operator-azure-image | ||
$(MAKE) docker-operator-generic-image | ||
|
||
docker-image-unstripped: clean docker-image-no-clean-unstripped docker-plugin-image-unstripped docker-hubble-relay-image-unstripped | ||
$(MAKE) docker-operator-image-unstripped | ||
$(MAKE) docker-operator-aws-image-unstripped | ||
$(MAKE) docker-operator-azure-image-unstripped | ||
$(MAKE) docker-operator-generic-image-unstripped |
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.
:) would've expected these to be largely equivalent. "If you didn't know, now you know"
👍
Focus passed 🎉 |
test-me-please |
This PR currently uses the brute-force method to resolve this. We should avoid building all operator images in the CI in a follow-up PR. |
Hit known flake #10929: |
The rest is green, merging. |
Update the helm charts for for generic, aws and azure operator images.
Fixes: #11800
This is a simplification of the minimal required changes from #11837 for the v1.8.0 release.