Skip to content

cilium-dbg: Add sysdump command#35370

Merged
joestringer merged 1 commit intomainfrom
pr/joe/cilium-dbg-sysdump
Nov 7, 2024
Merged

cilium-dbg: Add sysdump command#35370
joestringer merged 1 commit intomainfrom
pr/joe/cilium-dbg-sysdump

Conversation

@joestringer
Copy link
Copy Markdown
Member

This command, expected to be run inside the Cilium container, simply
informs users with a friendly message that they are using the wrong tool
to dump clusterwide state, and provides a link about how to do so.

Suggested-by: André Martins [email protected]

@joestringer joestringer requested a review from a team as a code owner October 11, 2024 21:34
@joestringer joestringer added the release-note/misc This PR makes changes that have no direct user impact. label Oct 11, 2024
@joestringer
Copy link
Copy Markdown
Member Author

The context here is we've started receiving bug reports from users using the wrong tool to run the sysdump. I figure we could mitigate that a little by providing a more user-friendly message to tell them how to do it correctly.

This command, expected to be run inside the Cilium container, simply
informs users with a friendly message that they are using the wrong tool
to dump clusterwide state, and provides a link about how to do so.

Suggested-by: André Martins <[email protected]>
Signed-off-by: Joe Stringer <[email protected]>
@joestringer joestringer force-pushed the pr/joe/cilium-dbg-sysdump branch from 136529f to b1eaec4 Compare October 21, 2024 15:03
@joestringer
Copy link
Copy Markdown
Member Author

/test

@joestringer joestringer merged commit 19daf59 into main Nov 7, 2024
@joestringer joestringer deleted the pr/joe/cilium-dbg-sysdump branch November 7, 2024 18:46
sayboras added a commit that referenced this pull request Nov 11, 2024
sayboras added a commit that referenced this pull request Nov 11, 2024
github-merge-queue Bot pushed a commit that referenced this pull request Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants