)]}'
{"tempest/api/compute/admin/test_aggregates.py":[{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"52bfbc9c8021dddc9359e9b913a2b673157d1c9f","unresolved":false,"context_lines":[{"line_number":105,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":106,"context_line":"        self.assertEqual(meta, body[\"metadata\"])"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"    @test.skip_because(bug\u003d\u00271274924\u0027)"},{"line_number":109,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":110,"context_line":"    def test_aggregate_create_update_metadata_update_name_and_zone(self):"},{"line_number":111,"context_line":"        # Create an aggregate and ensure its details are returned."}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWn%2F%2F%2FZs%3D","line":108,"updated":"2014-04-08 03:01:57.000000000","message":"IMHO we should remove the skip, I would prefer to wait until we get the main patch merged in nova than eventually merge this for nothing if the nova\u0027s one doesn\u0027t merge.\n(not that I think it\u0027s a wrong fix, it doesn\u0027t seems to me, but doesn\u0027t make sense either way).","commit_id":"83c5e8f0ef3edfcbf58f317a1c3305a79a107950"},{"author":{"_account_id":10247,"name":"Santiago Baldassin","email":"santiago.b.baldassin@intel.com","username":"santibaldassin"},"change_message_id":"3294c1aaa65343bba540b7c719cfb89c4dd8eece","unresolved":false,"context_lines":[{"line_number":105,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":106,"context_line":"        self.assertEqual(meta, body[\"metadata\"])"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"    @test.skip_because(bug\u003d\u00271274924\u0027)"},{"line_number":109,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":110,"context_line":"    def test_aggregate_create_update_metadata_update_name_and_zone(self):"},{"line_number":111,"context_line":"        # Create an aggregate and ensure its details are returned."}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAWn%2F%2F7sk%3D","line":108,"in_reply_to":"AAAAWn%2F%2F%2FZs%3D","updated":"2014-04-08 12:26:51.000000000","message":"I agree with you that we should wait until nova fix is merged. That\u0027s why I added the dependency in the commit description. This patch should not be +2 until that merge happen.","commit_id":"83c5e8f0ef3edfcbf58f317a1c3305a79a107950"},{"author":{"_account_id":8824,"name":"Sergey Murashov","email":"smurashov@mirantis.com","username":"SergeyMurashov"},"change_message_id":"0cf7dc2712b81a5e64f8ce422e393b3e3cc4206f","unresolved":false,"context_lines":[{"line_number":111,"context_line":"        # Create an aggregate and ensure its details are returned."},{"line_number":112,"context_line":"        aggregate_name \u003d data_utils.rand_name(self.aggregate_name_prefix)"},{"line_number":113,"context_line":"        az_name \u003d data_utils.rand_name(self.az_name_prefix)"},{"line_number":114,"context_line":"        resp, aggregate \u003d self.client.create_aggregate(name\u003daggregate_name,"},{"line_number":115,"context_line":"                                                       availability_zone\u003d"},{"line_number":116,"context_line":"                                                       az_name)"},{"line_number":117,"context_line":"        self.addCleanup(self.client.delete_aggregate, aggregate[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2F7yY%3D","line":114,"updated":"2014-04-09 11:52:16.000000000","message":"unused variable \"resp\"","commit_id":"3094242c458ec19360104e9a85952ca8362cf93a"},{"author":{"_account_id":10247,"name":"Santiago Baldassin","email":"santiago.b.baldassin@intel.com","username":"santibaldassin"},"change_message_id":"855446a88ea226173e69c90b6bedd7267fecdd76","unresolved":false,"context_lines":[{"line_number":111,"context_line":"        # Create an aggregate and ensure its details are returned."},{"line_number":112,"context_line":"        aggregate_name \u003d data_utils.rand_name(self.aggregate_name_prefix)"},{"line_number":113,"context_line":"        az_name \u003d data_utils.rand_name(self.az_name_prefix)"},{"line_number":114,"context_line":"        resp, aggregate \u003d self.client.create_aggregate(name\u003daggregate_name,"},{"line_number":115,"context_line":"                                                       availability_zone\u003d"},{"line_number":116,"context_line":"                                                       az_name)"},{"line_number":117,"context_line":"        self.addCleanup(self.client.delete_aggregate, aggregate[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAW3%2F%2F6bo%3D","line":114,"in_reply_to":"AAAAW3%2F%2F7yY%3D","updated":"2014-04-09 14:36:17.000000000","message":"Done","commit_id":"3094242c458ec19360104e9a85952ca8362cf93a"},{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"021d40f2048e4d1c25b038c9cc5f79ce0d44bb81","unresolved":false,"context_lines":[{"line_number":105,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":106,"context_line":"        self.assertEqual(meta, body[\"metadata\"])"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"    @test.skip_because(bug\u003d\u00271274924\u0027)"},{"line_number":109,"context_line":"    @test.attr(type\u003d\u0027gate\u0027)"},{"line_number":110,"context_line":"    def test_aggregate_create_update_metadata_update_name_and_zone(self):"},{"line_number":111,"context_line":"        # Create an aggregate and ensure its details are returned."}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXH%2F%2FybA%3D","line":108,"updated":"2014-04-17 13:27:39.000000000","message":"Santiago Baldassin, Apr 8,  said:\n\"I agree with you that we should wait until nova fix is merged. That\u0027s why I added the dependency in the commit description. This patch should not be +2 until that merge happen.\"\n\nSorry, but that\u0027s exactly what I meant, if this is not supposed to merge until the nova patch gets in I don\u0027t see a point to skip it here, it will only allow us to accidentally merge it.\nSo I would rather prefer to see Jenkins -1-ing this right now and recheck it later when nova\u0027s one is merged than skip and eventualy get a patch that never trully passed through the gate merged.","commit_id":"115f55dff98e1b85c057feb1047385899ddbcaab"}]}