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.

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

[6.15.z] Automate test_positive_ansible_variables_installed_with_collection #16629

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #16489

Problem Statement

  1. test_positive_ansible_variables_installed_with_collection is stubbed/Manual test.

Solution

  1. Automate test_positive_ansible_variables_imported_with_roles for admin and non-admin users
    Remove unrelated tests

Related Issues

This PR includes changes from #16431, this needs to be rebased after merge

@Satellite-QE Satellite-QE added 6.15.z Introduced in or relating directly to Satellite 6.15 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Oct 9, 2024
@Satellite-QE Satellite-QE requested a review from a team as a code owner October 9, 2024 13:49
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_ansible.py::TestAnsibleCfgMgmt::test_positive_ansible_variables_installed_with_collection

@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Oct 9, 2024
@devendra104
Copy link
Member

trigger: test-robottelo
pytest: tests/foreman/cli/test_ansible.py::TestAnsibleCfgMgmt::test_positive_ansible_variables_installed_with_collection

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 8932
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_ansible.py::TestAnsibleCfgMgmt::test_positive_ansible_variables_installed_with_collection --external-logging
Test Result : ================== 1 failed, 9 warnings in 573.73s (0:09:33) ===================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Oct 9, 2024
…16489)

* Automate test_positive_ansible_variables_imported_with_roles

Signed-off-by: Gaurav Talreja <[email protected]>

* Automate test_positive_ansible_variables_installed_with_collection

Signed-off-by: Gaurav Talreja <[email protected]>

---------

Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit b56eacd)
@Gauravtalreja1 Gauravtalreja1 force-pushed the cherry-pick-6.15.z-b56eacdea62d8529879838380b29aba06a03c4a6 branch from 6a61992 to 335a7aa Compare October 11, 2024 09:05
@Gauravtalreja1
Copy link
Collaborator

trigger: test-robottelo
pytest: tests/foreman/cli/test_ansible.py::TestAnsibleCfgMgmt::test_positive_ansible_variables_installed_with_collection

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 8955
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_ansible.py::TestAnsibleCfgMgmt::test_positive_ansible_variables_installed_with_collection --external-logging
Test Result : ================== 1 passed, 9 warnings in 745.42s (0:12:25) ===================

@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 Oct 11, 2024
@Satellite-QE Satellite-QE merged commit 0cdc448 into 6.15.z Oct 11, 2024
12 of 13 checks passed
@Satellite-QE Satellite-QE deleted the cherry-pick-6.15.z-b56eacdea62d8529879838380b29aba06a03c4a6 branch October 11, 2024 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.15.z Introduced in or relating directly to Satellite 6.15 Auto_Cherry_Picked Automatically cherrypicked PR using GHA AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants