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

ansible.cfg: improve ssh ControlPath #4705

Closed
wants to merge 1 commit into from
Closed

ansible.cfg: improve ssh ControlPath #4705

wants to merge 1 commit into from

Conversation

sosiouxme
Copy link
Member

AWS hosts and user names frequently make the default Ansible ControlPath
too long. This prevents ssh persistent connections, significantly
impacting runtime. More detail: ansible/ansible#11536

This config change shortens the ControlPath to make this less likely. It
would be better to change it to the %C hash, as extremely long hostnames
or usernames could still encounter this problem, but that is not yet
available with RHEL's openssh. This at least improves the situation.

AWS hosts and user names frequently make the default Ansible ControlPath
too long. This prevents ssh persistent connections, significantly
impacting runtime. More detail: ansible/ansible#11536

This config change shortens the ControlPath to make this less likely. It
would be better to change it to the %C hash, as extremely long hostnames
or usernames could still encounter this problem, but that is not yet
available with RHEL's openssh. This at least improves the situation.
@sosiouxme sosiouxme requested a review from tbielawa July 6, 2017 22:21
@tbielawa
Copy link
Contributor

tbielawa commented Jul 7, 2017

aos-ci-test

@tbielawa
Copy link
Contributor

tbielawa commented Jul 7, 2017

[test]

@openshift-bot
Copy link

Evaluated for openshift ansible test up to 94a5c89

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.6_NOT_containerized, aos-ci-jenkins/OS_3.6_NOT_containerized_e2e_tests" for 94a5c89 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_containerized, aos-ci-jenkins/OS_3.5_containerized_e2e_tests" for 94a5c89 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.6_containerized, aos-ci-jenkins/OS_3.6_containerized_e2e_tests" for 94a5c89 (logs)

@openshift-bot
Copy link

success: "aos-ci-jenkins/OS_3.5_NOT_containerized, aos-ci-jenkins/OS_3.5_NOT_containerized_e2e_tests" for 94a5c89 (logs)

@openshift-bot
Copy link

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_openshift_ansible/326/) (Base Commit: 5f49d32) (PR Branch Commit: 94a5c89)

@sosiouxme sosiouxme mentioned this pull request Jul 11, 2017
@sosiouxme
Copy link
Member Author

@tbielawa test failure was nothing... can I get a thumbs up?

@sosiouxme
Copy link
Member Author

merged in #4737

@sosiouxme sosiouxme closed this Jul 12, 2017
@sosiouxme sosiouxme deleted the 20170706-improve-controlpath branch July 12, 2017 11:16
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.

None yet

3 participants