)]}'
{"neutron/services/qos/qos_plugin.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"da0b690d7cf367ed534dce980cb3f819da1123de","unresolved":false,"context_lines":[{"line_number":103,"context_line":"                context.get_admin_context(), id\u003dport_res[\u0027network_id\u0027])"},{"line_number":104,"context_line":"            if not net:"},{"line_number":105,"context_line":"                LOG.debug(\"Could not look up network %s for port %s\","},{"line_number":106,"context_line":"                        port_res[\u0027network_id\u0027], port_res[\u0027id\u0027])"},{"line_number":107,"context_line":"                return"},{"line_number":108,"context_line":"            if net.qos_policy_id:"},{"line_number":109,"context_line":"                qos_policy \u003d policy_object.QosPolicy.get_network_policy("}],"source_content_type":"text/x-python","patch_set":1,"id":"ffd0ebdf_f78ef625","line":106,"updated":"2019-01-04 15:34:32.000000000","message":"nit: please fix alignment of this line :)","commit_id":"4a40bfa9282408025b4a55fd12ca4e6902a2593b"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"da0b690d7cf367ed534dce980cb3f819da1123de","unresolved":false,"context_lines":[{"line_number":104,"context_line":"            if not net:"},{"line_number":105,"context_line":"                LOG.debug(\"Could not look up network %s for port %s\","},{"line_number":106,"context_line":"                        port_res[\u0027network_id\u0027], port_res[\u0027id\u0027])"},{"line_number":107,"context_line":"                return"},{"line_number":108,"context_line":"            if net.qos_policy_id:"},{"line_number":109,"context_line":"                qos_policy \u003d policy_object.QosPolicy.get_network_policy("},{"line_number":110,"context_line":"                    context.get_admin_context(), net.id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"ffd0ebdf_977a0a22","line":107,"updated":"2019-01-04 15:34:32.000000000","message":"should be\n\n    return port_res\n\nin such case","commit_id":"4a40bfa9282408025b4a55fd12ca4e6902a2593b"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"d71900eca2fb5c4d1fcaa5ddedb4abec64d3e1e6","unresolved":false,"context_lines":[{"line_number":105,"context_line":"                LOG.debug(\"Could not look up network %s for port %s\","},{"line_number":106,"context_line":"                        port_res[\u0027network_id\u0027], port_res[\u0027id\u0027])"},{"line_number":107,"context_line":"                return"},{"line_number":108,"context_line":"            if net.qos_policy_id:"},{"line_number":109,"context_line":"                qos_policy \u003d policy_object.QosPolicy.get_network_policy("},{"line_number":110,"context_line":"                    context.get_admin_context(), net.id)"},{"line_number":111,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"ffd0ebdf_f7c6f616","line":108,"updated":"2019-01-04 15:47:06.000000000","message":"Why not just change this to \u0027if net and net.qos_policy_id:\u0027 ?  We just won\u0027t have the debug message, or can put it after L112 for all cases.","commit_id":"4a40bfa9282408025b4a55fd12ca4e6902a2593b"}]}