)]}'
{"neutron/api/rpc/handlers/l3_rpc.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"bee46954598e3661da8f499a09ecccca458d28d5","unresolved":false,"context_lines":[{"line_number":98,"context_line":"        if utils.is_extension_supported("},{"line_number":99,"context_line":"            self.plugin, constants.PORT_BINDING_EXT_ALIAS):"},{"line_number":100,"context_line":"            self._ensure_host_set_on_ports(context, host, routers)"},{"line_number":101,"context_line":"        LOG.debug(\"Routers returned to l3 agent %s:\\n %s\","},{"line_number":102,"context_line":"                  host,"},{"line_number":103,"context_line":"                  utils.DelayedStringRenderer(jsonutils.dumps,"},{"line_number":104,"context_line":"                                              routers, indent\u003d5))"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_74dcb712","line":101,"updated":"2016-08-27 17:42:18.000000000","message":"2 nits:\n1. In the debug message can you give a clue for the host, for example\nRouters return from host %s ... or whatever you think\n2. if you are changing this log message then please remove the space after the \\n","commit_id":"209883459c58b794d466a59d863e42cde9b0a0a2"},{"author":{"_account_id":8873,"name":"Assaf Muller","email":"amuller@redhat.com","username":"amuller"},"change_message_id":"fc909d7ea0dfac5f2f853d311da6bb4ccc4ae6ad","unresolved":false,"context_lines":[{"line_number":98,"context_line":"        if utils.is_extension_supported("},{"line_number":99,"context_line":"            self.plugin, constants.PORT_BINDING_EXT_ALIAS):"},{"line_number":100,"context_line":"            self._ensure_host_set_on_ports(context, host, routers)"},{"line_number":101,"context_line":"        LOG.debug(\"Routers returned to l3 agent %s:\\n %s\","},{"line_number":102,"context_line":"                  host,"},{"line_number":103,"context_line":"                  utils.DelayedStringRenderer(jsonutils.dumps,"},{"line_number":104,"context_line":"                                              routers, indent\u003d5))"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_3572f449","line":101,"in_reply_to":"fa7ab95a_74dcb712","updated":"2016-08-29 15:38:44.000000000","message":"Done.\n\nAlso, my personal opinion is that if a review contains solely nits then the reviewer should not use a -1, or should upload a new patchset him or herself.","commit_id":"209883459c58b794d466a59d863e42cde9b0a0a2"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"6da1f2acc13e04f0370d8c94f7ef110ce29f22e9","unresolved":false,"context_lines":[{"line_number":98,"context_line":"        if utils.is_extension_supported("},{"line_number":99,"context_line":"            self.plugin, constants.PORT_BINDING_EXT_ALIAS):"},{"line_number":100,"context_line":"            self._ensure_host_set_on_ports(context, host, routers)"},{"line_number":101,"context_line":"        LOG.debug(\"Routers returned to l3 agent on host %s:\\n%s\","},{"line_number":102,"context_line":"                  host,"},{"line_number":103,"context_line":"                  utils.DelayedStringRenderer(jsonutils.dumps,"},{"line_number":104,"context_line":"                                              routers, indent\u003d5))"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_fd55c7c1","line":101,"updated":"2016-08-29 18:50:27.000000000","message":"What, no dict? :)","commit_id":"16f5d50b526e8b53fc2bb81b1dd834a317001384"}]}