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

Fix test_negative_ipv6_update_check #17549

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

jameerpathan111
Copy link
Contributor

Problem Statement

  • test_negative_ipv6_update_check is failing because of change in assertion message.

Solution

  • Update assertion

Related Issues

  • SAT-30972
  • SAT-30154

@jameerpathan111 jameerpathan111 added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Feb 7, 2025
@jameerpathan111 jameerpathan111 self-assigned this Feb 7, 2025
@jameerpathan111 jameerpathan111 requested a review from a team as a code owner February 7, 2025 09:15
@jameerpathan111
Copy link
Contributor Author

jameerpathan111 commented Feb 7, 2025

trigger: test-robottelo
pytest: tests/foreman/destructive/test_fm_upgrade.py -k test_negative_ipv6_update_check

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10121
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/destructive/test_fm_upgrade.py -k test_negative_ipv6_update_check --external-logging
Test Result : ================= 2 passed, 242 warnings in 3212.21s (0:53:32) =================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 7, 2025
@Gauravtalreja1 Gauravtalreja1 merged commit 5c17f27 into SatelliteQE:master Feb 7, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No-CherryPick PR doesnt need 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
Development

Successfully merging this pull request may close these issues.

4 participants