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

Added ansible role syncing and delete ansible role... #16054

Merged
merged 1 commit into from
Sep 26, 2024
Merged

Added ansible role syncing and delete ansible role... #16054

merged 1 commit into from
Sep 26, 2024

Conversation

amolpati30
Copy link
Contributor

@amolpati30 amolpati30 commented Aug 22, 2024 •
edited
Loading

  1. While importing the Ansible role, it was noticed that the 'RedHatInsights.insights-client' role had already been imported. This caused issues with the Ansible test cases, resulting in errors due to conflicts between the synced role and the imported role. To resolve this, I used the sync option, which automatically syncs the selected role without requiring the import of all roles.

  2. The add_single_ansible_role method will now accept an additional parameter, role, which will determine the Ansible role to be added to the host.

Dependent PR: SatelliteQE/airgun#1542

@amolpati30 amolpati30 requested a review from a team as a code owner August 22, 2024 17:37
@amolpati30 amolpati30 added 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 6.16.z Introduced in or relating directly to Satellite 6.16 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.13.z Introduced in or relating directly to Satellite 6.13 CherryPick PR needs CherryPick to previous branches labels Aug 22, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k 'test_positive_assign_and_remove_ansible_role_to_hostgroup or test_positive_ansible_config_report_changes_notice_and_failed_tasks_errors'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8255
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_and_remove_ansible_role_to_hostgroup or test_positive_ansible_config_report_changes_notice_and_failed_tasks_errors --external-logging
Test Result : ==== 1 failed, 1 passed, 12 deselected, 145 warnings in 1087.78s (0:18:07) =====

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

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k 'test_positive_assign_and_remove_ansible_role_to_hostgroup or test_positive_ansible_config_report_changes_notice_and_failed_tasks_errors'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8256
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_and_remove_ansible_role_to_hostgroup or test_positive_ansible_config_report_changes_notice_and_failed_tasks_errors --external-logging
Test Result : ========= 2 passed, 12 deselected, 145 warnings in 1298.81s (0:21:38) ==========

@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 Aug 22, 2024
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 28, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k 'test_positive_assign_and_remove_ansible_role_to_hostgroup or test_positive_ansible_config_report_changes_notice_and_failed_tasks_errors'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8332
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_and_remove_ansible_role_to_hostgroup or test_positive_ansible_config_report_changes_notice_and_failed_tasks_errors --external-logging
Test Result : ========= 2 passed, 12 deselected, 145 warnings in 1339.19s (0:22:19) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 28, 2024
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 30, 2024
@amolpati30 amolpati30 removed the 6.13.z Introduced in or relating directly to Satellite 6.13 label Aug 30, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8702
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_non_admin_user_access_with_usergroup --external-logging
Test Result : ====== 1 passed, 13 deselected, 25 warnings, 1 error in 817.17s (0:13:37) ======

@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k 'test_positive_non_admin_user_access_with_usergroup'
airgun: 1542

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8704
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_non_admin_user_access_with_usergroup --external-logging
Test Result : ========== 1 passed, 13 deselected, 24 warnings in 827.30s (0:13:47) ===========

@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 Sep 18, 2024
shweta83
shweta83 requested changes Sep 18, 2024
View reviewed changes
tests/foreman/ui/test_ansible.py Outdated Show resolved Hide resolved
tests/foreman/ui/test_ansible.py Show resolved Hide resolved
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Sep 23, 2024
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k 'test_positive_assign_and_remove_ansible_role_to_hostgroup or test_positive_ansible_config_report_changes_notice_and_failed_tasks_errors or test_positive_assign_and_remove_ansible_role_to_host'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8770
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_ansible.py::TestAnsibleCfgMgmt -k test_positive_assign_and_remove_ansible_role_to_hostgroup or test_positive_ansible_config_report_changes_notice_and_failed_tasks_errors or test_positive_assign_and_remove_ansible_role_to_host --external-logging
Test Result : ========= 3 passed, 11 deselected, 153 warnings in 1247.12s (0:20:47) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Sep 23, 2024
@Gauravtalreja1 Gauravtalreja1 merged commit a565bd6 into SatelliteQE:master Sep 26, 2024
11 checks passed
github-actions bot pushed a commit that referenced this pull request Sep 26, 2024
@amolpati30 @web-flow
Added ansible role syncing and delete ansible role... (#16054)
5116018 
add ansible role sync and delete role

(cherry picked from commit a565bd6)
github-actions bot pushed a commit that referenced this pull request Sep 26, 2024
@amolpati30 @web-flow
Added ansible role syncing and delete ansible role... (#16054)
c8fdbd7 
add ansible role sync and delete role

(cherry picked from commit a565bd6)
github-actions bot pushed a commit that referenced this pull request Sep 26, 2024
@amolpati30 @web-flow
Added ansible role syncing and delete ansible role... (#16054)
b323e54 
add ansible role sync and delete role

(cherry picked from commit a565bd6)
This was referenced Sep 26, 2024
Merged
Merged
Merged
Gauravtalreja1 pushed a commit that referenced this pull request Sep 26, 2024
@Satellite-QE @amolpati30
[6.16.z] Added ansible role syncing and delete ansible role... (#16514)
f830a59 
Added ansible role syncing and delete ansible role... (#16054)

add ansible role sync and delete role

(cherry picked from commit a565bd6)

Co-authored-by: amolpati30 <[email protected]>
Gauravtalreja1 pushed a commit that referenced this pull request Sep 26, 2024
@Satellite-QE @amolpati30
[6.15.z] Added ansible role syncing and delete ansible role... (#16515)
6d0e082 
Added ansible role syncing and delete ansible role... (#16054)

add ansible role sync and delete role

(cherry picked from commit a565bd6)

Co-authored-by: amolpati30 <[email protected]>
Gauravtalreja1 pushed a commit that referenced this pull request Sep 27, 2024
@Satellite-QE @amolpati30
[6.14.z] Added ansible role syncing and delete ansible role... (#16513)
7d73ee6 
Added ansible role syncing and delete ansible role... (#16054)

add ansible role sync and delete role

(cherry picked from commit a565bd6)

Co-authored-by: amolpati30 <[email protected]>
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@amolpati30 @jyejare
Added ansible role syncing and delete ansible role... (SatelliteQE#16054
1187076 
)

add ansible role sync and delete role
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

@shweta83 shweta83 Awaiting requested review from shweta83

Assignees

@amolpati30 amolpati30

Labels
6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 6.16.z Introduced in or relating directly to Satellite 6.16 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
Projects
None yet
Milestone
No milestone
Development

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