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

The singularity package was renamed to apptainer #1081

Merged
merged 3 commits into from
Oct 13, 2022

Conversation

afbjorklund
Copy link
Member

@afbjorklund afbjorklund commented Oct 1, 2022

In fedora, singularity-3.8.7 was replaced with apptainer-1.0.3

https://singularity.hpcng.org/

https://apptainer.org/news/community-announcement-20211130

So the example should be renamed too, to reflect the reality.


SingularityCE has not been packaged as dnf, only from sylabs.io

https://sylabs.io/singularity/

https://docs.sylabs.io/guides/3.10/admin-guide/

If requested, it could be resurrected as an example (like e.g. centos stream)

@afbjorklund afbjorklund force-pushed the singularity-apptainer branch from 41bd3f7 to 2d02405 Compare October 1, 2022 10:27
@afbjorklund
Copy link
Member Author

Once the fedora package has been updated to 1.1.0, this PR replaces the experimental/apptainer.yaml example.

I believe that using Fedora is a better match than using Rocky, because of 1) download size 2) uses EPEL anyway

@afbjorklund
Copy link
Member Author

afbjorklund commented Oct 1, 2022

Currently there are problems with using tmpfs as /tmp, but a simple workaround is TMPDIR=/var/tmp

I will do a separate PR for this, once renamed... https://fedoraproject.org/wiki/Features/tmp-on-tmpfs

@afbjorklund afbjorklund force-pushed the singularity-apptainer branch 2 times, most recently from 845ad9e to e398f48 Compare October 1, 2022 10:37
@AkihiroSuda
Copy link
Member

Once the fedora package has been updated to 1.1.0, this PR replaces the experimental/apptainer.yaml example.

How long will it take?
Shall we wait until that happens or shall we merge this PR now?

@afbjorklund
Copy link
Member Author

afbjorklund commented Oct 2, 2022

Shall we wait until that happens or shall we merge this PR now?

I think the only difference was that apptainer-suid was split into a separate package:

-    dnf install --enablerepo=epel-testing -y apptainer apptainer-suid
+    dnf install -y apptainer

I tried enabling the updates-testing, but seems like the package only made it to fc38 ?

https://src.fedoraproject.org/rpms/apptainer


But you could merge it now, and we could tweak it later perhaps. It doesn't change anything.

The previous example has the same end result, with a symlink /usr/bin/singularity -> apptainer

$ limactl shell singularity singularity --version
apptainer version 1.0.3-2.fc36

So the change is more cosmetic and documentation, as it also has a new icon and home page.

@afbjorklund
Copy link
Member Author

@AkihiroSuda note that this also a new feature, in the new release: i.e. rootless by default

https://apptainer.org/news/apptainer-1-1-0-20220927

It is still possible to use rootless, by using flags or by changing the apptainer.conf configuration:

allow setuid = no

@AkihiroSuda
Copy link
Member

If we are going to merge this now, shall we remove experimental/apptainer.yaml?

@afbjorklund

This comment was marked as outdated.

@afbjorklund afbjorklund force-pushed the singularity-apptainer branch from e398f48 to 6351716 Compare October 6, 2022 18:23
@afbjorklund
Copy link
Member Author

afbjorklund commented Oct 6, 2022

@AkihiroSuda apptainer 1.1.0 has now been accepted in both Fedora and EPEL

https://src.fedoraproject.org/rpms/apptainer

Release Stable version Version in testing
Fedora 36 apptainer-1.1.0-1.fc36  
Fedora 35 apptainer-1.1.0-1.fc35  
Fedora EPEL 9 apptainer-1.1.0-1.el9  
Fedora EPEL 8 apptainer-1.1.0-1.el8

I moved the suid binary into a separate example, like with the other engines

I also deleted the Rocky-based example, since it is replaced with the Fedora-based

In fedora, singularity-3.8.7 was replaced with apptainer-1.1.0

The package was split into apptainer and apptainer-suid (root).

Use fedora instead of rocky, since it is smaller (and shorter).

SingularityCE has not been packaged as dnf, only from sylabs.io

Signed-off-by: Anders F Björklund <[email protected]>
@afbjorklund afbjorklund force-pushed the singularity-apptainer branch from 6351716 to 725d7dc Compare October 7, 2022 15:17
@afbjorklund
Copy link
Member Author

afbjorklund commented Oct 7, 2022

The rootless example doesn't need squashfs anymore, since it uses squashfuse instead.

It does need fuse, but that seems to be loaded by default in the fedora operating system.

@AkihiroSuda AkihiroSuda added this to the v0.12.1 milestone Oct 13, 2022
@AkihiroSuda
Copy link
Member

Added follow-up commits

@AkihiroSuda AkihiroSuda force-pushed the singularity-apptainer branch from ee77f0e to 29694c6 Compare October 13, 2022 01:57
Copy link
Member

@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

@AkihiroSuda AkihiroSuda merged commit d7193af into lima-vm:master Oct 13, 2022
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.

2 participants