Add read-only option for rootfs #74
Conversation
options.go
Outdated
| var newPath string | ||
| readOnly := false | ||
| if strings.HasSuffix(path, roDeviceSuffix) { | ||
| newPath = strings.TrimSuffix(path, roDeviceSuffix) | ||
| readOnly = true | ||
| } else { | ||
| newPath = strings.TrimSuffix(path, rwDeviceSuffix) | ||
| } |
There was a problem hiding this comment.
How about extracting this block? We could have something like newPath, readonly := parseDevice(...)
There was a problem hiding this comment.
@kzys thanks! Updated code
|
Hi @Zamony, Thanks for the PR! Before this can be merged, it will need a sign off in each of your commit messages like:
You can do this by committing with For more info: https://wiki.linuxfoundation.org/dco |
Signed-off-by: Nikita Mochalov <mochalov.n@yandex.ru>
Hi @Kern--. I've signed off the commit. |
Issue #, if available:
Description of changes:
Add read-only option for root filesystem
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.