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

Fix assertion failure due to a change in value #17675

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix assertion failure due to a change in value #17675

wants to merge 1 commit into from
+1 −1

Conversation

amolpati30
Copy link
Contributor

@amolpati30 amolpati30 commented Feb 25, 2025 •
edited
Loading

Fixing the assertion: We were getting the value as '× httpproxy', where '×' was unnecessary. So, removed it to ensure the assertion works correctly.

Dependent PR: SatelliteQE/airgun#1750

@amolpati30 amolpati30 added CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing Stream Introduced in or relating directly to Satellite Stream/Master 6.17.z labels Feb 25, 2025
@amolpati30 amolpati30 requested a review from a team as a code owner February 25, 2025 05:25
@amolpati30 amolpati30 mentioned this pull request Feb 25, 2025
Open
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_computeresource_gce.py
airgun: 1750
widgetastic.core: 270

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10300
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_computeresource_gce.py --external-logging
Test Result : ================= 72 warnings, 5 errors in 1976.46s (0:32:56) ==================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Feb 25, 2025
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_computeresource_ec2.py
airgun: 1750
widgetastic.core: 270

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10301
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_computeresource_ec2.py --external-logging
Test Result : ================== 2 failed, 50 warnings in 954.29s (0:15:54) ==================

@amolpati30 amolpati30 requested a review from shweta83 February 25, 2025 07:47
@amolpati30
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_computeresource_ec2.py
airgun: 1750

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10313
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_computeresource_ec2.py --external-logging
Test Result : ============= 1 failed, 1 passed, 34 warnings in 774.71s (0:12:54) =============

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shweta83 shweta83 Awaiting requested review from shweta83

Assignees

@amolpati30 amolpati30

Labels
6.17.z AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed 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.
2 participants
@amolpati30 @Satellite-QE