)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"170f8aa557bed90c4384a6e973744b23937a155d","unresolved":false,"context_lines":[{"line_number":10,"context_line":"1. Create a cell"},{"line_number":11,"context_line":"2. List all cells"},{"line_number":12,"context_line":"3. Delete created cell"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I753142f3b3518318821b71606765b1c04d8d9188"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"1ae5cdf2_fe679db5","line":13,"updated":"2014-06-06 14:17:50.000000000","message":"mention about the config option addition.","commit_id":"b15d535e160f1e83442d2e0efee310caa89a5c96"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"77222e5c75b1647ed954d0a989768ca16c5e4254","unresolved":false,"context_lines":[{"line_number":10,"context_line":"1. Create a cell"},{"line_number":11,"context_line":"2. List all cells"},{"line_number":12,"context_line":"3. Delete created cell"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I753142f3b3518318821b71606765b1c04d8d9188"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"1ae5cdf2_c1081d94","line":13,"in_reply_to":"1ae5cdf2_fe679db5","updated":"2014-06-09 10:10:41.000000000","message":"Done","commit_id":"b15d535e160f1e83442d2e0efee310caa89a5c96"}],"etc/tempest.conf.sample":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"690a3cf1156f0d6d34896900c84f5cc7d1df75a3","unresolved":false,"context_lines":[{"line_number":367,"context_line":""},{"line_number":368,"context_line":"# Does the test environment support n-cell ? (boolean value)"},{"line_number":369,"context_line":"#n-cell\u003dtrue"},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"# If false, skip all nova v3 tests. (boolean value)"},{"line_number":372,"context_line":"#api_v3\u003dtrue"},{"line_number":373,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":7,"id":"1ae5cdf2_c599697a","line":370,"updated":"2014-06-05 07:30:46.000000000","message":"this would not help just changing this file.\r\nFollowing needs to be done-\r\n1. change the ./tempest/config.py to set this flag\r\n2. run ./tool/generate\u003dsample.sh in your local enviornement. it will change the tempest.conf.sample automatically.","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ebbf067c62141d7d2bd553ed27c2e7d12239a2c7","unresolved":false,"context_lines":[{"line_number":367,"context_line":""},{"line_number":368,"context_line":"# Does the test environment support n-cell ? (boolean value)"},{"line_number":369,"context_line":"#n-cell\u003dtrue"},{"line_number":370,"context_line":""},{"line_number":371,"context_line":"# If false, skip all nova v3 tests. (boolean value)"},{"line_number":372,"context_line":"#api_v3\u003dtrue"},{"line_number":373,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":7,"id":"1ae5cdf2_f528ce34","line":370,"in_reply_to":"1ae5cdf2_c599697a","updated":"2014-06-05 11:33:57.000000000","message":"Done","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"}],"tempest/api/compute/admin/test_cells.py":[{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"8888119a965629bb34244c712a008767cb9cf09e","unresolved":false,"context_lines":[{"line_number":31,"context_line":"            msg \u003d \"cell extension not enabled.\""},{"line_number":32,"context_line":"            raise cls.skipException(msg)"},{"line_number":33,"context_line":"        cls.client \u003d cls.cells_client"},{"line_number":34,"context_line":"        cls.cell \u003d cls.create_cell(data_utils.rand_name(\"test-cell\"))"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def _try_delete_cell(self, cell_name):"},{"line_number":37,"context_line":"        # delete cell, if it exists"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXX%2F%2FwYI%3D","line":34,"updated":"2014-04-28 08:04:43.000000000","message":"This created cell in setUpClass() should also be cleaned-up.","commit_id":"2ed4bed905ad328bd52793fbced7f33c0312153d"},{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"bc3dd557566dd5458b0f6f82452ce5b018f9644b","unresolved":false,"context_lines":[{"line_number":31,"context_line":"            msg \u003d \"cell extension not enabled.\""},{"line_number":32,"context_line":"            raise cls.skipException(msg)"},{"line_number":33,"context_line":"        cls.client \u003d cls.cells_client"},{"line_number":34,"context_line":"        cls.cell \u003d cls.create_cell(data_utils.rand_name(\"test-cell\"))"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def _try_delete_cell(self, cell_name):"},{"line_number":37,"context_line":"        # delete cell, if it exists"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXX%2F%2FvpU%3D","line":34,"in_reply_to":"AAAAXX%2F%2FwDA%3D","updated":"2014-04-28 09:53:01.000000000","message":"Ya checked that. Thats Okie","commit_id":"2ed4bed905ad328bd52793fbced7f33c0312153d"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ccd1a77a3ad34ac36aed476a2a05fd9823f8685d","unresolved":false,"context_lines":[{"line_number":31,"context_line":"            msg \u003d \"cell extension not enabled.\""},{"line_number":32,"context_line":"            raise cls.skipException(msg)"},{"line_number":33,"context_line":"        cls.client \u003d cls.cells_client"},{"line_number":34,"context_line":"        cls.cell \u003d cls.create_cell(data_utils.rand_name(\"test-cell\"))"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"    def _try_delete_cell(self, cell_name):"},{"line_number":37,"context_line":"        # delete cell, if it exists"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAXX%2F%2FwDA%3D","line":34,"in_reply_to":"AAAAXX%2F%2FwYI%3D","updated":"2014-04-28 08:53:58.000000000","message":"this created cell will be deleted during tear down from base class.\nsee base.py","commit_id":"2ed4bed905ad328bd52793fbced7f33c0312153d"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"690a3cf1156f0d6d34896900c84f5cc7d1df75a3","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    @classmethod"},{"line_number":31,"context_line":"    def setUpClass(cls):"},{"line_number":32,"context_line":"        super(CellsAdminTestJSON, cls).setUpClass()"},{"line_number":33,"context_line":"        if not test.is_extension_enabled(\u0027cell\u0027, \u0027compute\u0027):"},{"line_number":34,"context_line":"            msg \u003d \"cell extension not enabled.\""},{"line_number":35,"context_line":"            raise cls.skipException(msg)"},{"line_number":36,"context_line":"        cls.client \u003d cls.os_adm.cells_client"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ae5cdf2_e592e54a","line":33,"updated":"2014-06-05 07:30:46.000000000","message":"feature flag needs to be used here to skip the test if it is disabled.","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"df87e40a5bb8220f41558f4844c4c2fb379a7d4b","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    @classmethod"},{"line_number":31,"context_line":"    def setUpClass(cls):"},{"line_number":32,"context_line":"        super(CellsAdminTestJSON, cls).setUpClass()"},{"line_number":33,"context_line":"        if not test.is_extension_enabled(\u0027cell\u0027, \u0027compute\u0027):"},{"line_number":34,"context_line":"            msg \u003d \"cell extension not enabled.\""},{"line_number":35,"context_line":"            raise cls.skipException(msg)"},{"line_number":36,"context_line":"        cls.client \u003d cls.os_adm.cells_client"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ae5cdf2_c26f7629","line":33,"in_reply_to":"1ae5cdf2_3594f689","updated":"2014-06-06 07:54:16.000000000","message":"where it is done?","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"fe8f85aa5901a625d1809152e02772b7515376a6","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    @classmethod"},{"line_number":31,"context_line":"    def setUpClass(cls):"},{"line_number":32,"context_line":"        super(CellsAdminTestJSON, cls).setUpClass()"},{"line_number":33,"context_line":"        if not test.is_extension_enabled(\u0027cell\u0027, \u0027compute\u0027):"},{"line_number":34,"context_line":"            msg \u003d \"cell extension not enabled.\""},{"line_number":35,"context_line":"            raise cls.skipException(msg)"},{"line_number":36,"context_line":"        cls.client \u003d cls.os_adm.cells_client"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ae5cdf2_450a8b8a","line":33,"in_reply_to":"1ae5cdf2_c26f7629","updated":"2014-06-06 10:43:49.000000000","message":"Sorry, It was left. Thanks :)","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ebbf067c62141d7d2bd553ed27c2e7d12239a2c7","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    @classmethod"},{"line_number":31,"context_line":"    def setUpClass(cls):"},{"line_number":32,"context_line":"        super(CellsAdminTestJSON, cls).setUpClass()"},{"line_number":33,"context_line":"        if not test.is_extension_enabled(\u0027cell\u0027, \u0027compute\u0027):"},{"line_number":34,"context_line":"            msg \u003d \"cell extension not enabled.\""},{"line_number":35,"context_line":"            raise cls.skipException(msg)"},{"line_number":36,"context_line":"        cls.client \u003d cls.os_adm.cells_client"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ae5cdf2_3594f689","line":33,"in_reply_to":"1ae5cdf2_e592e54a","updated":"2014-06-05 11:33:57.000000000","message":"Done","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"690a3cf1156f0d6d34896900c84f5cc7d1df75a3","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def setUp(self):"},{"line_number":39,"context_line":"        super(CellsAdminTestJSON, self).setUp()"},{"line_number":40,"context_line":"        cell_name \u003d data_utils.rand_name(\"test-cell\")"},{"line_number":41,"context_line":"        resp, body \u003d self.client.create_cell(cell_name)"},{"line_number":42,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":43,"context_line":"        self.cell \u003d body[\u0027cell\u0027][\u0027name\u0027]"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ae5cdf2_c51409eb","line":40,"updated":"2014-06-05 07:30:46.000000000","message":"I think we can move this to setUpclass. so that created cell can be used in other test like get cell in future.","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ebbf067c62141d7d2bd553ed27c2e7d12239a2c7","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    def setUp(self):"},{"line_number":39,"context_line":"        super(CellsAdminTestJSON, self).setUp()"},{"line_number":40,"context_line":"        cell_name \u003d data_utils.rand_name(\"test-cell\")"},{"line_number":41,"context_line":"        resp, body \u003d self.client.create_cell(cell_name)"},{"line_number":42,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":43,"context_line":"        self.cell \u003d body[\u0027cell\u0027][\u0027name\u0027]"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ae5cdf2_7b14ade8","line":40,"in_reply_to":"1ae5cdf2_c51409eb","updated":"2014-06-05 11:33:57.000000000","message":"Done","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"690a3cf1156f0d6d34896900c84f5cc7d1df75a3","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def tearDown(self):"},{"line_number":46,"context_line":"        try:"},{"line_number":47,"context_line":"            self.client.delete_cell(self.cell)"},{"line_number":48,"context_line":"        except exceptions.NotFound:"},{"line_number":49,"context_line":"            pass"},{"line_number":50,"context_line":"        except Exception:"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ae5cdf2_45e0f9f9","line":47,"updated":"2014-06-05 07:30:46.000000000","message":"ditto. can be moved to tearDownClass","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ebbf067c62141d7d2bd553ed27c2e7d12239a2c7","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def tearDown(self):"},{"line_number":46,"context_line":"        try:"},{"line_number":47,"context_line":"            self.client.delete_cell(self.cell)"},{"line_number":48,"context_line":"        except exceptions.NotFound:"},{"line_number":49,"context_line":"            pass"},{"line_number":50,"context_line":"        except Exception:"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ae5cdf2_9bd6d926","line":47,"in_reply_to":"1ae5cdf2_45e0f9f9","updated":"2014-06-05 11:33:57.000000000","message":"Done","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"690a3cf1156f0d6d34896900c84f5cc7d1df75a3","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        # list all cells"},{"line_number":71,"context_line":"        resp, cells \u003d self.client.list_cells()"},{"line_number":72,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":73,"context_line":"        self.assertIn(cell_name, [c[\u0027name\u0027] for c in cells[\u0027cells\u0027]])"},{"line_number":74,"context_line":"        # Delete created cell"},{"line_number":75,"context_line":"        resp, _ \u003d self.client.delete_cell(cell_name)"},{"line_number":76,"context_line":"        self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ae5cdf2_455b99f5","line":73,"updated":"2014-06-05 07:30:46.000000000","message":"check for cell created in setUp method too.","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"ebbf067c62141d7d2bd553ed27c2e7d12239a2c7","unresolved":false,"context_lines":[{"line_number":70,"context_line":"        # list all cells"},{"line_number":71,"context_line":"        resp, cells \u003d self.client.list_cells()"},{"line_number":72,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":73,"context_line":"        self.assertIn(cell_name, [c[\u0027name\u0027] for c in cells[\u0027cells\u0027]])"},{"line_number":74,"context_line":"        # Delete created cell"},{"line_number":75,"context_line":"        resp, _ \u003d self.client.delete_cell(cell_name)"},{"line_number":76,"context_line":"        self.assertEqual(200, resp.status)"}],"source_content_type":"text/x-python","patch_set":7,"id":"1ae5cdf2_bbd9d558","line":73,"in_reply_to":"1ae5cdf2_455b99f5","updated":"2014-06-05 11:33:57.000000000","message":"Done","commit_id":"d37467f563a47fc6ae197991426350b492b4dc8a"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"170f8aa557bed90c4384a6e973744b23937a155d","unresolved":false,"context_lines":[{"line_number":32,"context_line":"    @classmethod"},{"line_number":33,"context_line":"    def setUpClass(cls):"},{"line_number":34,"context_line":"        super(CellsAdminTestJSON, cls).setUpClass()"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        if not CONF.compute_feature_enabled.cell:"},{"line_number":37,"context_line":"            msg \u003d \"cell extension not enabled.\""},{"line_number":38,"context_line":"            raise cls.skipException(msg)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_deef41e8","line":35,"updated":"2014-06-06 14:17:50.000000000","message":"Why you removed the extension check?","commit_id":"b15d535e160f1e83442d2e0efee310caa89a5c96"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"170f8aa557bed90c4384a6e973744b23937a155d","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        super(CellsAdminTestJSON, cls).setUpClass()"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        if not CONF.compute_feature_enabled.cell:"},{"line_number":37,"context_line":"            msg \u003d \"cell extension not enabled.\""},{"line_number":38,"context_line":"            raise cls.skipException(msg)"},{"line_number":39,"context_line":"        cls.cell \u003d data_utils.rand_name(\"test-cell\")"},{"line_number":40,"context_line":"        resp, body \u003d cls.create_cell(cls.cell)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_9e1e990c","line":37,"updated":"2014-06-06 14:17:50.000000000","message":"this is not extension. this is feature flag.","commit_id":"b15d535e160f1e83442d2e0efee310caa89a5c96"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"170f8aa557bed90c4384a6e973744b23937a155d","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"        if not CONF.compute_feature_enabled.cell:"},{"line_number":37,"context_line":"            msg \u003d \"cell extension not enabled.\""},{"line_number":38,"context_line":"            raise cls.skipException(msg)"},{"line_number":39,"context_line":"        cls.cell \u003d data_utils.rand_name(\"test-cell\")"},{"line_number":40,"context_line":"        resp, body \u003d cls.create_cell(cls.cell)"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_1e8e0928","line":38,"updated":"2014-06-06 14:17:50.000000000","message":"Please move both as skip decorator to have skipped test list with  skip_tracker tool.","commit_id":"b15d535e160f1e83442d2e0efee310caa89a5c96"},{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"414f2b785a4ee5611a0e4295c6dc697baa19dc56","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    def setUpClass(cls):"},{"line_number":34,"context_line":"        if not CONF.compute_feature_enabled.cell:"},{"line_number":35,"context_line":"            msg \u003d \"cell tests are disabled.\""},{"line_number":36,"context_line":"            raise cls.skipException(msg)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        super(CellsAdminTestJSON, cls).setUpClass()"},{"line_number":39,"context_line":"        cls.cells_client \u003d cls.os_adm.cells_client"}],"source_content_type":"text/x-python","patch_set":10,"id":"1ae5cdf2_0bb91b15","line":36,"updated":"2014-06-10 06:13:56.000000000","message":"nit: it\u0027s not the cell\u0027s tests that are disabled, but the feature, I guess we should clarify that on the message.","commit_id":"1e148ce4039dbcec7fb437f15da4d98333645c1e"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"8e445b41c4e1756461299d05f1e77b1e6b5278ef","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        super(CellsAdminTestJSON, cls).setUpClass()"},{"line_number":39,"context_line":"        cls.cells_client \u003d cls.os_adm.cells_client"},{"line_number":40,"context_line":"        cls.cell \u003d data_utils.rand_name(\"test-cell\")"},{"line_number":41,"context_line":"        resp, body \u003d cls.create_cell(cls.cell)"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def _try_delete_cell(self, cell_name):"}],"source_content_type":"text/x-python","patch_set":11,"id":"1ae5cdf2_48a3ee9d","line":40,"updated":"2014-06-18 15:53:43.000000000","message":"Please, rename this variable. For example, to cls.cell_name.","commit_id":"bb71c937362b655040c3cc55d2e513e90e5a2f35"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"8e445b41c4e1756461299d05f1e77b1e6b5278ef","unresolved":false,"context_lines":[{"line_number":38,"context_line":"        super(CellsAdminTestJSON, cls).setUpClass()"},{"line_number":39,"context_line":"        cls.cells_client \u003d cls.os_adm.cells_client"},{"line_number":40,"context_line":"        cls.cell \u003d data_utils.rand_name(\"test-cell\")"},{"line_number":41,"context_line":"        resp, body \u003d cls.create_cell(cls.cell)"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    def _try_delete_cell(self, cell_name):"},{"line_number":44,"context_line":"        # delete cell, if it exists"}],"source_content_type":"text/x-python","patch_set":11,"id":"1ae5cdf2_68f50a82","line":41,"updated":"2014-06-18 15:53:43.000000000","message":"You can just write\n\n    cls.create_cell(cls.cell)\n\nThere is no need in extra variables.","commit_id":"bb71c937362b655040c3cc55d2e513e90e5a2f35"}],"tempest/api/compute/base.py":[{"author":{"_account_id":8824,"name":"Sergey Murashov","email":"smurashov@mirantis.com","username":"SergeyMurashov"},"change_message_id":"ba98459be69ea787f9585dc6d03ca25d742794ee","unresolved":false,"context_lines":[{"line_number":131,"context_line":"                LOG.info(\u0027Exception raised deleting cell %s\u0027,"},{"line_number":132,"context_line":"                         cell[\u0027name\u0027])"},{"line_number":133,"context_line":"                LOG.exception(exc)"},{"line_number":134,"context_line":"                pass"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"    @classmethod"},{"line_number":137,"context_line":"    def tearDownClass(cls):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXX%2F%2F5GU%3D","line":134,"updated":"2014-04-25 13:54:34.000000000","message":"remove \"pass\" please, it\u0027s redundant","commit_id":"e1b03c3bfc54b577d00a17f10e31e5d5d089dfe1"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"5e797fec96d64c5d312182c2a781a6fdc3ee0f32","unresolved":false,"context_lines":[{"line_number":131,"context_line":"                LOG.info(\u0027Exception raised deleting cell %s\u0027,"},{"line_number":132,"context_line":"                         cell[\u0027name\u0027])"},{"line_number":133,"context_line":"                LOG.exception(exc)"},{"line_number":134,"context_line":"                pass"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"    @classmethod"},{"line_number":137,"context_line":"    def tearDownClass(cls):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXX%2F%2F4uo%3D","line":134,"in_reply_to":"AAAAXX%2F%2F5GU%3D","updated":"2014-04-25 14:54:09.000000000","message":"Will do in next patch.","commit_id":"e1b03c3bfc54b577d00a17f10e31e5d5d089dfe1"},{"author":{"_account_id":8824,"name":"Sergey Murashov","email":"smurashov@mirantis.com","username":"SergeyMurashov"},"change_message_id":"89ccbbda2f44df03856df8931c5c6b545a1fbaaa","unresolved":false,"context_lines":[{"line_number":122,"context_line":"    def clear_cells(cls):"},{"line_number":123,"context_line":"        for cell in cls.cells:"},{"line_number":124,"context_line":"            try:"},{"line_number":125,"context_line":"                resp, body \u003d\\"},{"line_number":126,"context_line":"                    cls.client.delete_cell(cell)"},{"line_number":127,"context_line":"            except exceptions.NotFound:"},{"line_number":128,"context_line":"                # The cell may have already been deleted which is OK."}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAXX%2F%2Fwms%3D","line":125,"updated":"2014-04-28 07:22:33.000000000","message":"unused variables \"resp\" and \"body\", may be would be better:\n\ncls.client.delete_cell(cell)\n\ninstead\n\nresp, body \u003d cls.client.delete_cell(cell)\n\n?","commit_id":"3f77c1236b0a4b7e66f78624c9dfe7b6addc87c1"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"4650c80ee2bedc091a2cca416e5f702a2273bcf7","unresolved":false,"context_lines":[{"line_number":122,"context_line":"    def clear_cells(cls):"},{"line_number":123,"context_line":"        for cell in cls.cells:"},{"line_number":124,"context_line":"            try:"},{"line_number":125,"context_line":"                resp, body \u003d\\"},{"line_number":126,"context_line":"                    cls.client.delete_cell(cell)"},{"line_number":127,"context_line":"            except exceptions.NotFound:"},{"line_number":128,"context_line":"                # The cell may have already been deleted which is OK."}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAXX%2F%2Fwes%3D","line":125,"in_reply_to":"AAAAXX%2F%2Fwms%3D","updated":"2014-04-28 07:45:51.000000000","message":"Done","commit_id":"3f77c1236b0a4b7e66f78624c9dfe7b6addc87c1"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"170f8aa557bed90c4384a6e973744b23937a155d","unresolved":false,"context_lines":[{"line_number":428,"context_line":"            cls.agents_admin_client \u003d cls.os_adm.agents_v3_client"},{"line_number":429,"context_line":"            cls.migrations_admin_client \u003d cls.os_adm.migrations_v3_client"},{"line_number":430,"context_line":""},{"line_number":431,"context_line":"        cls.cells_client \u003d cls.os_adm.cells_client"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":""},{"line_number":434,"context_line":"class BaseV2ComputeAdminTest(BaseComputeAdminTest):"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_9ecf7957","line":431,"updated":"2014-06-06 14:17:50.000000000","message":"Please move this to BaseComputeTest under v2 check same way as other admin clients are initialized.","commit_id":"b15d535e160f1e83442d2e0efee310caa89a5c96"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"77222e5c75b1647ed954d0a989768ca16c5e4254","unresolved":false,"context_lines":[{"line_number":428,"context_line":"            cls.agents_admin_client \u003d cls.os_adm.agents_v3_client"},{"line_number":429,"context_line":"            cls.migrations_admin_client \u003d cls.os_adm.migrations_v3_client"},{"line_number":430,"context_line":""},{"line_number":431,"context_line":"        cls.cells_client \u003d cls.os_adm.cells_client"},{"line_number":432,"context_line":""},{"line_number":433,"context_line":""},{"line_number":434,"context_line":"class BaseV2ComputeAdminTest(BaseComputeAdminTest):"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_8e8f2775","line":431,"in_reply_to":"1ae5cdf2_9ecf7957","updated":"2014-06-09 10:10:41.000000000","message":"Done","commit_id":"b15d535e160f1e83442d2e0efee310caa89a5c96"}],"tempest/clients.py":[{"author":{"_account_id":10966,"name":"Abhijeet Jain","email":"abhijeetjpjain@gmail.com","username":"Abhijeet.Jain"},"change_message_id":"679e50d4212c094b1c7d89ee4ca7703b8f0bcb9f","unresolved":false,"context_lines":[{"line_number":303,"context_line":"                self.auth_provider)"},{"line_number":304,"context_line":"            self.security_groups_client \u003d SecurityGroupsClientJSON("},{"line_number":305,"context_line":"                self.auth_provider)"},{"line_number":306,"context_line":"            self.cells_client \u003d CellsClientJSON(self.auth_provider)"},{"line_number":307,"context_line":"            self.interfaces_v3_client \u003d InterfacesV3ClientJSON("},{"line_number":308,"context_line":"                self.auth_provider)"},{"line_number":309,"context_line":"            self.interfaces_client \u003d InterfacesClientJSON("}],"source_content_type":"text/x-python","patch_set":5,"id":"9ad9bd40_b133da60","line":306,"updated":"2014-05-07 06:43:09.000000000","message":"Hi raies,\nDeclare this client out of if/else part as you don\u0027t have any XML client. Its causing the failure of Jenkins.\ncheck here : http://logs.openstack.org/37/90337/5/check/check-tempest-dsvm-full/2c2c929/console.html.gz\n\nIt fails all the XML tests for other clients as it doesn\u0027t find  self.cells_client variable while running for XML.","commit_id":"7e6596f9edec373a8aab256207f5eb4484165301"}],"tempest/config.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6bfd1f293171171a43b0ccc529f32eb7244df14d","unresolved":false,"context_lines":[{"line_number":256,"context_line":"ComputeFeaturesGroup \u003d ["},{"line_number":257,"context_line":"    cfg.BoolOpt(\u0027cell\u0027,"},{"line_number":258,"context_line":"                default\u003dTrue,"},{"line_number":259,"context_line":"                help\u003d\"If false, skip all cell tests.\"),"},{"line_number":260,"context_line":"    cfg.BoolOpt(\u0027api_v3\u0027,"},{"line_number":261,"context_line":"                default\u003dTrue,"},{"line_number":262,"context_line":"                help\u003d\"If false, skip all nova v3 tests.\"),"}],"source_content_type":"text/x-python","patch_set":8,"id":"1ae5cdf2_62cdc230","line":259,"updated":"2014-06-06 07:54:10.000000000","message":"Give proper help description. something saying - Enable \u0027cell\u0027 feature. If environment is configured with \u0027cell\u0027 service then this should flag should be set as in nova configuration.","commit_id":"b8b1c19147a7f78cebdabf7937698a9c4cb96987"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"2325711e941e4bd44d6d03f6a0a998fe5783ba23","unresolved":false,"context_lines":[{"line_number":256,"context_line":"ComputeFeaturesGroup \u003d ["},{"line_number":257,"context_line":"    cfg.BoolOpt(\u0027cell\u0027,"},{"line_number":258,"context_line":"                default\u003dTrue,"},{"line_number":259,"context_line":"                help\u003d\"If false, skip all cell tests.\"),"},{"line_number":260,"context_line":"    cfg.BoolOpt(\u0027api_v3\u0027,"},{"line_number":261,"context_line":"                default\u003dTrue,"},{"line_number":262,"context_line":"                help\u003d\"If false, skip all nova v3 tests.\"),"}],"source_content_type":"text/x-python","patch_set":8,"id":"1ae5cdf2_65fdc76d","line":259,"in_reply_to":"1ae5cdf2_62cdc230","updated":"2014-06-06 10:43:42.000000000","message":"Done","commit_id":"b8b1c19147a7f78cebdabf7937698a9c4cb96987"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"170f8aa557bed90c4384a6e973744b23937a155d","unresolved":false,"context_lines":[{"line_number":257,"context_line":"    cfg.BoolOpt(\u0027cell\u0027,"},{"line_number":258,"context_line":"                default\u003dTrue,"},{"line_number":259,"context_line":"                help\u003d\u0027Enable cell feature,\u0027"},{"line_number":260,"context_line":"                     \u0027If environment is configured with cell service then this\u0027"},{"line_number":261,"context_line":"                     \u0027should flag should be set as in nova configuration.\u0027),"},{"line_number":262,"context_line":"    cfg.BoolOpt(\u0027api_v3\u0027,"},{"line_number":263,"context_line":"                default\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_be3a7532","line":260,"updated":"2014-06-06 14:17:50.000000000","message":"Space is needed here.","commit_id":"b15d535e160f1e83442d2e0efee310caa89a5c96"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"77222e5c75b1647ed954d0a989768ca16c5e4254","unresolved":false,"context_lines":[{"line_number":257,"context_line":"    cfg.BoolOpt(\u0027cell\u0027,"},{"line_number":258,"context_line":"                default\u003dTrue,"},{"line_number":259,"context_line":"                help\u003d\u0027Enable cell feature,\u0027"},{"line_number":260,"context_line":"                     \u0027If environment is configured with cell service then this\u0027"},{"line_number":261,"context_line":"                     \u0027should flag should be set as in nova configuration.\u0027),"},{"line_number":262,"context_line":"    cfg.BoolOpt(\u0027api_v3\u0027,"},{"line_number":263,"context_line":"                default\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_33875c11","line":260,"in_reply_to":"1ae5cdf2_be3a7532","updated":"2014-06-09 10:10:41.000000000","message":"Done","commit_id":"b15d535e160f1e83442d2e0efee310caa89a5c96"}],"tempest/services/compute/json/cells_client.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"170f8aa557bed90c4384a6e973744b23937a155d","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        return resp, body"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def create_cell(self, name, **kwargs):"},{"line_number":40,"context_line":"        \"\"\"Creates a new flavor or instance type.\"\"\""},{"line_number":41,"context_line":"        post_body \u003d {\u0027cell\u0027: kwargs}"},{"line_number":42,"context_line":"        post_body[\u0027cell\u0027][\u0027name\u0027] \u003d name"},{"line_number":43,"context_line":"        body \u003d json.dumps(post_body)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_fe1b1dc8","line":40,"updated":"2014-06-06 14:17:50.000000000","message":"Please correct the Doc string.","commit_id":"b15d535e160f1e83442d2e0efee310caa89a5c96"},{"author":{"_account_id":8205,"name":"Mh Raies","email":"raiesmh08@gmail.com","username":"raiesmh08"},"change_message_id":"77222e5c75b1647ed954d0a989768ca16c5e4254","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        return resp, body"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    def create_cell(self, name, **kwargs):"},{"line_number":40,"context_line":"        \"\"\"Creates a new flavor or instance type.\"\"\""},{"line_number":41,"context_line":"        post_body \u003d {\u0027cell\u0027: kwargs}"},{"line_number":42,"context_line":"        post_body[\u0027cell\u0027][\u0027name\u0027] \u003d name"},{"line_number":43,"context_line":"        body \u003d json.dumps(post_body)"}],"source_content_type":"text/x-python","patch_set":9,"id":"1ae5cdf2_7391e451","line":40,"in_reply_to":"1ae5cdf2_fe1b1dc8","updated":"2014-06-09 10:10:41.000000000","message":"Done","commit_id":"b15d535e160f1e83442d2e0efee310caa89a5c96"}]}