)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"bb4bcde6eb9626b44d71d0b97acbc2c33df0d77b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Retype support for CloudByte iSCSI cinder driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch will enable the QoS control on CloudByte iSCSI cinder driver."},{"line_number":10,"context_line":"Admin can control the IOPS, grace like many other QOS properties of a"},{"line_number":11,"context_line":"volumes via OpenStack dashboard."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4b05e49c545fb284e7abb90ad0661bcba5b646b7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da20952f_2b994d7c","line":10,"updated":"2015-08-31 14:30:20.000000000","message":"\"grace\" does not make sense to me in this context. Do you mean \"just\"?","commit_id":"b1f51dce9cb1c36066273df62f6f9cc7089093b7"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"93ddbcc988558711a887b535b6b4a9343d5b8099","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Retype support for CloudByte iSCSI cinder driver"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch will enable the QoS control on CloudByte iSCSI cinder driver."},{"line_number":10,"context_line":"Admin can control the IOPS, grace like many other QOS properties of a"},{"line_number":11,"context_line":"volumes via OpenStack dashboard."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4b05e49c545fb284e7abb90ad0661bcba5b646b7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da20952f_e505b845","line":10,"in_reply_to":"da20952f_2b994d7c","updated":"2015-08-31 18:07:46.000000000","message":"Thanks for comments\n\nActually cloudbyte has a QoS  properties named graceallowed.","commit_id":"b1f51dce9cb1c36066273df62f6f9cc7089093b7"},{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"bb4bcde6eb9626b44d71d0b97acbc2c33df0d77b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch will enable the QoS control on CloudByte iSCSI cinder driver."},{"line_number":10,"context_line":"Admin can control the IOPS, grace like many other QOS properties of a"},{"line_number":11,"context_line":"volumes via OpenStack dashboard."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4b05e49c545fb284e7abb90ad0661bcba5b646b7"},{"line_number":14,"context_line":"Implements: blueprint cloudbyte-driver-support-retype"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da20952f_2b806df4","line":11,"updated":"2015-08-31 14:30:20.000000000","message":"volumes -\u003e volume","commit_id":"b1f51dce9cb1c36066273df62f6f9cc7089093b7"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"93ddbcc988558711a887b535b6b4a9343d5b8099","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch will enable the QoS control on CloudByte iSCSI cinder driver."},{"line_number":10,"context_line":"Admin can control the IOPS, grace like many other QOS properties of a"},{"line_number":11,"context_line":"volumes via OpenStack dashboard."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4b05e49c545fb284e7abb90ad0661bcba5b646b7"},{"line_number":14,"context_line":"Implements: blueprint cloudbyte-driver-support-retype"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da20952f_25fdf06d","line":11,"in_reply_to":"da20952f_2b806df4","updated":"2015-08-31 18:07:46.000000000","message":"Done","commit_id":"b1f51dce9cb1c36066273df62f6f9cc7089093b7"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"7085072fda03ad727d715a1cbc100fa1f057a1dc","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch enables the volume retype function for CloudByte iSCSI cinder"},{"line_number":10,"context_line":"driver. Admin can control the IOPS, graceallowed, compression and many"},{"line_number":11,"context_line":"other QOS properties of a volume via OpenStack."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4b05e49c545fb284e7abb90ad0661bcba5b646b7"},{"line_number":14,"context_line":"Implements: blueprint cloudbyte-driver-support-retype"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"9a1a9d01_54fbbd47","line":12,"updated":"2015-10-01 15:41:11.000000000","message":"Need DocImpact tag please.","commit_id":"718a544e2d896d27c33cc91a5f486ba2c2fcd2fc"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"48447559d14fe5032fab2173f1d74b857c218805","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch enables the volume retype function for CloudByte iSCSI cinder"},{"line_number":10,"context_line":"driver. Admin can control the IOPS, graceallowed, compression and many"},{"line_number":11,"context_line":"other QOS properties of a volume via OpenStack."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4b05e49c545fb284e7abb90ad0661bcba5b646b7"},{"line_number":14,"context_line":"Implements: blueprint cloudbyte-driver-support-retype"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"9a1a9d01_a8cbc31d","line":12,"in_reply_to":"9a1a9d01_54fbbd47","updated":"2015-10-02 06:43:03.000000000","message":"Done","commit_id":"718a544e2d896d27c33cc91a5f486ba2c2fcd2fc"}],"cinder/tests/unit/test_cloudbyte.py":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"d4d60e90d80859d3eb466ed5f8bbbfb1d186554c","unresolved":false,"context_lines":[{"line_number":1517,"context_line":"                                           fake_new_type, None, None))"},{"line_number":1518,"context_line":""},{"line_number":1519,"context_line":"        # assert the invoked api calls"},{"line_number":1520,"context_line":"        self.assertEqual(3, mock_api_req.call_count)"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a1a9d01_0f2bf034","line":1520,"updated":"2015-10-01 15:50:16.000000000","message":"I am not sure if this test is thorough enough. Specifically I do not think we are testing every branch where an exception is raised.","commit_id":"718a544e2d896d27c33cc91a5f486ba2c2fcd2fc"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"48447559d14fe5032fab2173f1d74b857c218805","unresolved":false,"context_lines":[{"line_number":1517,"context_line":"                                           fake_new_type, None, None))"},{"line_number":1518,"context_line":""},{"line_number":1519,"context_line":"        # assert the invoked api calls"},{"line_number":1520,"context_line":"        self.assertEqual(3, mock_api_req.call_count)"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a1a9d01_4f932350","line":1520,"in_reply_to":"9a1a9d01_0f2bf034","updated":"2015-10-02 06:43:03.000000000","message":"Done","commit_id":"718a544e2d896d27c33cc91a5f486ba2c2fcd2fc"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"e3732b82a963a9a03280f24ef6d972381e6a7f6e","unresolved":false,"context_lines":[{"line_number":1544,"context_line":""},{"line_number":1545,"context_line":"        with testtools.ExpectedException("},{"line_number":1546,"context_line":"                exception.VolumeBackendAPIException,"},{"line_number":1547,"context_line":"                \"Bad or unexpected response from the storage volume \""},{"line_number":1548,"context_line":"                \"backend API: Volume \\[c93df32e-3a99-3491-8e10-cf318a7f9b7f\\] \""},{"line_number":1549,"context_line":"                \"not found at CloudByte storage.\"):"},{"line_number":1550,"context_line":"            self.driver.retype(self.ctxt, volume, fake_new_type, None, None)"}],"source_content_type":"text/x-python","patch_set":10,"id":"9a1a9d01_5035a443","line":1547,"updated":"2015-10-05 13:33:49.000000000","message":"I really would prefer not asserting on specific text. It would be better just to use assertRaises in my opinion. Text can be modified without someone realizing there is an unexpected dependency on it elsewhere in the code.","commit_id":"92c1c1f68621933b76d89ba897fffd8fac64cdd8"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"4038f99cf391a357ac8f87caafef7072299588ae","unresolved":false,"context_lines":[{"line_number":1544,"context_line":""},{"line_number":1545,"context_line":"        with testtools.ExpectedException("},{"line_number":1546,"context_line":"                exception.VolumeBackendAPIException,"},{"line_number":1547,"context_line":"                \"Bad or unexpected response from the storage volume \""},{"line_number":1548,"context_line":"                \"backend API: Volume \\[c93df32e-3a99-3491-8e10-cf318a7f9b7f\\] \""},{"line_number":1549,"context_line":"                \"not found at CloudByte storage.\"):"},{"line_number":1550,"context_line":"            self.driver.retype(self.ctxt, volume, fake_new_type, None, None)"}],"source_content_type":"text/x-python","patch_set":10,"id":"7a2fa921_705597c4","line":1547,"in_reply_to":"9a1a9d01_5035a443","updated":"2015-10-06 06:06:33.000000000","message":"I have made the changes according to you.\nPlease have a look.\n\nThanks!","commit_id":"92c1c1f68621933b76d89ba897fffd8fac64cdd8"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"04d67fd91547e9b9430b9c08afaff5c54fc05c7e","unresolved":false,"context_lines":[{"line_number":1528,"context_line":"        # assert the invoked api calls"},{"line_number":1529,"context_line":"        self.assertEqual(3, mock_api_req.call_count)"},{"line_number":1530,"context_line":""},{"line_number":1531,"context_line":"        # Test - II"},{"line_number":1532,"context_line":"        del volume[\u0027provider_id\u0027]"},{"line_number":1533,"context_line":""},{"line_number":1534,"context_line":"        # Now run the test \u0026 assert the exception"}],"source_content_type":"text/x-python","patch_set":15,"id":"5a2ca52d_cb7dac1d","line":1531,"updated":"2015-10-14 17:12:10.000000000","message":"Please, split this to 3 test methods","commit_id":"e15ee740af5fd7eb52114dd2ec98388f735b0165"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"878dfa4ea743f627db1e7ba5a30dd2f5d5c37798","unresolved":false,"context_lines":[{"line_number":1528,"context_line":"        # assert the invoked api calls"},{"line_number":1529,"context_line":"        self.assertEqual(3, mock_api_req.call_count)"},{"line_number":1530,"context_line":""},{"line_number":1531,"context_line":"        # Test - II"},{"line_number":1532,"context_line":"        del volume[\u0027provider_id\u0027]"},{"line_number":1533,"context_line":""},{"line_number":1534,"context_line":"        # Now run the test \u0026 assert the exception"}],"source_content_type":"text/x-python","patch_set":15,"id":"3a29b11f_f4176d52","line":1531,"in_reply_to":"5a2ca52d_cb7dac1d","updated":"2015-10-18 10:03:11.000000000","message":"done.\nPlease have a look.","commit_id":"e15ee740af5fd7eb52114dd2ec98388f735b0165"}],"cinder/volume/drivers/cloudbyte/cloudbyte.py":[{"author":{"_account_id":17405,"name":"Jacob Gregor","email":"jgregor@us.ibm.com","username":"jgregor"},"change_message_id":"bb4bcde6eb9626b44d71d0b97acbc2c33df0d77b","unresolved":false,"context_lines":[{"line_number":723,"context_line":"        type_id \u003d volume[\u0027volume_type_id\u0027]"},{"line_number":724,"context_line":""},{"line_number":725,"context_line":"        if type_id is not None:"},{"line_number":726,"context_line":"            qos_group_params, file_system_params \u003d\\"},{"line_number":727,"context_line":"                (self._get_qos_by_volume_type(ctxt, type_id))"},{"line_number":728,"context_line":""},{"line_number":729,"context_line":"        LOG.debug(\"Will create a volume [%(cb_vol)s] in TSM [%(tsm)s] \""}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_abf1dd6c","line":726,"updated":"2015-08-31 14:30:20.000000000","message":"Use () to wrap long lines instead of \\ for line continuation.","commit_id":"b1f51dce9cb1c36066273df62f6f9cc7089093b7"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"93ddbcc988558711a887b535b6b4a9343d5b8099","unresolved":false,"context_lines":[{"line_number":723,"context_line":"        type_id \u003d volume[\u0027volume_type_id\u0027]"},{"line_number":724,"context_line":""},{"line_number":725,"context_line":"        if type_id is not None:"},{"line_number":726,"context_line":"            qos_group_params, file_system_params \u003d\\"},{"line_number":727,"context_line":"                (self._get_qos_by_volume_type(ctxt, type_id))"},{"line_number":728,"context_line":""},{"line_number":729,"context_line":"        LOG.debug(\"Will create a volume [%(cb_vol)s] in TSM [%(tsm)s] \""}],"source_content_type":"text/x-python","patch_set":1,"id":"da20952f_45b82475","line":726,"in_reply_to":"da20952f_abf1dd6c","updated":"2015-08-31 18:07:46.000000000","message":"Done","commit_id":"b1f51dce9cb1c36066273df62f6f9cc7089093b7"},{"author":{"_account_id":17718,"name":"Lallana","email":"gaozhiqiang2@huawei.com","username":"Lallana"},"change_message_id":"9fd1d5ec148e7eb73ee7fb5fe2490805cc855d2c","unresolved":false,"context_lines":[{"line_number":1096,"context_line":""},{"line_number":1097,"context_line":"        if cb_volume_id is None:"},{"line_number":1098,"context_line":"            message \u003d _(\"Provider information not found for openstack \""},{"line_number":1099,"context_line":"                        \"volume [%s].\") % volume[\u0027display_name\u0027]"},{"line_number":1100,"context_line":"            raise exception.VolumeDriverException(message)"},{"line_number":1101,"context_line":""},{"line_number":1102,"context_line":"        update_qos_group_params, update_file_system_params \u003d ("}],"source_content_type":"text/x-python","patch_set":5,"id":"da20952f_217f04e1","line":1099,"updated":"2015-09-01 09:34:44.000000000","message":"\u0027display_name\u0027 can be modified, and there may be multiple volume using the same display_name, \u0027display_name\u0027 has uniqueness.","commit_id":"a6253715d7780182537bd41f0920861b8063a203"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"fb90d5c5d11e5f6d5b0b2a39498df4bbd8d75209","unresolved":false,"context_lines":[{"line_number":1096,"context_line":""},{"line_number":1097,"context_line":"        if cb_volume_id is None:"},{"line_number":1098,"context_line":"            message \u003d _(\"Provider information not found for openstack \""},{"line_number":1099,"context_line":"                        \"volume [%s].\") % volume[\u0027display_name\u0027]"},{"line_number":1100,"context_line":"            raise exception.VolumeDriverException(message)"},{"line_number":1101,"context_line":""},{"line_number":1102,"context_line":"        update_qos_group_params, update_file_system_params \u003d ("}],"source_content_type":"text/x-python","patch_set":5,"id":"da20952f_84093933","line":1099,"in_reply_to":"da20952f_217f04e1","updated":"2015-09-08 04:55:50.000000000","message":"Done","commit_id":"a6253715d7780182537bd41f0920861b8063a203"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"651f995ad3cd1455f344c81cf5e794e2b5a7a757","unresolved":false,"context_lines":[{"line_number":1143,"context_line":"            self._get_qos_by_volume_type(ctxt, new_type[\u0027id\u0027]))"},{"line_number":1144,"context_line":""},{"line_number":1145,"context_line":"        if update_qos_group_params:"},{"line_number":1146,"context_line":"            listFileSysParams \u003d {\u0027id\u0027: cb_volume_id}"},{"line_number":1147,"context_line":"            response \u003d self._api_request_for_cloudbyte("},{"line_number":1148,"context_line":"                \u0027listFileSystem\u0027, listFileSysParams)"},{"line_number":1149,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"9a1a9d01_f34ac2ab","line":1146,"updated":"2015-09-28 15:51:24.000000000","message":"According to PEP-8, should be list_file_sys_params","commit_id":"184b63b216552762f69f4785a11b3529ca69e6fa"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"989fa6e88df45ce55bf53bc7a4fa12b89547377f","unresolved":false,"context_lines":[{"line_number":1143,"context_line":"            self._get_qos_by_volume_type(ctxt, new_type[\u0027id\u0027]))"},{"line_number":1144,"context_line":""},{"line_number":1145,"context_line":"        if update_qos_group_params:"},{"line_number":1146,"context_line":"            listFileSysParams \u003d {\u0027id\u0027: cb_volume_id}"},{"line_number":1147,"context_line":"            response \u003d self._api_request_for_cloudbyte("},{"line_number":1148,"context_line":"                \u0027listFileSystem\u0027, listFileSysParams)"},{"line_number":1149,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"9a1a9d01_d0eb6711","line":1146,"in_reply_to":"9a1a9d01_f34ac2ab","updated":"2015-10-01 11:04:14.000000000","message":"Done","commit_id":"184b63b216552762f69f4785a11b3529ca69e6fa"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"d33289e93b29fd39dc6ba7d7aabfefe1dbcd18c4","unresolved":false,"context_lines":[{"line_number":192,"context_line":"                               volume,"},{"line_number":193,"context_line":"                               tsmid,"},{"line_number":194,"context_line":"                               volume_name,"},{"line_number":195,"context_line":"                               qos_group_params):"},{"line_number":196,"context_line":"        # Prepare the user input params"},{"line_number":197,"context_line":"        params \u003d {"},{"line_number":198,"context_line":"            \"name\": \"QoS_\" + volume_name,"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a1a9d01_9453956c","line":195,"updated":"2015-10-01 15:38:19.000000000","message":"this formatting is a bit clunky and hard to read.  please don\u0027t newline on every ,","commit_id":"718a544e2d896d27c33cc91a5f486ba2c2fcd2fc"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"48447559d14fe5032fab2173f1d74b857c218805","unresolved":false,"context_lines":[{"line_number":192,"context_line":"                               volume,"},{"line_number":193,"context_line":"                               tsmid,"},{"line_number":194,"context_line":"                               volume_name,"},{"line_number":195,"context_line":"                               qos_group_params):"},{"line_number":196,"context_line":"        # Prepare the user input params"},{"line_number":197,"context_line":"        params \u003d {"},{"line_number":198,"context_line":"            \"name\": \"QoS_\" + volume_name,"}],"source_content_type":"text/x-python","patch_set":8,"id":"9a1a9d01_e9b47783","line":195,"in_reply_to":"9a1a9d01_9453956c","updated":"2015-10-02 06:43:03.000000000","message":"Done","commit_id":"718a544e2d896d27c33cc91a5f486ba2c2fcd2fc"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"d33289e93b29fd39dc6ba7d7aabfefe1dbcd18c4","unresolved":false,"context_lines":[{"line_number":1199,"context_line":"            cb_volume \u003d cb_volume_list[0]"},{"line_number":1200,"context_line":""},{"line_number":1201,"context_line":"            if not cb_volume:"},{"line_number":1202,"context_line":"                msg \u003d (\"Volume [%s] not found at \""},{"line_number":1203,"context_line":"                       \"CloudByte storage.\") % cb_volume_id"},{"line_number":1204,"context_line":"                raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1205,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"9a1a9d01_74f8414a","line":1202,"updated":"2015-10-01 15:38:19.000000000","message":"missing _()","commit_id":"718a544e2d896d27c33cc91a5f486ba2c2fcd2fc"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"48447559d14fe5032fab2173f1d74b857c218805","unresolved":false,"context_lines":[{"line_number":1199,"context_line":"            cb_volume \u003d cb_volume_list[0]"},{"line_number":1200,"context_line":""},{"line_number":1201,"context_line":"            if not cb_volume:"},{"line_number":1202,"context_line":"                msg \u003d (\"Volume [%s] not found at \""},{"line_number":1203,"context_line":"                       \"CloudByte storage.\") % cb_volume_id"},{"line_number":1204,"context_line":"                raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1205,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"9a1a9d01_69762708","line":1202,"in_reply_to":"9a1a9d01_74f8414a","updated":"2015-10-02 06:43:03.000000000","message":"Done","commit_id":"718a544e2d896d27c33cc91a5f486ba2c2fcd2fc"},{"author":{"_account_id":14377,"name":"Apoorva","email":"apps.desh@gmail.com","username":"apoorvad"},"change_message_id":"a9f63ac33d48e109f6b9b7c2dad6f662781feb0f","unresolved":false,"context_lines":[{"line_number":1179,"context_line":"        cb_volume_id \u003d volume.get(\u0027provider_id\u0027)"},{"line_number":1180,"context_line":""},{"line_number":1181,"context_line":"        if cb_volume_id is None:"},{"line_number":1182,"context_line":"            message \u003d _(\"Provider information not found for openstack \""},{"line_number":1183,"context_line":"                        \"volume [%s].\") % volume[\u0027id\u0027]"},{"line_number":1184,"context_line":"            raise exception.VolumeBackendAPIException(message)"},{"line_number":1185,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"7a2fa921_1ec8d521","line":1182,"updated":"2015-10-06 18:37:06.000000000","message":"s/openstack/OpenStack","commit_id":"2db6728242a83d2f1515d8a489bb07f429c2e98b"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"742f0b177f04847e08ae314277458b1fc322cfb6","unresolved":false,"context_lines":[{"line_number":1179,"context_line":"        cb_volume_id \u003d volume.get(\u0027provider_id\u0027)"},{"line_number":1180,"context_line":""},{"line_number":1181,"context_line":"        if cb_volume_id is None:"},{"line_number":1182,"context_line":"            message \u003d _(\"Provider information not found for openstack \""},{"line_number":1183,"context_line":"                        \"volume [%s].\") % volume[\u0027id\u0027]"},{"line_number":1184,"context_line":"            raise exception.VolumeBackendAPIException(message)"},{"line_number":1185,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"7a2fa921_b53becea","line":1182,"in_reply_to":"7a2fa921_1ec8d521","updated":"2015-10-07 06:18:37.000000000","message":"Done.","commit_id":"2db6728242a83d2f1515d8a489bb07f429c2e98b"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"3d1378fecdedc6f556aef3f150fe677c40b97b7f","unresolved":false,"context_lines":[{"line_number":1197,"context_line":""},{"line_number":1198,"context_line":"            if not cb_volume:"},{"line_number":1199,"context_line":"                msg \u003d _(\"Volume [%s] not found at \""},{"line_number":1200,"context_line":"                        \"CloudByte storage.\") % cb_volume_id"},{"line_number":1201,"context_line":"                raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1202,"context_line":""},{"line_number":1203,"context_line":"            update_qos_group_params[\u0027id\u0027] \u003d cb_volume.get(\u0027groupid\u0027)"}],"source_content_type":"text/x-python","patch_set":11,"id":"7a2fa921_18eb05b2","line":1200,"updated":"2015-10-06 20:40:14.000000000","message":"Please, add cinder volume ID to the error message too. It will help to debug.","commit_id":"2db6728242a83d2f1515d8a489bb07f429c2e98b"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"742f0b177f04847e08ae314277458b1fc322cfb6","unresolved":false,"context_lines":[{"line_number":1197,"context_line":""},{"line_number":1198,"context_line":"            if not cb_volume:"},{"line_number":1199,"context_line":"                msg \u003d _(\"Volume [%s] not found at \""},{"line_number":1200,"context_line":"                        \"CloudByte storage.\") % cb_volume_id"},{"line_number":1201,"context_line":"                raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1202,"context_line":""},{"line_number":1203,"context_line":"            update_qos_group_params[\u0027id\u0027] \u003d cb_volume.get(\u0027groupid\u0027)"}],"source_content_type":"text/x-python","patch_set":11,"id":"7a2fa921_35303c01","line":1200,"in_reply_to":"7a2fa921_18eb05b2","updated":"2015-10-07 06:18:37.000000000","message":"Done.","commit_id":"2db6728242a83d2f1515d8a489bb07f429c2e98b"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"3d1378fecdedc6f556aef3f150fe677c40b97b7f","unresolved":false,"context_lines":[{"line_number":1211,"context_line":"                \u0027updateFileSystem\u0027, update_file_system_params)"},{"line_number":1212,"context_line":""},{"line_number":1213,"context_line":"        LOG.info(_LI(\"CloudByte volume [%s] updated successfully.\"),"},{"line_number":1214,"context_line":"                 cb_volume_id)"},{"line_number":1215,"context_line":""},{"line_number":1216,"context_line":"        return True"}],"source_content_type":"text/x-python","patch_set":11,"id":"7a2fa921_382cc101","line":1214,"updated":"2015-10-06 20:40:14.000000000","message":"The same as above.","commit_id":"2db6728242a83d2f1515d8a489bb07f429c2e98b"},{"author":{"_account_id":14753,"name":"yogeshprasad","email":"yogesh.prasad@cloudbyte.com","username":"yogeshprasad"},"change_message_id":"742f0b177f04847e08ae314277458b1fc322cfb6","unresolved":false,"context_lines":[{"line_number":1211,"context_line":"                \u0027updateFileSystem\u0027, update_file_system_params)"},{"line_number":1212,"context_line":""},{"line_number":1213,"context_line":"        LOG.info(_LI(\"CloudByte volume [%s] updated successfully.\"),"},{"line_number":1214,"context_line":"                 cb_volume_id)"},{"line_number":1215,"context_line":""},{"line_number":1216,"context_line":"        return True"}],"source_content_type":"text/x-python","patch_set":11,"id":"7a2fa921_750d543d","line":1214,"in_reply_to":"7a2fa921_382cc101","updated":"2015-10-07 06:18:37.000000000","message":"Done.","commit_id":"2db6728242a83d2f1515d8a489bb07f429c2e98b"}]}