internet/community.general
3
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2026-05-08 06:12:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

sudoers: display stderr raised while validation (#9871)

Browse Source 
* sudoers: display stderr raised while validation

* Print stdout and stderr raised from sudoers validation

Fixes: #9674

Signed-off-by: Abhijeet Kasurde <Akasurde@redhat.com>

* Apply suggestions from code review

Co-authored-by: Felix Fontein <felix@fontein.de>

---------

Signed-off-by: Abhijeet Kasurde <Akasurde@redhat.com>
Co-authored-by: Felix Fontein <felix@fontein.de>
This commit is contained in:
Abhijeet Kasurde
2025-03-14 23:36:54 -07:00
committed by GitHub
parent 3bd0ab4a49
commit dd0a2cbff9
3 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
changelogs/fragments/sudoers.yml Normal file
View File

@@ -0,0 +1,3 @@
---
bugfixes:
- sudoers - display stdout and stderr raised while failed validation (https://github.com/ansible-collections/community.general/issues/9674, https://github.com/ansible-collections/community.general/pull/9871).

2
plugins/modules/sudoers.py
View File

@@ -246,7 +246,7 @@ class Sudoers(object):
rc, stdout, stderr = self.module.run_command(check_command, data=self.content()) rc, stdout, stderr = self.module.run_command(check_command, data=self.content())
if rc != 0: if rc != 0:
raise Exception('Failed to validate sudoers rule:\n{stdout}'.format(stdout=stdout)) self.module.fail_json(msg='Failed to validate sudoers rule:\n{stdout}'.format(stdout=stdout or stderr), stdout=stdout, stderr=stderr)
def run(self): def run(self):
if self.state == 'absent': if self.state == 'absent':

2
tests/integration/targets/sudoers/tasks/main.yml
View File

@@ -284,6 +284,8 @@
that: that:
- edge_case_1 is failed - edge_case_1 is failed
- "'Failed to validate sudoers rule' in edge_case_1.msg" - "'Failed to validate sudoers rule' in edge_case_1.msg"
- edge_case_1.stdout is defined
- edge_case_1.stderr is defined
- edge_case_2 is not failed - edge_case_2 is not failed
- name: Check missing validation edge case - name: Check missing validation edge case