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.16.z] Fix test interference #16315

Merged

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #16267

Problem Statement

The test_negative_multi_cv_registration uses module_activation_key which uses module_sca_manifest_org which uploads a manifest to the module_org (which we don't actually need for the test case).
And there is another test in this module, test_syspurpose_end_to_end, which uses default_subscription fixture which uploads a manifest to the module_org too and here is where the interference happens since the second manifest upload fails with

['Owner has already imported from another subscription management application. The following conflicts were found: [ DISTRIBUTOR_CONFLICT ]']

Solution

Use module_ak_with_cv instead of module_activation_key - it does not upload any manifest and thus can't interfere with the other test.

PRT test Cases example

trigger: test-robottelo
pytest: tests/foreman/cli/test_host.py -k 'test_syspurpose_end_to_end or test_negative_multi_cv_registration'

* Fix test interference

* No containers to check PRT

(cherry picked from commit 4df4c11)
@Satellite-QE Satellite-QE requested a review from a team as a code owner September 11, 2024 06:47
@Satellite-QE Satellite-QE added 6.16.z Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Sep 11, 2024
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_host.py -k 'test_syspurpose_end_to_end or test_negative_multi_cv_registration'

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 8570
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_host.py -k test_syspurpose_end_to_end or test_negative_multi_cv_registration --external-logging
Test Result : = 1 failed, 2 passed, 93 deselected, 264 warnings, 1 error in 1208.40s (0:20:08) =

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Sep 11, 2024
@vsedmik
Copy link
Contributor

vsedmik commented Sep 11, 2024

Same PRT result, as expected #16267 (comment)

@vsedmik vsedmik merged commit c7e9f8c into 6.16.z Sep 11, 2024
15 of 16 checks passed
@vsedmik vsedmik deleted the cherry-pick-6.16.z-4df4c11603fc94c0ea9ff9eb3850184e19c2b35f branch September 11, 2024 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.16.z Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants