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

Await SSH tunnel init using docker exec #18

Closed
wants to merge 1 commit into from
Closed

Await SSH tunnel init using docker exec #18

wants to merge 1 commit into from

Conversation

samsonjs
Copy link

This change is copied from #10 which was never finished. It adds a 5-second timeout to the wget command as requested over there. I'm not sure why this change is necessary but it is necessary for my setup on macOS 12.

Before this change:

% docker-push-ssh -i ~/.ssh/id_ed25519 me@some.server.net vidjo:latest
[REQUIRED] Ensure localhost:5000 is added to your insecure registries.
Setting up secure private registry... 
Establishing SSH Tunnel...
Waiting for SSH Tunnel Initialization...
ERROR
SSH Tunnel failed to initialize.
('', "Warning: Permanently added 'some.server.net,1.2.3.4' (ECDSA) to the list of known hosts.\r\n")
Cleaning up...

and after:

% docker-push-ssh -i ~/.ssh/id_ed25519 me@some.server.net vidjo:latest
[REQUIRED] Ensure localhost:5000 is added to your insecure registries.
Setting up secure private registry... 
Establishing SSH Tunnel...
Waiting for SSH Tunnel Initialization...
Priming Registry with base images...
Tagging image(s) for push...
Pushing Image(s) from local host...
Pushed Image vidjo:latest Successfully...
Pulling and Retagging Image on remote host...
Pulled Image vidjo:latest Successfully...
Cleaning up...

This change is from a different pull-request that was never finished. It
adds a 5-second timeout to the wget command as requested in that PR.

#10
@samsonjs samsonjs closed this by deleting the head repository Aug 15, 2024
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