-
Notifications
You must be signed in to change notification settings - Fork 33
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
Support exporting backing image from volume #24
Conversation
Longhorn 2403 Signed-off-by: Shuo Wu <shuo.wu@suse.com>
6e8997d
to
ab70b1d
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.
LGTM
pkg/util/util.go
Outdated
|
||
func ExecuteWithTimeout(timeout time.Duration, envs []string, binary string, args ...string) (string, error) { | ||
var err error | ||
cmd := exec.Command(binary, args...) |
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.
NIT: You can use a CommandContext, then pass a context.withTimeout. This way you don't need the timeout go routine below.
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.
Addressed.
Longhorn 2403 Signed-off-by: Shuo Wu <shuo.wu@suse.com>
…ing ready Signed-off-by: Shuo Wu <shuo.wu@suse.com>
ab70b1d
to
1318216
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.
LGTM
longhorn/longhorn#2403