Skip to content

Fix SMS ssh #296

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 4, 2023
Merged

Fix SMS ssh #296

merged 1 commit into from
Aug 4, 2023

Conversation

sjpb
Copy link
Collaborator

@sjpb sjpb commented Aug 4, 2023 •
edited
Loading

ControlPath was missing from the original smslabs SSH config.

TODO: change environments/skeleton/{{cookiecutter.environment}}/ansible.cfg as well

@sjpb sjpb changed the base branch from main to fix/jupyter August 4, 2023 09:29
@sjpb
Copy link
Collaborator Author

sjpb commented Aug 4, 2023 •
edited
Loading

Without this PR: 2h 12m 47s (run)

@bae7255:
First run : 57m
Second run: 49m

@sjpb sjpb marked this pull request as ready for review August 4, 2023 11:58
@sjpb sjpb requested a review from a team as a code owner August 4, 2023 11:58
@sjpb sjpb requested a review from m-bull August 4, 2023 11:58
@sjpb sjpb merged commit 6df61ae into fix/jupyter Aug 4, 2023
@sjpb sjpb deleted the fix/sms-ssh branch August 4, 2023 11:59
@sjpb sjpb mentioned this pull request Aug 4, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m-bull m-bull Awaiting requested review from m-bull

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@sjpb