Skip to content

fuse-overlayfs: support running without upper layer#133

Merged
rhatdan merged 1 commit intocontainers:masterfrom
giuseppe:support-no-upperdir
Oct 20, 2019
Merged

fuse-overlayfs: support running without upper layer#133
rhatdan merged 1 commit intocontainers:masterfrom
giuseppe:support-no-upperdir

Conversation

@giuseppe
Copy link
Copy Markdown
Member

when there is no upper layer specified, fail every read operation with
EROFS.

Closes: #100

Signed-off-by: Giuseppe Scrivano [email protected]

Copy link
Copy Markdown
Contributor

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, but README needs update

when there is no upper layer specified, fail every read operation with
EROFS.

Closes: containers#100

Signed-off-by: Giuseppe Scrivano <[email protected]>
@giuseppe giuseppe force-pushed the support-no-upperdir branch from 2f8991a to c9bbc94 Compare October 19, 2019 19:33
@giuseppe
Copy link
Copy Markdown
Member Author

thanks, but README needs update

thanks, fixed now

AkihiroSuda added a commit to containerd/fuse-overlayfs-snapshotter that referenced this pull request Oct 20, 2019
Requires:

* containerd/containerd#3765
* containers/fuse-overlayfs#133

---

Older commits were imported from containerd with the following commands:

  git filter-branch --prune-empty --subdirectory-filter snapshots/overlay --msg-filter 'cat; echo; echo This commit was imported from https://github.com/containerd/containerd/commit/${GIT_COMMIT}' master
  git tag --list | xargs git tag -d

Signed-off-by: Akihiro Suda <[email protected]>
@AkihiroSuda
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Member

@rhatdan rhatdan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rhatdan
Copy link
Copy Markdown
Member

rhatdan commented Oct 20, 2019

@AkihiroSuda Could something like that be done for CRI-O.

@rhatdan rhatdan merged commit ae96c48 into containers:master Oct 20, 2019
@AkihiroSuda
Copy link
Copy Markdown
Contributor

CRI-O is using containers/storage which already supports fuse-overlayfs?

@rhatdan
Copy link
Copy Markdown
Member

rhatdan commented Oct 20, 2019

Ok, I was not quite understanding what your tool was doing. Great.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fuse-overlay needs to support readonly lower with no upper.

3 participants