)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"8c7ea406539505a3473ab5cdf425429f8e68ee9d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix test_server_metadata testcases"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change the code to tempest style as other classes."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Remove testcase setup function."},{"line_number":12,"context_line":"Use addCleanup for set_server_metadata"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5a8bc5a2_02cada1a","line":9,"updated":"2016-09-27 05:11:57.000000000","message":"I don\u0027t understand why this is better. Could you write the reason in this commit message?","commit_id":"2117e154466404096e24f3166a36015c029e5b4c"}],"tempest/api/compute/servers/test_server_metadata.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"8c7ea406539505a3473ab5cdf425429f8e68ee9d","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        server \u003d cls.create_test_server(metadata\u003d{}, wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":32,"context_line":"        cls.server_id \u003d server[\u0027id\u0027]"},{"line_number":33,"context_line":"        cls.meta \u003d {\u0027key1\u0027: \u0027value1\u0027, \u0027key2\u0027: \u0027value2\u0027}"},{"line_number":34,"context_line":"        cls.client.set_server_metadata(cls.server_id, cls.meta)[\u0027metadata\u0027]"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    @test.idempotent_id(\u0027479da087-92b3-4dcf-aeb3-fd293b2d14ce\u0027)"},{"line_number":37,"context_line":"    def test_list_server_metadata(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a8bc5a2_22cd1631","line":34,"updated":"2016-09-27 05:11:57.000000000","message":"why do we need to access [\u0027metadata\u0027] here??","commit_id":"2117e154466404096e24f3166a36015c029e5b4c"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"5f73bb924debd34190edc3cdbabf652d082f3a9c","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        server \u003d cls.create_test_server(metadata\u003d{}, wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":32,"context_line":"        cls.server_id \u003d server[\u0027id\u0027]"},{"line_number":33,"context_line":"        cls.meta \u003d {\u0027key1\u0027: \u0027value1\u0027, \u0027key2\u0027: \u0027value2\u0027}"},{"line_number":34,"context_line":"        cls.client.set_server_metadata(cls.server_id, cls.meta)[\u0027metadata\u0027]"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    @test.idempotent_id(\u0027479da087-92b3-4dcf-aeb3-fd293b2d14ce\u0027)"},{"line_number":37,"context_line":"    def test_list_server_metadata(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a8bc5a2_c8ac918e","line":34,"in_reply_to":"5a8bc5a2_22cd1631","updated":"2016-09-27 06:19:39.000000000","message":"I will remove it  , Thank you","commit_id":"2117e154466404096e24f3166a36015c029e5b4c"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"fae01ce74f491c3c6463046e3780cfa37759bb2f","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        # Verify the expected metadata items are in the list"},{"line_number":43,"context_line":"        expected \u003d {\u0027key1\u0027: \u0027value1\u0027, \u0027key2\u0027: \u0027value2\u0027}"},{"line_number":44,"context_line":"        self.assertEqual(expected, resp_metadata)"},{"line_number":45,"context_line":"        self.addCleanup(self.client.set_server_metadata,"},{"line_number":46,"context_line":"                        self.server_id, self.meta)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    @test.idempotent_id(\u0027211021f6-21de-4657-a68f-908878cfe251\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a8bc5a2_d0119753","line":45,"updated":"2016-09-27 08:19:48.000000000","message":"There\u0027s no need for this. The test is \"readonly\" on the server metadata, no cleanup to so since no action is actually performed.","commit_id":"5b48ac7acf5fcb7fba1e46af0361a341994af779"}]}