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] delete subscription manifest so it can re imported again #17151

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #17142

Problem Statement

  • test_sca_end_to_end and test_positive_expired_SCA_cert_handling tests are failing imported manifest issue.
  • Error: This subscription management application has already been imported by another owner
  • I guess test_positive_create test uses mdoule_manifest_org fixture and it is creating issue while importing manifest

Solution

Delete test_positive_create test, as it uses module_sca_manifest fixture and which is used inside any *_sca_manifest_org fixture that executed on so many places.

Local test execution

============================= test session starts ==============================
collecting ... 2024-12-10 22:29:23 - robottelo.collection - INFO - Processing test items to add testimony token markers
collected 11 items / 8 deselected / 3 selected

tests/foreman/api/test_subscription.py::test_positive_create 
tests/foreman/api/test_subscription.py::test_sca_end_to_end[rhel7-ipv4] 
tests/foreman/api/test_subscription.py::test_positive_expired_SCA_cert_handling[rhel7-ipv4] 

=========== 3 passed, 8 deselected, 58 warnings in 697.74s (0:11:37) ===========

PRT test Cases example

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

* delete subscription manifest so it can re imported again

* removing test case as discussed in review

(cherry picked from commit f1ca32b)
@Satellite-QE Satellite-QE requested a review from a team as a code owner December 11, 2024 14:47
@Satellite-QE Satellite-QE added 6.16.z Introduced in or relating directly to Satellite 6.16 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Dec 11, 2024
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/api/test_subscription.py -k 'test_sca_end_to_end or test_positive_expired_SCA_cert_handling'

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

PRT Result

Build Number: 9590
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_subscription.py -k test_sca_end_to_end or test_positive_expired_SCA_cert_handling --external-logging
Test Result : =========== 2 passed, 8 deselected, 54 warnings in 747.44s (0:12:27) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Dec 11, 2024
@Satellite-QE Satellite-QE merged commit c585a07 into 6.16.z Dec 11, 2024
17 of 18 checks passed
@Satellite-QE Satellite-QE deleted the cherry-pick-6.16.z-f1ca32be4cbc80b01753e71f3acc9efe08fe2284 branch December 11, 2024 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.16.z Introduced in or relating directly to Satellite 6.16 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.

2 participants