CFP-25694: Move cilium/cilium-cli code into cilium/cilium repository#34178
Merged
michi-covalent merged 2644 commits intomainfrom Aug 16, 2024
Merged
CFP-25694: Move cilium/cilium-cli code into cilium/cilium repository#34178michi-covalent merged 2644 commits intomainfrom
michi-covalent merged 2644 commits intomainfrom
Conversation
9879e79 to
cc94e7d
Compare
cc94e7d to
45852f4
Compare
45852f4 to
98081fe
Compare
Contributor
Author
|
/test |
98081fe to
d18d7ba
Compare
d18d7ba to
63a669c
Compare
63a669c to
f43b019
Compare
f43b019 to
251c9d1
Compare
Contributor
Author
|
/test |
251c9d1 to
459a346
Compare
[ cherry-picked from cilium/cilium-cli repository ] Signed-off-by: renovate[bot] <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Signed-off-by: renovate[bot] <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Signed-off-by: Michi Mutsuzaki <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] JUnit collector skip logic fixed. Signed-off-by: viktor-kurchenko <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Signed-off-by: Joe Stringer <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] This commit is to enable multicast status check and configuration from cilium-cli by adding multicast subcommand. The list subcommand show the list of multicast groups or their subscribers on all nodes. The add subcommand joins all nodes to the specified multicast group. The del subcommand deletes the specified multicast group from all nodes. This is follow up of cilium/cilium-cli#2574 Signed-off-by: Yusho Yamaguchi <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Also update kubeadm.k8s.io API version from v1beta2 to v1beta3. Ref: #2720 Suggested-by: Joe Stringer <[email protected]> Signed-off-by: Michi Mutsuzaki <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Signed-off-by: renovate[bot] <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Signed-off-by: renovate[bot] <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Save all the condition functions from WithCondition() instead on only remembering the one from the latest invocation. Run the test if and only if it satisfies all the conditions. Fixes: #2713 Signed-off-by: Michi Mutsuzaki <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] in client-egress-l7-http-external-node CNP instead of the fixed values to acconmodate the test namespace and external deployment port change. Signed-off-by: Yusuke Suzuki <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Signed-off-by: renovate[bot] <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Signed-off-by: renovate[bot] <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] - Enable local-redirect-policy-with-node-dns. - Disable local-redirect-policy until #2733 is fixed. Signed-off-by: renovate[bot] <[email protected]> Signed-off-by: Michi Mutsuzaki <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ]
Shorten cluster names to avoid getting this error:
Error: Unable to install Cilium: execution error at (cilium/templates/validate.yaml:100:5):
The cluster name is invalid: must not be more than 32 characters.
Configure 'upgradeCompatibility' to 1.15 or earlier to temporarily
skip this check at your own risk
Ref: #33939
Signed-off-by: Michi Mutsuzaki <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Following the upgrade notes [^1]: > With the default enablement of KVStoreMesh, if you use external > workloads, ensure that your cluster has a Cluster name and ID > specified before upgrading. Alternatively, you can explicitly opt out > of KVStoreMesh. [^1]: https://docs.cilium.io/en/stable/operations/upgrade/#upgrade-notes Signed-off-by: Michi Mutsuzaki <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Signed-off-by: Michi Mutsuzaki <[email protected]>
[ cherry-picked from cilium/cilium-cli repository ] Signed-off-by: Michi Mutsuzaki <[email protected]>
Remove go.mod from cilium-cli directory, and fix import paths to truly unite cilium-cli with cilium repo. Signed-off-by: Michi Mutsuzaki <[email protected]>
- Exclude cilium-cli in check-time.sh. - Fix an indentation warning. Signed-off-by: Michi Mutsuzaki <[email protected]>
These files will stay in cilium-cli repo for the time being. Signed-off-by: Michi Mutsuzaki <[email protected]>
Signed-off-by: Michi Mutsuzaki <[email protected]>
Build CI image for cilium-cli, and update all the workflows to use the CI image instead of the image built from a release tag. Signed-off-by: Michi Mutsuzaki <[email protected]>
Contributor
Author
|
/test |
This was referenced Aug 16, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
most of commits are from cilium-cli repo. please review these commits:
8b4d060 - cilium-cli: Remove go.mod
4bd9e9b - Fix lint errors
55a6ded - Remove redundant files
bc6d709 - Update CODEOWNERS for cilium-cli/ directory
e497a37 - Build cilium-cli CI Docker image
ignore the following failures:
BPF Checks / Check Patch: it's complaining about commit message format. i'm not going to rewrite commit messages at this point.Build Commits / Check if build works for every commit: too many commits to build before timing out.Ref: cilium/design-cfps#9
Fixes #25694