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

Update sudoers.sh #201

Closed
wants to merge 1 commit into from
Closed

Update sudoers.sh #201

wants to merge 1 commit into from

Conversation

mokevnin
Copy link

@mokevnin mokevnin commented Apr 3, 2014

"Sorry, user vagrant is not allowed to execute '/bin/sh -c echo SUDO-SUCCESS-sbawtdhzwzxpqlbsucskxmklqogloqyu; /usr/bin/python /tmp/ansible-tmp-1396539205.79-85165511766742/postgresql_user' as postgres on vagrant.vm."

"Sorry, user vagrant is not allowed to execute '/bin/sh -c echo SUDO-SUCCESS-sbawtdhzwzxpqlbsucskxmklqogloqyu; /usr/bin/python /tmp/ansible-tmp-1396539205.79-85165511766742/postgresql_user' as postgres on vagrant.vm."
@@ -13,5 +13,5 @@ then
sed -i -e 's/%admin\s*ALL=(ALL) ALL/%admin\tALL=NOPASSWD:ALL/g' /etc/sudoers
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be changed too, so that Ubuntus =< 12 are fixed?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also encountered the same problem.
It should be corrected.
Please see also hashicorp/vagrant#892

@stopiccot
Copy link

I'm running the same issue in my chef/12.04 vagrant box. Why this commit is neither accepted nor rejected?

@juliandunn
Copy link
Contributor

Because I was waiting for @mokevnin to advise if Ubuntus =< 12 are also impacted. The proposed change is in a clause that only deals with newer Ubuntus.

@stopiccot
Copy link

I'm experiencing this issue running https://vagrantcloud.com/chef/boxes/ubuntu-12.04

@stopiccot
Copy link

So Ubuntu 12.04 seems to be affected

@mokevnin
Copy link
Author

mokevnin commented Oct 7, 2014

sorry, i have no time for testing that (.

@legal90
Copy link
Contributor

legal90 commented Apr 3, 2015

I've found a time and verified these changes on Ubuntu 12 and 14 as well: #338

@cheeseplus
Copy link
Contributor

Looks like this was resolved by #338. Closing.

@cheeseplus cheeseplus closed this Jul 6, 2015
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 this pull request may close these issues.

6 participants