-
Notifications
You must be signed in to change notification settings - Fork 122
RSDK-9648: Add viam CLI command to grab all captured FTDC data #5019
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
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.
A couple small things but generally looks good to me. I suspect @benjirewis (or someone from netcode) should be included as a reviewer though, since he opened the initial ticket and probably has more context for the need here than team SDK does.
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.
tests seem to be failing (looks like it might be a flakey test?) otherwise lgtm!
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.
Nice! This LGTM. Works on MacOS from what I can tell. Do wish we had a progress bar for copying, but obviously not work for this PR.
Does what the title says. Internally the new command is reusing the file copy code with presets for the path and flags.