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

option -tty incompatible with vagrant #58

Closed
juan-cabrera opened this issue Jun 7, 2018 · 4 comments · Fixed by #59
Closed

option -tty incompatible with vagrant #58

juan-cabrera opened this issue Jun 7, 2018 · 4 comments · Fixed by #59

Comments

@juan-cabrera
Copy link

Hi,

I have a problem using ansible-silo with vagrant. When doing vagrant provision, I get

the input device is not a TTY
Ansible failed to complete successfully. Any error output should be
visible above. Please fix these errors and try again.

I see the option is hard coded in the run script

command="${docker_cmd} run --interactive --tty --rm"

For the moment I manually remove the --tty option. Is it possible to add a user option or an environment variable to chose if I want tty option or not?

Thanks

Juan

@udondan
Copy link
Collaborator

udondan commented Jun 7, 2018

Thanks Juan. Will take care of it in the next release. I recently stumbled over this myself when trying to run ansible-lint via VS Code plugin.

I assume tty detection should work for you? That way you don't have to set anything. The --tty will only be set if actually it is a tty.

@juan-cabrera
Copy link
Author

Yes, tty detection will be great. Thanks

@udondan
Copy link
Collaborator

udondan commented Jun 8, 2018

Juan, I released Silo 2.2.0 today. Let me know if you have problems with it.

@juan-cabrera
Copy link
Author

No problem. This works fine. Thanks a lot

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 a pull request may close this issue.

2 participants