)]}'
{"neutron/conf/common.py":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"1098cd36568af8997a901103afeaa6b8b4b8df57","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":10,"id":"e627beaa_0ecaafce","line":185,"range":{"start_line":185,"start_character":24,"end_line":185,"end_character":76},"updated":"2021-01-14 12:18:15.000000000","message":"Is this cfg option needed if both patches are merged in W? Do we officially support running Nona and Neutron of different versions (Nova V and Neutron W)?","commit_id":"f8a22c7d4aa654eaad3b683073849c873ea3beff"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"cf657d43ebd7c1d19375955b0a4fc3ecf3c0c1b5","unresolved":false,"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":10,"id":"2f345565_1f4211b7","line":185,"range":{"start_line":185,"start_character":24,"end_line":185,"end_character":76},"in_reply_to":"a8988e21_1efc6631","updated":"2021-01-14 12:48:45.000000000","message":"Ack, thanks","commit_id":"f8a22c7d4aa654eaad3b683073849c873ea3beff"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"de9eb63f9a2015d3e9558e9b5191ed8caa967b20","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":10,"id":"a8988e21_1efc6631","line":185,"range":{"start_line":185,"start_character":24,"end_line":185,"end_character":76},"in_reply_to":"e627beaa_0ecaafce","updated":"2021-01-14 12:30:08.000000000","message":"yes mixed version is offically supported in either order and the delta can be greater then 1 version.\n\nwe do not test that but its expected to work since both project follow policy around stable APIs.\n\ndepending on you installation tool its not that uncommon to do upgrades service by service so at least in that upgrade window there could be mixed versions but you can run it with mixed version indefinitely if you want.","commit_id":"f8a22c7d4aa654eaad3b683073849c873ea3beff"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"de9eb63f9a2015d3e9558e9b5191ed8caa967b20","unresolved":true,"context_lines":[{"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),"},{"line_number":189,"context_line":"                deprecated_for_removal\u003dTrue,"},{"line_number":190,"context_line":"                deprecated_reason\u003d("},{"line_number":191,"context_line":"                    \u0027In Y the Nova patch \u0027"},{"line_number":192,"context_line":"                    \u0027https://review.opendev.org/c/openstack/nova/+/767368 \u0027"},{"line_number":193,"context_line":"                    \u0027will be in the code even when running a Nova server in \u0027"},{"line_number":194,"context_line":"                    \u0027X.\u0027)),"},{"line_number":195,"context_line":"]"},{"line_number":196,"context_line":""},{"line_number":197,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"0b859f6f_6fdd9870","line":194,"range":{"start_line":190,"start_character":16,"end_line":194,"end_character":27},"updated":"2021-01-14 12:30:08.000000000","message":"we are going to backport that nova patch to all open branches by the way.\nso you could drop this in x or possibly not have the config option.\nthat said we cant be sure that the old nova is on the latest version fo stable/Whatever so it might not have the backport which is why this is useful to have.\n\nif this was nova we porbaly would have put this option in the workarounds section by the way.","commit_id":"f8a22c7d4aa654eaad3b683073849c873ea3beff"}],"neutron/db/provisioning_blocks.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"8089cf365729910309b93ae105d65a41f62be164","unresolved":true,"context_lines":[{"line_number":138,"context_line":"            context, standard_attr_id\u003dstandard_attr_id):"},{"line_number":139,"context_line":"        LOG.debug(\"Provisioning complete for %(otype)s %(oid)s triggered by \""},{"line_number":140,"context_line":"                  \"entity %(entity)s.\", log_dict)"},{"line_number":141,"context_line":"        registry.publish(object_type, PROVISIONING_COMPLETE, entity,"},{"line_number":142,"context_line":"                         payload\u003devents.DBEventPayload("},{"line_number":143,"context_line":"                             context, resource_id\u003dobject_id))"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"74f394e5_903452ec","line":141,"updated":"2021-01-14 10:24:44.000000000","message":"why You are changing that trigger passed here?","commit_id":"f8a22c7d4aa654eaad3b683073849c873ea3beff"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4582ceb72c3f1b5c215f260a05af5aeca801f8a6","unresolved":true,"context_lines":[{"line_number":138,"context_line":"            context, standard_attr_id\u003dstandard_attr_id):"},{"line_number":139,"context_line":"        LOG.debug(\"Provisioning complete for %(otype)s %(oid)s triggered by \""},{"line_number":140,"context_line":"                  \"entity %(entity)s.\", log_dict)"},{"line_number":141,"context_line":"        registry.publish(object_type, PROVISIONING_COMPLETE, entity,"},{"line_number":142,"context_line":"                         payload\u003devents.DBEventPayload("},{"line_number":143,"context_line":"                             context, resource_id\u003dobject_id))"},{"line_number":144,"context_line":""}],"source_content_type":"text/x-python","patch_set":10,"id":"38287d09_040eb2fd","line":141,"in_reply_to":"74f394e5_903452ec","updated":"2021-01-14 12:06:58.000000000","message":"The only method receiving this event is [1]. There I need to know who is sending it, the DHCP agent or the L2 agent [2].\n\nThen in [1], if:\n1) we are migrating,\n2) the \"live_migration_events\" flag is set,\n3) we have an active \"nova_notifier\"\n4) and the L2 agent (for example, in the related bug, the OVS agent) is sending the event (*)\n\nthe port is updated and the vif-plugged-event is sent.\n\n(*) Note that when the flag \"live_migration_events\" is enabled, the OVS agent will send only the event for the destination host port, not for the origin host one.\n\n[1]https://review.opendev.org/c/openstack/neutron/+/766277/10/neutron/plugins/ml2/plugin.py#351\n[2]https://github.com/openstack/neutron/blob/master/neutron/db/provisioning_blocks.py#L29-L30","commit_id":"f8a22c7d4aa654eaad3b683073849c873ea3beff"}],"neutron/plugins/ml2/plugin.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"8089cf365729910309b93ae105d65a41f62be164","unresolved":true,"context_lines":[{"line_number":348,"context_line":"            getattr(port, \u0027port_bindings\u0027, []))"},{"line_number":349,"context_line":"        if (host_migrating and cfg.CONF.nova.live_migration_events and"},{"line_number":350,"context_line":"                self.nova_notifier):"},{"line_number":351,"context_line":"            send_nova_event \u003d bool(trigger \u003d\u003d"},{"line_number":352,"context_line":"                                   provisioning_blocks.L2_AGENT_ENTITY)"},{"line_number":353,"context_line":"            with self.nova_notifier.context_enabled(send_nova_event):"},{"line_number":354,"context_line":"                self.update_port_status(payload.context, port_id,"}],"source_content_type":"text/x-python","patch_set":10,"id":"b1dec6fe_a9681f52","line":351,"range":{"start_line":351,"start_character":30,"end_line":351,"end_character":34},"updated":"2021-01-14 10:24:44.000000000","message":"is this really needed?","commit_id":"f8a22c7d4aa654eaad3b683073849c873ea3beff"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4582ceb72c3f1b5c215f260a05af5aeca801f8a6","unresolved":true,"context_lines":[{"line_number":348,"context_line":"            getattr(port, \u0027port_bindings\u0027, []))"},{"line_number":349,"context_line":"        if (host_migrating and cfg.CONF.nova.live_migration_events and"},{"line_number":350,"context_line":"                self.nova_notifier):"},{"line_number":351,"context_line":"            send_nova_event \u003d bool(trigger \u003d\u003d"},{"line_number":352,"context_line":"                                   provisioning_blocks.L2_AGENT_ENTITY)"},{"line_number":353,"context_line":"            with self.nova_notifier.context_enabled(send_nova_event):"},{"line_number":354,"context_line":"                self.update_port_status(payload.context, port_id,"}],"source_content_type":"text/x-python","patch_set":10,"id":"b3bd2055_ae5675ee","line":351,"range":{"start_line":351,"start_character":30,"end_line":351,"end_character":34},"in_reply_to":"b1dec6fe_a9681f52","updated":"2021-01-14 12:06:58.000000000","message":"Not really. Because zuul is happy, if I need to respin the patch, I\u0027ll remove it.","commit_id":"f8a22c7d4aa654eaad3b683073849c873ea3beff"}]}