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
composefs: enable it at runtime #1769
Conversation
a2c80b5
to
843af1b
Compare
@cgwalters PTAL |
afafaa4
to
3869129
Compare
So I was trying this out like this:
I see
Yet AFAICS |
it works only with the zstd:chunked code path now, you need to add this snippet to
|
3869129
to
ba301cb
Compare
Ah, OK only operating on zstd:chunked and requiring an opt-in and previously a build tag definitely makes it hard to use 😄 Anyways I think I've done what you suggested, but I am still not managing to trigger the composefs path. I do see
which is different, but apparently |
you already had the image in the storage? Was it re-pulled once you enabled the storage options? |
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
it solves the following error when pulling the alpine image: mkcomposefs: Parent directory missing for /bin/ Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
and honor the use_composefs configuration from the storage.conf file. Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
ba301cb
to
17bc7c5
Compare
@rhatdan PTAL |
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.
LGTM
@@ -387,6 +389,22 @@ func Init(home string, options graphdriver.Options) (graphdriver.Driver, error) | |||
} | |||
} | |||
|
|||
if opts.useComposefs { | |||
if unshare.IsRootless() { | |||
return nil, fmt.Errorf("composefs is not supported in user namespaces") |
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.
non blocking nit:
return nil, fmt.Errorf("composefs is not supported in user namespaces") | |
return nil, errors.New("composefs is not supported in user namespaces") |
@@ -122,6 +123,7 @@ type Driver struct { | |||
supportsDType bool | |||
supportsVolatile *bool | |||
usingMetacopy bool | |||
usingComposefs bool |
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.
non-blocking nit:
usingComposefs bool | |
useComposefs bool |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: giuseppe, saschagrunert The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@giuseppe could you fix @saschagrunert Nit's |
I am going to merge to fix CI/CD system on other PRs. @giuseppe if you care to fix the nits, please open a new PR. |
drop the build tag for building composefs and convert it to an option in
storage.conf