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

Expose a few more host locations inside the container under /run/host #264

Merged
merged 1 commit into from
Sep 18, 2019

Conversation

debarshiray
Copy link
Member

This is meant to alleviate some of the pain of not being able to modify
the list of bind mounts once a toolbox container has been created. For
some cases, where read-only access is enough, one can get by with
setting up symbolic links inside the toolbox container.

Based on an idea from Colin Walters.

This is meant to alleviate some of the pain of not being able to modify
the list of bind mounts once a toolbox container has been created. For
some cases, especially where read-only access is enough, one can get
by with setting up symbolic links inside the toolbox container.

Based on an idea from Colin Walters.

https://github.com/debarshiray/toolbox/pull/264
@debarshiray debarshiray force-pushed the wip/rishi/more-host-locations-run-host branch from e166e86 to d63b0a9 Compare September 18, 2019 15:12
@@ -872,6 +872,10 @@ create()
--volume /dev:/dev:rslave \
--volume /media:/media:rslave \
--volume /mnt:/mnt:rslave \
--volume /run:/run/host/run:rslave \
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hm, but I see you chose /run/host instead of /host. Standardizing that path as much as we can is useful because other tools may come to rely on it. For example, things like Systemtap may want to look at /host/usr/lib/modules to find the host kernel.

I chose /host because oc debug node from OpenShift did.

Copy link
Member Author

Choose a reason for hiding this comment

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

Interesting. I didn't know OpenShift uses /host. I did notice that you had went with /host, not /run/host.

I went with /run/host for no other reason than that we already had it for resolv.conf, and the original reason was just that it's the location that Flatpak uses.

I am happy to switch to /host or add some symbolic links or whatever.

Copy link
Member Author

Choose a reason for hiding this comment

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

Let's merge this, and think of /run/host versus /host in a separate PR or issue.

@debarshiray debarshiray merged commit d63b0a9 into master Sep 18, 2019
@debarshiray debarshiray deleted the wip/rishi/more-host-locations-run-host branch September 18, 2019 15:53
@debarshiray
Copy link
Member Author

For some reason, I cannot enter a toolbox that was created without that line removed
(on current F31 Silverblue).

You mean this line:

        --volume /usr:/run/host/usr:rslave \

We are also discussing it in containers/podman#4061

@debarshiray
Copy link
Member Author

For some reason, I cannot enter a toolbox that was created without that line
removed (on current F31 Silverblue).

Should be fixed by https://github.com/debarshiray/toolbox/pull/276

We should roll a new release as soon as possible to address this regression.

@debarshiray
Copy link
Member Author

debarshiray commented Oct 3, 2019

This is also affected by containers/crun#120 or https://github.com/debarshiray/toolbox/issues/282 on Fedora 31 Silverblue.

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.

3 participants