)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"113c8f51ce82af9ee1aca5d48020fc4ef7d1f588","unresolved":false,"context_lines":[{"line_number":12,"context_line":"Online copy is a feature in 3PAR whereby a copy initiated"},{"line_number":13,"context_line":"on a source volume , and when the data copy is in progress,"},{"line_number":14,"context_line":"the target volume of the copy is immediately available for"},{"line_number":15,"context_line":"volume attachment and performing I/O is allowed on that volume"},{"line_number":16,"context_line":"Resynching the source and the destination volume is"},{"line_number":17,"context_line":"done automatically by the 3PAR internally."},{"line_number":18,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a430d35_c648ba5e","line":15,"updated":"2017-02-03 11:06:52.000000000","message":"nit: .","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"5b09aa39acba45162f5476ac19f295bab1e667ad","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     William Durairaj \u003cwilliam.dur.sandanaraj@hpe.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-03 23:55:12 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"3PAR: Remove un-necessary snapshot in online copy operation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"While doing an online copy in the 3PAR, we create a temporary"},{"line_number":10,"context_line":"snapshot of source volume to do the data copy operation."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1a430d35_b66b5cf0","line":7,"updated":"2017-02-07 13:57:30.000000000","message":"For future reference, please keep summary lines within 50 characters when possible.","commit_id":"303e40121049cc88f462c990d0a53eea74cedb19"}],"cinder/tests/unit/volume/drivers/hpe/test_hpe3par.py":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"113c8f51ce82af9ee1aca5d48020fc4ef7d1f588","unresolved":false,"context_lines":[{"line_number":1843,"context_line":"            model_update \u003d self.driver.create_cloned_volume(volume, src_vref)"},{"line_number":1844,"context_line":"            self.assertIsNone(model_update)"},{"line_number":1845,"context_line":""},{"line_number":1846,"context_line":"            # snapshot name is random"},{"line_number":1847,"context_line":"            snap_name \u003d mock.ANY"},{"line_number":1848,"context_line":""},{"line_number":1849,"context_line":"            expected \u003d ["},{"line_number":1850,"context_line":"                mock.call.copyVolume("}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_66f9c6c8","line":1847,"range":{"start_line":1846,"start_character":11,"end_line":1847,"end_character":32},"updated":"2017-02-03 11:06:52.000000000","message":"Should not use this anymore.","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":21407,"name":"Kushal Wathore","email":"kushal.wathore@hpe.com","username":"kwathore"},"change_message_id":"83f256021ee5fd657e8e743fe06625fd11d5919c","unresolved":false,"context_lines":[{"line_number":1844,"context_line":"            self.assertIsNone(model_update)"},{"line_number":1845,"context_line":""},{"line_number":1846,"context_line":"            # snapshot name is random"},{"line_number":1847,"context_line":"            snap_name \u003d mock.ANY"},{"line_number":1848,"context_line":""},{"line_number":1849,"context_line":"            expected \u003d ["},{"line_number":1850,"context_line":"                mock.call.copyVolume("}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_e681364f","line":1847,"range":{"start_line":1847,"start_character":12,"end_line":1847,"end_character":21},"updated":"2017-02-03 11:03:21.000000000","message":"We do not need this variable now and above comments,\nInstead of using snap name in mock.call.copyVolume, use name of source volume as we are using this in actual driver code","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":22664,"name":"William Durairaj","email":"william.dur.sandanaraj@hpe.com","username":"william.durairaj"},"change_message_id":"6a00f9e0aaee53407c70e41365997f1fadeeff3b","unresolved":false,"context_lines":[{"line_number":1843,"context_line":"            model_update \u003d self.driver.create_cloned_volume(volume, src_vref)"},{"line_number":1844,"context_line":"            self.assertIsNone(model_update)"},{"line_number":1845,"context_line":""},{"line_number":1846,"context_line":"            # snapshot name is random"},{"line_number":1847,"context_line":"            snap_name \u003d mock.ANY"},{"line_number":1848,"context_line":""},{"line_number":1849,"context_line":"            expected \u003d ["},{"line_number":1850,"context_line":"                mock.call.copyVolume("}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_31cff6b1","line":1847,"range":{"start_line":1846,"start_character":11,"end_line":1847,"end_character":32},"in_reply_to":"1a430d35_66f9c6c8","updated":"2017-02-03 11:29:10.000000000","message":"Done","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":22664,"name":"William Durairaj","email":"william.dur.sandanaraj@hpe.com","username":"william.durairaj"},"change_message_id":"6a00f9e0aaee53407c70e41365997f1fadeeff3b","unresolved":false,"context_lines":[{"line_number":1844,"context_line":"            self.assertIsNone(model_update)"},{"line_number":1845,"context_line":""},{"line_number":1846,"context_line":"            # snapshot name is random"},{"line_number":1847,"context_line":"            snap_name \u003d mock.ANY"},{"line_number":1848,"context_line":""},{"line_number":1849,"context_line":"            expected \u003d ["},{"line_number":1850,"context_line":"                mock.call.copyVolume("}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_11cafaa1","line":1847,"range":{"start_line":1847,"start_character":12,"end_line":1847,"end_character":21},"in_reply_to":"1a430d35_e681364f","updated":"2017-02-03 11:29:10.000000000","message":"Done","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":21407,"name":"Kushal Wathore","email":"kushal.wathore@hpe.com","username":"kwathore"},"change_message_id":"83f256021ee5fd657e8e743fe06625fd11d5919c","unresolved":false,"context_lines":[{"line_number":1847,"context_line":"            snap_name \u003d mock.ANY"},{"line_number":1848,"context_line":""},{"line_number":1849,"context_line":"            expected \u003d ["},{"line_number":1850,"context_line":"                mock.call.copyVolume("},{"line_number":1851,"context_line":"                    snap_name,"},{"line_number":1852,"context_line":"                    \u0027osv-0DM4qZEVSKON-AAAAAAAAA\u0027,"},{"line_number":1853,"context_line":"                    HPE3PAR_CPG2,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_66962696","line":1850,"range":{"start_line":1850,"start_character":20,"end_line":1850,"end_character":21},"updated":"2017-02-03 11:03:21.000000000","message":"same as above","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"113c8f51ce82af9ee1aca5d48020fc4ef7d1f588","unresolved":false,"context_lines":[{"line_number":1848,"context_line":""},{"line_number":1849,"context_line":"            expected \u003d ["},{"line_number":1850,"context_line":"                mock.call.copyVolume("},{"line_number":1851,"context_line":"                    snap_name,"},{"line_number":1852,"context_line":"                    \u0027osv-0DM4qZEVSKON-AAAAAAAAA\u0027,"},{"line_number":1853,"context_line":"                    HPE3PAR_CPG2,"},{"line_number":1854,"context_line":"                    {\u0027snapCPG\u0027: \u0027OpenStackCPGSnap\u0027, \u0027tpvv\u0027: True,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_c600baad","line":1851,"range":{"start_line":1851,"start_character":20,"end_line":1851,"end_character":29},"updated":"2017-02-03 11:06:52.000000000","message":"src_vref","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":22664,"name":"William Durairaj","email":"william.dur.sandanaraj@hpe.com","username":"william.durairaj"},"change_message_id":"6a00f9e0aaee53407c70e41365997f1fadeeff3b","unresolved":false,"context_lines":[{"line_number":1848,"context_line":""},{"line_number":1849,"context_line":"            expected \u003d ["},{"line_number":1850,"context_line":"                mock.call.copyVolume("},{"line_number":1851,"context_line":"                    snap_name,"},{"line_number":1852,"context_line":"                    \u0027osv-0DM4qZEVSKON-AAAAAAAAA\u0027,"},{"line_number":1853,"context_line":"                    HPE3PAR_CPG2,"},{"line_number":1854,"context_line":"                    {\u0027snapCPG\u0027: \u0027OpenStackCPGSnap\u0027, \u0027tpvv\u0027: True,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_f1d48e48","line":1851,"range":{"start_line":1851,"start_character":20,"end_line":1851,"end_character":29},"in_reply_to":"1a430d35_c600baad","updated":"2017-02-03 11:29:10.000000000","message":"Done","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"113c8f51ce82af9ee1aca5d48020fc4ef7d1f588","unresolved":false,"context_lines":[{"line_number":1849,"context_line":"            expected \u003d ["},{"line_number":1850,"context_line":"                mock.call.copyVolume("},{"line_number":1851,"context_line":"                    snap_name,"},{"line_number":1852,"context_line":"                    \u0027osv-0DM4qZEVSKON-AAAAAAAAA\u0027,"},{"line_number":1853,"context_line":"                    HPE3PAR_CPG2,"},{"line_number":1854,"context_line":"                    {\u0027snapCPG\u0027: \u0027OpenStackCPGSnap\u0027, \u0027tpvv\u0027: True,"},{"line_number":1855,"context_line":"                     \u0027tdvv\u0027: False, \u0027online\u0027: True})]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_e6d21634","line":1852,"range":{"start_line":1852,"start_character":21,"end_line":1852,"end_character":47},"updated":"2017-02-03 11:06:52.000000000","message":"use: fakes.FAKE_UUID","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":21407,"name":"Kushal Wathore","email":"kushal.wathore@hpe.com","username":"kwathore"},"change_message_id":"83f256021ee5fd657e8e743fe06625fd11d5919c","unresolved":false,"context_lines":[{"line_number":1928,"context_line":"            common \u003d hpecommon.HPE3PARCommon(None)"},{"line_number":1929,"context_line":"            vol_name \u003d common._get_3par_vol_name(src_vref[\u0027id\u0027])"},{"line_number":1930,"context_line":"            # snapshot name is random"},{"line_number":1931,"context_line":"            snap_name \u003d mock.ANY"},{"line_number":1932,"context_line":""},{"line_number":1933,"context_line":"            expected \u003d ["},{"line_number":1934,"context_line":"                mock.call.getVolumeMetaData(vol_name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_e6e85619","line":1931,"range":{"start_line":1931,"start_character":12,"end_line":1931,"end_character":21},"updated":"2017-02-03 11:03:21.000000000","message":"same as above","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":22664,"name":"William Durairaj","email":"william.dur.sandanaraj@hpe.com","username":"william.durairaj"},"change_message_id":"6a00f9e0aaee53407c70e41365997f1fadeeff3b","unresolved":false,"context_lines":[{"line_number":1928,"context_line":"            common \u003d hpecommon.HPE3PARCommon(None)"},{"line_number":1929,"context_line":"            vol_name \u003d common._get_3par_vol_name(src_vref[\u0027id\u0027])"},{"line_number":1930,"context_line":"            # snapshot name is random"},{"line_number":1931,"context_line":"            snap_name \u003d mock.ANY"},{"line_number":1932,"context_line":""},{"line_number":1933,"context_line":"            expected \u003d ["},{"line_number":1934,"context_line":"                mock.call.getVolumeMetaData(vol_name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_b1e4c63b","line":1931,"range":{"start_line":1931,"start_character":12,"end_line":1931,"end_character":21},"in_reply_to":"1a430d35_e6e85619","updated":"2017-02-03 11:29:10.000000000","message":"Done","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":21407,"name":"Kushal Wathore","email":"kushal.wathore@hpe.com","username":"kwathore"},"change_message_id":"83f256021ee5fd657e8e743fe06625fd11d5919c","unresolved":false,"context_lines":[{"line_number":2070,"context_line":"            self.assertIsNone(model_update)"},{"line_number":2071,"context_line":""},{"line_number":2072,"context_line":"            # creation of the temp snapshot"},{"line_number":2073,"context_line":"            snap_name \u003d mock.ANY"},{"line_number":2074,"context_line":""},{"line_number":2075,"context_line":"            expected \u003d ["},{"line_number":2076,"context_line":"                mock.call.getCPG(expected_cpg),"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_26f33e0c","line":2073,"range":{"start_line":2073,"start_character":12,"end_line":2073,"end_character":21},"updated":"2017-02-03 11:03:21.000000000","message":"same as above","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":21407,"name":"Kushal Wathore","email":"kushal.wathore@hpe.com","username":"kwathore"},"change_message_id":"8d948207308b3de164fa623bb83860ee2585d564","unresolved":false,"context_lines":[{"line_number":2061,"context_line":"            model_update \u003d self.driver.create_cloned_volume(volume, src_vref)"},{"line_number":2062,"context_line":"            self.assertIsNone(model_update)"},{"line_number":2063,"context_line":""},{"line_number":2064,"context_line":"            # creation of the temp snapshot"},{"line_number":2065,"context_line":"            snap_name \u003d mock.ANY"},{"line_number":2066,"context_line":""},{"line_number":2067,"context_line":"            expected \u003d ["}],"source_content_type":"text/x-python","patch_set":2,"id":"1a430d35_57c47291","line":2064,"range":{"start_line":2064,"start_character":14,"end_line":2064,"end_character":22},"updated":"2017-02-03 12:17:23.000000000","message":"You missed here,Please do the changes","commit_id":"5521a65f39795ef2dcb0423c9e19b634235c328f"}],"cinder/volume/drivers/hpe/hpe_3par_common.py":[{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"113c8f51ce82af9ee1aca5d48020fc4ef7d1f588","unresolved":false,"context_lines":[{"line_number":2022,"context_line":"               back_up_process and vol_chap_enabled):"},{"line_number":2023,"context_line":"                LOG.debug(\"Creating a clone of volume, using online copy.\")"},{"line_number":2024,"context_line":"                # create a temporary snapshot"},{"line_number":2025,"context_line":"                snapshot \u003d self._create_temp_snapshot(src_vref)"},{"line_number":2026,"context_line":""},{"line_number":2027,"context_line":"                type_info \u003d self.get_volume_settings_from_type(volume)"},{"line_number":2028,"context_line":"                cpg \u003d type_info[\u0027cpg\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_46812a4a","side":"PARENT","line":2025,"range":{"start_line":2025,"start_character":16,"end_line":2025,"end_character":63},"updated":"2017-02-03 11:06:52.000000000","message":"so, this snapshot was always being left behind in all clone operations?","commit_id":"eb55e70a89f801523a8c50a8d09a6ad831e35bf2"},{"author":{"_account_id":22664,"name":"William Durairaj","email":"william.dur.sandanaraj@hpe.com","username":"william.durairaj"},"change_message_id":"6a00f9e0aaee53407c70e41365997f1fadeeff3b","unresolved":false,"context_lines":[{"line_number":2022,"context_line":"               back_up_process and vol_chap_enabled):"},{"line_number":2023,"context_line":"                LOG.debug(\"Creating a clone of volume, using online copy.\")"},{"line_number":2024,"context_line":"                # create a temporary snapshot"},{"line_number":2025,"context_line":"                snapshot \u003d self._create_temp_snapshot(src_vref)"},{"line_number":2026,"context_line":""},{"line_number":2027,"context_line":"                type_info \u003d self.get_volume_settings_from_type(volume)"},{"line_number":2028,"context_line":"                cpg \u003d type_info[\u0027cpg\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_46876a0f","side":"PARENT","line":2025,"range":{"start_line":2025,"start_character":16,"end_line":2025,"end_character":63},"in_reply_to":"1a430d35_46812a4a","updated":"2017-02-03 11:29:10.000000000","message":"yes. temporary snapshots are left as orphaned volumes on 3PAR Array.","commit_id":"eb55e70a89f801523a8c50a8d09a6ad831e35bf2"},{"author":{"_account_id":10058,"name":"Erlon R. Cruz","email":"erlon.rodrigues.cruz@canonical.com","username":"sombrafam"},"change_message_id":"113c8f51ce82af9ee1aca5d48020fc4ef7d1f588","unresolved":false,"context_lines":[{"line_number":252,"context_line":""},{"line_number":253,"context_line":"    \"\"\""},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"    VERSION \u003d \"3.0.27\""},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"    stats \u003d {}"},{"line_number":258,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1a430d35_c65fda0c","line":255,"range":{"start_line":255,"start_character":3,"end_line":255,"end_character":22},"updated":"2017-02-03 11:06:52.000000000","message":"-1: need to change here","commit_id":"8cc2b62c42c9d9c3798f8b810d6a400970f7b9c1"},{"author":{"_account_id":21406,"name":"Vijay Ladani","email":"vijay.ladani@hpe.com","username":"vladani"},"change_message_id":"f6db5891dc1dca07cdcb85688fa8167d991aa4d7","unresolved":false,"context_lines":[{"line_number":248,"context_line":"                 Bug #1646396 and also ,Fix backup of attached ISCSI"},{"line_number":249,"context_line":"                 and CHAP enabled volume.bug #1644238."},{"line_number":250,"context_line":"        3.0.28 - Remove un-necessary snapshot creation of source volume"},{"line_number":251,"context_line":"                 while doing online copy in create_cloned_volume call."},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"    \"\"\""},{"line_number":254,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1a430d35_92db481b","line":251,"updated":"2017-02-03 13:05:54.000000000","message":"Mention bug number at the end.","commit_id":"550c1635ff55abe18381546a0ddd31d9eaa0c5d4"},{"author":{"_account_id":22664,"name":"William Durairaj","email":"william.dur.sandanaraj@hpe.com","username":"william.durairaj"},"change_message_id":"006ebcc0022f5693c2dde2c0b4eba532ef3f0df1","unresolved":false,"context_lines":[{"line_number":248,"context_line":"                 Bug #1646396 and also ,Fix backup of attached ISCSI"},{"line_number":249,"context_line":"                 and CHAP enabled volume.bug #1644238."},{"line_number":250,"context_line":"        3.0.28 - Remove un-necessary snapshot creation of source volume"},{"line_number":251,"context_line":"                 while doing online copy in create_cloned_volume call."},{"line_number":252,"context_line":""},{"line_number":253,"context_line":"    \"\"\""},{"line_number":254,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"1a430d35_23b79e3a","line":251,"in_reply_to":"1a430d35_92db481b","updated":"2017-02-03 18:27:19.000000000","message":"Done","commit_id":"550c1635ff55abe18381546a0ddd31d9eaa0c5d4"},{"author":{"_account_id":23113,"name":"Hariprasad S Venkatasubramanian","email":"hari.venkat@hpe.com","username":"hari.venkat"},"change_message_id":"3ec08fbe6e4a57d8d726b05f3de93d2d05871f25","unresolved":false,"context_lines":[{"line_number":247,"context_line":"        3.0.27 - Fix snapCPG error during backup of attached volume."},{"line_number":248,"context_line":"                 Bug #1646396 and also ,Fix backup of attached ISCSI"},{"line_number":249,"context_line":"                 and CHAP enabled volume.bug #1644238."},{"line_number":250,"context_line":"        3.0.28 - Remove un-necessary snapshot creation of source volume"},{"line_number":251,"context_line":"                 while doing online copy in create_cloned_volume call."},{"line_number":252,"context_line":"                 Bug #1661541"},{"line_number":253,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1a430d35_425c7fed","line":250,"range":{"start_line":250,"start_character":17,"end_line":250,"end_character":71},"updated":"2017-02-06 04:18:12.000000000","message":"I understand that the smarts to create an internal RW/RO VV snapshot is available with some of the recent releases of 3PAR OS. \n\nHPE 3PAR drivers MUST support all versions of 3PAR OS 3.1.3 MU1 and upward. Is backward compatibility ensured for this change? If yes, changes look good to me. If not, a suitable switchable logic should be implemented.","commit_id":"303e40121049cc88f462c990d0a53eea74cedb19"},{"author":{"_account_id":22664,"name":"William Durairaj","email":"william.dur.sandanaraj@hpe.com","username":"william.durairaj"},"change_message_id":"d8344221276ddccfde585dd374969316aa6512bc","unresolved":false,"context_lines":[{"line_number":247,"context_line":"        3.0.27 - Fix snapCPG error during backup of attached volume."},{"line_number":248,"context_line":"                 Bug #1646396 and also ,Fix backup of attached ISCSI"},{"line_number":249,"context_line":"                 and CHAP enabled volume.bug #1644238."},{"line_number":250,"context_line":"        3.0.28 - Remove un-necessary snapshot creation of source volume"},{"line_number":251,"context_line":"                 while doing online copy in create_cloned_volume call."},{"line_number":252,"context_line":"                 Bug #1661541"},{"line_number":253,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1a430d35_45480988","line":250,"range":{"start_line":250,"start_character":17,"end_line":250,"end_character":71},"in_reply_to":"1a430d35_425c7fed","updated":"2017-02-06 06:09:38.000000000","message":"I\u0027m confirming this behaviour in older releases of 3PAR firmware with 3PAR Development Team. Waiting for their confirmation on this.","commit_id":"303e40121049cc88f462c990d0a53eea74cedb19"},{"author":{"_account_id":22664,"name":"William Durairaj","email":"william.dur.sandanaraj@hpe.com","username":"william.durairaj"},"change_message_id":"b1858d2dc5ee7237d20fc1f20ba9af153b5c3ac9","unresolved":false,"context_lines":[{"line_number":247,"context_line":"        3.0.27 - Fix snapCPG error during backup of attached volume."},{"line_number":248,"context_line":"                 Bug #1646396 and also ,Fix backup of attached ISCSI"},{"line_number":249,"context_line":"                 and CHAP enabled volume.bug #1644238."},{"line_number":250,"context_line":"        3.0.28 - Remove un-necessary snapshot creation of source volume"},{"line_number":251,"context_line":"                 while doing online copy in create_cloned_volume call."},{"line_number":252,"context_line":"                 Bug #1661541"},{"line_number":253,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1a430d35_47b7fb85","line":250,"range":{"start_line":250,"start_character":17,"end_line":250,"end_character":71},"in_reply_to":"1a430d35_45480988","updated":"2017-02-07 03:56:44.000000000","message":"3PAR Development team confirmed the behaviour of online copy as follows:\n\n\"Online copy was added in version 3.1.2, and from that initial version has always created internal snapshots to perform the copy\"","commit_id":"303e40121049cc88f462c990d0a53eea74cedb19"},{"author":{"_account_id":23113,"name":"Hariprasad S Venkatasubramanian","email":"hari.venkat@hpe.com","username":"hari.venkat"},"change_message_id":"7ca85e99cf9d4d1d9af71e215cd13e029e40e538","unresolved":false,"context_lines":[{"line_number":247,"context_line":"        3.0.27 - Fix snapCPG error during backup of attached volume."},{"line_number":248,"context_line":"                 Bug #1646396 and also ,Fix backup of attached ISCSI"},{"line_number":249,"context_line":"                 and CHAP enabled volume.bug #1644238."},{"line_number":250,"context_line":"        3.0.28 - Remove un-necessary snapshot creation of source volume"},{"line_number":251,"context_line":"                 while doing online copy in create_cloned_volume call."},{"line_number":252,"context_line":"                 Bug #1661541"},{"line_number":253,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"1a430d35_5bfc3372","line":250,"range":{"start_line":250,"start_character":17,"end_line":250,"end_character":71},"in_reply_to":"1a430d35_47b7fb85","updated":"2017-02-07 04:33:20.000000000","message":"Thanks for confirming this, William. All changes look good and I have given my review approval, with this.","commit_id":"303e40121049cc88f462c990d0a53eea74cedb19"}]}