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

[PR #5883/dcc3d4f5 backport][stable-6] Add support for setenv parameters #5920

Merged
merged 1 commit into from
Jan 30, 2023
Merged

[PR #5883/dcc3d4f5 backport][stable-6] Add support for setenv parameters #5920

merged 1 commit into from
Jan 30, 2023

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jan 30, 2023

This is a backport of PR #5883 as merged into main (dcc3d4f).

SUMMARY

During the configuration of sudo rights regarding a Python app I created, I needed to allow my user to keep their environments variables while running it. This can be done by using sudo -E [command to run].

To allow this in our sudoers files we need to pass a parameters. However while the sudoers modules allowed to pass nopasswd, it did not allowed to pass the setenv parameters. This PR aims to allow it.

minor_changes:
  - sudoers - add `setenv` parameters to support passing environments variables via sudo.
ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

sudoers

ADDITIONAL INFORMATION
%mygroup ALL=NOPASSWD:SETENV: /usr/local/bin/testing

@patchback patchback bot mentioned this pull request Jan 30, 2023
Merged
@ansibullbot
Copy link
Collaborator

cc @JonEllis @JonEllis0
click here for bot help

@ansibullbot ansibullbot added backport feature This issue/PR relates to a feature request integration tests/integration module module needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR new_contributor Help guide this first time contributor plugins plugin (any type) system tests tests and removed needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Jan 30, 2023
@felixfontein felixfontein merged commit e5930aa into stable-6 Jan 30, 2023
@felixfontein felixfontein deleted the patchback/backports/stable-6/dcc3d4f50843b54aee82a44b059466149d692fa5/pr-5883 branch January 30, 2023 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
feature This issue/PR relates to a feature request integration tests/integration module module new_contributor Help guide this first time contributor plugins plugin (any type) system tests tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ansibullbot @felixfontein @redat00