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

Check ansible playbook after job schedule... #15802

Merged
merged 1 commit into from
Aug 12, 2024
Merged

Check ansible playbook after job schedule... #15802

merged 1 commit into from
Aug 12, 2024

Conversation

amolpati30
Copy link
Contributor

  • After adding a new user and running the job, the system displayed the root user instead of the new user.

  • The automation test below is written to verify that the new user is correctly set after the job runs.

@amolpati30 amolpati30 requested review from a team as code owners July 30, 2024 15:42
@amolpati30 amolpati30 added CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 QETestCoverage Issues and PRs relating to a Satellite bug labels Jul 30, 2024
@amolpati30 amolpati30 force-pushed the privilege_escalation_ansible_playbook branch 3 times, most recently from c48735d to e0f6f3c Compare July 30, 2024 16:01
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_privilege_escalation_ansible_playbook

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7901
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_privilege_escalation_ansible_playbook --external-logging
Test Result : ========== 1 failed, 11 deselected, 24 warnings in 1277.09s (0:21:17) ==========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jul 30, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_privilege_escalation_ansible_playbook

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7902
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_privilege_escalation_ansible_playbook --external-logging
Test Result : ========== 1 failed, 11 deselected, 24 warnings in 1181.28s (0:19:41) ==========

@amolpati30 amolpati30 force-pushed the privilege_escalation_ansible_playbook branch 2 times, most recently from 526319a to aa91b5d Compare July 31, 2024 07:06
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_privilege_escalation_ansible_playbook

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7910
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_privilege_escalation_ansible_playbook --external-logging
Test Result : ========== 1 passed, 11 deselected, 24 warnings in 969.20s (0:16:09) ===========

@Satellite-QE Satellite-QE added PRT-Passed Indicates that latest PRT run is passed for the PR and removed PRT-Failed Indicates that latest PRT run is failed for the PR labels Jul 31, 2024
Gauravtalreja1
Gauravtalreja1 reviewed Jul 31, 2024
View reviewed changes
tests/foreman/api/test_ansible.py Outdated Show resolved Hide resolved
tests/foreman/api/test_ansible.py Outdated Show resolved Hide resolved
tests/foreman/api/test_ansible.py Outdated Show resolved Hide resolved
tests/foreman/api/test_ansible.py Outdated Show resolved Hide resolved
tests/foreman/api/test_ansible.py Outdated Show resolved Hide resolved
tests/foreman/api/test_ansible.py Outdated Show resolved Hide resolved
tests/foreman/api/test_ansible.py Outdated Show resolved Hide resolved
tests/foreman/api/test_ansible.py Show resolved Hide resolved
@amolpati30 amolpati30 force-pushed the privilege_escalation_ansible_playbook branch from aa91b5d to a8ff012 Compare July 31, 2024 12:17
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 31, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_privilege_escalation_ansible_playbook

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7922
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_privilege_escalation_ansible_playbook --external-logging
Test Result : ========== 1 passed, 11 deselected, 24 warnings in 980.51s (0:16:20) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 31, 2024
Gauravtalreja1
Gauravtalreja1 reviewed Aug 9, 2024
View reviewed changes
tests/foreman/api/test_ansible.py Outdated Show resolved Hide resolved
tests/foreman/api/test_ansible.py Outdated Show resolved Hide resolved
tests/foreman/api/test_ansible.py Outdated Show resolved Hide resolved
tests/foreman/api/test_ansible.py Outdated Show resolved Hide resolved
@amolpati30 amolpati30 force-pushed the privilege_escalation_ansible_playbook branch 2 times, most recently from 675aba6 to ae23433 Compare August 9, 2024 13:36
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 9, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_privilege_escalation

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8064
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_privilege_escalation --external-logging
Test Result : ========== 1 passed, 11 deselected, 24 warnings in 1053.58s (0:17:33) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 9, 2024
@amolpati30 amolpati30 force-pushed the privilege_escalation_ansible_playbook branch from ae23433 to 35c6150 Compare August 12, 2024 11:34
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 12, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_privilege_escalation

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8078
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_privilege_escalation --external-logging
Test Result : ========== 1 passed, 11 deselected, 24 warnings in 2020.15s (0:33:40) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 12, 2024
@amolpati30 amolpati30 added the 6.13.z Introduced in or relating directly to Satellite 6.13 label Aug 12, 2024
@amolpati30 amolpati30 force-pushed the privilege_escalation_ansible_playbook branch from 35c6150 to 823e9c2 Compare August 12, 2024 13:57
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 12, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_privilege_escalation

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8081
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_ansible.py::TestAnsibleREX -k test_positive_ansible_job_privilege_escalation --external-logging
Test Result : ========== 1 passed, 11 deselected, 24 warnings in 956.92s (0:15:56) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 12, 2024
@Gauravtalreja1 Gauravtalreja1 merged commit d4ceb5d into SatelliteQE:master Aug 12, 2024
11 checks passed
@Satellite-QE Satellite-QE mentioned this pull request Aug 12, 2024
Closed
github-actions bot pushed a commit that referenced this pull request Aug 12, 2024
@amolpati30 @web-flow
Check ansible playbook after job schedule... (#15802)
586e86d 
verify job invocation for different user should show after job schedule

(cherry picked from commit d4ceb5d)
github-actions bot pushed a commit that referenced this pull request Aug 12, 2024
@amolpati30 @web-flow
Check ansible playbook after job schedule... (#15802)
7247971 
verify job invocation for different user should show after job schedule

(cherry picked from commit d4ceb5d)
This was referenced Aug 12, 2024
Merged
Merged
@amolpati30 amolpati30 mentioned this pull request Aug 27, 2024
Merged
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@amolpati30 @jyejare
Check ansible playbook after job schedule... (SatelliteQE#15802)
cccd73d 
verify job invocation for different user should show after job schedule
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees

@amolpati30 amolpati30

Labels
6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR QETestCoverage Issues and PRs relating to a Satellite bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@amolpati30 @Satellite-QE @Gauravtalreja1