Skip to content
This repository has been archived by the owner on Feb 24, 2020. It is now read-only.

doc: dependencies: linux 3.18 #2282

Merged
merged 1 commit into from
Mar 16, 2016
Merged

Conversation

alban
Copy link
Member

@alban alban commented Mar 15, 2016

Linux 3.18 introduced the fix for unlinking files and directories that
are mount points in another mount namespace. Before Linux 3.18,
unlinking could return EBUSY:
torvalds/linux@8ed936b

With the introduction of rkt-fly, this becomes a visible issue with rkt
gc, see discussion on:
#1922 (comment)

This patch bumps the run-time dependency to Linux 3.18


/cc @jellonek @BrianHicks @jcollie

@jonboulle
Copy link
Contributor

Could we add this to the list of known issues? currently at bottom of readme.md

@alban alban force-pushed the alban/doc-dep-linux branch from 9604ba5 to 7cd7e81 Compare March 16, 2016 13:31
@alban
Copy link
Member Author

alban commented Mar 16, 2016

Updated.

@@ -73,3 +73,5 @@ Some portions of the codebase are derived from other projects under different li
## Known issues

Due to a bug in the Linux kernel, using rkt's overlay support on top of an overlay filesystem requires Linux 4.3+.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we link to actual issues?

Copy link
Contributor

Choose a reason for hiding this comment

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

(e.g. as an end user I'd find "In some configurations" a little frustratingly vague)

@alban alban force-pushed the alban/doc-dep-linux branch 3 times, most recently from 0bb03e0 to 8ae5421 Compare March 16, 2016 14:48
@alban
Copy link
Member Author

alban commented Mar 16, 2016

@jonboulle Updated. Is it a better wording?

@iaguis
Copy link
Member

iaguis commented Mar 16, 2016

s/bump/bumps/

@iaguis
Copy link
Member

iaguis commented Mar 16, 2016

Wording LGTM

@jonboulle
Copy link
Contributor

Sure seems fine. @joshix might have a tweak suggestion

@krnowak
Copy link
Collaborator

krnowak commented Mar 16, 2016

BTW, should we mention /var/lib/rkt on btrfs bug in the known issues section too?

@jonboulle
Copy link
Contributor

Yes!!!!

On Wed, Mar 16, 2016 at 4:23 PM Krzesimir Nowak notifications@github.com
wrote:

BTW, should we mention /var/lib/rkt on btrfs bug in the known issues
section too?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#2282 (comment)

@alban alban force-pushed the alban/doc-dep-linux branch 2 times, most recently from 958e313 to 4005e5a Compare March 16, 2016 15:41
@alban
Copy link
Member Author

alban commented Mar 16, 2016

Updated.

Linux 3.18 introduced the fix for unlinking files and directories that
are mount points in another mount namespace. Before Linux 3.18,
unlinking could return EBUSY:
torvalds/linux@8ed936b

With the introduction of rkt-fly, this becomes a visible issue with rkt
gc, see discussion on:
rkt#1922 (comment)

This patch bumps the run-time dependency to Linux 3.18. This also
documents the issues in README.md.
@jonboulle
Copy link
Contributor

Looks great. Thanks.

jonboulle added a commit that referenced this pull request Mar 16, 2016
@jonboulle jonboulle merged commit 95e6bcc into rkt:master Mar 16, 2016
@jcollie
Copy link

jcollie commented Mar 16, 2016

So unless Red Hat backports some patches RHEL and derivatives are a no-go since they use 3.10? Debian Jessie is on 3.16 although 4.3 appears to be available from the backports, but I'm much less familiar with the Debian ecosystem. Ubuntu 14.04 is on 3.13.

That seems like a pretty severe limitation.

@BrianHicks
Copy link

CentOS 7 is in the same boat. It's a very severe limitation. If this doc indicates that #1922 will not be fixed, we're not going to be able to adopt rkt in Mantl. So, is that the case?

@jonboulle
Copy link
Contributor

Note this is not to say that rkt won't work, just that certain functionality might be unavailable or have caveats (e.g. #1922 (comment)).

We do want rkt to be as functional as possible on older OSes and we're actively discussing how to improve this, stay tuned.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants