-
Notifications
You must be signed in to change notification settings - Fork 215
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
"Defaults requiretty" didn't work with Vagrant #101
Comments
a closed issue has been discussed before (#76) Seems you insisted not to change. But the fact is, you should comment it out from your code, which default requiretty is problematic and breaks valid usage, confirmed by Redhat. It is bug and fixed by redhat at March 2014.
|
I think, there should be some information in the docs, how to solve this problem, as it is easily solvable without any change in this module. Maybe you'll got some time to create a pull request? |
I have raised the pull request. #102 |
I'm good with the change, but this needs to be addressed in a better way:
I think this should solve the problem completely. |
My latest pull request has been passed the validation. Waiting for the merge. |
@saz , any updates for my pull request? |
For anyone who stuck with this issue, and still waiting the pull request to be merged. Run below command in your
|
This requiretty/vagrant issue is a major pain. I am still a puppet novice but the saz-sudo module looks to be one of the better forge module options that has with Hiera capability. Could a true/false parameter be added in enable/disable(comment out) requiretty for Centos/RHEL flavors? the following doesn't seem to work for me: |
This is pretty easy to fix by adding an entry overriding the default. Something like the below makes it a non-issue when added to hieradata or done via the manifest itself.
via the manifest it just becomes:
|
Also worth noting... if this issue is resolved, it should probably remove the !visiblepw default entries in all of the centos files as well since they are related configs. |
@vindir |
This should be fixed in the current master |
The option
Defaults requiretty
is enabled in bothfiles/sudoers.rhel5
andfiles/sudoers.rhel6
as default. But when apply this puppet module in Vagrant CentOS 6.5 instance, always get the error:the fix is here: comment it.
This is the explanation:
http://unix.stackexchange.com/questions/122616/why-do-i-need-a-tty-to-run-sudo-if-i-can-sudo-without-a-password
https://bugzilla.redhat.com/show_bug.cgi?id=1020147
Any suggestion that I can still
librarian-puppet
this module, but get that line commented?Or could you please comment this line in your code directly?
The text was updated successfully, but these errors were encountered: