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
CI: Local CI runs don't set CILIUM_REGISTRY #10077
Conversation
We need to indicate whether to build cilium. This is only relevant when running tests locally and cilium needs to be compiled. Historically, the image has been pushed to k8s1:5000, a repository that runs on the k8s1 host. We now use the global.registry value, piped into CILIUM_REGISTRY, to indicate whether to build cilium. This was being set to a default of "k8s1:5000" thus forcing compile.sh into pulling a prebuilt image from CILIUM_REGISTRY. This is the intended behaviour on CI and when running on managed k8s. In this one case, however, it is incorrect. Signed-off-by: Ray Bejjani <ray@isovalent.com>
Release note label not set, please set the appropriate release note. |
2 similar comments
Release note label not set, please set the appropriate release note. |
Release note label not set, please set the appropriate release note. |
test-focus K8sDemosTest* |
test-gke K8sDemosTest* |
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.
If it works locally, let's ship it
test-gke K8sDemosTest* |
1 similar comment
test-gke K8sDemosTest* |
test-gke K8sDemosTest* |
1 similar comment
test-gke K8sDemosTest* |
test-gke K8sDemosTest* |
7 similar comments
test-gke K8sDemosTest* |
test-gke K8sDemosTest* |
test-gke K8sDemosTest* |
test-gke K8sDemosTest* |
test-gke K8sDemosTest* |
test-gke K8sDemosTest* |
test-gke K8sDemosTest* |
ok, I'll join the party |
test-gke K8sDemosTest* |
test-gke K8sDemosTest* |
@raybejjani @nebril Does this need backporting to v1.7? |
We need to indicate whether to build cilium. This is only relevant when
running tests locally and cilium needs to be compiled. Historically,
the image has been pushed to k8s1:5000, a repository that runs on the
k8s1 host. We now use the global.registry value, piped into
CILIUM_REGISTRY, to indicate whether to build cilium. This was being set
to a default of "k8s1:5000" thus forcing compile.sh into pulling a
prebuilt image from CILIUM_REGISTRY. This is the intended behaviour on
CI and when running on managed k8s. In this one case, however, it is
incorrect.
This change is