Skip to content
This repository has been archived by the owner on Aug 8, 2024. It is now read-only.

Removing unneeded sshconfig #62

Merged
merged 1 commit into from
Mar 1, 2017
Merged

Removing unneeded sshconfig #62

merged 1 commit into from
Mar 1, 2017

Conversation

stevector
Copy link
Contributor

We have other repos that don't need this. @greg-1-anderson have you seen errors when this config isn't present?

@greg-1-anderson
Copy link
Member

@stevector Without that configuration, you will sometimes (often) get stuck at a prompt to accept the credentials for the remote server. This only happens when you are running remote Drush or WP-CLI commands. I think this configuration needs to stay.

I'm going to try to re-run the test without the cache, and see if it hangs up. I suspect it only worked this time because Circle ran a previous test with the no host key checking, and cached the credentials.

@greg-1-anderson
Copy link
Member

Hm, it said Warning: Permanently added '[appserver.pr-sshconfi.c401fd14-f745-4e51-9af2-f30b45146a0c.drush.in]:2222,[104.130.220.74]:2222' (RSA) to the list of known hosts., which implies that the no host ssh key configuration is active.

@stevector
Copy link
Contributor Author

@greg-1-anderson, have you ever seen a CircleCI build get hung on a prompt to accept the remote server? Our WordPress plugin and Drupal module repos call Drush and wp-cli commands from CircleCI on Pantheon. But they don't explicitly set StrictHostKeyChecking no

@stevector
Copy link
Contributor Author

Related to #61

@greg-1-anderson
Copy link
Member

Maybe this config line is needed in Travis; CircleCI sets this option in /etc/ssh/ssh_config.

@greg-1-anderson greg-1-anderson merged commit 1d38bb0 into master Mar 1, 2017
@greg-1-anderson greg-1-anderson deleted the sshconfig branch March 1, 2017 17:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants