Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

RFE: Audit log for add/remove Ansible host role #17594

Merged
merged 1 commit into from
Feb 17, 2025
Merged

RFE: Audit log for add/remove Ansible host role #17594

merged 1 commit into from
Feb 17, 2025

Conversation

pnovotny
Copy link
Contributor

New RFE test coverage

New UI test for RFE SAT-29715: Have audit information about changes in add/remove of ansible role on client system

@pnovotny
[SAT-29715 RFE] new test for Audit: Ansible host role add/remove
ffba5a2 
New UI test for RFE SAT-29715: Have audit information about changes in add/remove of ansible role on client system.
@pnovotny pnovotny added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Feb 14, 2025
@pnovotny pnovotny self-assigned this Feb 14, 2025
@pnovotny pnovotny requested review from a team February 14, 2025 00:20
@pnovotny
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_audit.py -k test_positive_add_remove_ansible_host_role_event

@pnovotny pnovotny requested review from a team February 14, 2025 00:22
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10178
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_audit.py -k test_positive_add_remove_ansible_host_role_event --external-logging
Test Result : =========== 1 passed, 5 deselected, 14 warnings in 794.51s (0:13:14) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 14, 2025
@pnovotny pnovotny marked this pull request as ready for review February 14, 2025 01:05
Gauravtalreja1
Gauravtalreja1 approved these changes Feb 14, 2025
View reviewed changes
Copy link
Collaborator

@Gauravtalreja1 Gauravtalreja1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK, merge pending on @SatelliteQE/team-endeavour 's review

@pondrejk pondrejk merged commit 17ef27b into SatelliteQE:master Feb 17, 2025
13 checks passed
@pnovotny pnovotny deleted the audit-host-ansible-role branch February 17, 2025 16:27
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@pondrejk pondrejk pondrejk approved these changes

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

@synkd synkd synkd approved these changes

Assignees

@pnovotny pnovotny

Labels
No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@pnovotny @Satellite-QE @pondrejk @Gauravtalreja1 @synkd