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: Add kvstore data to dumps. #972
sysdump: Add kvstore data to dumps. #972
Conversation
ps, err := c.Client.ListPods(ctx, c.Options.CiliumNamespace, metav1.ListOptions{ | ||
LabelSelector: c.Options.CiliumLabelSelector, | ||
}) | ||
if err != nil { |
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.
Suggestion: loop over pods, pick one that is scheduled and (ideally) has all containers running.
prefix: "state/services", | ||
}, | ||
} { | ||
stdout, stderr, err := c.Client.ExecInPodWithStderr(ctx, pod.Namespace, pod.Name, defaults.AgentContainerName, []string{ |
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.
I just committed a function, ensureExecTarget
, which you may find useful here (albeit a bit overkill).
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.
Oh very nice, ill take a look at.
20f0599
to
2f3f01b
Compare
2f3f01b
to
4eb9033
Compare
When running sysdump, if kvstore is detected then various kvstore paths are added to the dump. Signed-off-by: Tom Hadlaw <tom.hadlaw@isovalent.com>
4eb9033
to
f37f8a1
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.
Thanks
When running sysdump, if kvstore is detected then various kvstore paths are added to the dump.
This is intended to be used by the debug-ability tools to help diagnose issues.
Signed-off-by: Tom Hadlaw tom.hadlaw@isovalent.com