)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"b557ec3947072ee4c211de6315a22441ca89def7","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Bump driver version to 2.0.9 and updated changelog entry."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #1560654"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Iaeb2596ecd75b86a7a0d3da7fad0bf91b2a02fa0"},{"line_number":17,"context_line":"Signed-off-by: Danny Al-Gaaf \u003cdanny.al-gaaf@bisect.de\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3aaa91ec_7fc0107d","line":14,"updated":"2016-06-22 17:58:33.000000000","message":"Depends-on: I829c0e6f99fff0ec0bd5366c9d2e79b1b073f0a3\n\nTempest currently does not cover the path touched by this fix. So, we should run this extend test as dependency so we can assure they work for all backends after CIs runs.","commit_id":"a5aab6f22aea20592a18336ee46fc5e1edb09110"}],"cinder/tests/unit/test_hpelefthand.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"f032b926dd4d7155bb092bed7a9b5897ab55147b","unresolved":false,"context_lines":[{"line_number":826,"context_line":"                    port\u003dHPELEFTHAND_SSH_PORT,"},{"line_number":827,"context_line":"                    conn_timeout\u003dHPELEFTHAND_SAN_SSH_CON_TIMEOUT),"},{"line_number":828,"context_line":"                mock.call.getVolumeByName(\u0027fakevolume\u0027),"},{"line_number":829,"context_line":"                mock.call.modifyVolume(self.volume_id, {\u0027size\u0027: 5 * units.Gi}),"},{"line_number":830,"context_line":"                mock.call.logout(),"},{"line_number":831,"context_line":"                mock.call.logout()]"},{"line_number":832,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_119e5456","line":829,"range":{"start_line":829,"start_character":64,"end_line":829,"end_character":65},"updated":"2016-05-18 10:48:52.000000000","message":"nit: volume_extend[\u0027size\u0027]","commit_id":"a5aab6f22aea20592a18336ee46fc5e1edb09110"}],"cinder/tests/unit/volume/drivers/hpe/test_hpelefthand.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"aa0b5cd714615d07dce33b2a1fb7b718caebd27f","unresolved":false,"context_lines":[{"line_number":1005,"context_line":"                    port\u003dHPELEFTHAND_SSH_PORT,"},{"line_number":1006,"context_line":"                    conn_timeout\u003dHPELEFTHAND_SAN_SSH_CON_TIMEOUT),"},{"line_number":1007,"context_line":"                mock.call.getVolumeByName(\u0027fakevolume\u0027),"},{"line_number":1008,"context_line":"                mock.call.modifyVolume(self.volume_id, {\u0027size\u0027: 5 * units.Gi}),"},{"line_number":1009,"context_line":"                mock.call.logout(),"},{"line_number":1010,"context_line":"                mock.call.logout()]"},{"line_number":1011,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"5a22e58a_3d5efa45","line":1008,"range":{"start_line":1008,"start_character":64,"end_line":1008,"end_character":65},"updated":"2017-03-14 11:30:24.000000000","message":"-1: self.volume_extend[\u0027size\u0027]","commit_id":"85ef874d12dd71e2c326737db22e393f5d301ffc"}],"cinder/volume/drivers/hpe/hpe_lefthand_iscsi.py":[{"author":{"_account_id":16917,"name":"Alex O\u0027Rourke","email":"alex.orourke@hpe.com","username":"aorourke"},"change_message_id":"3ed7491c6aa44dd1d383afac28d28c5c711100e5","unresolved":false,"context_lines":[{"line_number":151,"context_line":"        2.0.5 - Changed minimum client version to be 2.1.0"},{"line_number":152,"context_line":"        2.0.6 - Update replication to version 2.1"},{"line_number":153,"context_line":"        2.0.7 - Fixed bug #1554746, Create clone volume with new size."},{"line_number":154,"context_line":"        2.0.8 - Add defaults for creating a replication client, bug #1556331"},{"line_number":155,"context_line":"    \"\"\""},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    VERSION \u003d \"2.0.8\""},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"    device_stats \u003d {}"},{"line_number":160,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa0719c6_8e62b23c","line":157,"range":{"start_line":154,"start_character":8,"end_line":157,"end_character":21},"updated":"2016-03-23 14:33:53.000000000","message":"We bump the version here when a modification is made to the driver. Can you please add:\n\n2.0.9 - Fix extend volume if larger than snapshot bug #1560654\n\nand set VERSION \u003d \"2.0.9\"","commit_id":"f94f8a805e82e93e3273404558fb27f20b01290d"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"f032b926dd4d7155bb092bed7a9b5897ab55147b","unresolved":false,"context_lines":[{"line_number":761,"context_line":"                snap_info[\u0027id\u0027])"},{"line_number":762,"context_line":""},{"line_number":763,"context_line":"            # Extend volume"},{"line_number":764,"context_line":"            if volume[\u0027size\u0027] \u003e snapshot[\u0027volume_size\u0027]:"},{"line_number":765,"context_line":"                LOG.debug(\"Resize the new volume to %s.\", volume[\u0027size\u0027])"},{"line_number":766,"context_line":"                self.extend_volume(volume, volume[\u0027size\u0027])"},{"line_number":767,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_51c1fc7f","line":764,"updated":"2016-05-18 10:48:52.000000000","message":"-1: Please use attribute notation for new code:\n\n if volume.size \u003e snapshot.volume_size","commit_id":"a5aab6f22aea20592a18336ee46fc5e1edb09110"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"9d776ee3f39a4c99cea937704082635e614ac9b6","unresolved":false,"context_lines":[{"line_number":761,"context_line":"                snap_info[\u0027id\u0027])"},{"line_number":762,"context_line":""},{"line_number":763,"context_line":"            # Extend volume"},{"line_number":764,"context_line":"            if volume[\u0027size\u0027] \u003e snapshot[\u0027volume_size\u0027]:"},{"line_number":765,"context_line":"                LOG.debug(\"Resize the new volume to %s.\", volume[\u0027size\u0027])"},{"line_number":766,"context_line":"                self.extend_volume(volume, volume[\u0027size\u0027])"},{"line_number":767,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1aa78d24_d97e3469","line":764,"in_reply_to":"bab6814e_51c1fc7f","updated":"2016-07-12 05:25:43.000000000","message":"I am going to do a follow up patch for the entire driver to switch over to the ovo from the dict.   This should be fine as is for now.","commit_id":"a5aab6f22aea20592a18336ee46fc5e1edb09110"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"f032b926dd4d7155bb092bed7a9b5897ab55147b","unresolved":false,"context_lines":[{"line_number":762,"context_line":""},{"line_number":763,"context_line":"            # Extend volume"},{"line_number":764,"context_line":"            if volume[\u0027size\u0027] \u003e snapshot[\u0027volume_size\u0027]:"},{"line_number":765,"context_line":"                LOG.debug(\"Resize the new volume to %s.\", volume[\u0027size\u0027])"},{"line_number":766,"context_line":"                self.extend_volume(volume, volume[\u0027size\u0027])"},{"line_number":767,"context_line":""},{"line_number":768,"context_line":"            model_update \u003d self._update_provider(volume_info)"}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_f15d90ed","line":765,"range":{"start_line":765,"start_character":58,"end_line":765,"end_character":72},"updated":"2016-05-18 10:48:52.000000000","message":"ditto","commit_id":"a5aab6f22aea20592a18336ee46fc5e1edb09110"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"f032b926dd4d7155bb092bed7a9b5897ab55147b","unresolved":false,"context_lines":[{"line_number":763,"context_line":"            # Extend volume"},{"line_number":764,"context_line":"            if volume[\u0027size\u0027] \u003e snapshot[\u0027volume_size\u0027]:"},{"line_number":765,"context_line":"                LOG.debug(\"Resize the new volume to %s.\", volume[\u0027size\u0027])"},{"line_number":766,"context_line":"                self.extend_volume(volume, volume[\u0027size\u0027])"},{"line_number":767,"context_line":""},{"line_number":768,"context_line":"            model_update \u003d self._update_provider(volume_info)"},{"line_number":769,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"bab6814e_9169a48e","line":766,"updated":"2016-05-18 10:48:52.000000000","message":"ditto","commit_id":"a5aab6f22aea20592a18336ee46fc5e1edb09110"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"aa0b5cd714615d07dce33b2a1fb7b718caebd27f","unresolved":false,"context_lines":[{"line_number":463,"context_line":"            # convert GB to bytes"},{"line_number":464,"context_line":"            options \u003d {\u0027size\u0027: int(new_size) * units.Gi}"},{"line_number":465,"context_line":"            vol_id \u003d volume_info[\u0027id\u0027]"},{"line_number":466,"context_line":"            client.modifyVolume(vol_id, options)"},{"line_number":467,"context_line":"        except Exception as ex:"},{"line_number":468,"context_line":"            raise exception.VolumeBackendAPIException(ex)"},{"line_number":469,"context_line":"        finally:"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a22e58a_5dff3634","line":466,"updated":"2017-03-14 11:30:24.000000000","message":"-1: Unrelated change","commit_id":"85ef874d12dd71e2c326737db22e393f5d301ffc"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"aa0b5cd714615d07dce33b2a1fb7b718caebd27f","unresolved":false,"context_lines":[{"line_number":809,"context_line":"                volume[\u0027name\u0027],"},{"line_number":810,"context_line":"                snap_info[\u0027id\u0027])"},{"line_number":811,"context_line":""},{"line_number":812,"context_line":"            # Extend volume"},{"line_number":813,"context_line":"            if volume[\u0027size\u0027] \u003e snapshot[\u0027volume_size\u0027]:"},{"line_number":814,"context_line":"                LOG.debug(\"Resize the new volume to %s.\", volume[\u0027size\u0027])"},{"line_number":815,"context_line":"                self.extend_volume(volume, volume[\u0027size\u0027])"},{"line_number":816,"context_line":""},{"line_number":817,"context_line":"            model_update \u003d self._update_provider(volume_info)"},{"line_number":818,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"5a22e58a_3841a83a","line":815,"range":{"start_line":812,"start_character":0,"end_line":815,"end_character":58},"updated":"2017-03-14 11:30:24.000000000","message":"-1: As mentioned in patch 5 I believe this should be using attribute notation, even if there will be a follow up patch to change the whole file to attribute notation later on.","commit_id":"85ef874d12dd71e2c326737db22e393f5d301ffc"}]}
