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-aks, ci-external-workloads: Use cilium-cli Helm mode #26382
Conversation
✅ ci-aks: https://github.com/cilium/cilium/actions/runs/5335633345/jobs/9669921350?pr=26382 removing the temporary commit |
5c20fe8
to
4d9d4d3
Compare
/test |
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.
one comment on enabling hubble relay as part of cilium installation, but not a blocker.
LGTM ✔️
Ref: #25156 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Ref: https://github.com/cilium/cilium-cli#experimental-helm-installation-mode Ref: #25156 Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
4d9d4d3
to
d6fb7cb
Compare
rebased. i'll re-test aks. moving to draft. |
/test |
d6fb7cb
to
57e6c71
Compare
/test |
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.
There's still a little discussion going on below, but as long as we've validated that the new version does the same testing as the old one on the correct version, LGTM 👍
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.
🚢 🚢 🚢
migrating ci-aks and ci-external-workloads to use Helm mode for #25156