)]}'
{"/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":"e71158e5457ea580855b203b05d05b7556160857","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"b4b1dc61_7f3e5f3a","updated":"2025-05-26 19:57:45.000000000","message":"So in the Ironic patch, the non-multinode passed: ironic-tempest-ovn-uefi-ipxe-ipv6 https://zuul.opendev.org/t/openstack/build/7c5aeb536d974aa7a0ff98866be52d6f : SUCCESS in 1h 01m 42s (non-voting)","commit_id":"19f7e72908c5b185e9076a803d4e4f862f8909da"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f0603972bd8a1f6a73fb9fe68d183f2060b723f7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"9e71dc15_76bbd29a","updated":"2025-05-27 13:51:19.000000000","message":"recheck additional data point required before merging","commit_id":"df74ecea5282367762c7c07923942b68750c1bd4"}],"devstack/plugin.sh":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"5fedf2b3ca9ab55f6ea5d234417abd996b82c7dd","unresolved":true,"context_lines":[{"line_number":168,"context_line":"    # neutron as it auto-identifies it\u0027s configuration files."},{"line_number":169,"context_line":"    neutron_server_config_add $GENERIC_SWITCH_INI_FILE"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    if [ -f /etc/neutron/neutron-api-uwsgi.ini ]; then"},{"line_number":172,"context_line":"        iniset -sudo /etc/neutron/neutron-api-uwsgi.ini uwsgi env OS_NEUTRON_CONFIG_FILES\u003d\u0027/etc/neutron/neutron.conf;/etc/neutron/plugins/ml2/ml2_conf.ini;/etc/neutron/plugins/ml2/ml2_conf_genericswitch.ini\u0027"},{"line_number":173,"context_line":"    fi"},{"line_number":174,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":16,"id":"9794bcda_6dcaef56","line":171,"updated":"2025-05-27 13:54:41.000000000","message":"So, there may be a race here with plugin execution, although we\u0027re likely hidden from it entirely. Specifically if this execution somehow gets here first. I think we might want to remove the file check entirely... But that is not blocking merging this as-is.","commit_id":"df74ecea5282367762c7c07923942b68750c1bd4"},{"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":"6d242e3befdad6f64f2c0785b97005709ec99b79","unresolved":true,"context_lines":[{"line_number":168,"context_line":"    # neutron as it auto-identifies it\u0027s configuration files."},{"line_number":169,"context_line":"    neutron_server_config_add $GENERIC_SWITCH_INI_FILE"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    if [ -f /etc/neutron/neutron-api-uwsgi.ini ]; then"},{"line_number":172,"context_line":"        iniset -sudo /etc/neutron/neutron-api-uwsgi.ini uwsgi env OS_NEUTRON_CONFIG_FILES\u003d\u0027/etc/neutron/neutron.conf;/etc/neutron/plugins/ml2/ml2_conf.ini;/etc/neutron/plugins/ml2/ml2_conf_genericswitch.ini\u0027"},{"line_number":173,"context_line":"    fi"},{"line_number":174,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":16,"id":"749ab148_524b731c","line":171,"in_reply_to":"4641b90f_ed31b3e1","updated":"2025-05-27 20:47:51.000000000","message":"https://review.opendev.org/c/openstack/networking-generic-switch/+/951027","commit_id":"df74ecea5282367762c7c07923942b68750c1bd4"},{"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":"35f0133baca9ad208ab747be6db8dbcce931bc06","unresolved":true,"context_lines":[{"line_number":168,"context_line":"    # neutron as it auto-identifies it\u0027s configuration files."},{"line_number":169,"context_line":"    neutron_server_config_add $GENERIC_SWITCH_INI_FILE"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":"    if [ -f /etc/neutron/neutron-api-uwsgi.ini ]; then"},{"line_number":172,"context_line":"        iniset -sudo /etc/neutron/neutron-api-uwsgi.ini uwsgi env OS_NEUTRON_CONFIG_FILES\u003d\u0027/etc/neutron/neutron.conf;/etc/neutron/plugins/ml2/ml2_conf.ini;/etc/neutron/plugins/ml2/ml2_conf_genericswitch.ini\u0027"},{"line_number":173,"context_line":"    fi"},{"line_number":174,"context_line":"}"}],"source_content_type":"text/x-sh","patch_set":16,"id":"4641b90f_ed31b3e1","line":171,"in_reply_to":"9794bcda_6dcaef56","updated":"2025-05-27 15:49:00.000000000","message":"I am going to do this single change as a followup once this lands; I\u0027m slightly concerned about the behavior of iniset if the file doesn\u0027t already exist, and I don\u0027t trust upstream devstack to use iniset instead of writing it out, so I\u0027m going to validate that in a separate patch so we can get the CI fix in ASAP.","commit_id":"df74ecea5282367762c7c07923942b68750c1bd4"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"5fedf2b3ca9ab55f6ea5d234417abd996b82c7dd","unresolved":true,"context_lines":[{"line_number":298,"context_line":"            # do too much. We likely need to implement our own less-disruptive restart if"},{"line_number":299,"context_line":"            # possible."},{"line_number":300,"context_line":"            echo_summary \"Skipping ngs_restart_neutron\""},{"line_number":301,"context_line":"            #ngs_restart_neutron"},{"line_number":302,"context_line":"        fi"},{"line_number":303,"context_line":""},{"line_number":304,"context_line":"    elif [[ \"$1\" \u003d\u003d \"stack\" \u0026\u0026 \"$2\" \u003d\u003d \"test-config\" ]]; then"}],"source_content_type":"text/x-sh","patch_set":16,"id":"d23b3b8c_71f31c6a","line":301,"updated":"2025-05-27 13:54:41.000000000","message":"So, no restart at all?","commit_id":"df74ecea5282367762c7c07923942b68750c1bd4"}]}