)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"12b36134b73929113e9b4694733f04266144e713","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b0cd9a9c_fe6a2add","updated":"2021-12-30 11:58:32.000000000","message":"recheck","commit_id":"7f2e9e8fb2f10549b2266531b9525968deff4a2a"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"50a5ffe3a09efc3b27910985d0df11ba8ebf317f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"cdb3d800_ca3cfbf2","updated":"2022-01-11 07:42:45.000000000","message":"recheck bug #1956344","commit_id":"8d8353e9f288870f070875a808b48b3bff10550d"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"da2c553225cb323e354156fab1c0523009b39fe4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9d64ff10_e43d21aa","updated":"2022-01-12 11:34:39.000000000","message":"added ability to configure static NAT for Local IP feature","commit_id":"49b64bceff4fd2d017ceff59ab919360a4b6d0ea"},{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"ddf02776e262ff72c3436dd2b7086be932bcb82d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"0d8e6ef9_47b0d5ce","updated":"2022-01-18 09:25:20.000000000","message":"recheck","commit_id":"66d50cdbd744cfacfa90905333e3b7dd9cb1ba90"},{"author":{"_account_id":32667,"name":"Mamatisa Nurmatov","email":"nurmatov.mamatisa@huawei.com","username":"isabek"},"change_message_id":"3a331bf7d26750f7f7e2098a6b1585c76dae3697","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"42e9b67f_f15c02c5","updated":"2022-01-18 13:58:34.000000000","message":"recheck neutron-tempest-plugin-scenario-openvswitch failure","commit_id":"66d50cdbd744cfacfa90905333e3b7dd9cb1ba90"}],"devstack/plugin.sh":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3b7ef2d850e8c1dd390be6ae1366860a2f090115","unresolved":true,"context_lines":[{"line_number":74,"context_line":"                    configure_ovs_distributed_dhcp"},{"line_number":75,"context_line":"                fi"},{"line_number":76,"context_line":"            fi"},{"line_number":77,"context_line":"            if is_service_enabled neutron-local-ip; then"},{"line_number":78,"context_line":"                configure_local_ip"},{"line_number":79,"context_line":"            fi"},{"line_number":80,"context_line":"            if is_service_enabled neutron-local-ip-static; then"}],"source_content_type":"text/x-sh","patch_set":8,"id":"217f91fb_24557f9d","line":77,"updated":"2022-01-12 14:41:21.000000000","message":"q-local-ip to keep q-* names ?","commit_id":"66d50cdbd744cfacfa90905333e3b7dd9cb1ba90"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"edb7c8f5158f05688ccbdbd690d9766a9d6e3d74","unresolved":true,"context_lines":[{"line_number":74,"context_line":"                    configure_ovs_distributed_dhcp"},{"line_number":75,"context_line":"                fi"},{"line_number":76,"context_line":"            fi"},{"line_number":77,"context_line":"            if is_service_enabled neutron-local-ip; then"},{"line_number":78,"context_line":"                configure_local_ip"},{"line_number":79,"context_line":"            fi"},{"line_number":80,"context_line":"            if is_service_enabled neutron-local-ip-static; then"}],"source_content_type":"text/x-sh","patch_set":8,"id":"13cb4855_1d9ee1fb","line":77,"in_reply_to":"217f91fb_24557f9d","updated":"2022-01-12 15:09:06.000000000","message":"But why need to keep this diversity? q- was for \"quantum\" long time ago)","commit_id":"66d50cdbd744cfacfa90905333e3b7dd9cb1ba90"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3b7ef2d850e8c1dd390be6ae1366860a2f090115","unresolved":true,"context_lines":[{"line_number":77,"context_line":"            if is_service_enabled neutron-local-ip; then"},{"line_number":78,"context_line":"                configure_local_ip"},{"line_number":79,"context_line":"            fi"},{"line_number":80,"context_line":"            if is_service_enabled neutron-local-ip-static; then"},{"line_number":81,"context_line":"                configure_local_ip_static"},{"line_number":82,"context_line":"            fi"},{"line_number":83,"context_line":"            if is_service_enabled q-agt neutron-agent; then"}],"source_content_type":"text/x-sh","patch_set":8,"id":"c6ccb6d5_973ccb14","line":80,"range":{"start_line":80,"start_character":34,"end_line":80,"end_character":41},"updated":"2022-01-12 14:41:21.000000000","message":"ditto","commit_id":"66d50cdbd744cfacfa90905333e3b7dd9cb1ba90"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"f7f6ac8e305006ffe14c5b3094d13873b3493280","unresolved":true,"context_lines":[{"line_number":77,"context_line":"            if is_service_enabled neutron-local-ip; then"},{"line_number":78,"context_line":"                configure_local_ip"},{"line_number":79,"context_line":"            fi"},{"line_number":80,"context_line":"            if is_service_enabled neutron-local-ip-static; then"},{"line_number":81,"context_line":"                configure_local_ip_static"},{"line_number":82,"context_line":"            fi"},{"line_number":83,"context_line":"            if is_service_enabled q-agt neutron-agent; then"}],"source_content_type":"text/x-sh","patch_set":8,"id":"c0d3db66_e6d7210f","line":80,"range":{"start_line":80,"start_character":34,"end_line":80,"end_character":41},"in_reply_to":"61225651_316f0249","updated":"2022-01-17 08:27:32.000000000","message":"So my point here is: adding support for \"q-local-ip\" now is simple, but removing it later is not that simple as will have to deal with bw compatibility. It\u0027s a new feature and devstack testing seems ok without \"q-\" - what if we postpone \"q-\" addition once we clearly see where/why it\u0027s needed for the Local IP feature?","commit_id":"66d50cdbd744cfacfa90905333e3b7dd9cb1ba90"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"0e8a2f0f1fed33826b30ac6201dd2d1d84fa0ea2","unresolved":true,"context_lines":[{"line_number":77,"context_line":"            if is_service_enabled neutron-local-ip; then"},{"line_number":78,"context_line":"                configure_local_ip"},{"line_number":79,"context_line":"            fi"},{"line_number":80,"context_line":"            if is_service_enabled neutron-local-ip-static; then"},{"line_number":81,"context_line":"                configure_local_ip_static"},{"line_number":82,"context_line":"            fi"},{"line_number":83,"context_line":"            if is_service_enabled q-agt neutron-agent; then"}],"source_content_type":"text/x-sh","patch_set":8,"id":"61225651_316f0249","line":80,"range":{"start_line":80,"start_character":34,"end_line":80,"end_character":41},"in_reply_to":"72a9210f_23b02604","updated":"2022-01-14 11:07:15.000000000","message":"Well, in scenario tests it works fine without \"q-\" [1]\n\n[1] https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/823007","commit_id":"66d50cdbd744cfacfa90905333e3b7dd9cb1ba90"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"9f926d04d0492d2bcad7683618ace2c208b38d0b","unresolved":true,"context_lines":[{"line_number":77,"context_line":"            if is_service_enabled neutron-local-ip; then"},{"line_number":78,"context_line":"                configure_local_ip"},{"line_number":79,"context_line":"            fi"},{"line_number":80,"context_line":"            if is_service_enabled neutron-local-ip-static; then"},{"line_number":81,"context_line":"                configure_local_ip_static"},{"line_number":82,"context_line":"            fi"},{"line_number":83,"context_line":"            if is_service_enabled q-agt neutron-agent; then"}],"source_content_type":"text/x-sh","patch_set":8,"id":"72a9210f_23b02604","line":80,"range":{"start_line":80,"start_character":34,"end_line":80,"end_character":41},"in_reply_to":"923c6e58_2d459d3b","updated":"2022-01-13 12:54:10.000000000","message":"As I know devsatck decides based on q-* service names to use functions from devstack/lib/neutron-legacy, so now q- namesare preferred.\nI started the \"deprecation process\" (there is no such) in devstack and it can be an iteresting discussion what kind on service names we should use in devstack/CI....","commit_id":"66d50cdbd744cfacfa90905333e3b7dd9cb1ba90"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"02b28867edf40ab85149ef2c3925809839475b0f","unresolved":true,"context_lines":[{"line_number":77,"context_line":"            if is_service_enabled neutron-local-ip; then"},{"line_number":78,"context_line":"                configure_local_ip"},{"line_number":79,"context_line":"            fi"},{"line_number":80,"context_line":"            if is_service_enabled neutron-local-ip-static; then"},{"line_number":81,"context_line":"                configure_local_ip_static"},{"line_number":82,"context_line":"            fi"},{"line_number":83,"context_line":"            if is_service_enabled q-agt neutron-agent; then"}],"source_content_type":"text/x-sh","patch_set":8,"id":"43e7548b_3336fc15","line":80,"range":{"start_line":80,"start_character":34,"end_line":80,"end_character":41},"in_reply_to":"c0d3db66_e6d7210f","updated":"2022-01-17 13:35:02.000000000","message":"ok","commit_id":"66d50cdbd744cfacfa90905333e3b7dd9cb1ba90"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"8aa3cd415c1983a77c48d4e506fdc42293a55f25","unresolved":true,"context_lines":[{"line_number":77,"context_line":"            if is_service_enabled neutron-local-ip; then"},{"line_number":78,"context_line":"                configure_local_ip"},{"line_number":79,"context_line":"            fi"},{"line_number":80,"context_line":"            if is_service_enabled neutron-local-ip-static; then"},{"line_number":81,"context_line":"                configure_local_ip_static"},{"line_number":82,"context_line":"            fi"},{"line_number":83,"context_line":"            if is_service_enabled q-agt neutron-agent; then"}],"source_content_type":"text/x-sh","patch_set":8,"id":"923c6e58_2d459d3b","line":80,"range":{"start_line":80,"start_character":34,"end_line":80,"end_character":41},"in_reply_to":"c6ccb6d5_973ccb14","updated":"2022-01-12 15:10:47.000000000","message":"AFAIR \"q-\"s were left for backward compatibility. Not sure we need it for new features","commit_id":"66d50cdbd744cfacfa90905333e3b7dd9cb1ba90"}]}
