)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"ee3dcbef0f3c910e97abddc4ef5e90e55d6b01a4","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2014-07-10 15:35:36 +0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Avoid involve other instances"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I51494c9f47745810583f682caadcd8772805733a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baada198_cd949446","line":8,"updated":"2014-07-10 13:21:43.000000000","message":"Can you improve this message? \nI mean, I understood your intention, but only looking into the code, the commit message is really vague. does this fix a bug?","commit_id":"d971ec7f978b99842d993f01255df68041f98cdb"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"94e81a7d1720ad36ac63b82eadde07cb92b789f5","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2014-07-10 15:35:36 +0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Avoid involve other instances"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I51494c9f47745810583f682caadcd8772805733a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baada198_a8d4aebd","line":8,"in_reply_to":"baada198_cd949446","updated":"2014-07-11 06:35:04.000000000","message":"Done","commit_id":"d971ec7f978b99842d993f01255df68041f98cdb"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"472a87c5fc35978317ad4a7f3d8d5ec9cdcc64b1","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Avoid conflicts with other tests instances"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Avoid conflicts with other instances that exist, for example when weren\u0027t deleted in other tests. Just filter out them in test that check instances amount"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I51494c9f47745810583f682caadcd8772805733a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"baada198_1a4bfc2b","line":10,"updated":"2014-07-11 06:24:50.000000000","message":"Commit msg subsequent line should be wrapped in 72 char. https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"42ec89bf8a20b0da7d22075bce9e20ed18b19640"}],"tempest/api/compute/servers/test_list_server_filters.py":[{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"ee3dcbef0f3c910e97abddc4ef5e90e55d6b01a4","unresolved":false,"context_lines":[{"line_number":115,"context_line":"    def test_list_servers_filter_by_server_status(self):"},{"line_number":116,"context_line":"        # Filter the list of servers by server status"},{"line_number":117,"context_line":"        params \u003d {\u0027status\u0027: \u0027active\u0027}"},{"line_number":118,"context_line":"        resp, body \u003d self.client.list_servers(params)"},{"line_number":119,"context_line":"        servers \u003d body[\u0027servers\u0027]"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        self.assertIn(self.s1[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], servers))"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_4d8fa423","line":118,"updated":"2014-07-10 13:21:43.000000000","message":"I guess the same piece of code should be included here as well","commit_id":"d971ec7f978b99842d993f01255df68041f98cdb"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"5343a45d656f1ff7d7954ff33887e22ad6c27535","unresolved":false,"context_lines":[{"line_number":115,"context_line":"    def test_list_servers_filter_by_server_status(self):"},{"line_number":116,"context_line":"        # Filter the list of servers by server status"},{"line_number":117,"context_line":"        params \u003d {\u0027status\u0027: \u0027active\u0027}"},{"line_number":118,"context_line":"        resp, body \u003d self.client.list_servers(params)"},{"line_number":119,"context_line":"        servers \u003d body[\u0027servers\u0027]"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"        self.assertIn(self.s1[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], servers))"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_689e16ba","line":118,"in_reply_to":"baada198_4d8fa423","updated":"2014-07-10 13:46:04.000000000","message":"not necessary, the server statuses AMOUNT is checked in one test only (test_list_servers_detailed_filter_by_server_status). I just want to filter out all other instances that may got stuck in other tests","commit_id":"d971ec7f978b99842d993f01255df68041f98cdb"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"ea4d1b9a819d23fa4f3f6f6b1408ac0487288eaf","unresolved":false,"context_lines":[{"line_number":203,"context_line":"        params \u003d {\u0027status\u0027: \u0027active\u0027}"},{"line_number":204,"context_line":"        resp, body \u003d self.client.list_servers_with_detail(params)"},{"line_number":205,"context_line":"        all_servers \u003d body[\u0027servers\u0027]"},{"line_number":206,"context_line":"        servers \u003d [i for i in all_servers"},{"line_number":207,"context_line":"                   if self.__class__.__name__ in i[\u0027name\u0027]]"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"        self.assertIn(self.s1[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], servers))"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_456e9402","line":206,"updated":"2014-07-10 15:56:11.000000000","message":"This is not very nice and it\u0027s fragile. There\u0027s no constraint that the server name is equal to the class name, this could change in the future","commit_id":"59c0cef7d69c5e55fa1b1f561bbb3efcb14c0ace"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"ea4d1b9a819d23fa4f3f6f6b1408ac0487288eaf","unresolved":false,"context_lines":[{"line_number":209,"context_line":"        self.assertIn(self.s1[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], servers))"},{"line_number":210,"context_line":"        self.assertIn(self.s2[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], servers))"},{"line_number":211,"context_line":"        self.assertIn(self.s3[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], servers))"},{"line_number":212,"context_line":"        self.assertEqual([\u0027ACTIVE\u0027] * 3, [x[\u0027status\u0027] for x in servers])"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":215,"context_line":"    def test_list_servers_filtered_by_name_wildcard(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_e5a5e847","line":212,"updated":"2014-07-10 15:56:11.000000000","message":"I think you should do :\n\nfor id in [self.s1[\u0027id\u0027], self.s2[\u0027id\u0027], self.s3[\u0027id\u0027]]:\n  self.assertEqual(\u0027ACTIVE\u0027, server[id][\u0027status\u0027])\n\nOr something like this.","commit_id":"59c0cef7d69c5e55fa1b1f561bbb3efcb14c0ace"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"16d17a6fefcadca1924e67e84f2e0ff3a0c471de","unresolved":false,"context_lines":[{"line_number":209,"context_line":"        self.assertIn(self.s1[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], servers))"},{"line_number":210,"context_line":"        self.assertIn(self.s2[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], servers))"},{"line_number":211,"context_line":"        self.assertIn(self.s3[\u0027id\u0027], map(lambda x: x[\u0027id\u0027], servers))"},{"line_number":212,"context_line":"        self.assertEqual([\u0027ACTIVE\u0027] * 3, [x[\u0027status\u0027] for x in servers])"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":215,"context_line":"    def test_list_servers_filtered_by_name_wildcard(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_d62098f9","line":212,"in_reply_to":"baada198_e5a5e847","updated":"2014-07-10 17:13:55.000000000","message":"you\u0027re right, the problem is in assert, I\u0027ll rewrite it","commit_id":"59c0cef7d69c5e55fa1b1f561bbb3efcb14c0ace"}]}