)]}'
{"cinder/volume/drivers/hpe/hpe_3par_common.py":[{"author":{"_account_id":6043,"name":"Kurt Martin","email":"kurt.f.martin@hpe.com","username":"kurt-f-martin"},"change_message_id":"c1728d65d58997b862daad3da8d224ab56e78a8b","unresolved":false,"context_lines":[{"line_number":217,"context_line":"        3.0.3 - Remove db access for consistency groups"},{"line_number":218,"context_line":"        3.0.4 - Adds v2 managed replication support"},{"line_number":219,"context_line":"        3.0.5 - Adds v2 unmanaged replication support"},{"line_number":220,"context_line":"        3.0.6 - Adding manage/unmanage snapshot supoort"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    \"\"\""},{"line_number":223,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"3a7e1126_41aa4072","line":220,"range":{"start_line":220,"start_character":48,"end_line":220,"end_character":55},"updated":"2015-12-17 00:04:37.000000000","message":"could you fix the spelling of support on your next patch set?","commit_id":"463541e3d9224a719d8d0e8659eea6ab0bbc3586"},{"author":{"_account_id":11903,"name":"Anthony Lee","email":"anthony.mic.lee@hpe.com","username":"leeantho"},"change_message_id":"a9a5ef160bec3eb6c5a503c5121b1cd6e94e2cc9","unresolved":false,"context_lines":[{"line_number":217,"context_line":"        3.0.3 - Remove db access for consistency groups"},{"line_number":218,"context_line":"        3.0.4 - Adds v2 managed replication support"},{"line_number":219,"context_line":"        3.0.5 - Adds v2 unmanaged replication support"},{"line_number":220,"context_line":"        3.0.6 - Adding manage/unmanage snapshot supoort"},{"line_number":221,"context_line":""},{"line_number":222,"context_line":"    \"\"\""},{"line_number":223,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"3a7e1126_966f0992","line":220,"range":{"start_line":220,"start_character":48,"end_line":220,"end_character":55},"in_reply_to":"3a7e1126_41aa4072","updated":"2015-12-17 00:55:03.000000000","message":"Done","commit_id":"463541e3d9224a719d8d0e8659eea6ab0bbc3586"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"ee68ecd418240cdc8e1f932f8e14c89350422d08","unresolved":false,"context_lines":[{"line_number":784,"context_line":"            err \u003d (_(\"Snapshot \u0027%(t_snap)s\u0027 can only be managed by the parent\""},{"line_number":785,"context_line":"                     \" volume \u0027%(p_name)s\u0027.\") %"},{"line_number":786,"context_line":"                   {\"t_snap\": target_snap_name,"},{"line_number":787,"context_line":"                    \"p_name\": snap[\u0027copyOf\u0027]})"},{"line_number":788,"context_line":"            LOG.error(err)"},{"line_number":789,"context_line":"            raise exception.InvalidInput(reason\u003derr)"},{"line_number":790,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"3a7e1126_abd4c91f","line":787,"updated":"2015-12-17 22:22:46.000000000","message":"I believe the snap[\u0027copyOf\u0027] will have the name of the parent volume on the 3PAR which will be in the osv-XXXXXX format.   That osv-XXXX volume might not even be part of the current driver\u0027s OpenStack deployment.   \n\nWe should not use that copyOf in the output.  We should simply say that the snapshot provided isn\u0027t a child/snapshot of the volume being passed in?","commit_id":"bdd817fa11018aa177673c281173c0b179b1fab0"},{"author":{"_account_id":11903,"name":"Anthony Lee","email":"anthony.mic.lee@hpe.com","username":"leeantho"},"change_message_id":"9a59ba48533c99b653b4cf65b097c4bcb09f106e","unresolved":false,"context_lines":[{"line_number":784,"context_line":"            err \u003d (_(\"Snapshot \u0027%(t_snap)s\u0027 can only be managed by the parent\""},{"line_number":785,"context_line":"                     \" volume \u0027%(p_name)s\u0027.\") %"},{"line_number":786,"context_line":"                   {\"t_snap\": target_snap_name,"},{"line_number":787,"context_line":"                    \"p_name\": snap[\u0027copyOf\u0027]})"},{"line_number":788,"context_line":"            LOG.error(err)"},{"line_number":789,"context_line":"            raise exception.InvalidInput(reason\u003derr)"},{"line_number":790,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"3a7e1126_71e9ee7f","line":787,"in_reply_to":"3a7e1126_abd4c91f","updated":"2015-12-17 23:26:05.000000000","message":"Done","commit_id":"bdd817fa11018aa177673c281173c0b179b1fab0"}],"cinder/volume/drivers/hpe/hpe_3par_fc.py":[{"author":{"_account_id":6043,"name":"Kurt Martin","email":"kurt.f.martin@hpe.com","username":"kurt-f-martin"},"change_message_id":"c1728d65d58997b862daad3da8d224ab56e78a8b","unresolved":false,"context_lines":[{"line_number":94,"context_line":"        3.0.1 - Remove db access for consistency groups"},{"line_number":95,"context_line":"        3.0.2 - Adds v2 managed replication support"},{"line_number":96,"context_line":"        3.0.3 - Adds v2 unmanaged replication support"},{"line_number":97,"context_line":"        3.0.4 - Adding manage/unmanage snapshot supoort"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    \"\"\""},{"line_number":100,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"3a7e1126_c1395043","line":97,"range":{"start_line":97,"start_character":48,"end_line":97,"end_character":55},"updated":"2015-12-17 00:04:37.000000000","message":"same here, support","commit_id":"463541e3d9224a719d8d0e8659eea6ab0bbc3586"},{"author":{"_account_id":11903,"name":"Anthony Lee","email":"anthony.mic.lee@hpe.com","username":"leeantho"},"change_message_id":"a9a5ef160bec3eb6c5a503c5121b1cd6e94e2cc9","unresolved":false,"context_lines":[{"line_number":94,"context_line":"        3.0.1 - Remove db access for consistency groups"},{"line_number":95,"context_line":"        3.0.2 - Adds v2 managed replication support"},{"line_number":96,"context_line":"        3.0.3 - Adds v2 unmanaged replication support"},{"line_number":97,"context_line":"        3.0.4 - Adding manage/unmanage snapshot supoort"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"    \"\"\""},{"line_number":100,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"3a7e1126_b60ec5e3","line":97,"range":{"start_line":97,"start_character":48,"end_line":97,"end_character":55},"in_reply_to":"3a7e1126_c1395043","updated":"2015-12-17 00:55:03.000000000","message":"Done","commit_id":"463541e3d9224a719d8d0e8659eea6ab0bbc3586"}],"cinder/volume/drivers/hpe/hpe_3par_iscsi.py":[{"author":{"_account_id":6043,"name":"Kurt Martin","email":"kurt.f.martin@hpe.com","username":"kurt-f-martin"},"change_message_id":"c1728d65d58997b862daad3da8d224ab56e78a8b","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        3.0.3 - Fix multipath dictionary key error. bug #1522062"},{"line_number":107,"context_line":"        3.0.4 - Adds v2 managed replication support"},{"line_number":108,"context_line":"        3.0.5 - Adds v2 unmanaged replication support"},{"line_number":109,"context_line":"        3.0.6 - Adding manage/unmanage snapshot supoort"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"    \"\"\""},{"line_number":112,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"3a7e1126_214934d1","line":109,"range":{"start_line":109,"start_character":48,"end_line":109,"end_character":55},"updated":"2015-12-17 00:04:37.000000000","message":"support","commit_id":"463541e3d9224a719d8d0e8659eea6ab0bbc3586"},{"author":{"_account_id":11903,"name":"Anthony Lee","email":"anthony.mic.lee@hpe.com","username":"leeantho"},"change_message_id":"a9a5ef160bec3eb6c5a503c5121b1cd6e94e2cc9","unresolved":false,"context_lines":[{"line_number":106,"context_line":"        3.0.3 - Fix multipath dictionary key error. bug #1522062"},{"line_number":107,"context_line":"        3.0.4 - Adds v2 managed replication support"},{"line_number":108,"context_line":"        3.0.5 - Adds v2 unmanaged replication support"},{"line_number":109,"context_line":"        3.0.6 - Adding manage/unmanage snapshot supoort"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"    \"\"\""},{"line_number":112,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"3a7e1126_5600c1aa","line":109,"range":{"start_line":109,"start_character":48,"end_line":109,"end_character":55},"in_reply_to":"3a7e1126_214934d1","updated":"2015-12-17 00:55:03.000000000","message":"Done","commit_id":"463541e3d9224a719d8d0e8659eea6ab0bbc3586"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"05890aff027af1ff57b7341a7e7c8139fe59899e","unresolved":false,"context_lines":[{"line_number":825,"context_line":"            self._logout(common)"},{"line_number":826,"context_line":""},{"line_number":827,"context_line":"    def manage_existing_snapshot(self, snapshot, existing_ref):"},{"line_number":828,"context_line":"        common \u003d self._login()"},{"line_number":829,"context_line":"        try:"},{"line_number":830,"context_line":"            return common.manage_existing_snapshot(snapshot, existing_ref)"},{"line_number":831,"context_line":"        finally:"}],"source_content_type":"text/x-python","patch_set":17,"id":"3a7e1126_58986078","line":828,"updated":"2015-12-18 20:35:03.000000000","message":"without passing in the snapshot object, this means we don\u0027t support managing a snapshot on the secondary 3PAR (replication).\n\nI suppose we can investigate this and add a follow up later if it makes sense.   But we should talk about this and see what we want to support and/or document wrt to managing a snapshot where the volume was failed over.","commit_id":"9c3cbdd90fbf4e462c23f640e68cd88034c873c2"},{"author":{"_account_id":11903,"name":"Anthony Lee","email":"anthony.mic.lee@hpe.com","username":"leeantho"},"change_message_id":"88b993aed57d1405eb3d6ac8c36e733e1bb3bc67","unresolved":false,"context_lines":[{"line_number":825,"context_line":"            self._logout(common)"},{"line_number":826,"context_line":""},{"line_number":827,"context_line":"    def manage_existing_snapshot(self, snapshot, existing_ref):"},{"line_number":828,"context_line":"        common \u003d self._login()"},{"line_number":829,"context_line":"        try:"},{"line_number":830,"context_line":"            return common.manage_existing_snapshot(snapshot, existing_ref)"},{"line_number":831,"context_line":"        finally:"}],"source_content_type":"text/x-python","patch_set":17,"id":"3a7e1126_78543c52","line":828,"in_reply_to":"3a7e1126_58986078","updated":"2015-12-18 20:44:56.000000000","message":"Walt, I agree we can discuss that support and maybe make a follow up to this patch later, if needed. This patch will just be focusing on support for the primary 3PAR.","commit_id":"9c3cbdd90fbf4e462c23f640e68cd88034c873c2"}]}