Skip to content
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

mock: use 'private' mount option, not 'rprivate' #536

Closed

Conversation

praiskup
Copy link
Member

The 'mount --rbind -o rprivate' on EL7 does weird things; it looks like
it actually reverses the '--rbind' option and does only '--bind'. Using
just 'private' seems to do the same (good) thing on both EL7 and modern
Fedora.

This fixes yum_chache plugin on EL7.

The 'mount --rbind -o rprivate' on EL7 does weird things;  it looks like
it actually reverses the '--rbind' option and does only '--bind'.  Using
just 'private' seems to do the same (good) thing on both EL7 and modern
Fedora.

This fixes local baseurl=file:// repositories with bootstrap, and
yum_chache plugin on EL7.

Fixes: rpm-software-management#381
@xsuchy
Copy link
Member

xsuchy commented Mar 13, 2020

I believe this was introduced as PR to make this work under Fedora Toolbox. Check discussion in that PR.

@praiskup
Copy link
Member Author

Hmm, I don't think we discussed private vs. rprivate. I'll take a look...

@praiskup
Copy link
Member Author

I think that we discussed --bind vs --rbind, but we talked about /proc and /sys mountpoints. Not about the /root -> /bootstrap/root mountpoint. WDYT?

@praiskup
Copy link
Member Author

Hm, it actually looks like we never tested bootstrap chroot with toolbox.

@praiskup
Copy link
Member Author

Ok, I tested:
$ mockchain --old-chroot --no-bootstrap-chroot python-copr-999-1.src.rpm dep-on-python-copr-999-1-0.src.rpm from testing repo and it succeded in toolbox:

...
INFO: Packages built: 2
INFO: Packages successfully built in this order:
INFO: python-copr-999-1.src.rpm
INFO: dep-on-python-copr-999-1-0.src.rpm

praiskup added a commit that referenced this pull request Mar 20, 2020
The 'mount --rbind -o rprivate' on EL7 does not work as we expected [1];
it looks like it actually reverses the '--rbind' option and does only
'--bind' (filled rhbz#1815534).  Using just 'private' seems to do the
same (good) thing on both EL7 and modern Fedora.

This fixes local baseurl=file:// repositories with bootstrap, and
yum_chache plugin on EL7.

Fixes: #381, #536
Relates: https://bugzilla.redhat.com/show_bug.cgi?id=1815534
@praiskup
Copy link
Member Author

Merged as 34fe0d5

@praiskup praiskup closed this Mar 20, 2020
@praiskup praiskup deleted the private-not-rprivate branch December 21, 2021 07:59
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.

None yet

2 participants