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

Fixes #25481 - Set ProxyCommand=none for Ansible #511

Merged
merged 1 commit into from
Jul 22, 2019

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Jun 6, 2019

In case of IPA there's a ProxyCommand set that handles known hosts, but foreman_ansible has its own way of handling known hosts. This sets the ssh args to force override this.

@ekohl
Copy link
Member Author

ekohl commented Jun 6, 2019

Note: I don't have the environment to verify this.

@ares
Copy link
Member

ares commented Jun 7, 2019

@alexjfisher would you be able to check whether it solves the problem for you?

@alexjfisher
Copy link
Contributor

@ares Sorry, not with that client anymore. Not using ansible at all at the moment.

Assuming someone can verify this actually works, I agree it's the better solution.

In case of IPA there's a ProxyCommand set that handles known hosts, but
foreman_ansible has its own way of handling known hosts. This sets the
ssh args to force override this.
@ekohl ekohl added the Bug label Jul 15, 2019
@ekohl ekohl merged commit 2e11f68 into theforeman:master Jul 22, 2019
@ekohl ekohl deleted the 25481-proxy-command-none branch July 22, 2019 16:30
@ekohl ekohl added this to the 12.0.0 milestone Jul 30, 2019
evgeni pushed a commit that referenced this pull request Aug 20, 2019
In case of IPA there's a ProxyCommand set that handles known hosts, but
foreman_ansible has its own way of handling known hosts. This sets the
ssh args to force override this.

(cherry picked from commit 2e11f68)
ekohl added a commit that referenced this pull request Aug 20, 2019
In case of IPA there's a ProxyCommand set that handles known hosts, but
foreman_ansible has its own way of handling known hosts. This sets the
ssh args to force override this.

(cherry picked from commit 2e11f68)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants