)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"982e47460c52232926a6f3ac7904bafbd25d0326","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0d3cf029_e2e47da8","updated":"2025-04-30 20:39:48.000000000","message":"I\u0027d like the release note revised since AFAICT this should be backported, and we need the detail for the stable branch.","commit_id":"99b4d565eb077d837a6d180ecf0d5e95e1b45193"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"fb73c5bd09a9d075da1dc660a8eef43fcd98134a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"927edc54_5251b9bb","updated":"2025-05-16 00:56:10.000000000","message":"recheck need current results for this item","commit_id":"42163f0c40260476350dfae4412f70a827bee3af"}],"ironic/drivers/modules/network/common.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"4e455d45156fad37b6c1eb9abadc60140f226d42","unresolved":true,"context_lines":[{"line_number":595,"context_line":"        # NOTE(TheJulia): Also ensure that we delete the vif when in"},{"line_number":596,"context_line":"        # DELETING state."},{"line_number":597,"context_line":"        if task.node.provision_state in [states.ACTIVE, states.DELETING,"},{"line_number":598,"context_line":"                                         states.AVAILABLE]:"},{"line_number":599,"context_line":"            neutron.unbind_neutron_port(vif_id, context\u003dtask.context)"},{"line_number":600,"context_line":""},{"line_number":601,"context_line":"    def get_node_network_data(self, task):"}],"source_content_type":"text/x-python","patch_set":2,"id":"e5f6adc8_ee873aaa","line":598,"updated":"2025-04-29 16:32:22.000000000","message":"I guess this makes sense to try and clear a possible conflict on the record if someone someone pre-bound a record.","commit_id":"99b4d565eb077d837a6d180ecf0d5e95e1b45193"}],"releasenotes/notes/vif-detach-in-available-acd744acde91fec9.yaml":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"982e47460c52232926a6f3ac7904bafbd25d0326","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fix an issue that mac may conflict on re-scheduling on node provision."},{"line_number":5,"context_line":"    See bug"},{"line_number":6,"context_line":"    `bug 2109300 \u003chttps://bugs.launchpad.net/ironic/+bug/2109300\u003e`_"},{"line_number":7,"context_line":"    for more details"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"4ab017ce_d204ba05","line":4,"updated":"2025-04-30 20:39:48.000000000","message":"Given this change is likely backportable, I\u0027d like us to be more specific here -- especially given Ironic has non-openstack users. I\u0027d suggest:\n\n\"Permit VIFs to be unplugged when a node is in AVAILABLE state. This fixes a race condition (I think it\u0027s a race?) when the OpenStack Nova driver reschedules to a different node.\"\n\nOr something similar. Essentially it should:\n- In Ironic-only terms talk about what we changed (allow VIFs to be unplugged while a node is AVAILABLE)\n- Be explicit about how the bug impacted OpenStack Nova-integrated installs (so that e.g. a metal3 customer might choose to not pull this patch in)","commit_id":"99b4d565eb077d837a6d180ecf0d5e95e1b45193"},{"author":{"_account_id":24828,"name":"Kaifeng Wang","email":"kaifeng.w@gmail.com","username":"wangkf"},"change_message_id":"01434b75f0b376c48a74f5c7650daa9d6acc28d1","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fix an issue that mac may conflict on re-scheduling on node provision."},{"line_number":5,"context_line":"    See bug"},{"line_number":6,"context_line":"    `bug 2109300 \u003chttps://bugs.launchpad.net/ironic/+bug/2109300\u003e`_"},{"line_number":7,"context_line":"    for more details"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"bf05d5c7_0819a345","line":4,"in_reply_to":"4ab017ce_d204ba05","updated":"2025-05-01 01:29:11.000000000","message":"I am not sure it can be called a race issue, but in short, attaching different vif to the same node during re-scheduling is likely to hit. Ironic doesn\u0027t configure network in metal3 AFAIK so there should be no impact there.","commit_id":"99b4d565eb077d837a6d180ecf0d5e95e1b45193"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"405ad5401a741932d22413cc34a5847213325278","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fix an issue that mac may conflict on re-scheduling on node provision."},{"line_number":5,"context_line":"    See bug"},{"line_number":6,"context_line":"    `bug 2109300 \u003chttps://bugs.launchpad.net/ironic/+bug/2109300\u003e`_"},{"line_number":7,"context_line":"    for more details"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5fa0b6f5_4d4182b9","line":4,"in_reply_to":"bf05d5c7_0819a345","updated":"2025-05-02 15:57:37.000000000","message":"The update looks good. I just like to flag for our standalone users if a backported change doesn\u0027t apply to them (and vice-versa).","commit_id":"99b4d565eb077d837a6d180ecf0d5e95e1b45193"}]}