)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"2b1cade89ca5e77db52f1cab0bed1c5b3b2c3640","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Move some router tests from Neutron to Tempest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Related-Bug: #1552960"},{"line_number":10,"context_line":"Depends-On: Ifb53037d92b247b01e4408f07aa8a4933be68587"},{"line_number":11,"context_line":"Change-Id: I12e69829dcc2e978a813e660364010b47d50f06c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9abb7d3a_a3bab2be","line":10,"updated":"2016-05-27 07:41:16.000000000","message":"IMO Depends-On should be in the opposite direction.\nduplicated tests is better than less coverage.\nhow do you think?","commit_id":"63655a806fd50afab2931209888be05338257889"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"02169f71cc13cb9d0592c2d7f06ebc82e201bea5","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Move some router tests from Neutron to Tempest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Related-Bug: #1552960"},{"line_number":10,"context_line":"Depends-On: Ifb53037d92b247b01e4408f07aa8a4933be68587"},{"line_number":11,"context_line":"Change-Id: I12e69829dcc2e978a813e660364010b47d50f06c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3a98d1f4_83ca24a0","line":10,"updated":"2016-09-28 21:32:27.000000000","message":"This depends on is also abandonded from what I can tell, so lets drop this, then I\u0027m fine with this","commit_id":"63655a806fd50afab2931209888be05338257889"},{"author":{"_account_id":8873,"name":"Assaf Muller","email":"amuller@redhat.com","username":"amuller"},"change_message_id":"47c78b2f309052b0ca8e1235ed4e34217508d172","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Move some router tests from Neutron to Tempest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Related-Bug: #1552960"},{"line_number":10,"context_line":"Depends-On: Ifb53037d92b247b01e4408f07aa8a4933be68587"},{"line_number":11,"context_line":"Change-Id: I12e69829dcc2e978a813e660364010b47d50f06c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"9abb7d3a_b15bea2d","line":10,"in_reply_to":"9abb7d3a_a3bab2be","updated":"2016-06-02 22:31:44.000000000","message":"I agree.","commit_id":"63655a806fd50afab2931209888be05338257889"}],"tempest/api/network/test_routers.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"10d541a921ba7691da57f7f724df2228b33d88c0","unresolved":false,"context_lines":[{"line_number":187,"context_line":"        self.routers_client.add_router_interface("},{"line_number":188,"context_line":"            router2[\u0027id\u0027],"},{"line_number":189,"context_line":"            port_id\u003dport2[\u0027id\u0027])"},{"line_number":190,"context_line":"        self.addCleanup(self._remove_router_interface_with_port_id,"},{"line_number":191,"context_line":"                        router1[\u0027id\u0027], port1[\u0027id\u0027])"},{"line_number":192,"context_line":"        self.addCleanup(self._remove_router_interface_with_port_id,"},{"line_number":193,"context_line":"                        router2[\u0027id\u0027], port2[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":5,"id":"dab17558_44a04c42","line":190,"updated":"2016-05-17 15:07:38.000000000","message":"Move this right after resource creation, so move this L187.","commit_id":"2fd84e669d49912d92e2c2c98e10221b2e0252b5"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"10d541a921ba7691da57f7f724df2228b33d88c0","unresolved":false,"context_lines":[{"line_number":191,"context_line":"                        router1[\u0027id\u0027], port1[\u0027id\u0027])"},{"line_number":192,"context_line":"        self.addCleanup(self._remove_router_interface_with_port_id,"},{"line_number":193,"context_line":"                        router2[\u0027id\u0027], port2[\u0027id\u0027])"},{"line_number":194,"context_line":"        body \u003d self.ports_client.show_port(port1[\u0027id\u0027])"},{"line_number":195,"context_line":"        port_show1 \u003d body[\u0027port\u0027]"},{"line_number":196,"context_line":"        body \u003d self.ports_client.show_port(port2[\u0027id\u0027])"},{"line_number":197,"context_line":"        port_show2 \u003d body[\u0027port\u0027]"}],"source_content_type":"text/x-python","patch_set":5,"id":"dab17558_044f6492","line":194,"updated":"2016-05-17 15:07:38.000000000","message":"Just do\n  port_show1 \u003d self.ports_client.show_port(port1[\u0027id\u0027])[\u0027port\u0027]\n\nI think it doesn\u0027t hurt readability and it would make the test a tiny bit more straight to the point.","commit_id":"2fd84e669d49912d92e2c2c98e10221b2e0252b5"}]}