)]}'
{"tempest/api/compute/servers/test_server_volume_actions.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"2b2f963c466fc1174dd0bdac06c9a688f24d795e","unresolved":false,"context_lines":[{"line_number":32,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"class ServerVolumeActionsTestJSON(base.BaseV2ComputeTest):"},{"line_number":36,"context_line":"    run_ssh \u003d CONF.validation.run_validation"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_1babee75","line":35,"updated":"2016-05-18 20:46:16.000000000","message":"No need to name your class with JSON, will only do JSON nowadays.","commit_id":"21cf53eeeeae2e9d83a0638c27b932595610ccab"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"a4b7902788c7f1113ecf7b5a9f41dbfa3aff104e","unresolved":false,"context_lines":[{"line_number":32,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"class ServerVolumeActionsTestJSON(base.BaseV2ComputeTest):"},{"line_number":36,"context_line":"    run_ssh \u003d CONF.validation.run_validation"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_e8857059","line":35,"in_reply_to":"bab6814e_1babee75","updated":"2016-05-19 07:33:04.000000000","message":"Done","commit_id":"21cf53eeeeae2e9d83a0638c27b932595610ccab"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"2b2f963c466fc1174dd0bdac06c9a688f24d795e","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"class ServerVolumeActionsTestJSON(base.BaseV2ComputeTest):"},{"line_number":36,"context_line":"    run_ssh \u003d CONF.validation.run_validation"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def setUp(self):"},{"line_number":39,"context_line":"        super(ServerVolumeActionsTestJSON, self).setUp()"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_fbca6a4e","line":36,"updated":"2016-05-18 20:46:16.000000000","message":"Is this used ?","commit_id":"21cf53eeeeae2e9d83a0638c27b932595610ccab"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"a4b7902788c7f1113ecf7b5a9f41dbfa3aff104e","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"class ServerVolumeActionsTestJSON(base.BaseV2ComputeTest):"},{"line_number":36,"context_line":"    run_ssh \u003d CONF.validation.run_validation"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def setUp(self):"},{"line_number":39,"context_line":"        super(ServerVolumeActionsTestJSON, self).setUp()"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_28801846","line":36,"in_reply_to":"bab6814e_fbca6a4e","updated":"2016-05-19 07:33:04.000000000","message":"Not at the moment so I\u0027ll remove it.","commit_id":"21cf53eeeeae2e9d83a0638c27b932595610ccab"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"2b2f963c466fc1174dd0bdac06c9a688f24d795e","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        super(ServerVolumeActionsTestJSON, self).setUp()"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        # Use a volume backed instance for these tests."},{"line_number":42,"context_line":"        server \u003d self.create_test_server(volume_backed\u003dTrue,"},{"line_number":43,"context_line":"                                         wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":44,"context_line":"        self.server_id \u003d server[\u0027id\u0027]"},{"line_number":45,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_1bc64e67","line":42,"updated":"2016-05-18 20:46:16.000000000","message":"You create a server here.","commit_id":"21cf53eeeeae2e9d83a0638c27b932595610ccab"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"2b2f963c466fc1174dd0bdac06c9a688f24d795e","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    @classmethod"},{"line_number":51,"context_line":"    def setup_credentials(cls):"},{"line_number":52,"context_line":"        cls.prepare_instance_network()"},{"line_number":53,"context_line":"        super(ServerVolumeActionsTestJSON, cls).setup_credentials()"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_5be296d6","line":52,"updated":"2016-05-18 20:46:16.000000000","message":"Would this test work without this line ? I think it should (not sure though)","commit_id":"21cf53eeeeae2e9d83a0638c27b932595610ccab"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"a4b7902788c7f1113ecf7b5a9f41dbfa3aff104e","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"    @classmethod"},{"line_number":51,"context_line":"    def setup_credentials(cls):"},{"line_number":52,"context_line":"        cls.prepare_instance_network()"},{"line_number":53,"context_line":"        super(ServerVolumeActionsTestJSON, cls).setup_credentials()"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    @classmethod"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_689220a1","line":52,"in_reply_to":"bab6814e_5be296d6","updated":"2016-05-19 07:33:04.000000000","message":"Done","commit_id":"21cf53eeeeae2e9d83a0638c27b932595610ccab"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"2b2f963c466fc1174dd0bdac06c9a688f24d795e","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        cls.set_validation_resources()"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"        super(ServerVolumeActionsTestJSON, cls).resource_setup()"},{"line_number":65,"context_line":"        cls.server_id \u003d cls.rebuild_server(None, validatable\u003dTrue)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    @test.idempotent_id(\u002779f058fa-e178-4d6d-8019-6708afabe9eb\u0027)"},{"line_number":68,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_fb954a40","line":65,"updated":"2016-05-18 20:46:16.000000000","message":"And another server here ?\n\nWhich one is actually used ?","commit_id":"21cf53eeeeae2e9d83a0638c27b932595610ccab"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"2b2f963c466fc1174dd0bdac06c9a688f24d795e","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    @test.idempotent_id(\u002779f058fa-e178-4d6d-8019-6708afabe9eb\u0027)"},{"line_number":68,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"},{"line_number":69,"context_line":"                          \u0027Resize is not available.\u0027)"},{"line_number":70,"context_line":"    @test.services(\u0027compute\u0027, \u0027volume\u0027)"},{"line_number":71,"context_line":"    def test_resize_volume_backed_server_revert(self):"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        server \u003d self.client.show_server(self.server_id)[\u0027server\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_5b945636","line":70,"updated":"2016-05-18 20:46:16.000000000","message":"You don\u0027t need to tag this test as \u0027compute\u0027 because this test lives in the \u0027compute\u0027 directory. More details here: https://github.com/openstack/tempest/blob/master/HACKING.rst#service-tagging","commit_id":"21cf53eeeeae2e9d83a0638c27b932595610ccab"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"a4b7902788c7f1113ecf7b5a9f41dbfa3aff104e","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    @test.idempotent_id(\u002779f058fa-e178-4d6d-8019-6708afabe9eb\u0027)"},{"line_number":68,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"},{"line_number":69,"context_line":"                          \u0027Resize is not available.\u0027)"},{"line_number":70,"context_line":"    @test.services(\u0027compute\u0027, \u0027volume\u0027)"},{"line_number":71,"context_line":"    def test_resize_volume_backed_server_revert(self):"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        server \u003d self.client.show_server(self.server_id)[\u0027server\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"bab6814e_28f778f6","line":70,"in_reply_to":"bab6814e_5b945636","updated":"2016-05-19 07:33:04.000000000","message":"Done","commit_id":"21cf53eeeeae2e9d83a0638c27b932595610ccab"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ae210600353136c04755767bd344721df4c2fcbc","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        super(ServerVolumeActionsTest, self).setUp()"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        # Use a volume backed instance for these tests."},{"line_number":34,"context_line":"        server \u003d self.create_test_server(volume_backed\u003dTrue,"},{"line_number":35,"context_line":"                                         wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":36,"context_line":"        self.server_id \u003d server[\u0027id\u0027]"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_67e8ac17","line":34,"range":{"start_line":34,"start_character":41,"end_line":34,"end_character":54},"updated":"2016-06-09 20:33:21.000000000","message":"You probably need something in the skip_checks method to make sure Cinder is available if you\u0027re going to create a volume-backed server.","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"6117ebef3faa1e484ad3da66fcbce0a82b28144b","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        super(ServerVolumeActionsTest, self).setUp()"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        # Use a volume backed instance for these tests."},{"line_number":34,"context_line":"        server \u003d self.create_test_server(volume_backed\u003dTrue,"},{"line_number":35,"context_line":"                                         wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":36,"context_line":"        self.server_id \u003d server[\u0027id\u0027]"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_8f5dd517","line":34,"range":{"start_line":34,"start_character":41,"end_line":34,"end_character":54},"in_reply_to":"7aa08908_67e8ac17","updated":"2016-06-10 10:03:08.000000000","message":"Done","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ae210600353136c04755767bd344721df4c2fcbc","unresolved":false,"context_lines":[{"line_number":35,"context_line":"                                         wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":36,"context_line":"        self.server_id \u003d server[\u0027id\u0027]"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def tearDown(self):"},{"line_number":39,"context_line":"        super(ServerVolumeActionsTest, self).tearDown()"},{"line_number":40,"context_line":"        self.server_check_teardown()"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @classmethod"},{"line_number":43,"context_line":"    def setup_credentials(cls):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_875b182e","line":40,"range":{"start_line":38,"start_character":4,"end_line":40,"end_character":36},"updated":"2016-06-09 20:33:21.000000000","message":"I\u0027m not sure this is necessary, when you call create_test_server is stores the server in a list in the parent class which is then cleaned up in it\u0027s tear down.","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"6117ebef3faa1e484ad3da66fcbce0a82b28144b","unresolved":false,"context_lines":[{"line_number":35,"context_line":"                                         wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":36,"context_line":"        self.server_id \u003d server[\u0027id\u0027]"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def tearDown(self):"},{"line_number":39,"context_line":"        super(ServerVolumeActionsTest, self).tearDown()"},{"line_number":40,"context_line":"        self.server_check_teardown()"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @classmethod"},{"line_number":43,"context_line":"    def setup_credentials(cls):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_cf9d1dbd","line":40,"range":{"start_line":38,"start_character":4,"end_line":40,"end_character":36},"in_reply_to":"7aa08908_875b182e","updated":"2016-06-10 10:03:08.000000000","message":"Done","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ae210600353136c04755767bd344721df4c2fcbc","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        self.server_check_teardown()"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @classmethod"},{"line_number":43,"context_line":"    def setup_credentials(cls):"},{"line_number":44,"context_line":"        super(ServerVolumeActionsTest, cls).setup_credentials()"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    @classmethod"},{"line_number":47,"context_line":"    def setup_clients(cls):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_27b9842f","line":44,"range":{"start_line":43,"start_character":4,"end_line":44,"end_character":63},"updated":"2016-06-09 20:33:21.000000000","message":"This is redundant.","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"6117ebef3faa1e484ad3da66fcbce0a82b28144b","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        self.server_check_teardown()"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"    @classmethod"},{"line_number":43,"context_line":"    def setup_credentials(cls):"},{"line_number":44,"context_line":"        super(ServerVolumeActionsTest, cls).setup_credentials()"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    @classmethod"},{"line_number":47,"context_line":"    def setup_clients(cls):"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_2fa5c103","line":44,"range":{"start_line":43,"start_character":4,"end_line":44,"end_character":63},"in_reply_to":"7aa08908_27b9842f","updated":"2016-06-10 10:03:08.000000000","message":"Done","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ae210600353136c04755767bd344721df4c2fcbc","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        cls.client \u003d cls.servers_client"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    @classmethod"},{"line_number":52,"context_line":"    def resource_setup(cls):"},{"line_number":53,"context_line":"        super(ServerVolumeActionsTest, cls).resource_setup()"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    @test.idempotent_id(\u002779f058fa-e178-4d6d-8019-6708afabe9eb\u0027)"},{"line_number":56,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_07b6083d","line":53,"range":{"start_line":52,"start_character":4,"end_line":53,"end_character":60},"updated":"2016-06-09 20:33:21.000000000","message":"This is redundant.","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"6117ebef3faa1e484ad3da66fcbce0a82b28144b","unresolved":false,"context_lines":[{"line_number":49,"context_line":"        cls.client \u003d cls.servers_client"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    @classmethod"},{"line_number":52,"context_line":"    def resource_setup(cls):"},{"line_number":53,"context_line":"        super(ServerVolumeActionsTest, cls).resource_setup()"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    @test.idempotent_id(\u002779f058fa-e178-4d6d-8019-6708afabe9eb\u0027)"},{"line_number":56,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7aa08908_0fa8c5da","line":53,"range":{"start_line":52,"start_character":4,"end_line":53,"end_character":60},"in_reply_to":"7aa08908_07b6083d","updated":"2016-06-10 10:03:08.000000000","message":"Done","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"ae55d5406023ab3b1ae86f970342018e1b83a9ab","unresolved":false,"context_lines":[{"line_number":56,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"},{"line_number":57,"context_line":"                          \u0027Resize is not available.\u0027)"},{"line_number":58,"context_line":"    @test.services(\u0027volume\u0027)"},{"line_number":59,"context_line":"    def test_resize_volume_backed_server_revert(self):"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        server \u003d self.client.show_server(self.server_id)[\u0027server\u0027]"},{"line_number":62,"context_line":"        resize_flavor \u003d CONF.compute.flavor_ref_alt"}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_05e70f13","line":59,"updated":"2016-05-24 16:35:42.000000000","message":"How is this test different than test_resize_volume_backed_server_confirm ?","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"0346fc636360817cf9793522a779f93c373e85f4","unresolved":false,"context_lines":[{"line_number":56,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"},{"line_number":57,"context_line":"                          \u0027Resize is not available.\u0027)"},{"line_number":58,"context_line":"    @test.services(\u0027volume\u0027)"},{"line_number":59,"context_line":"    def test_resize_volume_backed_server_revert(self):"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        server \u003d self.client.show_server(self.server_id)[\u0027server\u0027]"},{"line_number":62,"context_line":"        resize_flavor \u003d CONF.compute.flavor_ref_alt"}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_e8ac6237","line":59,"updated":"2016-05-24 16:59:51.000000000","message":"Well, this test reverts the resize if the instance instead of confirming it. Really there should be an assert at the end to confirm that the instance hasn\u0027t changed in this test.","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"fb893f6f776132ec0919f16c8356317c0d8bb751","unresolved":false,"context_lines":[{"line_number":56,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"},{"line_number":57,"context_line":"                          \u0027Resize is not available.\u0027)"},{"line_number":58,"context_line":"    @test.services(\u0027volume\u0027)"},{"line_number":59,"context_line":"    def test_resize_volume_backed_server_revert(self):"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        server \u003d self.client.show_server(self.server_id)[\u0027server\u0027]"},{"line_number":62,"context_line":"        resize_flavor \u003d CONF.compute.flavor_ref_alt"}],"source_content_type":"text/x-python","patch_set":5,"id":"9abb7d3a_b103130d","line":59,"updated":"2016-05-27 10:12:04.000000000","message":"You asked that I move this _revert test here in PS#2, I\u0027ll move the _confirm test here as well once this lands.","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"3a78946caea703ddeb4b8681196c21556886b4f5","unresolved":false,"context_lines":[{"line_number":56,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"},{"line_number":57,"context_line":"                          \u0027Resize is not available.\u0027)"},{"line_number":58,"context_line":"    @test.services(\u0027volume\u0027)"},{"line_number":59,"context_line":"    def test_resize_volume_backed_server_revert(self):"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        server \u003d self.client.show_server(self.server_id)[\u0027server\u0027]"},{"line_number":62,"context_line":"        resize_flavor \u003d CONF.compute.flavor_ref_alt"}],"source_content_type":"text/x-python","patch_set":5,"id":"9abb7d3a_b893f84e","line":59,"in_reply_to":"bab6814e_e8ac6237","updated":"2016-05-27 09:20:46.000000000","message":"I am confused on why the other test is in the scenario folder and this one in the API.","commit_id":"d5357fb55273ee5b8d1b5531d870247d9e173c8e"}],"tempest/scenario/test_server_advanced_ops.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"3b3d6272687979f8169e50d8825d4aa967a94937","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"},{"line_number":73,"context_line":"                          \u0027Resize is not available.\u0027)"},{"line_number":74,"context_line":"    @test.services(\u0027compute\u0027, \u0027volume\u0027)"},{"line_number":75,"context_line":"    def test_resize_volume_backed_server_revert(self):"},{"line_number":76,"context_line":"        # We create an instance for use in this test"},{"line_number":77,"context_line":"        instance \u003d self.create_server(wait_until\u003d\u0027ACTIVE\u0027, volume_backed\u003dTrue)"},{"line_number":78,"context_line":"        instance_id \u003d instance[\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_4face59e","line":75,"updated":"2016-05-17 15:16:09.000000000","message":"Why is this new test not in tempest/api/compute/servers/ ? It doesn\"t look like a scenario to me.","commit_id":"f815c80b4cd6005dcd04a9347973279b26058e4d"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"5868e40f8154baae7689c216ee487a6669282d94","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"},{"line_number":73,"context_line":"                          \u0027Resize is not available.\u0027)"},{"line_number":74,"context_line":"    @test.services(\u0027compute\u0027, \u0027volume\u0027)"},{"line_number":75,"context_line":"    def test_resize_volume_backed_server_revert(self):"},{"line_number":76,"context_line":"        # We create an instance for use in this test"},{"line_number":77,"context_line":"        instance \u003d self.create_server(wait_until\u003d\u0027ACTIVE\u0027, volume_backed\u003dTrue)"},{"line_number":78,"context_line":"        instance_id \u003d instance[\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"bab6814e_a2ef40af","line":75,"in_reply_to":"bab6814e_7c905eed","updated":"2016-05-18 09:30:00.000000000","message":"That\u0027s not a problem, I\u0027ll do this now, thanks!","commit_id":"f815c80b4cd6005dcd04a9347973279b26058e4d"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"3c11701fb3229352e3e8678d527912e3ef67a7ac","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"},{"line_number":73,"context_line":"                          \u0027Resize is not available.\u0027)"},{"line_number":74,"context_line":"    @test.services(\u0027compute\u0027, \u0027volume\u0027)"},{"line_number":75,"context_line":"    def test_resize_volume_backed_server_revert(self):"},{"line_number":76,"context_line":"        # We create an instance for use in this test"},{"line_number":77,"context_line":"        instance \u003d self.create_server(wait_until\u003d\u0027ACTIVE\u0027, volume_backed\u003dTrue)"},{"line_number":78,"context_line":"        instance_id \u003d instance[\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"dab17558_61ff3d34","line":75,"in_reply_to":"dab17558_4face59e","updated":"2016-05-17 16:08:36.000000000","message":"It just follows the above _confirm test, should both be moved elsewhere?","commit_id":"f815c80b4cd6005dcd04a9347973279b26058e4d"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"a38b0ba53c5f0215f8578ab4e9e6510fe848ed4a","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    @testtools.skipUnless(CONF.compute_feature_enabled.resize,"},{"line_number":73,"context_line":"                          \u0027Resize is not available.\u0027)"},{"line_number":74,"context_line":"    @test.services(\u0027compute\u0027, \u0027volume\u0027)"},{"line_number":75,"context_line":"    def test_resize_volume_backed_server_revert(self):"},{"line_number":76,"context_line":"        # We create an instance for use in this test"},{"line_number":77,"context_line":"        instance \u003d self.create_server(wait_until\u003d\u0027ACTIVE\u0027, volume_backed\u003dTrue)"},{"line_number":78,"context_line":"        instance_id \u003d instance[\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"bab6814e_7c905eed","line":75,"in_reply_to":"dab17558_61ff3d34","updated":"2016-05-18 09:19:14.000000000","message":"I think they should be moved indeed. But it should be done in 2 different patches. One patch to add the new test somewhere in tempest/api/compute/servers and another patch to move _confirm test.\n\nYou don\u0027t have to do both patches yourself.","commit_id":"f815c80b4cd6005dcd04a9347973279b26058e4d"}]}