)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":25547,"name":"jiapei","email":"jiapeish@gmail.com","username":"jeremy.jia"},"change_message_id":"fd9afd2c0f750ca39de47c573bde8d7ccb93bf45","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"HPE 3PAR: fix delete operation of replicated volume"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When QOS is set to the replicated volume, at Storage"},{"line_number":10,"context_line":"level the same volume becomes part of vvset"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch delete vvset if one volume per vvset"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7f515b1d_469e2708","line":9,"range":{"start_line":9,"start_character":5,"end_line":9,"end_character":8},"updated":"2017-09-22 02:56:49.000000000","message":"I think you means QoS, right?","commit_id":"63d9679ea23af6fb87b36c664b8b69630112fe8a"},{"author":{"_account_id":21407,"name":"Kushal Wathore","email":"kushal.wathore@hpe.com","username":"kwathore"},"change_message_id":"15d673dbcc4b05e7a4b5d48d5fd74e5b3d15a09f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"HPE 3PAR: fix delete operation of replicated volume"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When QOS is set to the replicated volume, at Storage"},{"line_number":10,"context_line":"level the same volume becomes part of vvset"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch delete vvset if one volume per vvset"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7f515b1d_3718f77f","line":9,"range":{"start_line":9,"start_character":5,"end_line":9,"end_character":8},"in_reply_to":"7f515b1d_469e2708","updated":"2017-09-22 05:43:37.000000000","message":"Yes, right.\nThanks","commit_id":"63d9679ea23af6fb87b36c664b8b69630112fe8a"}],"cinder/volume/drivers/hpe/hpe_3par_common.py":[{"author":{"_account_id":25547,"name":"jiapei","email":"jiapeish@gmail.com","username":"jeremy.jia"},"change_message_id":"fd9afd2c0f750ca39de47c573bde8d7ccb93bf45","unresolved":false,"context_lines":[{"line_number":3717,"context_line":"            # remove the volume set."},{"line_number":3718,"context_line":"            self.client.deleteVolumeSet("},{"line_number":3719,"context_line":"                self._get_3par_vvs_name(volume[\u0027id\u0027]))"},{"line_number":3720,"context_line":"        elif vvset_name is not None:"},{"line_number":3721,"context_line":"            # We have a pre-defined volume set just remove the"},{"line_number":3722,"context_line":"            # volume and leave the volume set."},{"line_number":3723,"context_line":"            self.client.removeVolumeFromVolumeSet(vvset_name,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f515b1d_6667e327","line":3720,"range":{"start_line":3720,"start_character":8,"end_line":3720,"end_character":35},"updated":"2017-09-22 02:56:49.000000000","message":"This \"vvset_name is not None\" and the last, can be reduced to one, like:\nif vvset_name is not None:\n  if vvset_name.startwith(\u0027vvs-\u0027):\n    ...\n  else:\n    ...","commit_id":"63d9679ea23af6fb87b36c664b8b69630112fe8a"},{"author":{"_account_id":21407,"name":"Kushal Wathore","email":"kushal.wathore@hpe.com","username":"kwathore"},"change_message_id":"15d673dbcc4b05e7a4b5d48d5fd74e5b3d15a09f","unresolved":false,"context_lines":[{"line_number":3717,"context_line":"            # remove the volume set."},{"line_number":3718,"context_line":"            self.client.deleteVolumeSet("},{"line_number":3719,"context_line":"                self._get_3par_vvs_name(volume[\u0027id\u0027]))"},{"line_number":3720,"context_line":"        elif vvset_name is not None:"},{"line_number":3721,"context_line":"            # We have a pre-defined volume set just remove the"},{"line_number":3722,"context_line":"            # volume and leave the volume set."},{"line_number":3723,"context_line":"            self.client.removeVolumeFromVolumeSet(vvset_name,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f515b1d_d7064321","line":3720,"range":{"start_line":3720,"start_character":8,"end_line":3720,"end_character":35},"in_reply_to":"7f515b1d_6667e327","updated":"2017-09-22 05:43:37.000000000","message":"Done, Thanks","commit_id":"63d9679ea23af6fb87b36c664b8b69630112fe8a"}]}