)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14885,"name":"Daniel Mellado","email":"dmellado@redhat.com","username":"daniel-mellado"},"change_message_id":"a2a7b8b33ebfe6f443ae066465566c8410e4261d","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     guo yunxian \u003cyunxian.guo@easystack.cn\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-09-27 19:42:49 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Trival: Remove redundant variable"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As I5b077f403e22ce1ff6833bc397afd21bf4d3e498, we can"},{"line_number":10,"context_line":"simply do self.***[\u0027id\u0027] instead of create a new varible. But"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a8bc5a2_41592934","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":6},"updated":"2016-09-27 12:43:35.000000000","message":"Trivial","commit_id":"aabb8ce9c7e39eed38c204ddb62917d176ca5b57"},{"author":{"_account_id":14885,"name":"Daniel Mellado","email":"dmellado@redhat.com","username":"daniel-mellado"},"change_message_id":"a2a7b8b33ebfe6f443ae066465566c8410e4261d","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Trival: Remove redundant variable"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As I5b077f403e22ce1ff6833bc397afd21bf4d3e498, we can"},{"line_number":10,"context_line":"simply do self.***[\u0027id\u0027] instead of create a new varible. But"},{"line_number":11,"context_line":"some other files are also the case. This patch changes these files"},{"line_number":12,"context_line":"to make the code consistent."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a8bc5a2_46b85f75","line":10,"range":{"start_line":10,"start_character":49,"end_line":10,"end_character":56},"updated":"2016-09-27 12:43:35.000000000","message":"small typo ;)","commit_id":"aabb8ce9c7e39eed38c204ddb62917d176ca5b57"}],"tempest/api/compute/servers/test_instance_actions.py":[{"author":{"_account_id":14885,"name":"Daniel Mellado","email":"dmellado@redhat.com","username":"daniel-mellado"},"change_message_id":"a2a7b8b33ebfe6f443ae066465566c8410e4261d","unresolved":false,"context_lines":[{"line_number":35,"context_line":"    def test_list_instance_actions(self):"},{"line_number":36,"context_line":"        # List actions of the provided server"},{"line_number":37,"context_line":"        self.client.reboot_server(self.server[\u0027id\u0027], type\u003d\u0027HARD\u0027)"},{"line_number":38,"context_line":"        waiters.wait_for_server_status(self.client, self.server[\u0027id\u0027], \u0027ACTIVE\u0027)"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"        body \u003d (self.client.list_instance_actions(self.server[\u0027id\u0027])"},{"line_number":41,"context_line":"                [\u0027instanceActions\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a8bc5a2_c12e198b","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":80},"updated":"2016-09-27 12:43:35.000000000","message":"This line became too long after the change ;)","commit_id":"aabb8ce9c7e39eed38c204ddb62917d176ca5b57"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"9e10fa0da5d5ce2de72cd2d171fce4cbeb4616a7","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    @classmethod"},{"line_number":29,"context_line":"    def resource_setup(cls):"},{"line_number":30,"context_line":"        super(InstanceActionsTestJSON, cls).resource_setup()"},{"line_number":31,"context_line":"        server \u003d cls.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":32,"context_line":"        cls.request_id \u003d server.response[\u0027x-compute-request-id\u0027]"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    @test.idempotent_id(\u002777ca5cc5-9990-45e0-ab98-1de8fead201a\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a8bc5a2_2511605e","line":31,"range":{"start_line":31,"start_character":8,"end_line":31,"end_character":9},"updated":"2016-09-28 04:26:18.000000000","message":"missing cls.\n\nAnyway why to keep the complete server object as a class variable if TCs only need the id?","commit_id":"a5b48ab1789746d43c000adbb86476cbaeda2755"},{"author":{"_account_id":22692,"name":"guo yunxian","email":"yunxian.guo@easystack.cn","username":"guoyunxian"},"change_message_id":"ee92937ed1fccef72e887467e4bd03de4a0df248","unresolved":false,"context_lines":[{"line_number":28,"context_line":"    @classmethod"},{"line_number":29,"context_line":"    def resource_setup(cls):"},{"line_number":30,"context_line":"        super(InstanceActionsTestJSON, cls).resource_setup()"},{"line_number":31,"context_line":"        server \u003d cls.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":32,"context_line":"        cls.request_id \u003d server.response[\u0027x-compute-request-id\u0027]"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    @test.idempotent_id(\u002777ca5cc5-9990-45e0-ab98-1de8fead201a\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a8bc5a2_54dc5f46","line":31,"range":{"start_line":31,"start_character":8,"end_line":31,"end_character":9},"in_reply_to":"5a8bc5a2_2511605e","updated":"2016-09-28 05:16:40.000000000","message":"Oh, I am sorry I forgot this cls. Thank for your review. \nAs https://review.openstack.org/#/c/376321/ said, we can use self.server[\u0027id\u0027] directly and no need to create a new variable named server_id. I do this to make the code consistent.","commit_id":"a5b48ab1789746d43c000adbb86476cbaeda2755"}]}