-
Notifications
You must be signed in to change notification settings - Fork 235
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
types/options: check prior drivers using readdir #1637
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.
Thanks
types/options.go
Outdated
if strings.HasSuffix(entry.Name(), "-images") { | ||
res := strings.Split(entry.Name(), "-images") | ||
if len(res) > 0 { | ||
opts.GraphDriverName = res[0] |
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.
add a break here
types/options.go
Outdated
if err == nil { | ||
for _, entry := range dirEntries { | ||
if strings.HasSuffix(entry.Name(), "-images") { | ||
res := strings.Split(entry.Name(), "-images") |
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 think using TrimSuffix() is better than using Split().
Fetch prior drivers if any using `ReadDir` and looking for traces of `%s-images` dirs if present already on the system where `%s` is the prior driver. Signed-off-by: Aditya R <arajan@redhat.com>
9bc084c
to
e31225e
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
LGTM |
This reverts commit ebf7474. With the c/storage change[1] we no longer need this workaround. [1] containers/storage#1637
This reverts commit ebf7474. With the c/storage change[1] we no longer need this workaround. [1] containers/storage#1637 Signed-off-by: Paul Holzinger <pholzing@redhat.com>
This reverts commit ebf7474. With the c/storage change[1] we no longer need this workaround. [1] containers/storage#1637 Signed-off-by: Paul Holzinger <pholzing@redhat.com>
This reverts commit ebf7474. With the c/storage change[1] we no longer need this workaround. [1] containers/storage#1637 Signed-off-by: Paul Holzinger <pholzing@redhat.com>
This reverts commit ebf7474. With the c/storage change[1] we no longer need this workaround. [1] containers/storage#1637 Signed-off-by: Paul Holzinger <pholzing@redhat.com>
This reverts commit ebf7474. With the c/storage change[1] we no longer need this workaround. [1] containers/storage#1637 Signed-off-by: Paul Holzinger <pholzing@redhat.com>
This reverts commit ebf7474. With the c/storage change[1] we no longer need this workaround. [1] containers/storage#1637 Signed-off-by: Paul Holzinger <pholzing@redhat.com>
Fetch prior drivers if any using
ReadDir
and looking for traces of%s-images
dirs if present already on the system where%s
is the prior driver.Closes: #1618 (comment)