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: fix clustermesh worfklows on stable branches #25089
Conversation
My fault, sorry for the inconvenience. The checkout step is required to retrieve the helm chart used during installation, but it can be moved after cluster creation. |
CI passed: https://github.com/cilium/cilium/actions/runs/4788606197 |
ed930a3
to
a3db4b4
Compare
The clustermesh workflows currently fail on stable branches as they expect `.github/kind-config.yaml.tmpl` to be present, but this file is only present on `main`. We still need to checkout the branch code so as to retrieve the Helm chart that will be used in the cluster via `cilium install`. We move the checkout part just right before we install Cilium. Thus, the Kind template file on `main` is used for all worfkows. Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
a3db4b4
to
d7534a4
Compare
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.
Thanks!
CI passed: https://github.com/cilium/cilium/actions/runs/4809296042 |
d7534a4
to
13fb6a8
Compare
The clustermesh workflows currently fail on stable branches as they expect
.github/kind-config.yaml.tmpl
to be present, but this file is only present onmain
.It seems the template file is the only reason we need to checkout the repository, as we do not use any other file from the repository in subsequent steps, so we fix this by removing the checkout of the current branch since we already checkout the
main
branch in a prior step.Thus, the template file on
main
is used for all worfkows.