)]}'
{"tempest/api/compute/admin/test_flavors_extra_specs.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"ac8a2eda344eeb2a6a242b838891e1486e6a559d","unresolved":false,"context_lines":[{"line_number":92,"context_line":"                                                          \"key1\")"},{"line_number":93,"context_line":"        self.admin_flavors_client.unset_flavor_extra_spec(self.flavor[\u0027id\u0027],"},{"line_number":94,"context_line":"                                                          \"key2\")"},{"line_number":95,"context_line":"        get_body \u003d (self.admin_flavors_client.list_flavor_extra_specs("},{"line_number":96,"context_line":"            self.flavor[\u0027id\u0027])[\u0027extra_specs\u0027])"},{"line_number":97,"context_line":"        self.assertEmpty(get_body)"},{"line_number":98,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1f013ff3_4739813f","line":95,"updated":"2017-05-13 08:46:41.000000000","message":"You don\u0027t need the extra parenthesis.","commit_id":"99d0529c8df07804db439985c6dfbdc2c8253ab4"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"ad0b3060cc9e7cf428880fabc20f3086f0fd9021","unresolved":false,"context_lines":[{"line_number":71,"context_line":"            self.flavor[\u0027id\u0027], **specs)[\u0027extra_specs\u0027]"},{"line_number":72,"context_line":"        self.assertEqual(set_body, specs)"},{"line_number":73,"context_line":"        # GET extra specs and verify"},{"line_number":74,"context_line":"        get_body \u003d (self.admin_flavors_client.list_flavor_extra_specs("},{"line_number":75,"context_line":"            self.flavor[\u0027id\u0027])[\u0027extra_specs\u0027])"},{"line_number":76,"context_line":"        self.assertEqual(get_body, specs)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        # UPDATE the value of the extra specs key1"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f013ff3_24199dd6","line":75,"range":{"start_line":74,"start_character":8,"end_line":75,"end_character":46},"updated":"2017-05-16 11:49:34.000000000","message":"There still has some extra parenthesis in this test, could you remove them in the patch, :)","commit_id":"93d7011c90255bca6bf7d47b9e0c0b971c9552c7"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"fadb403a913308a0b273d1cd73a32b6c4d519f90","unresolved":false,"context_lines":[{"line_number":71,"context_line":"            self.flavor[\u0027id\u0027], **specs)[\u0027extra_specs\u0027]"},{"line_number":72,"context_line":"        self.assertEqual(set_body, specs)"},{"line_number":73,"context_line":"        # GET extra specs and verify"},{"line_number":74,"context_line":"        get_body \u003d (self.admin_flavors_client.list_flavor_extra_specs("},{"line_number":75,"context_line":"            self.flavor[\u0027id\u0027])[\u0027extra_specs\u0027])"},{"line_number":76,"context_line":"        self.assertEqual(get_body, specs)"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        # UPDATE the value of the extra specs key1"}],"source_content_type":"text/x-python","patch_set":2,"id":"ff0f0b1f_f98fc52b","line":75,"range":{"start_line":74,"start_character":8,"end_line":75,"end_character":46},"in_reply_to":"1f013ff3_24199dd6","updated":"2017-05-19 14:02:34.000000000","message":"Changing this in this patch would be an unrelated change, like the one in L86 :)\nI don\u0027t think it\u0027s worth re-spinning for this though. \nCould you propose a follow-up change to cleanup extra parenthesis?","commit_id":"93d7011c90255bca6bf7d47b9e0c0b971c9552c7"},{"author":{"_account_id":23081,"name":"Jeremy Zhang","email":"zhangjun_inspur@163.com","username":"jeremy.zhang"},"change_message_id":"ad0b3060cc9e7cf428880fabc20f3086f0fd9021","unresolved":false,"context_lines":[{"line_number":101,"context_line":"        specs \u003d {\"key1\": \"value1\", \"key2\": \"value2\"}"},{"line_number":102,"context_line":"        self.admin_flavors_client.set_flavor_extra_spec(self.flavor[\u0027id\u0027],"},{"line_number":103,"context_line":"                                                        **specs)"},{"line_number":104,"context_line":"        body \u003d (self.flavors_client.list_flavor_extra_specs("},{"line_number":105,"context_line":"            self.flavor[\u0027id\u0027])[\u0027extra_specs\u0027])"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"        for key in specs:"},{"line_number":108,"context_line":"            self.assertEqual(body[key], specs[key])"}],"source_content_type":"text/x-python","patch_set":2,"id":"1f013ff3_04358146","line":105,"range":{"start_line":104,"start_character":8,"end_line":105,"end_character":46},"updated":"2017-05-16 11:49:34.000000000","message":"ditto","commit_id":"93d7011c90255bca6bf7d47b9e0c0b971c9552c7"}]}