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

Docs: describe data files apptainer #10020

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

tara-det-ai
Copy link
Contributor

@tara-det-ai tara-det-ai commented Oct 4, 2024

eng-det-oncall-support 3 Oct 2024

The issue described revolves around where the user is trying to run a Determined AI shell with Singularity/Apptainer to perform interactive inference tasks. How Singularity/Apptainer handles home directories and file bindings, differs from how Docker containers might handle similar configurations. The files are actually copied to directories including /run/determined/workdir (which is where we should point users), but because of the automatic home directory binding, the user is placed in their home directory upon shell launch, which might be confusing.

Outside of /run/determined/workdir the users should not use anything in /run/determined.

@cla-bot cla-bot bot added the cla-signed label Oct 4, 2024
@determined-ci determined-ci requested a review from a team October 4, 2024 14:36
@determined-ci determined-ci added the documentation Improvements or additions to documentation label Oct 4, 2024
Copy link

netlify bot commented Oct 4, 2024

Deploy Preview for determined-ui canceled.

Name Link
🔨 Latest commit 08653c4
🔍 Latest deploy log https://app.netlify.com/sites/determined-ui/deploys/670fd0778562bc0008c09ace

@tara-det-ai tara-det-ai marked this pull request as draft October 4, 2024 16:11
@tara-det-ai tara-det-ai force-pushed the docs/describe-data-files-apptainer branch from 91ecd96 to a81364d Compare October 11, 2024 15:26
@determined-ci determined-ci requested a review from a team October 11, 2024 15:26
@tara-det-ai tara-det-ai force-pushed the docs/describe-data-files-apptainer branch 2 times, most recently from 478d3ab to 1fa4e2e Compare October 11, 2024 18:10
@tara-det-ai tara-det-ai marked this pull request as ready for review October 11, 2024 18:11
@tara-det-ai tara-det-ai force-pushed the docs/describe-data-files-apptainer branch from 1fa4e2e to 15c55f1 Compare October 14, 2024 14:52
@determined-ci determined-ci requested a review from a team October 14, 2024 14:53
@tara-det-ai tara-det-ai force-pushed the docs/describe-data-files-apptainer branch from 15c55f1 to 7754210 Compare October 14, 2024 16:58
Copy link
Contributor

@rb-determined-ai rb-determined-ai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great!

eng-det-oncall-support 3 Oct 2024
@tara-det-ai tara-det-ai force-pushed the docs/describe-data-files-apptainer branch from 7754210 to 08653c4 Compare October 16, 2024 14:40
@determined-ci determined-ci requested a review from a team October 16, 2024 14:41
@tara-det-ai tara-det-ai merged commit 3ca3418 into main Oct 16, 2024
78 of 91 checks passed
@tara-det-ai tara-det-ai deleted the docs/describe-data-files-apptainer branch October 16, 2024 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants