)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"06629a5f6d920a8d4d51a8fc616390b8dedf5b69","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix pep8 errors found by hacking 2.0.0"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The new hacking release found some pep8 errors, fix them."},{"line_number":10,"context_line":"Unfortunately, moving to 2.0.0 pulled a thread with other"},{"line_number":11,"context_line":"dependencies causing issues, so for now pin it below"},{"line_number":12,"context_line":"1.2.0 to get the gate functioning again."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_d9a8dfdb","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":57},"updated":"2019-12-14 19:11:12.000000000","message":"What\u0027s the problem with v2.0.0? I tried to install it and pep8 is passing without any problem.\n\nMaybe something related to other CI jobs...","commit_id":"36727e3463ca94d25cac1ab715097c778b3f7736"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"c5921c1bdbe414952676486704015d401ae28800","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix pep8 errors found by hacking 2.0.0"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The new hacking release found some pep8 errors, fix them."},{"line_number":10,"context_line":"Unfortunately, moving to 2.0.0 pulled a thread with other"},{"line_number":11,"context_line":"dependencies causing issues, so for now pin it below"},{"line_number":12,"context_line":"1.2.0 to get the gate functioning again."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_25421a0f","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":57},"in_reply_to":"3fa7e38b_39f25307","updated":"2019-12-16 15:21:23.000000000","message":"Yes, and even though flake8 3.0 added a compat shim in flake8.api.legacy my quick change showed we\u0027ll need other library updates to support 2.0.0","commit_id":"36727e3463ca94d25cac1ab715097c778b3f7736"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"744e4c3deea99e1b1f2a10d0a85ef22422053c71","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix pep8 errors found by hacking 2.0.0"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The new hacking release found some pep8 errors, fix them."},{"line_number":10,"context_line":"Unfortunately, moving to 2.0.0 pulled a thread with other"},{"line_number":11,"context_line":"dependencies causing issues, so for now pin it below"},{"line_number":12,"context_line":"1.2.0 to get the gate functioning again."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_39f25307","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":57},"in_reply_to":"3fa7e38b_d9a8dfdb","updated":"2019-12-14 19:20:19.000000000","message":"Never mind, I found the error:\n\nFile \"/home/zuul/src/opendev.org/openstack/neutron/neutron/tests/unit/hacking/test_checks.py\", line 15, in \u003cmodule\u003e\n\nI\u0027ll change my vote to +2+W, we need to patch now.","commit_id":"36727e3463ca94d25cac1ab715097c778b3f7736"}],"neutron/tests/unit/common/ovn/test_hash_ring_manager.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"b54b4f7bb6e436e63e06c54dc907432279033fc3","unresolved":false,"context_lines":[{"line_number":100,"context_line":"        # hash_dict_after_rebalance \u003d {another_host_node: \u0027fake-uuid\u0027,"},{"line_number":101,"context_line":"        #                              another_host_node: \u0027fake-uuid-0\u0027,"},{"line_number":102,"context_line":"        #                              another_host_node: \u0027fake-uuid-ABCDE\u0027}"},{"line_number":103,"context_line":"        # self._verify_hashes(hash_dict_after_rebalance)"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"        # Now touch the nodes so they appear active again"},{"line_number":106,"context_line":"        db_hash_ring.touch_nodes_from_host("}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_67d10c57","line":103,"updated":"2019-12-13 21:11:17.000000000","message":"This obviously needs some more work, I just didn\u0027t have time to investigate further.  I first wanted to make sure the hashring code didn\u0027t have some bug, but guessing it\u0027s just this test.","commit_id":"7eb8f7215524db3591c190953834b6dd046c7956"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"06629a5f6d920a8d4d51a8fc616390b8dedf5b69","unresolved":false,"context_lines":[{"line_number":96,"context_line":"                         list(self.hash_ring_manager._hash_ring.nodes.keys()))"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        # Now only \"another_host_node\" is alive, all values should hash to it"},{"line_number":99,"context_line":"        # NOTE(haleyb) needs further work, need to ask lucasgomes"},{"line_number":100,"context_line":"        # hash_dict_after_rebalance \u003d {another_host_node: \u0027fake-uuid\u0027,"},{"line_number":101,"context_line":"        #                              another_host_node: \u0027fake-uuid-0\u0027,"},{"line_number":102,"context_line":"        #                              another_host_node: \u0027fake-uuid-ABCDE\u0027}"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_1978b77c","line":99,"updated":"2019-12-14 19:11:12.000000000","message":"I\u0027ll push a patch on top of this one solving this problem. Instead of building a dict with {node:uuid}, this should be {uuid:node}","commit_id":"36727e3463ca94d25cac1ab715097c778b3f7736"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ba443cd16a47a4f224a4a02bb522c128af386b56","unresolved":false,"context_lines":[{"line_number":96,"context_line":"                         list(self.hash_ring_manager._hash_ring.nodes.keys()))"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        # Now only \"another_host_node\" is alive, all values should hash to it"},{"line_number":99,"context_line":"        # NOTE(haleyb) needs further work, need to ask lucasgomes"},{"line_number":100,"context_line":"        # hash_dict_after_rebalance \u003d {another_host_node: \u0027fake-uuid\u0027,"},{"line_number":101,"context_line":"        #                              another_host_node: \u0027fake-uuid-0\u0027,"},{"line_number":102,"context_line":"        #                              another_host_node: \u0027fake-uuid-ABCDE\u0027}"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_59e38f3b","line":99,"in_reply_to":"3fa7e38b_1978b77c","updated":"2019-12-14 19:15:54.000000000","message":"https://review.opendev.org/#/c/699083/","commit_id":"36727e3463ca94d25cac1ab715097c778b3f7736"}]}