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

6.14.z changes - increse timeout for read subscription #1420

Merged
merged 2 commits into from
Jun 11, 2024
Merged

6.14.z changes - increse timeout for read subscription #1420

merged 2 commits into from
Jun 11, 2024

Conversation

vijaysawant
Copy link
Contributor

Sub-task of 6.14.z Airgun PR #1412 *

Problem
PRT is failing for 6.14.z CP PR SatelliteQE/robottelo#15276 (Original 6.15.z robottelo PR SatelliteQE/robottelo#15237)

Actual issue
Subscription table is not loading within existing 10s of timeout

Solution
Increasing timeout value would help to fix this issue

Related PR
6.15.z Airgun PR #1405
6.14.z Airgun PR #1412 *

PRT Comment

trigger: test-robottelo
pytest: tests/foreman/ui/test_subscription.py -k 'test_positive_access_with_non_admin_user_with_manifest'
robottelo: 15276

@vijaysawant
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_subscription.py -k 'test_positive_access_with_non_admin_user_with_manifest'
robottelo: 15276

@Satellite-QE
Copy link
Contributor

PRT Result

Build Number: 290
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_subscription.py -k test_positive_access_with_non_admin_user_with_manifest --external-logging
Test Result : =========== 1 failed, 8 deselected, 17 warnings in 709.27s (0:11:49) ===========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jun 5, 2024
@vijaysawant vijaysawant force-pushed the 6.14.z-increase-timeout-of-read-subscription branch from bbc770e to 41d5e5f Compare June 5, 2024 11:52
@Satellite-QE
Copy link
Contributor

PRT Result

Build Number: 291
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_subscription.py -k test_positive_access_with_non_admin_user_with_manifest --external-logging
Test Result : =========== 1 failed, 8 deselected, 17 warnings in 728.62s (0:12:08) ===========

@vijaysawant vijaysawant force-pushed the 6.14.z-increase-timeout-of-read-subscription branch from 41d5e5f to 17476b6 Compare June 5, 2024 12:13
@Satellite-QE
Copy link
Contributor

PRT Result

Build Number: 292
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_subscription.py -k test_positive_access_with_non_admin_user_with_manifest --external-logging
Test Result : =========== 8 deselected, 17 warnings, 1 error in 660.35s (0:11:00) ============

@Satellite-QE
Copy link
Contributor

PRT Result

Build Number: 293
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_subscription.py -k test_positive_access_with_non_admin_user_with_manifest --external-logging
Test Result : =========== 1 failed, 8 deselected, 17 warnings in 660.66s (0:11:00) ===========

@vijaysawant vijaysawant marked this pull request as ready for review June 10, 2024 13:24
@vsedmik vsedmik added No-CherryPick PR doesnt need CherryPick to previous branches 6.14.z labels Jun 10, 2024
vsedmik
vsedmik approved these changes Jun 10, 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.

PRT passed at SatelliteQE/robottelo#15276 (comment)

@vijaysawant vijaysawant added the AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing label Jun 10, 2024
@vijaysawant
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_subscription.py -k 'test_positive_access_with_non_admin_user_with_manifest'
robottelo: 15276

@Satellite-QE
Copy link
Contributor

PRT Result

Build Number: 294
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_subscription.py -k test_positive_access_with_non_admin_user_with_manifest --external-logging
Test Result : =========== 1 failed, 8 deselected, 17 warnings in 711.79s (0:11:51) ===========

@vijaysawant
Copy link
Contributor Author 

trigger: test-robottelo
pytest: tests/foreman/ui/test_subscription.py -k 'test_positive_access_with_non_admin_user_with_manifest'
robottelo: 15276
env:
    ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.14.0'

@Satellite-QE
Copy link
Contributor

PRT Result

Build Number: 295
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_subscription.py -k test_positive_access_with_non_admin_user_with_manifest --external-logging
Test Result : ================= 8 deselected, 14 warnings, 1 error in 49.81s =================

@vijaysawant
Copy link
Contributor Author 

trigger: test-robottelo
pytest: tests/foreman/ui/test_subscription.py -k 'test_positive_access_with_non_admin_user_with_manifest'
robottelo: 15276
env:
    ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.14'

@Satellite-QE
Copy link
Contributor

PRT Result

Build Number: 296
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_subscription.py -k test_positive_access_with_non_admin_user_with_manifest --external-logging
Test Result : =========== 1 failed, 8 deselected, 17 warnings in 671.77s (0:11:11) ===========

@vijaysawant
Copy link
Contributor Author 

trigger: test-robottelo
pytest: tests/foreman/ui/test_subscription.py -k 'test_positive_access_with_non_admin_user_with_manifest'
robottelo: 15276
env:
    ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.14'

@omkarkhatavkar
Copy link

trigger: test-robottelo
pytest: tests/foreman/ui/test_subscription.py -k 'test_positive_access_with_non_admin_user_with_manifest'

1 similar comment
@vijaysawant
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_subscription.py -k 'test_positive_access_with_non_admin_user_with_manifest'

@Satellite-QE
Copy link
Contributor

PRT Result

Build Number: 299
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_subscription.py -k test_positive_access_with_non_admin_user_with_manifest --external-logging
Test Result : =========== 1 passed, 8 deselected, 17 warnings in 698.14s (0:11:38) ===========

@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 Jun 11, 2024
@omkarkhatavkar omkarkhatavkar merged commit 51a96a8 into SatelliteQE:6.14.z Jun 11, 2024
10 of 11 checks passed
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

@omkarkhatavkar omkarkhatavkar omkarkhatavkar approved these changes

Assignees

@vijaysawant vijaysawant

Labels
6.14.z AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are 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
Milestone
No milestone
Development

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