Skip to content
This repository was archived by the owner on Jul 11, 2019. It is now read-only.

[Provisioner] Added ansible_ssh_private_key_file var.#326

Open
mwardRH wants to merge 1 commit intoansible:masterfrom
mwardRH:patch-1
Open

[Provisioner] Added ansible_ssh_private_key_file var.#326
mwardRH wants to merge 1 commit intoansible:masterfrom
mwardRH:patch-1

Conversation

@mwardRH
Copy link

@mwardRH mwardRH commented Apr 28, 2018

Adding in this into extra vars options lets them user set the variable. I ran into an issue where the provisioner would provision the instances using my ec2_key_name pem file but would fail when trying to connect post provision to gather facts. I had to point to the directory of my ansible_ssh_private_key_file to the correct file using the full file path. Now it provisions correctly.

Adding in this into extra vars options lets them user set the variable. I ran into an issue where the provisioner would provision the instances using my ec2_key_name pem file but would fail when trying to connect post provision to gather facts. I had to point to the directory of my ansible_ssh_private_key_file to the correct file using the full file path. Now it provisions correctly.
@tima tima requested a review from liquidat April 30, 2018 09:00
@liquidat
Copy link
Contributor

@mwardRH Just to understand the issue - did this happen when your key was in $HOME/.ssh? Or does the issue by chance only happens when the key is not in $HOME/.ssh?

@tima
Copy link
Collaborator

tima commented Jun 4, 2018

@mwardRH: This is waiting on your response to @liquidat.

@gundalow
Copy link
Collaborator

gundalow commented Dec 3, 2018

@mwardRH Just reviewing old PRs, could you reply to

Just to understand the issue - did this happen when your key was in $HOME/.ssh? Or does the issue by chance only happens when the key is not in $HOME/.ssh?

@gundalow gundalow changed the title Added ansible_ssh_private_key_file var. [Provisioner] Added ansible_ssh_private_key_file var. Dec 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants