)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"aa832ed0241908d53f609c8c2fd9e361ad495d12","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4aa00cdc_08ae24d0","updated":"2026-01-08 17:52:48.000000000","message":"+1 over all but i woudl split the job api change (changing the devstack vaiable name) form the rest of this change.","commit_id":"68b0368b3fa08cffcbbd0dc327700107c5bed7f3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"37413da6c92daba87b7e60f9d34dc13839ff6278","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ee70c58c_6a2d8bb7","updated":"2026-01-08 02:36:46.000000000","message":"recheck dependent patch fixed","commit_id":"68b0368b3fa08cffcbbd0dc327700107c5bed7f3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ef3fb3c898545cd36bdd95787fe93b7a00fa5842","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dbf38fb3_4a692642","updated":"2026-02-24 16:23:04.000000000","message":"The neutron patch was just +W\u0027d, would be good to get this done as well. Let me know if it needs any updates, thanks","commit_id":"03db96dc9e8c17e284e201d2350627a2ebfd4a1a"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"152db47d764276d76f06e7cc86fadf50dbcd1843","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3bf84d0d_d21e8e44","updated":"2026-03-04 18:53:07.000000000","message":"lgtm","commit_id":"03db96dc9e8c17e284e201d2350627a2ebfd4a1a"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"975b0992ba7621d2327f464986b45e64353ae30b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d7c5e693_9b86ad69","updated":"2026-02-24 16:27:47.000000000","message":"no i think this is ok as is\nyou previously address my feedback about doing this two parts so looks good to me","commit_id":"03db96dc9e8c17e284e201d2350627a2ebfd4a1a"}],"lib/neutron":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"aa832ed0241908d53f609c8c2fd9e361ad495d12","unresolved":true,"context_lines":[{"line_number":207,"context_line":"# will be available for the allocation of provider networks."},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"# To disable tunnels (GRE or VXLAN) for project networks,"},{"line_number":210,"context_line":"# set to False in ``local.conf``."},{"line_number":211,"context_line":"# GRE tunnels are only supported by the openvswitch."},{"line_number":212,"context_line":"ENABLE_TENANT_TUNNELS\u003d${ENABLE_TENANT_TUNNELS:-True}"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"# If using GRE, VXLAN or GENEVE tunnels for project networks,"},{"line_number":215,"context_line":"# specify the range of IDs from which project networks are"},{"line_number":216,"context_line":"# allocated. Can be overridden in ``localrc`` if necessary."},{"line_number":217,"context_line":"TENANT_TUNNEL_RANGES\u003d${TENANT_TUNNEL_RANGES:-1:1000}"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"# To use VLANs for project networks, set to True in localrc. VLANs"},{"line_number":220,"context_line":"# are supported by the ML2 plugins, requiring additional configuration"},{"line_number":221,"context_line":"# described below."},{"line_number":222,"context_line":"ENABLE_TENANT_VLANS\u003d${ENABLE_TENANT_VLANS:-False}"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"# If using VLANs for project networks, set in ``localrc`` to specify"},{"line_number":225,"context_line":"# the range of VLAN VIDs from which project networks are"},{"line_number":226,"context_line":"# allocated. An external network switch must be configured to"},{"line_number":227,"context_line":"# trunk these VLANs between hosts for multi-host connectivity."},{"line_number":228,"context_line":"#"},{"line_number":229,"context_line":"# Example: ``TENANT_VLAN_RANGE\u003d1000:1999``"},{"line_number":230,"context_line":"TENANT_VLAN_RANGE\u003d${TENANT_VLAN_RANGE:-}"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"# If using VLANs for project networks, or if using flat or VLAN"},{"line_number":233,"context_line":"# provider networks, set in ``localrc`` to the name of the physical"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"43f5563f_5aaf3eb6","line":230,"range":{"start_line":210,"start_character":0,"end_line":230,"end_character":40},"updated":"2026-01-08 17:52:48.000000000","message":"im not sure if it makes sense to change\nQ_ML2_TENANT_NETWORK_TYPE to Q_ML2_PROJECT_NETWORK_TYPE\n\nif we are going to continut to use the term TENANT for these.","commit_id":"68b0368b3fa08cffcbbd0dc327700107c5bed7f3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"573c18573394f78c126fb34cf135fa7b033ad725","unresolved":false,"context_lines":[{"line_number":207,"context_line":"# will be available for the allocation of provider networks."},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"# To disable tunnels (GRE or VXLAN) for project networks,"},{"line_number":210,"context_line":"# set to False in ``local.conf``."},{"line_number":211,"context_line":"# GRE tunnels are only supported by the openvswitch."},{"line_number":212,"context_line":"ENABLE_TENANT_TUNNELS\u003d${ENABLE_TENANT_TUNNELS:-True}"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"# If using GRE, VXLAN or GENEVE tunnels for project networks,"},{"line_number":215,"context_line":"# specify the range of IDs from which project networks are"},{"line_number":216,"context_line":"# allocated. Can be overridden in ``localrc`` if necessary."},{"line_number":217,"context_line":"TENANT_TUNNEL_RANGES\u003d${TENANT_TUNNEL_RANGES:-1:1000}"},{"line_number":218,"context_line":""},{"line_number":219,"context_line":"# To use VLANs for project networks, set to True in localrc. VLANs"},{"line_number":220,"context_line":"# are supported by the ML2 plugins, requiring additional configuration"},{"line_number":221,"context_line":"# described below."},{"line_number":222,"context_line":"ENABLE_TENANT_VLANS\u003d${ENABLE_TENANT_VLANS:-False}"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"# If using VLANs for project networks, set in ``localrc`` to specify"},{"line_number":225,"context_line":"# the range of VLAN VIDs from which project networks are"},{"line_number":226,"context_line":"# allocated. An external network switch must be configured to"},{"line_number":227,"context_line":"# trunk these VLANs between hosts for multi-host connectivity."},{"line_number":228,"context_line":"#"},{"line_number":229,"context_line":"# Example: ``TENANT_VLAN_RANGE\u003d1000:1999``"},{"line_number":230,"context_line":"TENANT_VLAN_RANGE\u003d${TENANT_VLAN_RANGE:-}"},{"line_number":231,"context_line":""},{"line_number":232,"context_line":"# If using VLANs for project networks, or if using flat or VLAN"},{"line_number":233,"context_line":"# provider networks, set in ``localrc`` to the name of the physical"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"e3ffb5a7_b4d0d098","line":230,"range":{"start_line":210,"start_character":0,"end_line":230,"end_character":40},"in_reply_to":"43f5563f_5aaf3eb6","updated":"2026-01-08 20:21:38.000000000","message":"Done","commit_id":"68b0368b3fa08cffcbbd0dc327700107c5bed7f3"}],"lib/neutron_plugins/openvswitch":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"aa832ed0241908d53f609c8c2fd9e361ad495d12","unresolved":true,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"function neutron_plugin_configure_service {"},{"line_number":22,"context_line":"    if [[ \"$ENABLE_TENANT_TUNNELS\" \u003d\u003d \"True\" ]]; then"},{"line_number":23,"context_line":"        iniset /$Q_PLUGIN_CONF_FILE ovs project_network_type gre"},{"line_number":24,"context_line":"        iniset /$Q_PLUGIN_CONF_FILE ovs tunnel_id_ranges $TENANT_TUNNEL_RANGES"},{"line_number":25,"context_line":"    elif [[ \"$ENABLE_TENANT_VLANS\" \u003d\u003d \"True\" ]]; then"},{"line_number":26,"context_line":"        iniset /$Q_PLUGIN_CONF_FILE ovs project_network_type vlan"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"4216e447_66bfdae3","line":23,"range":{"start_line":23,"start_character":40,"end_line":23,"end_character":60},"updated":"2026-01-08 17:52:48.000000000","message":"updating this i think is fine but i woudl keep \n\nQ_ML2_TENANT_NETWORK_TYPE until we are ready to update all the other devstack vars to use PROJECT instead of TENANT\n\nso +1 to using the non deprecated neturon value in config  and -1 to channg the Q_ML2_TENANT_NETWORK_TYPE to Q_ML2_PROJECT_NETWORK_TYPE while all the rest of the flags are using PROJECT.","commit_id":"68b0368b3fa08cffcbbd0dc327700107c5bed7f3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"573c18573394f78c126fb34cf135fa7b033ad725","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"function neutron_plugin_configure_service {"},{"line_number":22,"context_line":"    if [[ \"$ENABLE_TENANT_TUNNELS\" \u003d\u003d \"True\" ]]; then"},{"line_number":23,"context_line":"        iniset /$Q_PLUGIN_CONF_FILE ovs project_network_type gre"},{"line_number":24,"context_line":"        iniset /$Q_PLUGIN_CONF_FILE ovs tunnel_id_ranges $TENANT_TUNNEL_RANGES"},{"line_number":25,"context_line":"    elif [[ \"$ENABLE_TENANT_VLANS\" \u003d\u003d \"True\" ]]; then"},{"line_number":26,"context_line":"        iniset /$Q_PLUGIN_CONF_FILE ovs project_network_type vlan"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"d24076a2_b0918f07","line":23,"range":{"start_line":23,"start_character":40,"end_line":23,"end_character":60},"in_reply_to":"4216e447_66bfdae3","updated":"2026-01-08 20:21:38.000000000","message":"Yeah, I\u0027ll update to leave the variable alone for now.","commit_id":"68b0368b3fa08cffcbbd0dc327700107c5bed7f3"}]}