Skip to content

Read sudo_prefix from config file #127

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

Merged
merged 1 commit into from
Jan 4, 2013
Merged

Conversation

paneq
Copy link
Contributor

@paneq paneq commented Dec 19, 2012

This is first step to allow to overwrite some Fabric default values via config file. I personally needed to set sudo_prefix in config file:

[ssh]
sudo_prefix =sudo -E -S -p '%(sudo_prompt)s'

The additional -E switch for sudo preserves environment variable which in my case was SSH_AUTH_SOCK that I needed so that chef user can log into another machines using ForwardAgent ssh option when running one of the recipe.

@tobami
Copy link
Owner

tobami commented Dec 20, 2012

Would't this be fixed once #59 is implemented?

@paneq
Copy link
Contributor Author

paneq commented Jan 4, 2013

I guess not. You still need to preserve environment to have agent forwarding working under sudo command.

tobami added a commit that referenced this pull request Jan 4, 2013
Read sudo_prefix from config file
@tobami tobami merged commit 2d5e8ed into tobami:master Jan 4, 2013
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.

2 participants