)]}'
{"neutron/db/l3_dvr_db.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"3f05668beefa2b3bb22ebacfeed67f77f1daeeb0","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"# TODO(slaweq): this should be moved to neutron_lib.plugins.utils module"},{"line_number":61,"context_line":"def is_port_bound(port):"},{"line_number":62,"context_line":"    active_binding \u003d plugin_utils.get_port_binding_by_status_and_host("},{"line_number":63,"context_line":"        port.get(\"port_bindings\", []), const.ACTIVE)"},{"line_number":64,"context_line":"    if not active_binding:"},{"line_number":65,"context_line":"        LOG.warning(\"Binding for port %s was not found.\", port)"}],"source_content_type":"text/x-python","patch_set":1,"id":"df33271e_039a056d","line":62,"updated":"2020-04-03 20:14:13.000000000","message":"pep8: F821 undefined name \u0027plugin_utils\u0027","commit_id":"5ee1c0f7d467d9550fa28199406663b3bee04c7c"},{"author":{"_account_id":25290,"name":"Mikhail Ushanov","email":"gm.mephisto@gmail.com","username":"gmmephisto"},"change_message_id":"8a14235bb70c9e21d4a446ebec4cd01ddee89769","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"# TODO(slaweq): this should be moved to neutron_lib.plugins.utils module"},{"line_number":61,"context_line":"def is_port_bound(port):"},{"line_number":62,"context_line":"    if port.port_binding:"},{"line_number":63,"context_line":"        LOG.warning(\"Binding for port %s was not found.\", port)"},{"line_number":64,"context_line":"        return False"},{"line_number":65,"context_line":"    return port.port_binding[portbindings_extended.VIF_TYPE] not in ["}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_f1e4e2ae","line":62,"updated":"2020-04-22 14:51:57.000000000","message":"Maybe this condition should be:\n`if not port.port_binding`?","commit_id":"09aaa520f00af4498610d91a64e8b37988168312"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"57f9e5c2b73155c7a71f36ff1dd0d6b79ef41f80","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"# TODO(slaweq): this should be moved to neutron_lib.plugins.utils module"},{"line_number":61,"context_line":"def is_port_bound(port):"},{"line_number":62,"context_line":"    if port.port_binding:"},{"line_number":63,"context_line":"        LOG.warning(\"Binding for port %s was not found.\", port)"},{"line_number":64,"context_line":"        return False"},{"line_number":65,"context_line":"    return port.port_binding[portbindings_extended.VIF_TYPE] not in ["}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_9c3e41d8","line":62,"in_reply_to":"1f493fa4_2b63731c","updated":"2020-04-23 09:13:30.000000000","message":"Argh. Sorry for that. And thx for pointing this out.\nFix is proposed: https://review.opendev.org/722235","commit_id":"09aaa520f00af4498610d91a64e8b37988168312"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"bbb7bbfadf8cceab6a154576a15fc5c0850a4d7d","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"# TODO(slaweq): this should be moved to neutron_lib.plugins.utils module"},{"line_number":61,"context_line":"def is_port_bound(port):"},{"line_number":62,"context_line":"    if port.port_binding:"},{"line_number":63,"context_line":"        LOG.warning(\"Binding for port %s was not found.\", port)"},{"line_number":64,"context_line":"        return False"},{"line_number":65,"context_line":"    return port.port_binding[portbindings_extended.VIF_TYPE] not in ["}],"source_content_type":"text/x-python","patch_set":2,"id":"1f493fa4_2b63731c","line":62,"in_reply_to":"1f493fa4_f1e4e2ae","updated":"2020-04-22 18:27:03.000000000","message":"Yes, I think you\u0027re right.  Looks like this was something that had to be tweaked to get this to apply to Queens and we missed this.","commit_id":"09aaa520f00af4498610d91a64e8b37988168312"}]}