)]}'
{"cinder/volume/drivers/san/hp/hp_lefthand_iscsi.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e00436cd9948cd297cd1816b6a1513fa89982c96","unresolved":false,"context_lines":[{"line_number":111,"context_line":"        1.0.11 - Add over subscription support"},{"line_number":112,"context_line":"        1.0.12 - Adds consistency group support"},{"line_number":113,"context_line":"        1.0.13 - Added update_migrated_volume #1493546"},{"line_number":114,"context_line":"        1.0.14 - Removed the old CLIQ based driver"},{"line_number":115,"context_line":"    \"\"\""},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"    VERSION \u003d \"1.0.14\""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa80f949_2b2b3232","line":114,"updated":"2015-11-04 05:22:41.000000000","message":"Where do all these new versions come from?  I assume those new features were already supported before the removal of the proxy driver?","commit_id":"472f9d6091fa39d3b19c06520d4610072704c895"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"d5ae4d300607db4ef2d9ea6eb6f6e95b9cb60439","unresolved":false,"context_lines":[{"line_number":111,"context_line":"        1.0.11 - Add over subscription support"},{"line_number":112,"context_line":"        1.0.12 - Adds consistency group support"},{"line_number":113,"context_line":"        1.0.13 - Added update_migrated_volume #1493546"},{"line_number":114,"context_line":"        1.0.14 - Removed the old CLIQ based driver"},{"line_number":115,"context_line":"    \"\"\""},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"    VERSION \u003d \"1.0.14\""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa80f949_f95babf4","line":114,"in_reply_to":"fa80f949_2b2b3232","updated":"2015-11-04 21:27:07.000000000","message":"They came from the old rest proxy.   This file is nothing but a rename of the rest proxy to hp_lefthand_iscsi","commit_id":"472f9d6091fa39d3b19c06520d4610072704c895"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e00436cd9948cd297cd1816b6a1513fa89982c96","unresolved":false,"context_lines":[{"line_number":229,"context_line":""},{"line_number":230,"context_line":"            return self._update_provider(volume_info)"},{"line_number":231,"context_line":"        except Exception as ex:"},{"line_number":232,"context_line":"            raise exception.VolumeBackendAPIException(ex)"},{"line_number":233,"context_line":"        finally:"},{"line_number":234,"context_line":"            self._logout(client)"},{"line_number":235,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa80f949_abc5e25b","line":232,"updated":"2015-11-04 05:22:41.000000000","message":"exception.VolumeBackendAPIException(data\u003dsix.text_type(ex))\n\nSame comment goes for all the other places using VolumeBackendAPIException as well.","commit_id":"472f9d6091fa39d3b19c06520d4610072704c895"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"f16b765e610c783a3b2c21ba8f1d572bb6d9f76a","unresolved":false,"context_lines":[{"line_number":229,"context_line":""},{"line_number":230,"context_line":"            return self._update_provider(volume_info)"},{"line_number":231,"context_line":"        except Exception as ex:"},{"line_number":232,"context_line":"            raise exception.VolumeBackendAPIException(ex)"},{"line_number":233,"context_line":"        finally:"},{"line_number":234,"context_line":"            self._logout(client)"},{"line_number":235,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa80f949_fb20789d","line":232,"in_reply_to":"fa80f949_abc5e25b","updated":"2015-11-04 21:36:15.000000000","message":"Actually I don\u0027t think the six wrapping is needed.\n\nhttps://github.com/openstack/cinder/blob/master/cinder/exception.py#L100-L101","commit_id":"472f9d6091fa39d3b19c06520d4610072704c895"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"d5ae4d300607db4ef2d9ea6eb6f6e95b9cb60439","unresolved":false,"context_lines":[{"line_number":229,"context_line":""},{"line_number":230,"context_line":"            return self._update_provider(volume_info)"},{"line_number":231,"context_line":"        except Exception as ex:"},{"line_number":232,"context_line":"            raise exception.VolumeBackendAPIException(ex)"},{"line_number":233,"context_line":"        finally:"},{"line_number":234,"context_line":"            self._logout(client)"},{"line_number":235,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa80f949_c027bf10","line":232,"in_reply_to":"fa80f949_abc5e25b","updated":"2015-11-04 21:27:07.000000000","message":"Done","commit_id":"472f9d6091fa39d3b19c06520d4610072704c895"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e00436cd9948cd297cd1816b6a1513fa89982c96","unresolved":false,"context_lines":[{"line_number":271,"context_line":"        \"\"\"Creates a consistency group from a source\"\"\""},{"line_number":272,"context_line":"        LOG.error(_LE(\"Creating a consistency group from a source is not \""},{"line_number":273,"context_line":"                      \"currently supported.\"))"},{"line_number":274,"context_line":"        raise NotImplementedError()"},{"line_number":275,"context_line":""},{"line_number":276,"context_line":"    def delete_consistencygroup(self, context, group):"},{"line_number":277,"context_line":"        \"\"\"Deletes a consistency group.\"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa80f949_cb7ce622","line":274,"updated":"2015-11-04 05:22:41.000000000","message":"Can you add a message so that by reading the logs we know what is not implemented?","commit_id":"472f9d6091fa39d3b19c06520d4610072704c895"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"d5ae4d300607db4ef2d9ea6eb6f6e95b9cb60439","unresolved":false,"context_lines":[{"line_number":271,"context_line":"        \"\"\"Creates a consistency group from a source\"\"\""},{"line_number":272,"context_line":"        LOG.error(_LE(\"Creating a consistency group from a source is not \""},{"line_number":273,"context_line":"                      \"currently supported.\"))"},{"line_number":274,"context_line":"        raise NotImplementedError()"},{"line_number":275,"context_line":""},{"line_number":276,"context_line":"    def delete_consistencygroup(self, context, group):"},{"line_number":277,"context_line":"        \"\"\"Deletes a consistency group.\"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa80f949_e069a326","line":274,"in_reply_to":"fa80f949_cb7ce622","updated":"2015-11-04 21:27:07.000000000","message":"Done","commit_id":"472f9d6091fa39d3b19c06520d4610072704c895"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"e00436cd9948cd297cd1816b6a1513fa89982c96","unresolved":false,"context_lines":[{"line_number":419,"context_line":"        except hpexceptions.HTTPServerError as ex:"},{"line_number":420,"context_line":"            in_use_msg \u003d \u0027cannot be deleted because it is a clone point\u0027"},{"line_number":421,"context_line":"            if in_use_msg in ex.get_description():"},{"line_number":422,"context_line":"                raise exception.SnapshotIsBusy(ex)"},{"line_number":423,"context_line":""},{"line_number":424,"context_line":"            raise exception.VolumeBackendAPIException(ex)"},{"line_number":425,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa80f949_6b94da36","line":422,"updated":"2015-11-04 05:22:41.000000000","message":"exception.SnapshotIsBusy(snapshot_name\u003dsnapshot[\u0027name\u0027])","commit_id":"472f9d6091fa39d3b19c06520d4610072704c895"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"d5ae4d300607db4ef2d9ea6eb6f6e95b9cb60439","unresolved":false,"context_lines":[{"line_number":419,"context_line":"        except hpexceptions.HTTPServerError as ex:"},{"line_number":420,"context_line":"            in_use_msg \u003d \u0027cannot be deleted because it is a clone point\u0027"},{"line_number":421,"context_line":"            if in_use_msg in ex.get_description():"},{"line_number":422,"context_line":"                raise exception.SnapshotIsBusy(ex)"},{"line_number":423,"context_line":""},{"line_number":424,"context_line":"            raise exception.VolumeBackendAPIException(ex)"},{"line_number":425,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa80f949_804f77bc","line":422,"in_reply_to":"fa80f949_6b94da36","updated":"2015-11-04 21:27:07.000000000","message":"Done","commit_id":"472f9d6091fa39d3b19c06520d4610072704c895"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"79c7eec44d7a791054e4511a6ecc52d4407ac663","unresolved":false,"context_lines":[{"line_number":29,"context_line":"from cinder.volume import utils"},{"line_number":30,"context_line":"from cinder.volume import volume_types"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"import six"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"import math"},{"line_number":35,"context_line":"import re"}],"source_content_type":"text/x-python","patch_set":6,"id":"da85f559_a45d05ee","line":32,"updated":"2015-11-13 18:17:15.000000000","message":"Import order should be stdlibs, extralibs, local imports, so these should be first. Not a huge deal, but just pointing it out in case you need to do another update.","commit_id":"7f50df6db4978edd84b6d076d036392245efa75d"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"4afa69dd98f57c18cd6f16d470735391379c3c95","unresolved":false,"context_lines":[{"line_number":29,"context_line":"from cinder.volume import utils"},{"line_number":30,"context_line":"from cinder.volume import volume_types"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"import six"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"import math"},{"line_number":35,"context_line":"import re"}],"source_content_type":"text/x-python","patch_set":6,"id":"da85f559_7d025071","line":32,"in_reply_to":"da85f559_a45d05ee","updated":"2015-11-13 19:38:42.000000000","message":"I thought this will be picked up by the hacking check.","commit_id":"7f50df6db4978edd84b6d076d036392245efa75d"}]}