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

add support for sesu become method #27376

Closed
wants to merge 1 commit into from
Closed

add support for sesu become method #27376

wants to merge 1 commit into from

Conversation

nekonyuu
Copy link

SUMMARY

This is a simple new addition to the supported list of become methods, which is sesu, a tool provided by CA Privileged Access Manager for privilege control. The tool has a similar behavior to the su command, although it has no i18n.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

Privilege Escalation system.

ANSIBLE VERSION
2.4.0.0 (based on devel branch)
ADDITIONAL INFORMATION

None, summary is self-explanatory.

@bcoca
Copy link
Member

bcoca commented Jul 27, 2017

lgtm, but have no way to test

@ansibot
Copy link
Contributor

ansibot commented Jul 27, 2017

cc @ghjm @jlaska @matburt @wwitzel3
click here for bot help

@ansibot ansibot added affects_2.4 This issue/PR affects Ansible v2.4 c:constants c:playbook/play_context community_review In order to be merged, this PR must follow the community review workflow. feature_pull_request module This issue/PR relates to a module. support:community This issue/PR relates to code supported by the Ansible community. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. and removed community_review In order to be merged, this PR must follow the community review workflow. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Jul 27, 2017
@nekonyuu
Copy link
Author

I checked if I could update any test around this, without success though, I'd take any help about doing it if you got some pointers for testing that part !

@ansibot ansibot added needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. and removed community_review In order to be merged, this PR must follow the community review workflow. labels Aug 4, 2017
@ansibot ansibot added the new_contributor This PR is the first contribution by a new community member. label Oct 18, 2017
@ansibot ansibot removed the new_contributor This PR is the first contribution by a new community member. label Nov 3, 2017
@ansibot ansibot added the support:core This issue/PR relates to code supported by the Ansible Engineering Team. label Nov 19, 2017
@zigarn
Copy link
Contributor

zigarn commented Dec 6, 2017

Link #10499

@ansibot ansibot added the new_contributor This PR is the first contribution by a new community member. label Jan 23, 2018
@UnitedMarsupials-zz
Copy link

UnitedMarsupials-zz commented Feb 23, 2018 •
edited

My new employer is using sesu enterprise-wide. It would be most useful for us to have this feature working. Can I help getting it into tree?

Meanwhile, I adapted the patch for 2.4.3.0 and will try to use it here.

@ansibot ansibot added feature This issue/PR relates to a feature request. and removed feature_pull_request labels Mar 2, 2018
@ansibot
Copy link
Contributor

ansibot commented Mar 2, 2018

cc @AlanCoding @rooftopcellist @ryanpetrello @simfarm
click here for bot help

@wenottingham wenottingham mentioned this pull request Mar 2, 2018
Closed
@UnitedMarsupials-zz
Copy link

I'm using this version of the patch with Ansible-2.5.2.

uqs pushed a commit to freebsd/freebsd-ports that referenced this pull request May 3, 2018
sysutils/ansible: add 'sesu' become method
0ce62b7 
This adds CA sesu as a "become" method.

Pull request: ansible/ansible#27376

PR:		227934
Submitted by:	mi


git-svn-id: svn+ssh://svn.freebsd.org/ports/head@468946 35697150-7ecd-e111-bb59-0022644237b5
uqs pushed a commit to freebsd/freebsd-ports that referenced this pull request May 3, 2018
@lifanov
sysutils/ansible: add 'sesu' become method
51d6f86 
This adds CA sesu as a "become" method.

Pull request: ansible/ansible#27376

PR:		227934
Submitted by:	mi
swills pushed a commit to swills/freebsd-ports that referenced this pull request May 3, 2018
@lifanov
sysutils/ansible: add 'sesu' become method
fbf37f2 
This adds CA sesu as a "become" method.

Pull request: ansible/ansible#27376

PR:		227934
Submitted by:	mi


git-svn-id: svn+ssh://svn.freebsd.org/ports/head@468946 35697150-7ecd-e111-bb59-0022644237b5
Jehops pushed a commit to Jehops/freebsd-ports-legacy that referenced this pull request May 3, 2018
@lifanov
sysutils/ansible: add 'sesu' become method
63b41ee 
This adds CA sesu as a "become" method.

Pull request: ansible/ansible#27376

PR:		227934
Submitted by:	mi


git-svn-id: svn+ssh://svn.freebsd.org/ports/head@468946 35697150-7ecd-e111-bb59-0022644237b5
@zemian
Copy link

zemian commented May 29, 2018

Hi, I am facing same issue and need 'sesu' support with Ansible. The current PR has conflict against latest devel branch. And despite I resolved it, it's still missing one of the error msg map. I have fixed all these and it's now working for me from my fork here: https://github.com/zemian/ansible under "zemian_v2.5.3p20180519" branch.

Note that my fix is based on v2.5.3, not the latest "devel" branch since I need something that's stable and working in my workplace right now. But if you guys want the contribution, I will be happy to make any changes. I would love to see this be part of official Ansible.

@sakof
Copy link

sakof commented Jun 8, 2018

@zemian: have you heard anything from anyone about getting your code merged? I'd be very interested in seeing this available.

@zemian
Copy link

zemian commented Jun 8, 2018

Hi @sakof, no I have not hear any feedback yet.

@ansibot ansibot removed the support:community This issue/PR relates to code supported by the Ansible community. label Sep 21, 2018
@ansibot ansibot added the support:community This issue/PR relates to code supported by the Ansible community. label Oct 7, 2018
@AlanCoding
Copy link
Member

#50421 will almost certainly land before this.

Given that, I think it will have to be changed into the new plugin structure. Ping @abadger, because it seems timely and relevant for his work.

@ansibot ansibot added the web_infrastructure Web-infrastructure category label Feb 16, 2019
@ansibot ansibot added the tower Tower community label Feb 24, 2019
@ansibot ansibot added the commands Commands category label Mar 4, 2019
@ansibot
Copy link
Contributor

ansibot commented Mar 12, 2019

cc @jladdjr @kdelee
click here for bot help

@ansibot
Copy link
Contributor

ansibot commented Jun 8, 2019

cc @chrismeyersfsu
click here for bot help

@AlanCoding
Copy link
Member

-label tower

@Akasurde
Copy link
Member

superseded by #50991

@Akasurde Akasurde closed this Oct 20, 2019
@Akasurde
Copy link
Member

@nekonyuu Thanks for the contribution.

@ansible ansible locked and limited conversation to collaborators Nov 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
affects_2.4 This issue/PR affects Ansible v2.4 c:constants c:playbook/play_context commands Commands category constants feature This issue/PR relates to a feature request. has_issue module This issue/PR relates to a module. needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. new_contributor This PR is the first contribution by a new community member. playbook/play_context stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. support:community This issue/PR relates to code supported by the Ansible community. support:core This issue/PR relates to code supported by the Ansible Engineering Team. tower Tower community web_infrastructure Web-infrastructure category
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
9 participants
@nekonyuu @bcoca @ansibot @zigarn @UnitedMarsupials-zz @zemian @sakof @AlanCoding @Akasurde