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

Use APPTAINER_BINDPATH instead of APPTAINER_HOME #1125

Merged
merged 1 commit into from
Aug 5, 2023

Conversation

afbjorklund
Copy link
Contributor

@afbjorklund afbjorklund commented Oct 20, 2022

Not all software was happy with a read-only $HOME directory.

Using APPTAINER_HOME changed the $HOME in the container.


So use the regular VM $HOME, but do bind-mount host $HOME.

Using APPTAINER_BINDPATH is the same as using the -B flag.

cmd/apptainer.lima Outdated Show resolved Hide resolved
@afbjorklund
Copy link
Contributor Author

afbjorklund commented Oct 23, 2022

@AkihiroSuda : it would be nice to have this bugfix in 0.13.0, if it is possible

Fix for:

The workaround is running limactl shell apptainer apptainer run -B $HOME

DrDaveD
DrDaveD previously approved these changes Oct 24, 2022
cmd/apptainer.lima Outdated Show resolved Hide resolved
DrDaveD
DrDaveD previously approved these changes Oct 25, 2022
@afbjorklund
Copy link
Contributor Author

Rebased on top of v0.13.0 release, will recommend limactl shell apptainer until it is fixed.

https://apptainer.org/docs/admin/main/installation.html#id3

Not all software was happy with a read-only $HOME directory.

So use the regular VM $HOME, but do bind-mount host $HOME.

Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
@AkihiroSuda AkihiroSuda added this to the v0.18.0 milestone Aug 5, 2023
@AkihiroSuda
Copy link
Member

Sorry, I should have approved this much earlier

@AkihiroSuda AkihiroSuda merged commit 7de95ca into lima-vm:master Aug 5, 2023
22 checks passed
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.

None yet

3 participants