)]}'
{"neutron/db/l3_dvr_db.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2f105b7fca746e450fcfd6249acd5805a7329c28","unresolved":false,"context_lines":[{"line_number":50,"context_line":"cfg.CONF.register_opts(router_distributed_opts)"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"dvr_unbound_port_fip_use_cvr_opts \u003d ["},{"line_number":53,"context_line":"    cfg.BoolOpt(\u0027dvr_unbound_port_fip_use_cvr\u0027,"},{"line_number":54,"context_line":"                default\u003dFalse,"},{"line_number":55,"context_line":"                help\u003d_(\"System-wide flag to determine the behavior of the \""},{"line_number":56,"context_line":"                       \"unbound ports that are associated with a floatingip \""}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_432d4845","line":53,"updated":"2016-05-25 20:06:36.000000000","message":"I think this can be combined with the above one, no need for a new list.","commit_id":"7b138103fbd6e2384a66daf858d20a264488009b"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2f105b7fca746e450fcfd6249acd5805a7329c28","unresolved":false,"context_lines":[{"line_number":53,"context_line":"    cfg.BoolOpt(\u0027dvr_unbound_port_fip_use_cvr\u0027,"},{"line_number":54,"context_line":"                default\u003dFalse,"},{"line_number":55,"context_line":"                help\u003d_(\"System-wide flag to determine the behavior of the \""},{"line_number":56,"context_line":"                       \"unbound ports that are associated with a floatingip \""},{"line_number":57,"context_line":"                       \"and connected to a DVR Router.\")),"},{"line_number":58,"context_line":"]"},{"line_number":59,"context_line":"cfg.CONF.register_opts(dvr_unbound_port_fip_use_cvr_opts)"}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_e31a7cc5","line":56,"updated":"2016-05-25 20:06:36.000000000","message":"s/floatingip/floating IP","commit_id":"7b138103fbd6e2384a66daf858d20a264488009b"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2f105b7fca746e450fcfd6249acd5805a7329c28","unresolved":false,"context_lines":[{"line_number":54,"context_line":"                default\u003dFalse,"},{"line_number":55,"context_line":"                help\u003d_(\"System-wide flag to determine the behavior of the \""},{"line_number":56,"context_line":"                       \"unbound ports that are associated with a floatingip \""},{"line_number":57,"context_line":"                       \"and connected to a DVR Router.\")),"},{"line_number":58,"context_line":"]"},{"line_number":59,"context_line":"cfg.CONF.register_opts(dvr_unbound_port_fip_use_cvr_opts)"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_c3277810","line":57,"updated":"2016-05-25 20:06:36.000000000","message":"Need some more text around the CVR impact - i.e. explaining an unbound port will be hosted on the CVR router.","commit_id":"7b138103fbd6e2384a66daf858d20a264488009b"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2f105b7fca746e450fcfd6249acd5805a7329c28","unresolved":false,"context_lines":[{"line_number":501,"context_line":"                        if host_mismatch and no_gateway_host:"},{"line_number":502,"context_line":"                            continue"},{"line_number":503,"context_line":"                        if (host_mismatch and not no_gateway_host and"},{"line_number":504,"context_line":"                            not cfg.CONF.dvr_unbound_port_fip_use_cvr):"},{"line_number":505,"context_line":"                            continue"},{"line_number":506,"context_line":"                    else:"},{"line_number":507,"context_line":"                        if host_mismatch:"}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_63004c67","line":504,"updated":"2016-05-25 20:06:36.000000000","message":"Seems like the new value should be what\u0027s checked on L494, since otherwise we\u0027ll do some work for nothing.","commit_id":"7b138103fbd6e2384a66daf858d20a264488009b"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2f105b7fca746e450fcfd6249acd5805a7329c28","unresolved":false,"context_lines":[{"line_number":905,"context_line":"            if dest_host and dest_host !\u003d host:"},{"line_number":906,"context_line":"                self.l3_rpc_notifier.routers_updated_on_host("},{"line_number":907,"context_line":"                    context, [router_id], dest_host)"},{"line_number":908,"context_line":"            rtr_gateway_port \u003d self._core_plugin.get_port("},{"line_number":909,"context_line":"                context, router.get(\u0027gw_port_id\u0027))"},{"line_number":910,"context_line":"            if (host is None and dest_host is None and"},{"line_number":911,"context_line":"                cfg.CONF.dvr_unbound_port_fip_use_cvr):"}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_c37db81d","line":908,"updated":"2016-05-25 20:06:36.000000000","message":"Doesn\u0027t look like rtr_gateway_port is used unless we get into the if(), so let\u0027s defer it until then to skip the get_port() call unnecessarily.","commit_id":"7b138103fbd6e2384a66daf858d20a264488009b"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"cc3df2ecdb513a240b065f07a8735d5bcf926834","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                default\u003dFalse,"},{"line_number":47,"context_line":"                help\u003d_(\"System-wide flag to determine the type of router \""},{"line_number":48,"context_line":"                       \"that tenants can create. Only admin can override.\")),"},{"line_number":49,"context_line":"    cfg.BoolOpt(\u0027dvr_unbound_port_fip_use_cvr\u0027,"},{"line_number":50,"context_line":"                default\u003dFalse,"},{"line_number":51,"context_line":"                help\u003d_(\"System-wide flag to determine the behavior of the \""},{"line_number":52,"context_line":"                       \"unbound ports that are associated with a floating IP \""}],"source_content_type":"text/x-python","patch_set":9,"id":"9abb7d3a_0ead8788","line":49,"updated":"2016-06-02 16:26:11.000000000","message":"As suggested on the bug report. I am against the config option.","commit_id":"39057e0fcb56a5bf001af612375d7b48c3847310"},{"author":{"_account_id":7016,"name":"Swaminathan Vasudevan","email":"swvasude@cisco.com","username":"souminathan"},"change_message_id":"f7759b3e0b8255305b78c3186befdb43ba32856f","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                default\u003dFalse,"},{"line_number":47,"context_line":"                help\u003d_(\"System-wide flag to determine the type of router \""},{"line_number":48,"context_line":"                       \"that tenants can create. Only admin can override.\")),"},{"line_number":49,"context_line":"    cfg.BoolOpt(\u0027dvr_unbound_port_fip_use_cvr\u0027,"},{"line_number":50,"context_line":"                default\u003dFalse,"},{"line_number":51,"context_line":"                help\u003d_(\"System-wide flag to determine the behavior of the \""},{"line_number":52,"context_line":"                       \"unbound ports that are associated with a floating IP \""}],"source_content_type":"text/x-python","patch_set":9,"id":"9abb7d3a_ee4ac3ab","line":49,"in_reply_to":"9abb7d3a_0ead8788","updated":"2016-06-02 16:28:53.000000000","message":"Yes will try to address it based on the device_owners if possible.","commit_id":"39057e0fcb56a5bf001af612375d7b48c3847310"},{"author":{"_account_id":7016,"name":"Swaminathan Vasudevan","email":"swvasude@cisco.com","username":"souminathan"},"change_message_id":"bb296799d8de2a8fc770ab095db35d663a2b51de","unresolved":false,"context_lines":[{"line_number":46,"context_line":"                default\u003dFalse,"},{"line_number":47,"context_line":"                help\u003d_(\"System-wide flag to determine the type of router \""},{"line_number":48,"context_line":"                       \"that tenants can create. Only admin can override.\")),"},{"line_number":49,"context_line":"    cfg.BoolOpt(\u0027dvr_unbound_port_fip_use_cvr\u0027,"},{"line_number":50,"context_line":"                default\u003dFalse,"},{"line_number":51,"context_line":"                help\u003d_(\"System-wide flag to determine the behavior of the \""},{"line_number":52,"context_line":"                       \"unbound ports that are associated with a floating IP \""}],"source_content_type":"text/x-python","patch_set":9,"id":"9abb7d3a_31038c0f","line":49,"in_reply_to":"9abb7d3a_ee4ac3ab","updated":"2016-06-02 16:34:05.000000000","message":"Armando, I did see your comment in the RFE regarding the config option, since I already had uploaded the patch, I was just test it, I will try to see if there are any other options to address this scenario.","commit_id":"39057e0fcb56a5bf001af612375d7b48c3847310"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0248c59b71f425a5e46644b5fbc150acfe0be7a6","unresolved":false,"context_lines":[{"line_number":881,"context_line":"            dest_host \u003d self._get_dvr_migrating_service_port_hostid("},{"line_number":882,"context_line":"                context, fixed_port_id)"},{"line_number":883,"context_line":"            self.l3_rpc_notifier.routers_updated_on_host("},{"line_number":884,"context_line":"                context, [router_id], host)"},{"line_number":885,"context_line":"            if dest_host and dest_host !\u003d host:"},{"line_number":886,"context_line":"                self.l3_rpc_notifier.routers_updated_on_host("},{"line_number":887,"context_line":"                    context, [router_id], dest_host)"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_1a96d33e","side":"PARENT","line":884,"updated":"2016-07-06 17:47:38.000000000","message":"I would leave this code as-is, then...","commit_id":"805e2391f6314d3fb78ad1fc190a08f8dbfb7a20"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0248c59b71f425a5e46644b5fbc150acfe0be7a6","unresolved":false,"context_lines":[{"line_number":489,"context_line":"                    host_mismatch \u003d ("},{"line_number":490,"context_line":"                        fip_host !\u003d host and"},{"line_number":491,"context_line":"                        floating_ip.get(\u0027dest_host\u0027) is None)"},{"line_number":492,"context_line":"                    if floating_ip.get(\u0027host\u0027) is None:"},{"line_number":493,"context_line":"                        gw_port_id \u003d router.get(\u0027gw_port_id\u0027)"},{"line_number":494,"context_line":"                        if gw_port_id:"},{"line_number":495,"context_line":"                            gateway_port_db \u003d self._core_plugin.get_port("}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_5a7fbb65","line":492,"updated":"2016-07-06 17:47:38.000000000","message":"Use fip_host here as it was just assigned on L488","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0248c59b71f425a5e46644b5fbc150acfe0be7a6","unresolved":false,"context_lines":[{"line_number":493,"context_line":"                        gw_port_id \u003d router.get(\u0027gw_port_id\u0027)"},{"line_number":494,"context_line":"                        if gw_port_id:"},{"line_number":495,"context_line":"                            gateway_port_db \u003d self._core_plugin.get_port("},{"line_number":496,"context_line":"                                context, router[\u0027gw_port_id\u0027])"},{"line_number":497,"context_line":"                            # Condition to check if the requesting agent is"},{"line_number":498,"context_line":"                            # a dvr_snat agent"},{"line_number":499,"context_line":"                            gateway_host_request \u003d (gateway_port_db and"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_5acdfbb5","line":496,"updated":"2016-07-06 17:47:38.000000000","message":"Use gw_port_id here","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0248c59b71f425a5e46644b5fbc150acfe0be7a6","unresolved":false,"context_lines":[{"line_number":500,"context_line":"                                host !\u003d gateway_port_db[portbindings.HOST_ID])"},{"line_number":501,"context_line":"                            if not gateway_host_request:"},{"line_number":502,"context_line":"                                continue"},{"line_number":503,"context_line":"                    if fip_host and host_mismatch:"},{"line_number":504,"context_line":"                        continue"},{"line_number":505,"context_line":"                    LOG.debug(\"Floating IP host: %s\", floating_ip.get(\u0027host\u0027))"},{"line_number":506,"context_line":"                router_floatingips.append(floating_ip)"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_7aa9d78f","line":503,"updated":"2016-07-06 17:47:38.000000000","message":"Might as well move this to L491+","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"948bf0e0b13bad441c81652dabfc6ff64896c99d","unresolved":false,"context_lines":[{"line_number":502,"context_line":"                                continue"},{"line_number":503,"context_line":"                    if fip_host and host_mismatch:"},{"line_number":504,"context_line":"                        continue"},{"line_number":505,"context_line":"                    LOG.debug(\"Floating IP host: %s\", floating_ip.get(\u0027host\u0027))"},{"line_number":506,"context_line":"                router_floatingips.append(floating_ip)"},{"line_number":507,"context_line":"                router[const.FLOATINGIP_KEY] \u003d router_floatingips"},{"line_number":508,"context_line":"                if not fip_sync_interfaces:"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_850216b3","line":505,"updated":"2016-07-08 00:15:27.000000000","message":"You can use fip_host here too","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0248c59b71f425a5e46644b5fbc150acfe0be7a6","unresolved":false,"context_lines":[{"line_number":547,"context_line":"                    port_profile.get(\u0027migrating_to\u0027) \u003d\u003d host)"},{"line_number":548,"context_line":"                unbound_lbaas_ports \u003d (port[\u0027device_owner\u0027] in ("},{"line_number":549,"context_line":"                    const.DEVICE_OWNER_LOADBALANCER,"},{"line_number":550,"context_line":"                    const.DEVICE_OWNER_LOADBALANCERV2) and"},{"line_number":551,"context_line":"                    not port[portbindings.HOST_ID])"},{"line_number":552,"context_line":"                if (port[portbindings.HOST_ID] \u003d\u003d host or port_in_migration or"},{"line_number":553,"context_line":"                    unbound_lbaas_ports):"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_1a0b738f","line":550,"updated":"2016-07-06 17:47:38.000000000","message":"I\u0027d rather have these in a method, I thought that was in a previous patch?","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"948bf0e0b13bad441c81652dabfc6ff64896c99d","unresolved":false,"context_lines":[{"line_number":547,"context_line":"                    port_profile.get(\u0027migrating_to\u0027) \u003d\u003d host)"},{"line_number":548,"context_line":"                unbound_lbaas_ports \u003d (port[\u0027device_owner\u0027] in ("},{"line_number":549,"context_line":"                    const.DEVICE_OWNER_LOADBALANCER,"},{"line_number":550,"context_line":"                    const.DEVICE_OWNER_LOADBALANCERV2) and"},{"line_number":551,"context_line":"                    not port[portbindings.HOST_ID])"},{"line_number":552,"context_line":"                if (port[portbindings.HOST_ID] \u003d\u003d host or port_in_migration or"},{"line_number":553,"context_line":"                    unbound_lbaas_ports):"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_a57c72ec","line":550,"in_reply_to":"1aa78d24_1a0b738f","updated":"2016-07-08 00:15:27.000000000","message":"Why are we doing this patch only for LBaaS?\nThis issue impacts any Allowed Address Pairs port with a VRRP service behind it.","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"948bf0e0b13bad441c81652dabfc6ff64896c99d","unresolved":false,"context_lines":[{"line_number":561,"context_line":"                    fip[\u0027dest_host\u0027] \u003d ("},{"line_number":562,"context_line":"                        self._get_dvr_migrating_service_port_hostid("},{"line_number":563,"context_line":"                            context, fip[\u0027port_id\u0027], port\u003dvm_port))"},{"line_number":564,"context_line":"                    # Handle the case were there is no host binding"},{"line_number":565,"context_line":"                    # for the fip port and if configured to host the"},{"line_number":566,"context_line":"                    # fip in cvr, then pass in the flag to the agent"},{"line_number":567,"context_line":"                    # for the specified fip."}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_20c0707c","line":564,"updated":"2016-07-08 00:15:27.000000000","message":"were -\u003e where","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0248c59b71f425a5e46644b5fbc150acfe0be7a6","unresolved":false,"context_lines":[{"line_number":566,"context_line":"                    # fip in cvr, then pass in the flag to the agent"},{"line_number":567,"context_line":"                    # for the specified fip."},{"line_number":568,"context_line":"                    if (not fip[\u0027host\u0027]):"},{"line_number":569,"context_line":"                        fip[\u0027unbound\u0027] \u003d \u0027cvr\u0027"},{"line_number":570,"context_line":"        routers_dict \u003d self._process_routers(context, routers)"},{"line_number":571,"context_line":"        self._process_floating_ips_dvr(context, routers_dict,"},{"line_number":572,"context_line":"                                       floating_ips, host, agent)"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_970bbe29","line":569,"updated":"2016-07-06 17:47:38.000000000","message":"\u0027cvr\u0027 needs to be in constants file","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"948bf0e0b13bad441c81652dabfc6ff64896c99d","unresolved":false,"context_lines":[{"line_number":566,"context_line":"                    # fip in cvr, then pass in the flag to the agent"},{"line_number":567,"context_line":"                    # for the specified fip."},{"line_number":568,"context_line":"                    if (not fip[\u0027host\u0027]):"},{"line_number":569,"context_line":"                        fip[\u0027unbound\u0027] \u003d \u0027cvr\u0027"},{"line_number":570,"context_line":"        routers_dict \u003d self._process_routers(context, routers)"},{"line_number":571,"context_line":"        self._process_floating_ips_dvr(context, routers_dict,"},{"line_number":572,"context_line":"                                       floating_ips, host, agent)"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_a0d62025","line":569,"in_reply_to":"1aa78d24_970bbe29","updated":"2016-07-08 00:15:27.000000000","message":"All of these probably should be.","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"948bf0e0b13bad441c81652dabfc6ff64896c99d","unresolved":false,"context_lines":[{"line_number":915,"context_line":"                    context.elevated(), fixed_port_id)"},{"line_number":916,"context_line":"                if (unbound_port[\u0027device_owner\u0027] in ("},{"line_number":917,"context_line":"                    const.DEVICE_OWNER_LOADBALANCER or"},{"line_number":918,"context_line":"                    const.DEVICE_OWNER_LOADBALANCERV2)):"},{"line_number":919,"context_line":"                    rtr_gateway_port \u003d self._core_plugin.get_port("},{"line_number":920,"context_line":"                        context, router.get(\u0027gw_port_id\u0027))"},{"line_number":921,"context_line":"                    rtr_gateway_host \u003d rtr_gateway_port.get("}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_a0bd80d9","line":918,"updated":"2016-07-08 00:15:27.000000000","message":"Again, why not fix this for everyone?","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0248c59b71f425a5e46644b5fbc150acfe0be7a6","unresolved":false,"context_lines":[{"line_number":919,"context_line":"                    rtr_gateway_port \u003d self._core_plugin.get_port("},{"line_number":920,"context_line":"                        context, router.get(\u0027gw_port_id\u0027))"},{"line_number":921,"context_line":"                    rtr_gateway_host \u003d rtr_gateway_port.get("},{"line_number":922,"context_line":"                        portbindings.HOST_ID)"},{"line_number":923,"context_line":"                    self.l3_rpc_notifier.routers_updated_on_host("},{"line_number":924,"context_line":"                        context, [router_id], rtr_gateway_host)"},{"line_number":925,"context_line":"            if dest_host and dest_host !\u003d host:"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_1ae413df","line":922,"updated":"2016-07-06 17:47:38.000000000","message":"... have this set host \u003d ... so the other code doesn\u0027t have to change.","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"}],"neutron/tests/functional/services/l3_router/test_l3_dvr_router_plugin.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"948bf0e0b13bad441c81652dabfc6ff64896c99d","unresolved":false,"context_lines":[{"line_number":486,"context_line":"        vrrp_port \u003d self._make_port("},{"line_number":487,"context_line":"            self.fmt,"},{"line_number":488,"context_line":"            private_net1[\u0027network\u0027][\u0027id\u0027],"},{"line_number":489,"context_line":"            device_owner\u003dconstants.DEVICE_OWNER_LOADBALANCER,"},{"line_number":490,"context_line":"            fixed_ips\u003dfixed_vrrp_ip)"},{"line_number":491,"context_line":"        allowed_address_pairs \u003d ["},{"line_number":492,"context_line":"            {\u0027ip_address\u0027: \u002710.1.0.201\u0027,"}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_40c93c5a","line":489,"updated":"2016-07-08 00:15:27.000000000","message":"This impacts not just LBaaS and doesn\u0027t test the other V2 option.","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"948bf0e0b13bad441c81652dabfc6ff64896c99d","unresolved":false,"context_lines":[{"line_number":512,"context_line":"                    {\u0027port\u0027: {portbindings.HOST_ID: HOST1}})"},{"line_number":513,"context_line":"                vm_port2 \u003d self.core_plugin.update_port("},{"line_number":514,"context_line":"                    self.context, int_port2[\u0027port\u0027][\u0027id\u0027],"},{"line_number":515,"context_line":"                    {\u0027port\u0027: {portbindings.HOST_ID: HOST2}}) "},{"line_number":516,"context_line":"                self.assertTrue(vm_port1[\u0027admin_state_up\u0027])"},{"line_number":517,"context_line":"                self.assertTrue(vm_port2[\u0027admin_state_up\u0027])"},{"line_number":518,"context_line":"                vrrp_port_db \u003d self.core_plugin.get_port("}],"source_content_type":"text/x-python","patch_set":12,"id":"1aa78d24_e09ca85f","line":515,"updated":"2016-07-08 00:15:27.000000000","message":"space","commit_id":"88011231b581b4174ebc07389e4defde9b2432c6"}]}