-
Notifications
You must be signed in to change notification settings - Fork 210
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
sysdump: Remove the Hidden flag #481
Conversation
We've been using the sysdump command for a while in both cilium and cilium-cli CI workflows and it's been stable. Signed-off-by: Michi Mutsuzaki <[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.
🚢
Multicluster tests failed on known issue #361: https://github.com/cilium/cilium-cli/runs/3309372422?check_suite_focus=true This is unrelated to the changes in this PR, so let's merge. |
Looks like this PR introduced a Not sure why this wasn't caught on the PR 🤔 |
The change in PR #481 wasn't properly formatted, leading to a golangci-lint failure on master: https://github.com/cilium/cilium-cli/runs/3310670546 Fix it by running `goimports -w internal/cli/cmd/sysdump.go`. Signed-off-by: Tobias Klauser <[email protected]>
Fix for the formatting issue here: #483 Still not sure why it wasn't caught on the PR, probably something with the |
The change in PR #481 wasn't properly formatted, leading to a golangci-lint failure on master: https://github.com/cilium/cilium-cli/runs/3310670546 Fix it by running `goimports -w internal/cli/cmd/sysdump.go`. Signed-off-by: Tobias Klauser <[email protected]>
The change in PR cilium#481 wasn't properly formatted, leading to a golangci-lint failure on master: https://github.com/cilium/cilium-cli/runs/3310670546 Fix it by running `goimports -w internal/cli/cmd/sysdump.go`. Signed-off-by: Tobias Klauser <[email protected]>
We've been using the sysdump command for a while in both cilium and
cilium-cli CI workflows and it's been stable.
Signed-off-by: Michi Mutsuzaki [email protected]