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

oci: Support --no-mount, to extent possible, from sylabs 1785 #341

Merged
merged 1 commit into from
Jul 11, 2023

Conversation

edytuk
Copy link
Collaborator

@edytuk edytuk commented Jul 10, 2023

This pulls in sylabs PR

  • sylabs/singularity# 1785
    which fixed
  • sylabs/singularity# 1781

The original PR description was:

Allow the --no-mount flag to be specified in --oci mode. This allows disabling the following mounts:

  • proc
  • sys
  • devpts
  • tmp
  • home

Note that dev cannot be supported in --oci mode, as an OCI runtime requires that certain devices are present, and will include them in a /dev tmpfs.

We currently run similar to native mode --compat, so we don't mount the current working directory. Therefore, --no-mount cwd is not supported.

Similarly, --compat infers that bind path entries from singularity.conf are ignored. We may handle them in some way, in future.

Allow the `--no-mount` flag to be specified in `--oci` mode. This
allows disabling the following mounts:

* proc
* sys
* devpts
* tmp
* home

Note that `dev` cannot be supported in `--oci` mode, as an OCI
runtime *requires* that certain devices are present, and will include
them in a `/dev` tmpfs.

We currently run similar to native mode `--compat`, so we don't mount
the current working directory. Therefore, `--no-mount cwd` is not
supported.

Similarly, `--compat` infers that `bind path` entries from
`singularity.conf` are ignored. We may handle them in some way, in future.

Fixes sylabs/singularity#1781

Signed-off-by: Edita Kizinevic <edita.kizinevic@cern.ch>
@edytuk edytuk requested a review from DrDaveD July 10, 2023 14:39
@edytuk edytuk merged commit 0e64d0c into oci-action Jul 11, 2023
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