)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"9ce2ac28c14ac4ac4857f87e0432f96318911c36","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In _update_volume_stats in dell_storagecenter_common.py totalcapacitygb"},{"line_number":10,"context_line":"and freespacegb are not necessarily initialized in the LOG.debug call."},{"line_number":11,"context_line":"Changed to data[\u0027total_capacity_gb\u0027] and data.get(\u0027free_capacity_gb\u0027)."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib699e0d1cf67645298628aa96656888193843509"},{"line_number":14,"context_line":"Closes-Bug: #1461230"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5afe65bd_56174b51","line":11,"updated":"2015-06-02 19:28:21.000000000","message":"Not actually data.get(...) but close enough. Update to match if you need to do another update.","commit_id":"6c3af1c06124478a8992a0e72f182548b2cd7647"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"2244df2c2c7dea72f798b909b4d36902d45c5788","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In _update_volume_stats in dell_storagecenter_common.py totalcapacitygb"},{"line_number":10,"context_line":"and freespacegb are not necessarily initialized in the LOG.debug call."},{"line_number":11,"context_line":"Changed to data[\u0027total_capacity_gb\u0027] and data.get(\u0027free_capacity_gb\u0027)."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib699e0d1cf67645298628aa96656888193843509"},{"line_number":14,"context_line":"Closes-Bug: #1461230"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5afe65bd_4de1f199","line":11,"in_reply_to":"5afe65bd_56174b51","updated":"2015-06-02 21:58:06.000000000","message":"Done","commit_id":"6c3af1c06124478a8992a0e72f182548b2cd7647"}],"cinder/volume/drivers/dell/dell_storagecenter_common.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"175f8b95825827be7f2ac138e8386dccdf8f933b","unresolved":false,"context_lines":[{"line_number":307,"context_line":"            data[\u0027driver_version\u0027] \u003d self.VERSION"},{"line_number":308,"context_line":"            data[\u0027storage_protocol\u0027] \u003d \u0027iSCSI\u0027"},{"line_number":309,"context_line":"            data[\u0027reserved_percentage\u0027] \u003d 0"},{"line_number":310,"context_line":"            # In theory if storageusage is None then we should have"},{"line_number":311,"context_line":"            # blown up getting it.  If not just report unavailable."},{"line_number":312,"context_line":"            if storageusage is not None:"},{"line_number":313,"context_line":"                totalcapacity \u003d storageusage.get(\u0027availableSpace\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5afe65bd_8a7eb0e2","line":310,"updated":"2015-06-02 20:51:06.000000000","message":"Just put\n\ndata[\u0027total_capacity_gb\u0027] \u003d \u0027unavailable\u0027\n\ndata[\u0027free_capacity_gb\u0027] \u003d \u0027unavailable\u0027\n\nhere and then you can eliminate two \"if\"s and simplify this whole piece a bit.","commit_id":"6c3af1c06124478a8992a0e72f182548b2cd7647"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"2244df2c2c7dea72f798b909b4d36902d45c5788","unresolved":false,"context_lines":[{"line_number":307,"context_line":"            data[\u0027driver_version\u0027] \u003d self.VERSION"},{"line_number":308,"context_line":"            data[\u0027storage_protocol\u0027] \u003d \u0027iSCSI\u0027"},{"line_number":309,"context_line":"            data[\u0027reserved_percentage\u0027] \u003d 0"},{"line_number":310,"context_line":"            # In theory if storageusage is None then we should have"},{"line_number":311,"context_line":"            # blown up getting it.  If not just report unavailable."},{"line_number":312,"context_line":"            if storageusage is not None:"},{"line_number":313,"context_line":"                totalcapacity \u003d storageusage.get(\u0027availableSpace\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5afe65bd_4d1c51c1","line":310,"in_reply_to":"5afe65bd_0dd77924","updated":"2015-06-02 21:58:06.000000000","message":"Done.","commit_id":"6c3af1c06124478a8992a0e72f182548b2cd7647"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"30fb32194c7f17f688c81f6f8682708694241558","unresolved":false,"context_lines":[{"line_number":307,"context_line":"            data[\u0027driver_version\u0027] \u003d self.VERSION"},{"line_number":308,"context_line":"            data[\u0027storage_protocol\u0027] \u003d \u0027iSCSI\u0027"},{"line_number":309,"context_line":"            data[\u0027reserved_percentage\u0027] \u003d 0"},{"line_number":310,"context_line":"            # In theory if storageusage is None then we should have"},{"line_number":311,"context_line":"            # blown up getting it.  If not just report unavailable."},{"line_number":312,"context_line":"            if storageusage is not None:"},{"line_number":313,"context_line":"                totalcapacity \u003d storageusage.get(\u0027availableSpace\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5afe65bd_0dd77924","line":310,"in_reply_to":"5afe65bd_8a7eb0e2","updated":"2015-06-02 21:41:03.000000000","message":"I like this idea better.","commit_id":"6c3af1c06124478a8992a0e72f182548b2cd7647"}]}