)]}'
{"neutron/tests/functional/agent/test_l2_lb_agent.py":[{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"2802a99ef476c1f85621235cc825e592d61feab9","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    def test_vlan_QinQ_subinterfaces(self):"},{"line_number":101,"context_line":"        # the trunk model does not support this right now, but this is to"},{"line_number":102,"context_line":"        # the plumber on the agent side doesn\u0027t explode in their presence"},{"line_number":103,"context_line":"        # in case an operator does something fancy or we have a race where"},{"line_number":104,"context_line":"        # a trunk\u0027s parent port is converted to a subport while the agent"},{"line_number":105,"context_line":"        # is offline."}],"source_content_type":"text/x-python","patch_set":2,"id":"5a3905b3_a6292ee8","line":102,"range":{"start_line":102,"start_character":10,"end_line":102,"end_character":73},"updated":"2017-01-23 23:33:27.000000000","message":"I think this line can be improved even more. I think what is meant is the following:\n\nensure the plumber on the agent side doesn\u0027t explode in their presence\n\nDo you mind fixing it?","commit_id":"1be563c8c8ba23e92ffcb11b3007f0f33929bb69"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"04977f83d0a34a59c7936933e6a2c5d5146482eb","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    def test_vlan_QinQ_subinterfaces(self):"},{"line_number":101,"context_line":"        # the trunk model does not support this right now, but this is to"},{"line_number":102,"context_line":"        # ensure the plumber on the agent side doesn\u0027t explode in their presence"},{"line_number":103,"context_line":"        # in case an operator does something fancy or we have a race where"},{"line_number":104,"context_line":"        # a trunk\u0027s parent port is converted to a subport while the agent"},{"line_number":105,"context_line":"        # is offline."}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_a6ebc511","line":102,"range":{"start_line":102,"start_character":0,"end_line":102,"end_character":80},"updated":"2017-01-24 15:56:00.000000000","message":"The line seems too long","commit_id":"117eb282ea86c2248af07dea75b654abd0a5fc74"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"0a22786793362e827863c6e6dc7c39e4cba56a5f","unresolved":false,"context_lines":[{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    def test_vlan_QinQ_subinterfaces(self):"},{"line_number":101,"context_line":"        # the trunk model does not support this right now, but this is to"},{"line_number":102,"context_line":"        # ensure the plumber on the agent side doesn\u0027t explode in their presence"},{"line_number":103,"context_line":"        # in case an operator does something fancy or we have a race where"},{"line_number":104,"context_line":"        # a trunk\u0027s parent port is converted to a subport while the agent"},{"line_number":105,"context_line":"        # is offline."}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_b50a09d0","line":102,"range":{"start_line":102,"start_character":0,"end_line":102,"end_character":80},"in_reply_to":"5a3905b3_a6ebc511","updated":"2017-01-24 17:09:51.000000000","message":"I should have stated in my previous comment. You will have to re-distribute words in lines in this comment block, making sure no line is longer than 79 characters. That is why you have this failure in the pep8 job: http://logs.openstack.org/27/423727/3/check/gate-neutron-pep8-ubuntu-xenial/b08b0d8/console.html#_2017-01-24_00_32_14_415634","commit_id":"117eb282ea86c2248af07dea75b654abd0a5fc74"}]}