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] Move ansible API tests to Ansible-Cfgmgmt and REX classes #14635

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #14626

Problem Statement

Ansible tests are mixed in API ansible module, which need to be consistent as CLI/UI module

Solution

Move Ansible API tests to Ansible-ConfigurationManagement and Ansible-RemoteExecution classes, And few other minor fixes.

Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 4ad590a)
@Satellite-QE Satellite-QE requested a review from a team as a code owner April 5, 2024 07:48
@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 Apr 5, 2024
@Satellite-QE
Copy link
Collaborator Author

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

@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Apr 5, 2024
@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 6321
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_ansible.py --external-logging
Test Result : =========== 12 passed, 1330 warnings, 1 error in 5883.20s (1:38:03) ============

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Apr 5, 2024
@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 6325
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_ansible.py --external-logging
Test Result : =========== 12 passed, 1228 warnings, 1 error in 5335.51s (1:28:55) ============

@Gauravtalreja1
Copy link
Collaborator

PRT looks due to an intermittent issue, so I think this PR is ready for review/merge

@omkarkhatavkar
Copy link

omkarkhatavkar commented Apr 9, 2024

both the time the single test tests.foreman.api.test_ansible.TestAnsibleCfgMgmt.test_positive_read_facts_with_filter[rhel8] is getting failed same reason as 500 error

  "error": {"message":"PG::ForeignKeyViolation: ERROR:  update or delete on table \"hosts\" violates foreign key constraint \"fact_values_host_id_fk\" on table \"fact_values\"\nDETAIL:  Key (id)=(6) is still referenced from table \"fact_values\".\n"}
}

@Gauravtalreja1
Copy link
Collaborator

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

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 6444
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_ansible.py::TestAnsibleCfgMgmt --external-logging
Test Result : ================= 5 passed, 301 warnings in 1508.93s (0:25:08) =================

@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 Apr 11, 2024
@Satellite-QE Satellite-QE merged commit db87cdf into 6.15.z Apr 11, 2024
19 of 21 checks passed
@Satellite-QE Satellite-QE deleted the cherry-pick-6.15.z-4ad590a12682216e02c5122bfc509448c38df624 branch April 11, 2024 10:47
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