)]}'
{"neutron/plugins/ml2/plugin.py":[{"author":{"_account_id":10267,"name":"venkata anil","email":"anilvenkata@redhat.com","username":"anil"},"change_message_id":"8f7ab8d1d0ce9e9b25b0c66f170913fb6e9f46c6","unresolved":false,"context_lines":[{"line_number":233,"context_line":"        if updated_port[\u0027device_owner\u0027] \u003d\u003d const.DEVICE_OWNER_ROUTER_HA_INTF:"},{"line_number":234,"context_line":"            updated_port[portbindings.HOST_ID] \u003d host"},{"line_number":235,"context_line":"            updated_port[\u0027status\u0027] \u003d status"},{"line_number":236,"context_line":"        port_context \u003d self.get_bound_port_context("},{"line_number":237,"context_line":"            context, updated_port[\u0027id\u0027], host)"},{"line_number":238,"context_line":"        if port_context:"},{"line_number":239,"context_line":"            self.mechanism_manager.update_port_postcommit(port_context)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_822ae48e","line":236,"updated":"2016-09-01 05:22:39.000000000","message":"I should modify get_bound_port_context() to accept orinial_port and update port_context with it.\nNeed to update host and status directly on port context\u0027s _port for HA router port.","commit_id":"83d5f2443302087d071409b7ee6d209fdab421ec"},{"author":{"_account_id":10267,"name":"venkata anil","email":"anilvenkata@redhat.com","username":"anil"},"change_message_id":"6f6ed57fc5abb83048682597befc60ae4a3e2eb5","unresolved":false,"context_lines":[{"line_number":1738,"context_line":"                binding.update(binding)"},{"line_number":1739,"context_line":"                updated \u003d True"},{"line_number":1740,"context_line":""},{"line_number":1741,"context_line":"        if updated:"},{"line_number":1742,"context_line":"            kwargs \u003d {\u0027context\u0027: context, \u0027port\u0027: mech_context.current,"},{"line_number":1743,"context_line":"                      \u0027original_port\u0027: original_port}"},{"line_number":1744,"context_line":"            if status \u003d\u003d const.PORT_STATUS_ACTIVE:"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_c6dc2049","line":1741,"updated":"2016-08-31 16:26:12.000000000","message":"This notification code is moved to [1] and update_port_status will only update status. Is it worth of having separate notifier for postcommit when status updated?\n\n[1] https://review.openstack.org/#/c/363777/1/neutron/plugins/ml2/plugin.py@216","commit_id":"83d5f2443302087d071409b7ee6d209fdab421ec"}],"neutron/plugins/ml2/rpc.py":[{"author":{"_account_id":10267,"name":"venkata anil","email":"anilvenkata@redhat.com","username":"anil"},"change_message_id":"6f6ed57fc5abb83048682597befc60ae4a3e2eb5","unresolved":false,"context_lines":[{"line_number":195,"context_line":""},{"line_number":196,"context_line":"        try:"},{"line_number":197,"context_line":"            port_exists \u003d bool(plugin.update_port_status("},{"line_number":198,"context_line":"                rpc_context, port_id, n_const.PORT_STATUS_DOWN, host))"},{"line_number":199,"context_line":"            updated_port \u003d plugin.get_port(rpc_context, port_id)"},{"line_number":200,"context_line":"            registry.notify(resources.PORT, events.DATAPLANE_INACTIVE,"},{"line_number":201,"context_line":"                self, context\u003drpc_context, host\u003dhost,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_a699dc21","line":198,"updated":"2016-08-31 16:26:12.000000000","message":"Still we need to call update_port_status with \"host\" as DVR will fetch the binding record with \"host\" and update status in this record. So DATAPLANE event notification can\u0027t remove this update_port_status calls for DVR and use provisioning blocks for updating status.","commit_id":"83d5f2443302087d071409b7ee6d209fdab421ec"},{"author":{"_account_id":10267,"name":"venkata anil","email":"anilvenkata@redhat.com","username":"anil"},"change_message_id":"6f6ed57fc5abb83048682597befc60ae4a3e2eb5","unresolved":false,"context_lines":[{"line_number":258,"context_line":"            provisioning_blocks.provisioning_complete("},{"line_number":259,"context_line":"                rpc_context, port[\u0027id\u0027], resources.PORT,"},{"line_number":260,"context_line":"                provisioning_blocks.L2_AGENT_ENTITY)"},{"line_number":261,"context_line":"            updated_port \u003d plugin.get_port(rpc_context, port_id)"},{"line_number":262,"context_line":"            registry.notify(resources.PORT, events.DATAPLANE_ACTIVE,"},{"line_number":263,"context_line":"                self, context\u003drpc_context, host\u003dhost,"},{"line_number":264,"context_line":"                status\u003dn_const.PORT_STATUS_ACTIVE,"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_c63a60e7","line":261,"updated":"2016-08-31 16:26:12.000000000","message":"Should we still notify DATAPLANE_ACTIVE event if above provising_complete fails to to update DB status?","commit_id":"83d5f2443302087d071409b7ee6d209fdab421ec"}]}