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

Migrate Old CV UI coverage (Upgrade) #16283

Merged
merged 1 commit into from
Oct 24, 2024
Merged

Migrate Old CV UI coverage (Upgrade) #16283

merged 1 commit into from
Oct 24, 2024

Conversation

damoore044
Copy link
Contributor

@damoore044 damoore044 commented Sep 9, 2024 •
edited
Loading

Purpose

  • Migrate upgrade case from test_contentview_old.py (not collected during test sessions)
  • Promoted CVV in multiple environments, using UI to Delete the CVV and/or CV, removes it from the environments.
  • Needs Airgun 1592)

PRT Case (for Upgrade)

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env
airgun: 1592
env:
    ROBOTTELO_server__deploy_workflow: 'deploy-satellite-upgrade'
    ROBOTTELO_server__deploy_arguments__deploy_rhel_version: '9'
    ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.16'
    ROBOTTELO_server__deploy_arguments__deploy_snap_version: 'ystream'
    BROKER_AnsibleTower__inventory: 'osp-rhos01-satellite-upgrade'

@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview_old.py::test_positive_delete_cv_promoted_to_multi_env

@damoore044 damoore044 added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Sep 9, 2024
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview_old.py::test_positive_delete_cv_promoted_to_multi_env
env:
ROBOTTELO_server__deploy_workflow: 'deploy-satellite-upgrade'
ROBOTTELO_server__deploy_arguments__deploy_rhel_version: '9'
ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.16'
ROBOTTELO_server__deploy_arguments__deploy_snap_version: 'ystream'
BROKER_AnsibleTower__inventory: 'osp-rhos01-satellite-upgrade'

@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview_old.py -k test_positive_delete_cv_promoted_to_multi_env
env:
ROBOTTELO_server__deploy_workflow: 'deploy-satellite-upgrade'
ROBOTTELO_server__deploy_arguments__deploy_rhel_version: '9'
ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.16'
ROBOTTELO_server__deploy_arguments__deploy_snap_version: 'ystream'
BROKER_AnsibleTower__inventory: 'osp-rhos01-satellite-upgrade'

@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env

@damoore044 damoore044 changed the title Troubleshooting Old CV UI case (Upgrade) Migrate Old CV UI coverage (Upgrade) Oct 2, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8849
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env --external-logging
Test Result : ================== 1 failed, 10 warnings in 608.02s (0:10:08) ==================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Oct 2, 2024
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env
airgun: 939

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8850
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env --external-logging
Test Result : ================== 1 failed, 11 warnings in 681.52s (0:11:21) ==================

@damoore044 damoore044 mentioned this pull request Oct 9, 2024
Merged
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env
airgun: 1592

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8926
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env --external-logging
Test Result : ================== 1 failed, 11 warnings in 678.11s (0:11:18) ==================

@damoore044 damoore044 force-pushed the temp-master-PRT branch 2 times, most recently from a863a7b to 825362d Compare October 9, 2024 19:00
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env
airgun: 1592

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8934
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env --external-logging
Test Result : ================= 1 failed, 20 warnings in 2428.34s (0:40:28) ==================

@damoore044 damoore044 mentioned this pull request Oct 15, 2024
Merged
@damoore044 damoore044 requested a review from a team October 16, 2024 17:01
@damoore044 damoore044 self-assigned this Oct 16, 2024
@damoore044 damoore044 added the 6.16.z Introduced in or relating directly to Satellite 6.16 label Oct 16, 2024
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env
airgun: 1592

@damoore044 damoore044 requested review from a team and removed request for a team October 17, 2024 12:57
@Satellite-QE
Copy link
Collaborator

Satellite-QE commented Oct 17, 2024 •
edited by damoore044
Loading

PRT Result

Build Number: 8991
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env --external-logging
Test Result : ================= 1 failed, 15 warnings in 1100.11s (0:18:20) ==================

^ cancelled this build

@damoore044 damoore044 force-pushed the temp-master-PRT branch 2 times, most recently from 494b470 to 9d6a520 Compare October 17, 2024 13:50
@damoore044 damoore044 marked this pull request as ready for review October 23, 2024 13:34
@damoore044 damoore044 requested a review from a team as a code owner October 23, 2024 13:34
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9114
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env --external-logging
Test Result : ================== 1 failed, 26 warnings in 854.22s (0:14:14) ==================

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9117
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env --external-logging
Test Result : ================== 1 passed, 32 warnings in 932.85s (0:15:32) ==================

@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 23, 2024
@damoore044
Copy link
Contributor Author 

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env
airgun: 1592
env:
    ROBOTTELO_server__deploy_workflow: 'deploy-satellite-upgrade'
    ROBOTTELO_server__deploy_arguments__deploy_rhel_version: '9'
    ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.16'
    ROBOTTELO_server__deploy_arguments__deploy_snap_version: 'ystream'
    BROKER_AnsibleTower__inventory: 'osp-rhos01-satellite-upgrade'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9119
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env --external-logging
Test Result : ================== 32 warnings, 1 error in 4760.18s (1:19:20) ==================

@Satellite-QE Satellite-QE added PRT-Failed Indicates that latest PRT run is failed for the PR and removed PRT-Passed Indicates that latest PRT run is passed for the PR labels Oct 23, 2024
@damoore044
Copy link
Contributor Author

pre-commit.ci autofix

@Satellite-QE
Copy link
Collaborator

Satellite-QE commented Oct 23, 2024 •
edited by damoore044
Loading

PRT Result

Build Number: 9121
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env --external-logging
Test Result : ================= 1 passed, 32 warnings in 1009.52s (0:16:49) ==================

^ Non-Upgrade run

@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 23, 2024
@Satellite-QE Satellite-QE mentioned this pull request Oct 23, 2024
Merged
vsedmik
vsedmik approved these changes Oct 24, 2024
View reviewed changes
Copy link
Contributor

@vsedmik vsedmik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me so giving ACK. One minor comment pending.

tests/foreman/ui/test_contentview.py Outdated Show resolved Hide resolved
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Oct 24, 2024
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env
airgun: 1592

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9143
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_contentview.py::test_positive_delete_cv_promoted_to_multi_env --external-logging
Test Result : ================= 1 passed, 32 warnings in 2935.41s (0:48:55) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Oct 24, 2024
@sambible sambible added CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing and removed No-CherryPick PR doesnt need CherryPick to previous branches labels Oct 24, 2024
sambible
sambible approved these changes Oct 24, 2024
View reviewed changes
Copy link
Contributor

@sambible sambible left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changed some labels, looks good

@sambible sambible merged commit 8a2c367 into SatelliteQE:master Oct 24, 2024
14 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 24, 2024
@damoore044 @web-flow
Migrate Old CV UI coverage (Upgrade) (#16283)
67ee4e2 
CV_old upgrade case brought into new UI

(cherry picked from commit 8a2c367)
@Satellite-QE Satellite-QE mentioned this pull request Oct 24, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vsedmik vsedmik vsedmik approved these changes

@sambible sambible sambible approved these changes

Assignees

@damoore044 damoore044

Labels
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 Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@damoore044 @Satellite-QE @sambible @vsedmik