-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
api, cli: Show srv6 status in cilium status #28700
Conversation
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 for your contribution! Overall, looks nice, but I have some questions + change request.
Also, I want you to cleanup the commit history. Currently there are commits without any message there. Please put them into the single commit (because they are basically touching to the same files) and description of the background of this commit.
Commit be9a09d does not match "(?m)^Signed-off-by:". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
be9a09d
to
42ab222
Compare
42ab222
to
22bee4a
Compare
Updated to single commit |
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.
Now looks much better! Sorry, one last cosmetic change request.
22bee4a
to
37a6a29
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.
Now looks good to me. Thanks!
/test |
Signed-off-by: Husni Alhamdani <dhanielluis@gmail.com>
37a6a29
to
00c1fdc
Compare
Pushed a commit that include generated |
/test |
Cilium IPsec upgrade: #28088 |
ConformanceEKS: #28693 |
Okay, now all tests are passing. @nathanjsweet Could you give this approval? It's a small change, so shouldn't take time. |
Please ensure your pull request adheres to the following guidelines:
description and a
Fixes: #XXX
line if the commit addresses a particularGitHub issue.
Fixes: <commit-id>
tag, thenplease add the commit author[s] as reviewer[s] to this issue.
With this pull request, the
cilium status
will show the status of SRv6 (enabled or not and the encap-mode)Fixes: #21872