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

podspec: add additional features for podspec #231

Merged
merged 4 commits into from
Jul 23, 2024

Conversation

vsoch
Copy link
Member

@vsoch vsoch commented Jul 21, 2024

runtimeClassName can be used to designate nvidia for skypilot.

It looks like we also need:

  • an ability to designate specific labels for the leader (index 0) vs the workers. (doesn't seem hugely important)
  • the running user must be sky (we have root) and thus the container needs sudo

vsoch added 3 commits July 21, 2024 17:05
runtimeClassName can be used to designate nvidia
for skypilot

Signed-off-by: vsoch <[email protected]>
Problem: skypilot (and likely others) do not run with a root user
Solution: allow a non-root user that has sudo
Signed-off-by: vsoch <[email protected]>
@vsoch vsoch force-pushed the add-additional-skypilot-features branch from 909b97f to 6d4b7b9 Compare July 22, 2024 01:58
@vsoch vsoch merged commit 7c4006f into main Jul 23, 2024
22 checks passed
@vsoch vsoch deleted the add-additional-skypilot-features branch July 23, 2024 20:04
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.

1 participant