[bug#71552] [PATCH] pack: Create /tmp in Apptainer images.

[bug#71552] [PATCH] pack: Create /tmp in Apptainer images.

[Lars Bilke]

Related to <https://bugs.gnu.org/37161> and
<https://git.savannah.gnu.org/cgit/guix.git/commit/?id=7979a287f8eb84cbbfa44629951572408939a756>.

Change-Id: I576aaa6ba8cb8478acf4c3144d492ae5caf411ca
---
 guix/scripts/pack.scm | 3 ++-
 tests/pack.scm        | 1 +
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/guix/scripts/pack.scm b/guix/scripts/pack.scm
index fe4df042d7..7c5fe76fe0 100644
--- a/guix/scripts/pack.scm
+++ b/guix/scripts/pack.scm
@@ -493,7 +493,8 @@ (define* (squashfs-image name profile
                "-p" "/proc d 555 0 0"
                "-p" "/sys d 555 0 0"
                "-p" "/dev d 555 0 0"
-               "-p" "/home d 555 0 0"))
+               "-p" "/home d 555 0 0"
+               "-p" "/tmp d 555 0 0"))
 
             (when database
               ;; Initialize /var/guix.
diff --git a/tests/pack.scm b/tests/pack.scm
index 40897a5589..f8a9e09c28 100644
--- a/tests/pack.scm
+++ b/tests/pack.scm
@@ -328,6 +328,7 @@ (define rpm-for-tests
                            (when (and (file-exists? (string-append bin
                                                                    "/guile"))
                                       (file-exists? "var/guix/db/db.sqlite")
+                                      (file-is-directory? "tmp")
                                       (string=? (string-append #$%bootstrap-guile "/bin")
                                                 (pk 'binlink (readlink bin)))
 

base-commit: 0beb0dbfe036763e7b5d6c999fe2b3fc89faa6cb
-- 
2.44.0

[bug#71552] Acknowledgement ([PATCH] pack: Create /tmp in Apptainer images.)

[Lars Bilke]

Partly fixes <https://debbugs.gnu.org/cgi/bugreport.cgi?bug=70316>.

We could add the /var/tmp mount point as well.

On 14 Jun 2024, at 15:25, GNU bug Tracking System wrote:

> Thank you for filing a new bug report with debbugs.gnu.org.
>
> This is an automatically generated reply to let you know your message
> has been received.
>
> Your message is being forwarded to the package maintainers and other
> interested parties for their attention; they will reply in due course.
>
> As you requested using X-Debbugs-CC, your message was also forwarded to
>   Christopher Baines <guix@cbaines.net>, Josselin Poiret <dev@jpoiret.xyz>, Ludovic Court**s <ludo@gnu.org>, Mathieu Othacehe <othacehe@gnu.org>, Ricardo Wurmus <rekado@elephly.net>, Simon Tournier <zimon.toutoune@gmail.com>, Tobias Geerinckx-Rice <me@tobias.gr>
> (after having been given a bug report number, if it did not have one).
>
> Your message has been sent to the package maintainer(s):
>  guix-patches@gnu.org
>
> If you wish to submit further information on this problem, please
> send it to 71552@debbugs.gnu.org.
>
> Please do not send mail to help-debbugs@gnu.org unless you wish
> to report a problem with the Bug-tracking system.
>
> -- 
> 71552: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=71552
> GNU Bug Tracking System
> Contact help-debbugs@gnu.org with problems

bug#71552: [PATCH] pack: Create /tmp in Apptainer images.

[Ludovic Courtès]

Hi Lars,

Lars Bilke <lars.bilke@ufz.de> skribis:

> Related to <https://bugs.gnu.org/37161> and
> <https://git.savannah.gnu.org/cgit/guix.git/commit/?id=7979a287f8eb84cbbfa44629951572408939a756>.
>
> Change-Id: I576aaa6ba8cb8478acf4c3144d492ae5caf411ca

I tweaked the commit log to match our conventions and applied it.
Thanks!

I don’t recall having problems with images that lack /tmp (we use ‘guix
pack -f squashfs’ regularly at work), but I guess having it cannot hurt.

Ludo’.

[bug#71552] [PATCH] pack: Create /tmp in Apptainer images.

[Lars Bilke]

[-- Attachment #1: Type: text/plain, Size: 852 bytes --]

Hi Ludo,

thanks a lot for merging!

Without this patch I could not run OpenMPI / PETSc applications via 
apptainer, I had to specify `apptainer exec --bind /tmp:/tmp ...` but I 
can't recall the actual error message anymore...

Sincerely,
Lars

On 10 Jul 2024, at 16:03, Ludovic Courtès wrote:

> Hi Lars,
>
> Lars Bilke <lars.bilke@ufz.de> skribis:
>
>> Related to <https://bugs.gnu.org/37161> and
>> <https://git.savannah.gnu.org/cgit/guix.git/commit/?id=7979a287f8eb84cbbfa44629951572408939a756> 
>> .
>>
>> Change-Id: I576aaa6ba8cb8478acf4c3144d492ae5caf411ca
>
> I tweaked the commit log to match our conventions and applied it.
> Thanks!
>
> I don’t recall having problems with images that lack /tmp (we use 
> ‘guix
> pack -f squashfs’ regularly at work), but I guess having it cannot 
> hurt.
>
> Ludo’.

[-- Attachment #2: Type: text/html, Size: 2090 bytes --]

[bug#71552] [PATCH] pack: Create /tmp in Apptainer images.

[Ludovic Courtès]

Hello,

Lars Bilke <lars.bilke@ufz.de> skribis:

> Without this patch I could not run OpenMPI / PETSc applications via
> apptainer, I had to specify `apptainer exec --bind /tmp:/tmp ...` but
> I can't recall the actual error message anymore...

OK (I believe we only used the older ‘singularity’ command, which might
be different in this regard).

Thanks for explaining,
Ludo’.