Skip to content

Conversation

@cardoe
Copy link
Contributor

@cardoe cardoe commented Nov 18, 2025

If we are able to read the physical_network from the port binding information then we should do that over looking it up.

Uses: https://review.opendev.org/c/openstack/ironic/+/964570

@cardoe cardoe requested a review from a team November 18, 2025 20:01
@cardoe cardoe force-pushed the update-port-binding branch from 9c63592 to a2eca74 Compare November 18, 2025 21:36
@cardoe cardoe enabled auto-merge November 18, 2025 22:31
@cardoe cardoe force-pushed the update-port-binding branch 3 times, most recently from ddacb45 to 0c0fd25 Compare November 24, 2025 22:08
If we are able to read the physical_network from the port binding
information then we should do that over looking it up.

Uses: https://review.opendev.org/c/openstack/ironic/+/964570
@cardoe cardoe force-pushed the update-port-binding branch from 0c0fd25 to 27ed1fa Compare November 25, 2025 18:32
Copy link
Contributor

@stevekeay stevekeay left a comment

Choose a reason for hiding this comment

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

seems like a good idea

@cardoe cardoe added this pull request to the merge queue Nov 25, 2025
Merged via the queue into main with commit ff6cff4 Nov 25, 2025
38 checks passed
@cardoe cardoe deleted the update-port-binding branch November 25, 2025 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants