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.

Sign up for GitHub

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

Jump to bottom

Feature Idea: Run syntax/file/role/etc checks before asking for passwords/user inupt #9904

Closed
jerrac opened this issue Dec 28, 2014 · 2 comments · Fixed by #18265
Closed

Feature Idea: Run syntax/file/role/etc checks before asking for passwords/user inupt #9904

jerrac opened this issue Dec 28, 2014 · 2 comments · Fixed by #18265
Assignees
@jctanner
Labels
affects_2.1 This issue/PR affects Ansible v2.1 bug This issue/PR relates to a bug.

Comments

@jerrac
Copy link

jerrac commented Dec 28, 2014

For example, if I'm running a play than needs the --ask-sudo-pass flag, then ansible should not ask for the password until after it's checked for syntax errors, that all the roles exist, and other pre-task checks.
@jerrac jerrac changed the title Feature Idea: Can the yaml syntax check run before asking for passwords, when using --ask-pass or --ask-sudo-pass? Feature Idea: Run syntax/file/role/etc checks before asking for passwords/user inupt Dec 28, 2014
@bcoca bcoca added P3 labels Dec 29, 2014
@lielran
Copy link

lielran commented Jan 19, 2015

👍

@jimi-c jimi-c removed the P3 label Dec 7, 2015
@baldurmen
Copy link

This is a very good idea. It's very frustrating to have to type a long password to then have ansible tell you the playbook cannot be found since your path was wrong.

@haroldTlan haroldTlan mentioned this issue Sep 2, 2016
Closed
@ansibot ansibot added triage affects_2.1 This issue/PR affects Ansible v2.1 labels Sep 7, 2016
@jctanner jctanner mentioned this issue Oct 31, 2016
Merged
@jctanner jctanner self-assigned this Oct 31, 2016
@bcoca bcoca removed the triage label Oct 31, 2016
bcoca pushed a commit that referenced this issue Oct 31, 2016
@jctanner @bcoca
Move the check for playbook files above the password prompting.
6d9771b 
Fixes #9904
@ansibot ansibot added bug This issue/PR relates to a bug. and removed bug_report labels Mar 6, 2018
@ansible ansible locked and limited conversation to collaborators Apr 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jctanner jctanner

Labels
affects_2.1 This issue/PR affects Ansible v2.1 bug This issue/PR relates to a bug.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Move the check for playbook files above the password prompting.
7 participants
@jimi-c @bcoca @jerrac @lielran @jctanner @baldurmen @ansibot