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
Fix missing operator-generic in upstream k8s tests #12055
Conversation
test-upstream-k8s |
c3c10f5
to
8b03c05
Compare
test-upstream-k8s |
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.
Let's see if it passes this time.
test/provision/compile.sh
Outdated
@@ -45,13 +48,28 @@ then | |||
|
|||
if [[ "${CILIUM_OPERATOR_IMAGE}" == "" ]]; then | |||
echo "building cilium-operator image..." | |||
make LOCKDEBUG=1 docker-operator-image | |||
echo "tagging cilium-operator image..." | |||
make LOCKDEBUG=1 docker-operator-image docker-operator-aws-image docker-operator-azure-image docker-operator-generic-image |
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.
We should probably reference #12035 here as well
(The equivalent change for the regular cilium build is in the makefile)
Also build all operator images by default so the upstream tests will have the generic image available. Fixes: 2773d06 ("Implement per-provider operator deployments") Signed-off-by: André Martins <andre@cilium.io>
Fixes: 2773d06 ("Implement per-provider operator deployments") Signed-off-by: André Martins <andre@cilium.io>
8b03c05
to
dac9ffa
Compare
test-upstream-k8s |
test-me-please |
This PR fixes some bugs introduced by #12029