)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"7f950698fdcbc3b000ff9e55504c3ea875b58b60","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix resize tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If we run tests in single thread tempest and we use one"},{"line_number":10,"context_line":"instance for two resize operations than we have a bug."},{"line_number":11,"context_line":"First test case test_resize_server_from_manual_to_auto"},{"line_number":12,"context_line":"resizes the instance from flavor_ref to flavor_ref_alt."},{"line_number":13,"context_line":"After that we reuse this instance for the next test case"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3aaa91ec_ac661827","line":10,"updated":"2016-06-27 14:53:59.000000000","message":"This doesn\u0027t matter. Tempest serializes tests within a class so it doesn\u0027t matter how many worker processes you use to run things within a class it\u0027ll be serialized. The issue you\u0027re having here is actually just the order things are executed in (which is something tempest doesn\u0027t control and is the domain of the test runner)","commit_id":"0504cc0d42ef37d5ef0c84cd0384a2fed07b3d02"},{"author":{"_account_id":12408,"name":"Maxim Nestratov","email":"mnestratov@virtuozzo.com","username":"mnestratov"},"change_message_id":"7205a15491a3d670a6e154c82571f3447539aec0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix resize tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If we run tests in single thread tempest and we use one"},{"line_number":10,"context_line":"instance for two resize operations than we have a bug."},{"line_number":11,"context_line":"First test case test_resize_server_from_manual_to_auto"},{"line_number":12,"context_line":"resizes the instance from flavor_ref to flavor_ref_alt."},{"line_number":13,"context_line":"After that we reuse this instance for the next test case"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3aaa91ec_5d675cea","line":10,"in_reply_to":"3aaa91ec_a219ff24","updated":"2016-06-27 15:28:03.000000000","message":"Ok. Got you. We made an incorrect inference then regarding order influence.","commit_id":"0504cc0d42ef37d5ef0c84cd0384a2fed07b3d02"},{"author":{"_account_id":12408,"name":"Maxim Nestratov","email":"mnestratov@virtuozzo.com","username":"mnestratov"},"change_message_id":"959851e56a80937b5381170a62dbe946f306fd84","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix resize tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If we run tests in single thread tempest and we use one"},{"line_number":10,"context_line":"instance for two resize operations than we have a bug."},{"line_number":11,"context_line":"First test case test_resize_server_from_manual_to_auto"},{"line_number":12,"context_line":"resizes the instance from flavor_ref to flavor_ref_alt."},{"line_number":13,"context_line":"After that we reuse this instance for the next test case"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3aaa91ec_c77c1dee","line":10,"in_reply_to":"3aaa91ec_ac661827","updated":"2016-06-27 15:04:12.000000000","message":"Then I don\u0027t get why Jenkins doesn\u0027t hit this for qemu virt_type as it doesn\u0027t support resize down also?","commit_id":"0504cc0d42ef37d5ef0c84cd0384a2fed07b3d02"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"eb99b7b517303166e78bb10ba2225cda01267d9f","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix resize tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"If we run tests in single thread tempest and we use one"},{"line_number":10,"context_line":"instance for two resize operations than we have a bug."},{"line_number":11,"context_line":"First test case test_resize_server_from_manual_to_auto"},{"line_number":12,"context_line":"resizes the instance from flavor_ref to flavor_ref_alt."},{"line_number":13,"context_line":"After that we reuse this instance for the next test case"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3aaa91ec_a219ff24","line":10,"in_reply_to":"3aaa91ec_c77c1dee","updated":"2016-06-27 15:14:56.000000000","message":"Well specifically in the case of your third party ci it\u0027s actually failing because of the lack of the rebuild tests in that class. The way the gate orders these tests (just by chance) is so that the rebuild tests end up being run between the resize tests ensuring we don\u0027t resize down (just rebuild down)\n\nThis is a fragile mechanism and not something we actually want to depend on, Which is why I\u0027m generally in favor with removing the server reuse.","commit_id":"0504cc0d42ef37d5ef0c84cd0384a2fed07b3d02"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"7f950698fdcbc3b000ff9e55504c3ea875b58b60","unresolved":false,"context_lines":[{"line_number":19,"context_line":"disk_config and try to resize this instance. Also we added"},{"line_number":20,"context_line":"server_id as parameter for _update_server_with_disk_config()"},{"line_number":21,"context_line":"to set disk_config for specified instance."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I04c4379459c2fc1fd4801ec2aad53d0f6053b6d6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3aaa91ec_6c2b9095","line":22,"updated":"2016-06-27 14:53:59.000000000","message":"There should be a bug filed on this issue, it doesn\u0027t sound unique (I\u0027m pretty sure it\u0027s come up before) so documenting it in the bug tracker will let people see it","commit_id":"0504cc0d42ef37d5ef0c84cd0384a2fed07b3d02"},{"author":{"_account_id":15847,"name":"Mikhail Feoktistov","email":"mfeoktistov@virtuozzo.com","username":"mfeoktistov"},"change_message_id":"10342df4610025744666c576f0fe5f0763874015","unresolved":false,"context_lines":[{"line_number":19,"context_line":"disk_config and try to resize this instance. Also we added"},{"line_number":20,"context_line":"server_id as parameter for _update_server_with_disk_config()"},{"line_number":21,"context_line":"to set disk_config for specified instance."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Change-Id: I04c4379459c2fc1fd4801ec2aad53d0f6053b6d6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3aaa91ec_1dd034c9","line":22,"in_reply_to":"3aaa91ec_6c2b9095","updated":"2016-06-27 15:41:13.000000000","message":"I think we should close the following bug\nhttps://bugs.launchpad.net/tempest/+bug/1358221\nBecause in other test cases we always resize from flavor_ref to flavor_ref_alt. So from small disk to bigger one.","commit_id":"0504cc0d42ef37d5ef0c84cd0384a2fed07b3d02"}],"tempest/api/compute/servers/test_disk_config.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"4fab1aa1c8a176ef183b639bec902c083074d041","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        super(ServerDiskConfigTestJSON, cls).setup_clients()"},{"line_number":38,"context_line":"        cls.client \u003d cls.os.servers_client"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @classmethod"},{"line_number":41,"context_line":"    def resource_setup(cls):"},{"line_number":42,"context_line":"        super(ServerDiskConfigTestJSON, cls).resource_setup()"},{"line_number":43,"context_line":"        server \u003d cls.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":44,"context_line":"        cls.server_id \u003d server[\u0027id\u0027]"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    def _update_server_with_disk_config(self, server_id, disk_config):"},{"line_number":47,"context_line":"        server \u003d self.client.show_server(server_id)[\u0027server\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_4c717477","line":44,"range":{"start_line":40,"start_character":4,"end_line":44,"end_character":36},"updated":"2016-06-27 14:44:14.000000000","message":"I feel like we should just remove this altogether, sharing resources between tests is tricky and usually leads to bugs, as you are seeing. I think Tempest as a whole is trying to get away from class variables for resources used in tests, so I\u0027d think we should just remove this and make each test self-sufficient for creating/deleting resources.","commit_id":"0504cc0d42ef37d5ef0c84cd0384a2fed07b3d02"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"7f950698fdcbc3b000ff9e55504c3ea875b58b60","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        super(ServerDiskConfigTestJSON, cls).setup_clients()"},{"line_number":38,"context_line":"        cls.client \u003d cls.os.servers_client"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @classmethod"},{"line_number":41,"context_line":"    def resource_setup(cls):"},{"line_number":42,"context_line":"        super(ServerDiskConfigTestJSON, cls).resource_setup()"},{"line_number":43,"context_line":"        server \u003d cls.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":44,"context_line":"        cls.server_id \u003d server[\u0027id\u0027]"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    def _update_server_with_disk_config(self, server_id, disk_config):"},{"line_number":47,"context_line":"        server \u003d self.client.show_server(server_id)[\u0027server\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_075bc52f","line":44,"range":{"start_line":40,"start_character":4,"end_line":44,"end_character":36},"in_reply_to":"3aaa91ec_4c717477","updated":"2016-06-27 14:53:59.000000000","message":"Yes, this. If we\u0027re having an issue because of test ordering and resource reuse, lets just remove the resource reuse. It\u0027ll make things a bit slower here but that\u0027s the tradeoff for making it more resilient","commit_id":"0504cc0d42ef37d5ef0c84cd0384a2fed07b3d02"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fd5e7801ad15b79b7b40666e8dd22e026a4ae09f","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        super(ServerDiskConfigTestJSON, cls).setup_clients()"},{"line_number":38,"context_line":"        cls.client \u003d cls.os.servers_client"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @classmethod"},{"line_number":41,"context_line":"    def resource_setup(cls):"},{"line_number":42,"context_line":"        super(ServerDiskConfigTestJSON, cls).resource_setup()"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def _update_server_with_disk_config(self, server_id, disk_config):"},{"line_number":45,"context_line":"        server \u003d self.client.show_server(server_id)[\u0027server\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_ca9ed262","line":42,"range":{"start_line":40,"start_character":4,"end_line":42,"end_character":61},"updated":"2016-06-27 20:49:18.000000000","message":"This is redundant, just remove the override method.","commit_id":"672ce64eab50d1bfca53c377ea969642dc329593"},{"author":{"_account_id":15847,"name":"Mikhail Feoktistov","email":"mfeoktistov@virtuozzo.com","username":"mfeoktistov"},"change_message_id":"5af2702754b1cb1fbe00272c882157bc08614915","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        super(ServerDiskConfigTestJSON, cls).setup_clients()"},{"line_number":38,"context_line":"        cls.client \u003d cls.os.servers_client"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    @classmethod"},{"line_number":41,"context_line":"    def resource_setup(cls):"},{"line_number":42,"context_line":"        super(ServerDiskConfigTestJSON, cls).resource_setup()"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    def _update_server_with_disk_config(self, server_id, disk_config):"},{"line_number":45,"context_line":"        server \u003d self.client.show_server(server_id)[\u0027server\u0027]"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_c887e9d5","line":42,"range":{"start_line":40,"start_character":4,"end_line":42,"end_character":61},"in_reply_to":"3aaa91ec_ca9ed262","updated":"2016-06-28 07:49:48.000000000","message":"Done","commit_id":"672ce64eab50d1bfca53c377ea969642dc329593"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fd5e7801ad15b79b7b40666e8dd22e026a4ae09f","unresolved":false,"context_lines":[{"line_number":54,"context_line":"    def test_rebuild_server_with_manual_disk_config(self):"},{"line_number":55,"context_line":"        # A server should be rebuilt using the manual disk config option"},{"line_number":56,"context_line":"        server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":57,"context_line":"        self.addCleanup(self.client.delete_server, server[\u0027id\u0027])"},{"line_number":58,"context_line":"        self._update_server_with_disk_config(server[\u0027id\u0027],"},{"line_number":59,"context_line":"                                             disk_config\u003d\u0027AUTO\u0027)"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_6a9fde50","line":57,"range":{"start_line":57,"start_character":8,"end_line":57,"end_character":64},"updated":"2016-06-27 20:49:18.000000000","message":"This is actually redundant also, self.create_test_server adds the created server (id) to a running list that the parent class deletes on teardown. It might be fine to leave this though as the code in the base class should be handling 404s.","commit_id":"672ce64eab50d1bfca53c377ea969642dc329593"},{"author":{"_account_id":15847,"name":"Mikhail Feoktistov","email":"mfeoktistov@virtuozzo.com","username":"mfeoktistov"},"change_message_id":"5af2702754b1cb1fbe00272c882157bc08614915","unresolved":false,"context_lines":[{"line_number":54,"context_line":"    def test_rebuild_server_with_manual_disk_config(self):"},{"line_number":55,"context_line":"        # A server should be rebuilt using the manual disk config option"},{"line_number":56,"context_line":"        server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":57,"context_line":"        self.addCleanup(self.client.delete_server, server[\u0027id\u0027])"},{"line_number":58,"context_line":"        self._update_server_with_disk_config(server[\u0027id\u0027],"},{"line_number":59,"context_line":"                                             disk_config\u003d\u0027AUTO\u0027)"},{"line_number":60,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_4be4933a","line":57,"range":{"start_line":57,"start_character":8,"end_line":57,"end_character":64},"in_reply_to":"3aaa91ec_6a9fde50","updated":"2016-06-28 07:49:48.000000000","message":"It\u0027s not necessary to keep running instance, because we create a new one in each test method. I think it helps us to save memory. Otherwise we will have 5 running instances in the end.","commit_id":"672ce64eab50d1bfca53c377ea969642dc329593"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fd5e7801ad15b79b7b40666e8dd22e026a4ae09f","unresolved":false,"context_lines":[{"line_number":73,"context_line":"    def test_rebuild_server_with_auto_disk_config(self):"},{"line_number":74,"context_line":"        # A server should be rebuilt using the auto disk config option"},{"line_number":75,"context_line":"        server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":76,"context_line":"        self.addCleanup(self.client.delete_server, server[\u0027id\u0027])"},{"line_number":77,"context_line":"        self._update_server_with_disk_config(server[\u0027id\u0027],"},{"line_number":78,"context_line":"                                             disk_config\u003d\u0027MANUAL\u0027)"},{"line_number":79,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_4aaf827d","line":76,"range":{"start_line":76,"start_character":8,"end_line":76,"end_character":64},"updated":"2016-06-27 20:49:18.000000000","message":"same","commit_id":"672ce64eab50d1bfca53c377ea969642dc329593"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fd5e7801ad15b79b7b40666e8dd22e026a4ae09f","unresolved":false,"context_lines":[{"line_number":94,"context_line":"    def test_resize_server_from_manual_to_auto(self):"},{"line_number":95,"context_line":"        # A server should be resized from manual to auto disk config"},{"line_number":96,"context_line":"        server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":97,"context_line":"        self.addCleanup(self.client.delete_server, server[\u0027id\u0027])"},{"line_number":98,"context_line":"        self._update_server_with_disk_config(server[\u0027id\u0027],"},{"line_number":99,"context_line":"                                             disk_config\u003d\u0027MANUAL\u0027)"},{"line_number":100,"context_line":"        # Resize with auto option"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_8a3e8a27","line":97,"range":{"start_line":97,"start_character":8,"end_line":97,"end_character":64},"updated":"2016-06-27 20:49:18.000000000","message":"same","commit_id":"672ce64eab50d1bfca53c377ea969642dc329593"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fd5e7801ad15b79b7b40666e8dd22e026a4ae09f","unresolved":false,"context_lines":[{"line_number":105,"context_line":"        self.client.confirm_resize_server(server[\u0027id\u0027])"},{"line_number":106,"context_line":"        waiters.wait_for_server_status(self.client, server[\u0027id\u0027], \u0027ACTIVE\u0027)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        server_info \u003d self.client.show_server(server[\u0027id\u0027])[\u0027server\u0027]"},{"line_number":109,"context_line":"        self.assertEqual(\u0027AUTO\u0027, server_info[\u0027OS-DCF:diskConfig\u0027])"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"    @test.idempotent_id(\u0027693d16f3-556c-489a-8bac-3d0ca2490bad\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_8ada0ae8","line":108,"range":{"start_line":108,"start_character":8,"end_line":108,"end_character":19},"updated":"2016-06-27 20:49:18.000000000","message":"nit: you didn\u0027t really need to rename this.","commit_id":"672ce64eab50d1bfca53c377ea969642dc329593"},{"author":{"_account_id":15847,"name":"Mikhail Feoktistov","email":"mfeoktistov@virtuozzo.com","username":"mfeoktistov"},"change_message_id":"5af2702754b1cb1fbe00272c882157bc08614915","unresolved":false,"context_lines":[{"line_number":105,"context_line":"        self.client.confirm_resize_server(server[\u0027id\u0027])"},{"line_number":106,"context_line":"        waiters.wait_for_server_status(self.client, server[\u0027id\u0027], \u0027ACTIVE\u0027)"},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        server_info \u003d self.client.show_server(server[\u0027id\u0027])[\u0027server\u0027]"},{"line_number":109,"context_line":"        self.assertEqual(\u0027AUTO\u0027, server_info[\u0027OS-DCF:diskConfig\u0027])"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"    @test.idempotent_id(\u0027693d16f3-556c-489a-8bac-3d0ca2490bad\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_682075f8","line":108,"range":{"start_line":108,"start_character":8,"end_line":108,"end_character":19},"in_reply_to":"3aaa91ec_8ada0ae8","updated":"2016-06-28 07:49:48.000000000","message":"Done","commit_id":"672ce64eab50d1bfca53c377ea969642dc329593"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"fd5e7801ad15b79b7b40666e8dd22e026a4ae09f","unresolved":false,"context_lines":[{"line_number":125,"context_line":"        self.client.confirm_resize_server(server[\u0027id\u0027])"},{"line_number":126,"context_line":"        waiters.wait_for_server_status(self.client, server[\u0027id\u0027], \u0027ACTIVE\u0027)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"        server_info \u003d self.client.show_server(server[\u0027id\u0027])[\u0027server\u0027]"},{"line_number":129,"context_line":"        self.assertEqual(\u0027MANUAL\u0027, server_info[\u0027OS-DCF:diskConfig\u0027])"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"    @test.idempotent_id(\u00275ef18867-358d-4de9-b3c9-94d4ba35742f\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_2aa31651","line":128,"range":{"start_line":128,"start_character":8,"end_line":128,"end_character":19},"updated":"2016-06-27 20:49:18.000000000","message":"same","commit_id":"672ce64eab50d1bfca53c377ea969642dc329593"},{"author":{"_account_id":15847,"name":"Mikhail Feoktistov","email":"mfeoktistov@virtuozzo.com","username":"mfeoktistov"},"change_message_id":"5af2702754b1cb1fbe00272c882157bc08614915","unresolved":false,"context_lines":[{"line_number":125,"context_line":"        self.client.confirm_resize_server(server[\u0027id\u0027])"},{"line_number":126,"context_line":"        waiters.wait_for_server_status(self.client, server[\u0027id\u0027], \u0027ACTIVE\u0027)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"        server_info \u003d self.client.show_server(server[\u0027id\u0027])[\u0027server\u0027]"},{"line_number":129,"context_line":"        self.assertEqual(\u0027MANUAL\u0027, server_info[\u0027OS-DCF:diskConfig\u0027])"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"    @test.idempotent_id(\u00275ef18867-358d-4de9-b3c9-94d4ba35742f\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_481db9bc","line":128,"range":{"start_line":128,"start_character":8,"end_line":128,"end_character":19},"in_reply_to":"3aaa91ec_2aa31651","updated":"2016-06-28 07:49:48.000000000","message":"Done","commit_id":"672ce64eab50d1bfca53c377ea969642dc329593"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"0f5143b44a190d1b694534c553a1aa5b268eb995","unresolved":false,"context_lines":[{"line_number":50,"context_line":"    def test_rebuild_server_with_manual_disk_config(self):"},{"line_number":51,"context_line":"        # A server should be rebuilt using the manual disk config option"},{"line_number":52,"context_line":"        server \u003d self.create_test_server(wait_until\u003d\u0027ACTIVE\u0027)"},{"line_number":53,"context_line":"        self.addCleanup(self.client.delete_server, server[\u0027id\u0027])"},{"line_number":54,"context_line":"        self._update_server_with_disk_config(server[\u0027id\u0027],"},{"line_number":55,"context_line":"                                             disk_config\u003d\u0027AUTO\u0027)"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"3aaa91ec_ddfe536a","line":53,"range":{"start_line":53,"start_character":9,"end_line":53,"end_character":64},"updated":"2016-06-29 01:15:28.000000000","message":"these actually not needed but no harm as we ignore 404 if already deleted.","commit_id":"7a71ff535a036f8f72fb54c1102147ae12badb22"}]}