search

apptainer / singularity

Singularity has been renamed to Apptainer as part of us moving the project to the Linux Foundation. This repo has been persisted as a snapshot right before the changes.
https://github.com/apptainer/apptainer
Other
2.53k stars 424 forks source link

SINGULARITY_TMPDIR=/var/singularity/tmp singularity build not using the env variable, but failing to /tmp with not enough space #2305

Closed titansmc closed 5 years ago

titansmc commented 6 years ago

Version of Singularity:

latest pull from master

Expected behavior

build image in specific tmpdir

Actual behavior

SINGULARITY_TMPDIR=/var/singularity/tmp  singularity --verbose  build ngless Singularity
WARNING: Authentication token file not found : Only pulls of public images will succeed
INFO:    Starting build...
Getting image source signatures
Skipping fetch of repeat blob sha256:f9f73d801f0558b085ffa505240a065319269c4cefbe9c2e60103d58761edfa8
Copying config sha256:a364696682bd9b114799e2764041ee44f7e0573d263e899641bf66c642615f2d
 857 B / 857 B [============================================================] 0s
Writing manifest to image destination
Storing signatures
FATAL:   While performing build: conveyor failed to get: Error writing blob: write /tmp/sbuild-256120010/oci-put-blob738714252: no space left on device

Steps to reproduce behavior

Set diferent path like: SINGULARITY_TMPDIR=/var/singularity/tmp singularity --verbose build ngless Singularity

jmstover commented 6 years ago

@titansmc I believe that you use: TMPDIR=/var/singularity/tmp singularity build [...]

Because of the change to Go, we're using a different code path than we previously did, so just setting TMPDIR works.

jscook2345 commented 6 years ago

TMPDIR is the correct option to use here @titansmc . Please let us know if it does not work for you!

Side note, this should be documented as deprecated or put in the changelog as removed.

titansmc commented 6 years ago

Works like a charm. Thank.

On Sat, Oct 27, 2018, 6:54 PM Justin notifications@github.com wrote:

TMPDIR is the correct option to use here @titansmc https://github.com/titansmc . Please let us know if it does not work for you!

Side note, this should be documented as deprecated or put in the changelog as removed.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sylabs/singularity/issues/2305#issuecomment-433636893, or mute the thread https://github.com/notifications/unsubscribe-auth/AH0AYOxoMErbb3BKrKgMJMAbxp6lf6d7ks5upI-sgaJpZM4X8tKv .

jscook2345 commented 6 years ago

@bauerm97

Where should this go in the changelog? A new section on removed environment variables?

ikaneshiro commented 5 years ago

Hey @titansmc! This is actually a regression, it should be SINGULARITY_TMPDIR and this will be fixed by PR #2396