Skip to content
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

make: rework kind-install-cilium-clustermesh for Cilium CLI Helm mode #26753

Merged

Conversation

giorio94
Copy link
Member

Cilium CLI v0.15 leverages the Helm-based installation mode by default, and it is the one which supports Cilium v1.14 and above. Let's adapt the kind-install-cilium-clustermesh Makefile target to work properly with it.

While being there, let's also add the possibility to enable kvstoremesh if the dedicated variable is set.

Marking for backport to v1.14 because Cilium CLI v0.15 supports Cilium 1.14 and newer.

@giorio94 giorio94 added area/clustermesh Relates to multi-cluster routing functionality in Cilium. release-note/misc This PR makes changes that have no direct user impact. sig/contributing Impacts contribution workflow, guidelines, and tools. needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch labels Jul 11, 2023
@giorio94 giorio94 requested review from a team as code owners July 11, 2023 08:07
@giorio94 giorio94 requested a review from qmonnet July 11, 2023 08:07
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.14.0 Jul 11, 2023
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Cilium CLI v0.15 leverages the Helm-based installation mode by default,
and it is the one which supports Cilium v1.14 and above. Let's adapt the
kind-install-cilium-clustermesh Makefile target to work properly with it.

While being there, let's also add the possibility to enable kvstoremesh
if the dedicated variable is set.

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 force-pushed the mio/make-kind-install-cilium-clustermesh branch from 9f723dc to 7a4ed37 Compare July 11, 2023 14:36
@giorio94 giorio94 requested a review from qmonnet July 11, 2023 14:38
Copy link
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@giorio94
Copy link
Member Author

/test

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jul 11, 2023
@julianwiedmann julianwiedmann merged commit c25b96d into cilium:main Jul 12, 2023
65 checks passed
@giorio94 giorio94 deleted the mio/make-kind-install-cilium-clustermesh branch July 12, 2023 06:25
@jibi jibi mentioned this pull request Jul 13, 2023
13 tasks
@jibi jibi added backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. and removed needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch labels Jul 13, 2023
@aanm aanm added backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. and removed backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. labels Jul 14, 2023
@aanm aanm moved this from Needs backport from main to Backport done to v1.14 in 1.14.0 Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/clustermesh Relates to multi-cluster routing functionality in Cilium. backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact. sig/contributing Impacts contribution workflow, guidelines, and tools.
Projects
No open projects
1.14.0
Backport done to v1.14
Development

Successfully merging this pull request may close these issues.

None yet

5 participants