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.17.z] Fix configure_kea_dhcp6_server fixture for IPv4 #17662

Merged
merged 1 commit into from
Feb 24, 2025
Merged

[6.17.z] Fix configure_kea_dhcp6_server fixture for IPv4 #17662

merged 1 commit into from
Feb 24, 2025

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #17656

Problem Statement

When setup configure_kea_dhcp6_server fixture is run for IPv4, it fails with "failed on setup with "ValueError: configure_kea_dhcp6_server did not yield a value"

Solution

Fix configure_kea_dhcp6_server fixture for IPv4 by yielding None value since this fixture intended for IPv6 only

Related Issues

@Gauravtalreja1 @web-flow
Fix configure_kea_dhcp6_server fixture for IPv4 (#17656)
7bd5403 
Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit b1ed2ac)
@Satellite-QE Satellite-QE requested a review from a team as a code owner February 24, 2025 10:10
@Satellite-QE Satellite-QE added 6.17.z Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Feb 24, 2025
@Satellite-QE
Copy link
Collaborator Author 

trigger: test-robottelo
pytest: tests/foreman/api/test_provisioning.py::test_rhel_pxe_provisioning
provisioning: true
env:
     BROKER_SSH__BACKEND: hussh

@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Feb 24, 2025
shweta83
shweta83 reviewed Feb 24, 2025
View reviewed changes
Copy link
Contributor

@shweta83 shweta83 left a comment

Choose a reason for hiding this comment

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

ack.

@shweta83 shweta83 merged commit 5c1b2db into 6.17.z Feb 24, 2025
15 of 17 checks passed
@shweta83 shweta83 deleted the cherry-pick-6.17.z-b1ed2ac0ea58de0b64dcc652fec2b6218ff739ea branch February 24, 2025 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shweta83 shweta83 shweta83 left review comments

Assignees

@Gauravtalreja1 Gauravtalreja1

Labels
6.17.z 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
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Satellite-QE @shweta83 @Gauravtalreja1