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

SSH Flexibility for SageMaker clusters #105

Merged
merged 3 commits into from
Sep 16, 2023
Merged

Conversation

jlewitt1
Copy link
Collaborator

Improve the flexibility for connecting to SageMaker clusters by:

  • Maintaining an authorized keys list with public keys stored in S3
  • Copying the authorized keys file onto the cluster when connecting
  • Generating a fresh key pair and adding the public key to the authorized list If user with the relevant SageMaker role does not already have one

@jlewitt1 jlewitt1 marked this pull request as draft September 10, 2023 19:01
@jlewitt1 jlewitt1 changed the title SSH Flexibility for SageMaker clusters [WIP] SSH Flexibility for SageMaker clusters Sep 10, 2023
@jlewitt1 jlewitt1 force-pushed the sagemaker-ssh-flexibility branch from 83dc977 to 22f9b25 Compare September 11, 2023 03:05
@jlewitt1 jlewitt1 force-pushed the sagemaker-ssh-flexibility branch from 22f9b25 to dd3eecc Compare September 11, 2023 03:24
@jlewitt1 jlewitt1 changed the title [WIP] SSH Flexibility for SageMaker clusters SSH Flexibility for SageMaker clusters Sep 13, 2023
@jlewitt1 jlewitt1 marked this pull request as ready for review September 13, 2023 18:40
@jlewitt1 jlewitt1 force-pushed the sagemaker-ssh-flexibility branch 3 times, most recently from 0393dd8 to 86ff7d1 Compare September 14, 2023 20:31
@jlewitt1 jlewitt1 force-pushed the sagemaker-ssh-flexibility branch from 86ff7d1 to ae7c48d Compare September 16, 2023 01:00
@jlewitt1 jlewitt1 merged commit 85fcef0 into main Sep 16, 2023
@jlewitt1 jlewitt1 deleted the sagemaker-ssh-flexibility branch September 29, 2023 22:44
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