)]}'
{"networking_odl/ml2/port_status_update.py":[{"author":{"_account_id":27068,"name":"Trinh Nguyen","email":"dangtrinhnt@gmail.com","username":"dangtrinhnt"},"change_message_id":"212fb143915346ec11c907106917e5713dff9e6c","unresolved":false,"context_lines":[{"line_number":122,"context_line":"        for port in ports:"},{"line_number":123,"context_line":"            port_id \u003d port[\"id\"]"},{"line_number":124,"context_line":"            response \u003d client.get(port_id)"},{"line_number":125,"context_line":"            if response.status_code !\u003d 200:"},{"line_number":126,"context_line":"                LOG.warning(\"Non-200 response code %s\", str(response))"},{"line_number":127,"context_line":"                continue"},{"line_number":128,"context_line":"            odl_status \u003d response.json()[\u0027port\u0027][0][\u0027status\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf659307_b63e1dd8","line":125,"range":{"start_line":125,"start_character":39,"end_line":125,"end_character":42},"updated":"2018-03-27 11:15:19.000000000","message":"I suggest using six.moves.http_client.OK instead of int value here for better future maintenance.","commit_id":"239409b844097608e8e81a4f0f04f5b5f5d2bd94"},{"author":{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},"change_message_id":"3dd5d8b303b447c285951a80cf19e610a433a7aa","unresolved":false,"context_lines":[{"line_number":122,"context_line":"        for port in ports:"},{"line_number":123,"context_line":"            port_id \u003d port[\"id\"]"},{"line_number":124,"context_line":"            response \u003d client.get(port_id)"},{"line_number":125,"context_line":"            if response.status_code !\u003d 200:"},{"line_number":126,"context_line":"                LOG.warning(\"Non-200 response code %s\", str(response))"},{"line_number":127,"context_line":"                continue"},{"line_number":128,"context_line":"            odl_status \u003d response.json()[\u0027port\u0027][0][\u0027status\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf659307_f131ffa3","line":125,"range":{"start_line":125,"start_character":39,"end_line":125,"end_character":42},"in_reply_to":"bf659307_b63e1dd8","updated":"2018-03-27 11:32:40.000000000","message":"It’s unrelated to the change and therefore a -1 is not expexted. Could you please remove the -1? and if you want you are welcome to submit a patch with such change.","commit_id":"239409b844097608e8e81a4f0f04f5b5f5d2bd94"}],"networking_odl/tests/functional/test_bgpvpn.py":[{"author":{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},"change_message_id":"1129c8a4c472697a308ffb1fac1d9b91d7b9fccb","unresolved":false,"context_lines":[{"line_number":97,"context_line":"            net_id1 \u003d net1[\u0027network\u0027][\u0027id\u0027]"},{"line_number":98,"context_line":"            net_id2 \u003d net2[\u0027network\u0027][\u0027id\u0027]"},{"line_number":99,"context_line":"            bgpvpn_id \u003d bgpvpn[\u0027bgpvpn\u0027][\u0027id\u0027]"},{"line_number":100,"context_line":"            with (self.assoc_net(bgpvpn_id, net_id1),"},{"line_number":101,"context_line":"                  self.assoc_net(bgpvpn_id, net_id2)):"},{"line_number":102,"context_line":"                self._assert_networks_associated([net_id1, net_id2], bgpvpn)"},{"line_number":103,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"bf659307_9609992d","line":100,"updated":"2018-03-27 10:43:52.000000000","message":"Oh, I didn\u0027t see this small detail...\n\nYou can\u0027t do \n\nwith (foo(), \n      bar()):\n    pass\n\nAs you are then passing a tuple of the result of foo() and bar() to the with. Therefore, you need to do:\n\nwith foo(), \\\n     bar(): \n    pass","commit_id":"6d96fd5e16877d8371c0bf5bd3c4f2e7c1d90773"},{"author":{"_account_id":26507,"name":"Michel Peterson","email":"michel@redhat.com","username":"mpeterson"},"change_message_id":"1129c8a4c472697a308ffb1fac1d9b91d7b9fccb","unresolved":false,"context_lines":[{"line_number":158,"context_line":"            router_id \u003d router[\u0027router\u0027][\u0027id\u0027]"},{"line_number":159,"context_line":"            net_id \u003d net1[\u0027network\u0027][\u0027id\u0027]"},{"line_number":160,"context_line":"            bgpvpn_id \u003d bgpvpn[\u0027bgpvpn\u0027][\u0027id\u0027]"},{"line_number":161,"context_line":"            with (self.assoc_router(bgpvpn_id, router_id),"},{"line_number":162,"context_line":"                  self.assoc_net(bgpvpn_id, net_id)):"},{"line_number":163,"context_line":"                response \u003d self.get_odl_resource(odl_const.ODL_BGPVPN, bgpvpn)"},{"line_number":164,"context_line":"                self.assertItemsEqual([router_id],"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf659307_b60cdd3b","line":161,"updated":"2018-03-27 10:43:52.000000000","message":"Same","commit_id":"6d96fd5e16877d8371c0bf5bd3c4f2e7c1d90773"}]}