)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"be74f7e2d0b0a19859ebd580074ab4dc417342c9","unresolved":true,"context_lines":[{"line_number":57,"context_line":"Conflicts:"},{"line_number":58,"context_line":"      zuul.d/tempest-multinode.yaml"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Change-Id: Iee323943ac66e566e5a5e92de1861832e86fc7fc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"b80f6c2a_9e76f007","line":60,"updated":"2021-06-11 08:06:42.000000000","message":"You missed \"cherry-picked from ...\" line here","commit_id":"9d424ca27155874b70a86f5ea8f745d764987424"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"1cbbf461ccf7e5c03d09b3bf989f843fec0f320e","unresolved":true,"context_lines":[{"line_number":57,"context_line":"Conflicts:"},{"line_number":58,"context_line":"      zuul.d/tempest-multinode.yaml"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"Change-Id: Iee323943ac66e566e5a5e92de1861832e86fc7fc"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ea60d8dd_53ccbdee","line":60,"in_reply_to":"b80f6c2a_9e76f007","updated":"2021-06-11 15:16:25.000000000","message":"Right! I forgot the -x in the cherry pick","commit_id":"9d424ca27155874b70a86f5ea8f745d764987424"}],"neutron/conf/common.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"63dd09f2e0277cc704366bc292e0817eba3c87f5","unresolved":true,"context_lines":[{"line_number":182,"context_line":"                       \u0027other agent (like DHCP) to send to Nova this event \u0027"},{"line_number":183,"context_line":"                       \u0027when the port is provisioned.\u0027"},{"line_number":184,"context_line":"                       \u0027This option can be enabled if Nova patch \u0027"},{"line_number":185,"context_line":"                       \u0027https://review.opendev.org/c/openstack/nova/+/767368 \u0027"},{"line_number":186,"context_line":"                       \u0027is in place.\u0027"},{"line_number":187,"context_line":"                       \u0027This option is temporary and will be removed in Y and \u0027"},{"line_number":188,"context_line":"                       \u0027the behavior will be \"True\".\u0027),"}],"source_content_type":"text/x-python","patch_set":1,"id":"1c2dfc5a_cbc80e80","line":185,"updated":"2021-05-12 19:53:43.000000000","message":"I see now that this patch is already merged in stable/ussuri in Nova","commit_id":"476d90f9f6c5a7736873e123d05e0233d73d8750"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"3152c4c793788f443e92bdd40e27e8ecd1ef8ce8","unresolved":true,"context_lines":[{"line_number":174,"context_line":"               help\u003d_(\u0027Type of the nova endpoint to use.  This endpoint will\u0027"},{"line_number":175,"context_line":"                      \u0027 be looked up in the keystone catalog and should be\u0027"},{"line_number":176,"context_line":"                      \u0027 one of public, internal or admin.\u0027)),"},{"line_number":177,"context_line":"    cfg.BoolOpt(\u0027live_migration_events\u0027, default\u003dFalse,"},{"line_number":178,"context_line":"                help\u003d_(\u0027When this option is enabled, during the live \u0027"},{"line_number":179,"context_line":"                       \u0027migration, the OVS agent will only send the \u0027"},{"line_number":180,"context_line":"                       \u0027\"vif-plugged-event\" when the destination host \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"beadb144_0e0cbf56","line":177,"range":{"start_line":177,"start_character":17,"end_line":177,"end_character":38},"updated":"2021-06-10 08:53:29.000000000","message":"New config option but OK for backport as default is old behaviour (and mentioned nova patch is available in victoria branch)","commit_id":"9d424ca27155874b70a86f5ea8f745d764987424"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b56a141a094df9fc0b420a1fe39273942485fd86","unresolved":true,"context_lines":[{"line_number":174,"context_line":"               help\u003d_(\u0027Type of the nova endpoint to use.  This endpoint will\u0027"},{"line_number":175,"context_line":"                      \u0027 be looked up in the keystone catalog and should be\u0027"},{"line_number":176,"context_line":"                      \u0027 one of public, internal or admin.\u0027)),"},{"line_number":177,"context_line":"    cfg.BoolOpt(\u0027live_migration_events\u0027, default\u003dFalse,"},{"line_number":178,"context_line":"                help\u003d_(\u0027When this option is enabled, during the live \u0027"},{"line_number":179,"context_line":"                       \u0027migration, the OVS agent will only send the \u0027"},{"line_number":180,"context_line":"                       \u0027\"vif-plugged-event\" when the destination host \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"607282db_1b504e76","line":177,"range":{"start_line":177,"start_character":17,"end_line":177,"end_character":38},"in_reply_to":"beadb144_0e0cbf56","updated":"2021-06-10 10:49:02.000000000","message":"Thanks for the exception. Yes, the patch is in V too so I think is ok to merge it.\n\nThis option also works for OVS hybrid-plugin out of the box.","commit_id":"9d424ca27155874b70a86f5ea8f745d764987424"}],"zuul.d/tempest-multinode.yaml":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"3152c4c793788f443e92bdd40e27e8ecd1ef8ce8","unresolved":true,"context_lines":[{"line_number":145,"context_line":"          # Cinder services"},{"line_number":146,"context_line":"          c-bak: false"},{"line_number":147,"context_line":"          c-vol: false"},{"line_number":148,"context_line":"        devstack_local_conf:"},{"line_number":149,"context_line":"          post-config:"},{"line_number":150,"context_line":"            $NEUTRON_CONF:"},{"line_number":151,"context_line":"              nova:"},{"line_number":152,"context_line":"                live_migration_events: True"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"- job:"},{"line_number":155,"context_line":"    name: neutron-tempest-slow-py3"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"967824fc_4f8e681f","line":152,"range":{"start_line":148,"start_character":0,"end_line":152,"end_character":43},"updated":"2021-06-10 08:53:29.000000000","message":"Do we need this param set twice for victoria? In master we only have the section in vars","commit_id":"9d424ca27155874b70a86f5ea8f745d764987424"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"b56a141a094df9fc0b420a1fe39273942485fd86","unresolved":true,"context_lines":[{"line_number":145,"context_line":"          # Cinder services"},{"line_number":146,"context_line":"          c-bak: false"},{"line_number":147,"context_line":"          c-vol: false"},{"line_number":148,"context_line":"        devstack_local_conf:"},{"line_number":149,"context_line":"          post-config:"},{"line_number":150,"context_line":"            $NEUTRON_CONF:"},{"line_number":151,"context_line":"              nova:"},{"line_number":152,"context_line":"                live_migration_events: True"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"- job:"},{"line_number":155,"context_line":"    name: neutron-tempest-slow-py3"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"ee983639_8466e54c","line":152,"range":{"start_line":148,"start_character":0,"end_line":152,"end_character":43},"in_reply_to":"967824fc_4f8e681f","updated":"2021-06-10 10:49:02.000000000","message":"As discussed in IRC: the master branch patch is incorrect. The subned config is independent from the controller node and needs this variable.\n\nThis was initially a bad cherry-pick that became a fixed version of the original patch!","commit_id":"9d424ca27155874b70a86f5ea8f745d764987424"}]}