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
Add commands for suspending and resuming drives and volumes #842
Conversation
7ed713b
to
076bf88
Compare
resumeVolumesCmd.PersistentFlags().SortFlags = false | ||
|
||
addNodesFlag(resumeVolumesCmd, "If present, resume volumes from given nodes") | ||
addDrivesFlag(resumeVolumesCmd, "If present, resume volumes by given drive names") |
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.
As these two flags are also available in resume_drives.go
, you could add commonly to resume.go
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.
we can't have generic messages if we move it to resume.go
like resume objects by given drive names
suspendDrivesCmd.PersistentFlags().SortFlags = false | ||
|
||
addNodesFlag(suspendDrivesCmd, "If present, suspend drives from given nodes") | ||
addDrivesFlag(suspendDrivesCmd, "If present, suspend drives by given names") |
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.
Same as above commands
addDrivesFlag(suspendVolumesCmd, "If present, suspend volumes by given drive names") | ||
addPodNameFlag(suspendVolumesCmd, "If present, suspend volumes by given pod names") | ||
addPodNSFlag(suspendVolumesCmd, "If present, suspend volumes by given pod namespaces") | ||
addDangerousFlag(suspendVolumesCmd, "Suspending the volumes will make them as read-only") |
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.
We could add this flag to suspend.go
076bf88
to
80e6a22
Compare
80e6a22
to
2ab8026
Compare
Fixes #836