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

Create separate ssh tunnel class #1171

Merged
merged 2 commits into from
Aug 29, 2024
Merged

Create separate ssh tunnel class #1171

merged 2 commits into from
Aug 29, 2024

Conversation

carolineechen
Copy link
Collaborator

No description provided.

Copy link

sentry-io bot commented Aug 19, 2024

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: runhouse/resources/hardware/sky_ssh_runner.py

Function Unhandled Issue
rsync CommandError: Command rsync -Pavz --filter='dir-merge,- .gitignore' -e "ssh -o Port=22 -o StrictHostKeyChecking... ...
Event Count: 1

Did you find this useful? React with a 👍 or 👎

@carolineechen carolineechen force-pushed the cc/ssh-tunnel-class branch 3 times, most recently from 53a9407 to bf31898 Compare August 20, 2024 04:38
@carolineechen carolineechen force-pushed the cc/ssh-utils branch 2 times, most recently from 2fda42d to 7d447e5 Compare August 21, 2024 18:20
@carolineechen carolineechen force-pushed the cc/ssh-tunnel-class branch 2 times, most recently from 7aeb3d5 to e507c23 Compare August 21, 2024 18:33
@carolineechen carolineechen marked this pull request as ready for review August 21, 2024 20:36
@carolineechen carolineechen force-pushed the cc/ssh-tunnel-class branch 2 times, most recently from 29c0ff0 to 7412d6b Compare August 23, 2024 00:12
@carolineechen carolineechen force-pushed the cc/ssh-tunnel-class branch 2 times, most recently from e6eda28 to ca58d4c Compare August 23, 2024 15:38
@carolineechen carolineechen force-pushed the cc/ssh-tunnel-class branch 2 times, most recently from 291206d to 2583bdf Compare August 29, 2024 16:12
@carolineechen carolineechen force-pushed the cc/ssh-utils branch 2 times, most recently from 153e16f to f1ec92e Compare August 29, 2024 16:32
@carolineechen carolineechen force-pushed the cc/ssh-tunnel-class branch 2 times, most recently from ab4b3fd to edb892c Compare August 29, 2024 16:46
Base automatically changed from cc/ssh-utils to main August 29, 2024 17:15
@carolineechen carolineechen merged commit 7905995 into main Aug 29, 2024
12 checks passed
@carolineechen carolineechen deleted the cc/ssh-tunnel-class branch August 29, 2024 17:20
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.

2 participants