-
Notifications
You must be signed in to change notification settings - Fork 390
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
Support antctl command for packetcapture #6884
base: main
Are you sure you want to change the base?
Conversation
4a616d6
to
bf4cb68
Compare
cc @antoninbas Can you help review this? also maybe merge #3659 first cc @luolanzone |
b27b72b
to
43a9451
Compare
Please fix the antctl e2e test failures: https://github.com/antrea-io/antrea/actions/runs/13198383877?pr=6884 |
@hangyan there are lots of failures in github checks, please check and fix them. |
e1ba1b2
to
3436bfe
Compare
I disable the flag to run packetcapture sub-command in agentMode but controllerMode will still fail. One must be chosen, or this sub-command wont be registered to the rootCmd. However, seems packetcapture doens't make much sense in agentMode or controllerMode(controllerMode also contains outside of cluster case I think, but the e2e will check the command in controller pod). The tests expect run Can we make an exception in the test-case or we need to provide some sane defaults for it to working in the controllerMode? |
5ac92d9
to
e44de62
Compare
0c4cc8b
to
c4a4235
Compare
Signed-off-by: Hang Yan <[email protected]>
fc46f32
to
86609ff
Compare
Signed-off-by: Hang Yan <[email protected]>
Signed-off-by: Hang Yan <[email protected]>
Co-authored-by: Antonin Bas <[email protected]>
Signed-off-by: Hang Yan <[email protected]>
Signed-off-by: Hang Yan <[email protected]>
Signed-off-by: Hang Yan <[email protected]>
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.
Please rename option
to options
everywhere for consistency, otherwise it looks good to me
sure, will do.
Any thoughts on this? cc @antoninbas |
Signed-off-by: Hang Yan <[email protected]>
TODO items:
after #3659 , the CodeQL alert should be fixed and UT coverage will also be improved.