)]}'
{"cinder/volume/drivers/ibm/gpfs.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"53d30439ef40acc55e5f7239e308de0d7201865c","unresolved":false,"context_lines":[{"line_number":861,"context_line":"                                 {\u0027cluster_id\u0027: self._cluster_id,"},{"line_number":862,"context_line":"                                  \u0027root_path\u0027: gpfs_base})"},{"line_number":863,"context_line":""},{"line_number":864,"context_line":"        data[\u0027consistencygroup_support\u0027] \u003d \u0027True\u0027"},{"line_number":865,"context_line":""},{"line_number":866,"context_line":"        if self._encryption_state.lower() \u003d\u003d \u0027yes\u0027:"},{"line_number":867,"context_line":"            data[\u0027gpfs_encryption_rest\u0027] \u003d \u0027True\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_e8e0e4ce","line":864,"updated":"2017-06-01 15:57:15.000000000","message":"You should add data[\u0027consistent_group_snapshot_enabled\u0027] \u003d True here.  Otherwise you shouldn\u0027t be able to create a consistency group in your testing.","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"64ba523cc01d4c747068f01fe3f26a7d1eba1de9","unresolved":false,"context_lines":[{"line_number":861,"context_line":"                                 {\u0027cluster_id\u0027: self._cluster_id,"},{"line_number":862,"context_line":"                                  \u0027root_path\u0027: gpfs_base})"},{"line_number":863,"context_line":""},{"line_number":864,"context_line":"        data[\u0027consistencygroup_support\u0027] \u003d \u0027True\u0027"},{"line_number":865,"context_line":""},{"line_number":866,"context_line":"        if self._encryption_state.lower() \u003d\u003d \u0027yes\u0027:"},{"line_number":867,"context_line":"            data[\u0027gpfs_encryption_rest\u0027] \u003d \u0027True\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_e93e58bf","line":864,"in_reply_to":"df140735_e8e0e4ce","updated":"2017-06-05 10:25:29.000000000","message":"Done","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"53d30439ef40acc55e5f7239e308de0d7201865c","unresolved":false,"context_lines":[{"line_number":1124,"context_line":"            LOG.error(msg)"},{"line_number":1125,"context_line":"            raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1126,"context_line":""},{"line_number":1127,"context_line":"    def create_consistencygroup(self, context, group):"},{"line_number":1128,"context_line":"        \"\"\"Create consistency group of GPFS volumes.\"\"\""},{"line_number":1129,"context_line":"        cgname \u003d \"consisgroup-%s\" % group[\u0027id\u0027]"},{"line_number":1130,"context_line":"        fsdev \u003d self._gpfs_device"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_28a81c75","line":1127,"range":{"start_line":1127,"start_character":8,"end_line":1127,"end_character":31},"updated":"2017-06-01 15:57:15.000000000","message":"Rename this to _create_consistencygroup","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"64ba523cc01d4c747068f01fe3f26a7d1eba1de9","unresolved":false,"context_lines":[{"line_number":1124,"context_line":"            LOG.error(msg)"},{"line_number":1125,"context_line":"            raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1126,"context_line":""},{"line_number":1127,"context_line":"    def create_consistencygroup(self, context, group):"},{"line_number":1128,"context_line":"        \"\"\"Create consistency group of GPFS volumes.\"\"\""},{"line_number":1129,"context_line":"        cgname \u003d \"consisgroup-%s\" % group[\u0027id\u0027]"},{"line_number":1130,"context_line":"        fsdev \u003d self._gpfs_device"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_a934e0de","line":1127,"range":{"start_line":1127,"start_character":8,"end_line":1127,"end_character":31},"in_reply_to":"df140735_28a81c75","updated":"2017-06-05 10:25:29.000000000","message":"Done","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"53d30439ef40acc55e5f7239e308de0d7201865c","unresolved":false,"context_lines":[{"line_number":1163,"context_line":"        model_update \u003d {\u0027status\u0027: fields.ConsistencyGroupStatus.AVAILABLE}"},{"line_number":1164,"context_line":"        return model_update"},{"line_number":1165,"context_line":""},{"line_number":1166,"context_line":"    def delete_consistencygroup(self, context, group, volumes):"},{"line_number":1167,"context_line":"        \"\"\"Delete consistency group of GPFS volumes.\"\"\""},{"line_number":1168,"context_line":"        cgname \u003d \"consisgroup-%s\" % group[\u0027id\u0027]"},{"line_number":1169,"context_line":"        fsdev \u003d self._gpfs_device"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_c899a06e","line":1166,"range":{"start_line":1166,"start_character":8,"end_line":1166,"end_character":31},"updated":"2017-06-01 15:57:15.000000000","message":"Rename to _delete_consistencygroup","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"64ba523cc01d4c747068f01fe3f26a7d1eba1de9","unresolved":false,"context_lines":[{"line_number":1163,"context_line":"        model_update \u003d {\u0027status\u0027: fields.ConsistencyGroupStatus.AVAILABLE}"},{"line_number":1164,"context_line":"        return model_update"},{"line_number":1165,"context_line":""},{"line_number":1166,"context_line":"    def delete_consistencygroup(self, context, group, volumes):"},{"line_number":1167,"context_line":"        \"\"\"Delete consistency group of GPFS volumes.\"\"\""},{"line_number":1168,"context_line":"        cgname \u003d \"consisgroup-%s\" % group[\u0027id\u0027]"},{"line_number":1169,"context_line":"        fsdev \u003d self._gpfs_device"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_6950080c","line":1166,"range":{"start_line":1166,"start_character":8,"end_line":1166,"end_character":31},"in_reply_to":"df140735_c899a06e","updated":"2017-06-05 10:25:29.000000000","message":"Done","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"53d30439ef40acc55e5f7239e308de0d7201865c","unresolved":false,"context_lines":[{"line_number":1212,"context_line":""},{"line_number":1213,"context_line":"        return None, None"},{"line_number":1214,"context_line":""},{"line_number":1215,"context_line":"    def create_cgsnapshot(self, context, cgsnapshot, snapshots):"},{"line_number":1216,"context_line":"        \"\"\"Create snapshot of a consistency group of GPFS volumes.\"\"\""},{"line_number":1217,"context_line":"        model_update \u003d {\u0027status\u0027: fields.ConsistencyGroupStatus.AVAILABLE}"},{"line_number":1218,"context_line":"        snapshots_model_update \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_e892244b","line":1215,"range":{"start_line":1215,"start_character":8,"end_line":1215,"end_character":25},"updated":"2017-06-01 15:57:15.000000000","message":"Rename to _create_cgsnapshot","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"64ba523cc01d4c747068f01fe3f26a7d1eba1de9","unresolved":false,"context_lines":[{"line_number":1212,"context_line":""},{"line_number":1213,"context_line":"        return None, None"},{"line_number":1214,"context_line":""},{"line_number":1215,"context_line":"    def create_cgsnapshot(self, context, cgsnapshot, snapshots):"},{"line_number":1216,"context_line":"        \"\"\"Create snapshot of a consistency group of GPFS volumes.\"\"\""},{"line_number":1217,"context_line":"        model_update \u003d {\u0027status\u0027: fields.ConsistencyGroupStatus.AVAILABLE}"},{"line_number":1218,"context_line":"        snapshots_model_update \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_49530418","line":1215,"range":{"start_line":1215,"start_character":8,"end_line":1215,"end_character":25},"in_reply_to":"df140735_e892244b","updated":"2017-06-05 10:25:29.000000000","message":"Done","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"53d30439ef40acc55e5f7239e308de0d7201865c","unresolved":false,"context_lines":[{"line_number":1234,"context_line":""},{"line_number":1235,"context_line":"        return model_update, snapshots_model_update"},{"line_number":1236,"context_line":""},{"line_number":1237,"context_line":"    def delete_cgsnapshot(self, context, cgsnapshot, snapshots):"},{"line_number":1238,"context_line":"        \"\"\"Delete snapshot of a consistency group of GPFS volumes.\"\"\""},{"line_number":1239,"context_line":"        model_update \u003d {\u0027status\u0027: fields.ConsistencyGroupStatus.DELETED}"},{"line_number":1240,"context_line":"        snapshots_model_update \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_088b181f","line":1237,"range":{"start_line":1237,"start_character":8,"end_line":1237,"end_character":25},"updated":"2017-06-01 15:57:15.000000000","message":"Rename this to _delete_cgsnapshot","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"64ba523cc01d4c747068f01fe3f26a7d1eba1de9","unresolved":false,"context_lines":[{"line_number":1234,"context_line":""},{"line_number":1235,"context_line":"        return model_update, snapshots_model_update"},{"line_number":1236,"context_line":""},{"line_number":1237,"context_line":"    def delete_cgsnapshot(self, context, cgsnapshot, snapshots):"},{"line_number":1238,"context_line":"        \"\"\"Delete snapshot of a consistency group of GPFS volumes.\"\"\""},{"line_number":1239,"context_line":"        model_update \u003d {\u0027status\u0027: fields.ConsistencyGroupStatus.DELETED}"},{"line_number":1240,"context_line":"        snapshots_model_update \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_e943f840","line":1237,"range":{"start_line":1237,"start_character":8,"end_line":1237,"end_character":25},"in_reply_to":"df140735_088b181f","updated":"2017-06-05 10:25:29.000000000","message":"Done","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"53d30439ef40acc55e5f7239e308de0d7201865c","unresolved":false,"context_lines":[{"line_number":1256,"context_line":""},{"line_number":1257,"context_line":"        return model_update, snapshots_model_update"},{"line_number":1258,"context_line":""},{"line_number":1259,"context_line":"    def update_consistencygroup(self, context, group,"},{"line_number":1260,"context_line":"                                add_volumes\u003dNone, remove_volumes\u003dNone):"},{"line_number":1261,"context_line":"        msg \u003d _(\u0027Updating a consistency group is not supported.\u0027)"},{"line_number":1262,"context_line":"        LOG.error(msg)"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_c884000f","line":1259,"range":{"start_line":1259,"start_character":8,"end_line":1259,"end_character":31},"updated":"2017-06-01 15:57:15.000000000","message":"Rename this to _update_consistencygroup","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"64ba523cc01d4c747068f01fe3f26a7d1eba1de9","unresolved":false,"context_lines":[{"line_number":1256,"context_line":""},{"line_number":1257,"context_line":"        return model_update, snapshots_model_update"},{"line_number":1258,"context_line":""},{"line_number":1259,"context_line":"    def update_consistencygroup(self, context, group,"},{"line_number":1260,"context_line":"                                add_volumes\u003dNone, remove_volumes\u003dNone):"},{"line_number":1261,"context_line":"        msg \u003d _(\u0027Updating a consistency group is not supported.\u0027)"},{"line_number":1262,"context_line":"        LOG.error(msg)"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_c93ef4b6","line":1259,"range":{"start_line":1259,"start_character":8,"end_line":1259,"end_character":31},"in_reply_to":"df140735_c884000f","updated":"2017-06-05 10:25:29.000000000","message":"Done","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"53d30439ef40acc55e5f7239e308de0d7201865c","unresolved":false,"context_lines":[{"line_number":1262,"context_line":"        LOG.error(msg)"},{"line_number":1263,"context_line":"        raise exception.GPFSDriverUnsupportedOperation(msg\u003dmsg)"},{"line_number":1264,"context_line":""},{"line_number":1265,"context_line":"    def create_consistencygroup_from_src(self, context, group, volumes,"},{"line_number":1266,"context_line":"                                         cgsnapshot\u003dNone, snapshots\u003dNone,"},{"line_number":1267,"context_line":"                                         source_cg\u003dNone, source_vols\u003dNone):"},{"line_number":1268,"context_line":"        msg \u003d _(\u0027Creating a consistency group from any source consistency \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_08bd58d7","line":1265,"range":{"start_line":1265,"start_character":8,"end_line":1265,"end_character":40},"updated":"2017-06-01 15:57:15.000000000","message":"Please rename this to _create_consistencygroup_from_src","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"64ba523cc01d4c747068f01fe3f26a7d1eba1de9","unresolved":false,"context_lines":[{"line_number":1262,"context_line":"        LOG.error(msg)"},{"line_number":1263,"context_line":"        raise exception.GPFSDriverUnsupportedOperation(msg\u003dmsg)"},{"line_number":1264,"context_line":""},{"line_number":1265,"context_line":"    def create_consistencygroup_from_src(self, context, group, volumes,"},{"line_number":1266,"context_line":"                                         cgsnapshot\u003dNone, snapshots\u003dNone,"},{"line_number":1267,"context_line":"                                         source_cg\u003dNone, source_vols\u003dNone):"},{"line_number":1268,"context_line":"        msg \u003d _(\u0027Creating a consistency group from any source consistency \u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_a9498060","line":1265,"range":{"start_line":1265,"start_character":8,"end_line":1265,"end_character":40},"in_reply_to":"df140735_08bd58d7","updated":"2017-06-05 10:25:29.000000000","message":"Done","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"53d30439ef40acc55e5f7239e308de0d7201865c","unresolved":false,"context_lines":[{"line_number":1577,"context_line":"                                 {\u0027cluster_id\u0027: self._cluster_id,"},{"line_number":1578,"context_line":"                                  \u0027root_path\u0027: gpfs_base})"},{"line_number":1579,"context_line":""},{"line_number":1580,"context_line":"        data[\u0027consistencygroup_support\u0027] \u003d \u0027True\u0027"},{"line_number":1581,"context_line":"        self._stats \u003d data"},{"line_number":1582,"context_line":"        LOG.debug(\"Exit _update_volume_stats.\")"},{"line_number":1583,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_e8090495","line":1580,"updated":"2017-06-01 15:57:15.000000000","message":"Add data[\u0027consistent_group_snapshot_enabled\u0027] \u003d True","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"64ba523cc01d4c747068f01fe3f26a7d1eba1de9","unresolved":false,"context_lines":[{"line_number":1577,"context_line":"                                 {\u0027cluster_id\u0027: self._cluster_id,"},{"line_number":1578,"context_line":"                                  \u0027root_path\u0027: gpfs_base})"},{"line_number":1579,"context_line":""},{"line_number":1580,"context_line":"        data[\u0027consistencygroup_support\u0027] \u003d \u0027True\u0027"},{"line_number":1581,"context_line":"        self._stats \u003d data"},{"line_number":1582,"context_line":"        LOG.debug(\"Exit _update_volume_stats.\")"},{"line_number":1583,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"df140735_497c2482","line":1580,"in_reply_to":"df140735_e8090495","updated":"2017-06-05 10:25:29.000000000","message":"Done","commit_id":"e44a712a7cb3b404e486397cb7c129537e2b5cbb"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"1af49e1883c92461f88bd78862d36c126127f718","unresolved":false,"context_lines":[{"line_number":1161,"context_line":"            LOG.error(msg)"},{"line_number":1162,"context_line":"            raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1163,"context_line":""},{"line_number":1164,"context_line":"        model_update \u003d {\u0027status\u0027: fields.ConsistencyGroupStatus.AVAILABLE}"},{"line_number":1165,"context_line":"        return model_update"},{"line_number":1166,"context_line":""},{"line_number":1167,"context_line":"    def _delete_consistencygroup(self, context, group, volumes):"}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_2bd55a44","line":1164,"range":{"start_line":1164,"start_character":34,"end_line":1164,"end_character":63},"updated":"2017-06-05 16:56:14.000000000","message":"Please change all fields.ConsistencyGroupStatus to fields.GroupStatus","commit_id":"67ecf6c45682cda88b320a8a6a80ba870fcff0f0"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"cd924bef4a86a94c146b1d78bd9d507d9f5e77f5","unresolved":false,"context_lines":[{"line_number":1161,"context_line":"            LOG.error(msg)"},{"line_number":1162,"context_line":"            raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1163,"context_line":""},{"line_number":1164,"context_line":"        model_update \u003d {\u0027status\u0027: fields.ConsistencyGroupStatus.AVAILABLE}"},{"line_number":1165,"context_line":"        return model_update"},{"line_number":1166,"context_line":""},{"line_number":1167,"context_line":"    def _delete_consistencygroup(self, context, group, volumes):"}],"source_content_type":"text/x-python","patch_set":2,"id":"df140735_be5beaf3","line":1164,"range":{"start_line":1164,"start_character":34,"end_line":1164,"end_character":63},"in_reply_to":"df140735_2bd55a44","updated":"2017-06-05 17:23:47.000000000","message":"Done","commit_id":"67ecf6c45682cda88b320a8a6a80ba870fcff0f0"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"6c15c5b4089b53802989e03d040f0b2a2cd793af","unresolved":false,"context_lines":[{"line_number":772,"context_line":"        \"\"\"Return the local path for the specified volume.\"\"\""},{"line_number":773,"context_line":"        # Check if the volume is part of a consistency group and return"},{"line_number":774,"context_line":"        # the local_path accordingly."},{"line_number":775,"context_line":"        if volume[\u0027group_id\u0027] is not None:"},{"line_number":776,"context_line":"            group \u003d volume_utils.group_get_by_id(volume[\u0027group_id\u0027])"},{"line_number":777,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":778,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume[\u0027group_id\u0027]"},{"line_number":779,"context_line":"                volume_path \u003d os.path.join("},{"line_number":780,"context_line":"                    self.configuration.gpfs_mount_point_base,"},{"line_number":781,"context_line":"                    cgname,"},{"line_number":782,"context_line":"                    volume[\u0027name\u0027]"},{"line_number":783,"context_line":"                )"},{"line_number":784,"context_line":"                return volume_path"},{"line_number":785,"context_line":""},{"line_number":786,"context_line":"        volume_path \u003d os.path.join("},{"line_number":787,"context_line":"            self.configuration.gpfs_mount_point_base,"},{"line_number":788,"context_line":"            volume[\u0027name\u0027]"},{"line_number":789,"context_line":"        )"},{"line_number":790,"context_line":"        return volume_path"},{"line_number":791,"context_line":""},{"line_number":792,"context_line":"    def _get_gpfs_encryption_status(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"df140735_c1291f37","line":789,"range":{"start_line":775,"start_character":0,"end_line":789,"end_character":9},"updated":"2017-06-05 18:04:25.000000000","message":"Please use volume.group_id and volume.name because volume is an object here.","commit_id":"42b476670c17dc0eca24101b32672efd52d23a80"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"15d9295c9a5149ef57e9133f41e28eb670bd8df9","unresolved":false,"context_lines":[{"line_number":772,"context_line":"        \"\"\"Return the local path for the specified volume.\"\"\""},{"line_number":773,"context_line":"        # Check if the volume is part of a consistency group and return"},{"line_number":774,"context_line":"        # the local_path accordingly."},{"line_number":775,"context_line":"        if volume[\u0027group_id\u0027] is not None:"},{"line_number":776,"context_line":"            group \u003d volume_utils.group_get_by_id(volume[\u0027group_id\u0027])"},{"line_number":777,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":778,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume[\u0027group_id\u0027]"},{"line_number":779,"context_line":"                volume_path \u003d os.path.join("},{"line_number":780,"context_line":"                    self.configuration.gpfs_mount_point_base,"},{"line_number":781,"context_line":"                    cgname,"},{"line_number":782,"context_line":"                    volume[\u0027name\u0027]"},{"line_number":783,"context_line":"                )"},{"line_number":784,"context_line":"                return volume_path"},{"line_number":785,"context_line":""},{"line_number":786,"context_line":"        volume_path \u003d os.path.join("},{"line_number":787,"context_line":"            self.configuration.gpfs_mount_point_base,"},{"line_number":788,"context_line":"            volume[\u0027name\u0027]"},{"line_number":789,"context_line":"        )"},{"line_number":790,"context_line":"        return volume_path"},{"line_number":791,"context_line":""},{"line_number":792,"context_line":"    def _get_gpfs_encryption_status(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf091321_f35040d7","line":789,"range":{"start_line":775,"start_character":0,"end_line":789,"end_character":9},"in_reply_to":"df140735_c1291f37","updated":"2017-06-08 09:41:12.000000000","message":"Done","commit_id":"42b476670c17dc0eca24101b32672efd52d23a80"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"6c15c5b4089b53802989e03d040f0b2a2cd793af","unresolved":false,"context_lines":[{"line_number":1586,"context_line":"    def _get_volume_path(self, volume):"},{"line_number":1587,"context_line":"        \"\"\"Returns remote GPFS path for the given volume.\"\"\""},{"line_number":1588,"context_line":"        export_path \u003d self.configuration.gpfs_mount_point_base"},{"line_number":1589,"context_line":"        if volume[\u0027group_id\u0027] is not None:"},{"line_number":1590,"context_line":"            group \u003d volume_utils.group_get_by_id(volume[\u0027group_id\u0027])"},{"line_number":1591,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1592,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume[\u0027group_id\u0027]"},{"line_number":1593,"context_line":"                return os.path.join(export_path, cgname, volume[\u0027name\u0027])"},{"line_number":1594,"context_line":"        return os.path.join(export_path, volume[\u0027name\u0027])"},{"line_number":1595,"context_line":""},{"line_number":1596,"context_line":"    def local_path(self, volume):"},{"line_number":1597,"context_line":"        \"\"\"Returns the local path for the specified volume.\"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"df140735_c1debf3c","line":1594,"range":{"start_line":1589,"start_character":1,"end_line":1594,"end_character":56},"updated":"2017-06-05 18:04:25.000000000","message":"Please use volume.group_id and volume.name because volume is an object.","commit_id":"42b476670c17dc0eca24101b32672efd52d23a80"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"15d9295c9a5149ef57e9133f41e28eb670bd8df9","unresolved":false,"context_lines":[{"line_number":1586,"context_line":"    def _get_volume_path(self, volume):"},{"line_number":1587,"context_line":"        \"\"\"Returns remote GPFS path for the given volume.\"\"\""},{"line_number":1588,"context_line":"        export_path \u003d self.configuration.gpfs_mount_point_base"},{"line_number":1589,"context_line":"        if volume[\u0027group_id\u0027] is not None:"},{"line_number":1590,"context_line":"            group \u003d volume_utils.group_get_by_id(volume[\u0027group_id\u0027])"},{"line_number":1591,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1592,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume[\u0027group_id\u0027]"},{"line_number":1593,"context_line":"                return os.path.join(export_path, cgname, volume[\u0027name\u0027])"},{"line_number":1594,"context_line":"        return os.path.join(export_path, volume[\u0027name\u0027])"},{"line_number":1595,"context_line":""},{"line_number":1596,"context_line":"    def local_path(self, volume):"},{"line_number":1597,"context_line":"        \"\"\"Returns the local path for the specified volume.\"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"bf091321_f32a808e","line":1594,"range":{"start_line":1589,"start_character":1,"end_line":1594,"end_character":56},"in_reply_to":"df140735_c1debf3c","updated":"2017-06-08 09:41:12.000000000","message":"Done","commit_id":"42b476670c17dc0eca24101b32672efd52d23a80"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"6c15c5b4089b53802989e03d040f0b2a2cd793af","unresolved":false,"context_lines":[{"line_number":1600,"context_line":""},{"line_number":1601,"context_line":"        # Check if the volume is part of a consistency group and return"},{"line_number":1602,"context_line":"        # the local_path accordingly."},{"line_number":1603,"context_line":"        if volume[\u0027group_id\u0027] is not None:"},{"line_number":1604,"context_line":"            group \u003d volume_utils.group_get_by_id(volume[\u0027group_id\u0027])"},{"line_number":1605,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1606,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume[\u0027group_id\u0027]"},{"line_number":1607,"context_line":"                return os.path.join(base_local_path, cgname, volume[\u0027name\u0027])"},{"line_number":1608,"context_line":"        return os.path.join(base_local_path, volume[\u0027name\u0027])"},{"line_number":1609,"context_line":""},{"line_number":1610,"context_line":"    def _get_snapshot_path(self, snapshot):"},{"line_number":1611,"context_line":"        \"\"\"Returns remote GPFS path for the given snapshot.\"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"df140735_21bfbbd1","line":1608,"range":{"start_line":1603,"start_character":0,"end_line":1608,"end_character":60},"updated":"2017-06-05 18:04:25.000000000","message":"Use volume.group_id and volume.name.","commit_id":"42b476670c17dc0eca24101b32672efd52d23a80"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"15d9295c9a5149ef57e9133f41e28eb670bd8df9","unresolved":false,"context_lines":[{"line_number":1600,"context_line":""},{"line_number":1601,"context_line":"        # Check if the volume is part of a consistency group and return"},{"line_number":1602,"context_line":"        # the local_path accordingly."},{"line_number":1603,"context_line":"        if volume[\u0027group_id\u0027] is not None:"},{"line_number":1604,"context_line":"            group \u003d volume_utils.group_get_by_id(volume[\u0027group_id\u0027])"},{"line_number":1605,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1606,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume[\u0027group_id\u0027]"},{"line_number":1607,"context_line":"                return os.path.join(base_local_path, cgname, volume[\u0027name\u0027])"},{"line_number":1608,"context_line":"        return os.path.join(base_local_path, volume[\u0027name\u0027])"},{"line_number":1609,"context_line":""},{"line_number":1610,"context_line":"    def _get_snapshot_path(self, snapshot):"},{"line_number":1611,"context_line":"        \"\"\"Returns remote GPFS path for the given snapshot.\"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"bf091321_d319bc88","line":1608,"range":{"start_line":1603,"start_character":0,"end_line":1608,"end_character":60},"in_reply_to":"df140735_21bfbbd1","updated":"2017-06-08 09:41:12.000000000","message":"Done","commit_id":"42b476670c17dc0eca24101b32672efd52d23a80"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"c6c0990dfe4fd94611b77fd68d43156f25b638a4","unresolved":false,"context_lines":[{"line_number":773,"context_line":"        # Check if the volume is part of a consistency group and return"},{"line_number":774,"context_line":"        # the local_path accordingly."},{"line_number":775,"context_line":"        if volume.group_id is not None:"},{"line_number":776,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":777,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":778,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":779,"context_line":"                volume_path \u003d os.path.join("}],"source_content_type":"text/x-python","patch_set":4,"id":"bf091321_37e1f69b","line":776,"range":{"start_line":776,"start_character":12,"end_line":776,"end_character":65},"updated":"2017-06-08 15:45:17.000000000","message":"Actually this call is not needed.  volume.group will give you this directly.","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"c8c79404884fcc0b707f7a0b7100b16ce5f91c94","unresolved":false,"context_lines":[{"line_number":773,"context_line":"        # Check if the volume is part of a consistency group and return"},{"line_number":774,"context_line":"        # the local_path accordingly."},{"line_number":775,"context_line":"        if volume.group_id is not None:"},{"line_number":776,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":777,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":778,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":779,"context_line":"                volume_path \u003d os.path.join("}],"source_content_type":"text/x-python","patch_set":4,"id":"bf091321_be6f1ac4","line":776,"range":{"start_line":776,"start_character":12,"end_line":776,"end_character":65},"in_reply_to":"bf091321_37e1f69b","updated":"2017-06-12 12:04:50.000000000","message":"Done","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"c6c0990dfe4fd94611b77fd68d43156f25b638a4","unresolved":false,"context_lines":[{"line_number":774,"context_line":"        # the local_path accordingly."},{"line_number":775,"context_line":"        if volume.group_id is not None:"},{"line_number":776,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":777,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":778,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":779,"context_line":"                volume_path \u003d os.path.join("},{"line_number":780,"context_line":"                    self.configuration.gpfs_mount_point_base,"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf091321_3d9b1773","line":777,"range":{"start_line":777,"start_character":12,"end_line":777,"end_character":63},"updated":"2017-06-08 15:45:17.000000000","message":"if volume_utils.is_group_a_cg_snapshot_type(volume.group):","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"c8c79404884fcc0b707f7a0b7100b16ce5f91c94","unresolved":false,"context_lines":[{"line_number":774,"context_line":"        # the local_path accordingly."},{"line_number":775,"context_line":"        if volume.group_id is not None:"},{"line_number":776,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":777,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":778,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":779,"context_line":"                volume_path \u003d os.path.join("},{"line_number":780,"context_line":"                    self.configuration.gpfs_mount_point_base,"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf091321_9e74569b","line":777,"range":{"start_line":777,"start_character":12,"end_line":777,"end_character":63},"in_reply_to":"bf091321_3d9b1773","updated":"2017-06-12 12:04:50.000000000","message":"Done","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"c6c0990dfe4fd94611b77fd68d43156f25b638a4","unresolved":false,"context_lines":[{"line_number":1587,"context_line":"        \"\"\"Returns remote GPFS path for the given volume.\"\"\""},{"line_number":1588,"context_line":"        export_path \u003d self.configuration.gpfs_mount_point_base"},{"line_number":1589,"context_line":"        if volume.group_id is not None:"},{"line_number":1590,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":1591,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1592,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":1593,"context_line":"                return os.path.join(export_path, cgname, volume.name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf091321_7d9f4f50","line":1590,"range":{"start_line":1590,"start_character":12,"end_line":1590,"end_character":65},"updated":"2017-06-08 15:45:17.000000000","message":"Same as above.  This call is not needed.","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"c8c79404884fcc0b707f7a0b7100b16ce5f91c94","unresolved":false,"context_lines":[{"line_number":1587,"context_line":"        \"\"\"Returns remote GPFS path for the given volume.\"\"\""},{"line_number":1588,"context_line":"        export_path \u003d self.configuration.gpfs_mount_point_base"},{"line_number":1589,"context_line":"        if volume.group_id is not None:"},{"line_number":1590,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":1591,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1592,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":1593,"context_line":"                return os.path.join(export_path, cgname, volume.name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f231b9d_b7085b28","line":1590,"range":{"start_line":1590,"start_character":12,"end_line":1590,"end_character":65},"in_reply_to":"bf091321_7d9f4f50","updated":"2017-06-12 12:04:50.000000000","message":"Done","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"c6c0990dfe4fd94611b77fd68d43156f25b638a4","unresolved":false,"context_lines":[{"line_number":1588,"context_line":"        export_path \u003d self.configuration.gpfs_mount_point_base"},{"line_number":1589,"context_line":"        if volume.group_id is not None:"},{"line_number":1590,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":1591,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1592,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":1593,"context_line":"                return os.path.join(export_path, cgname, volume.name)"},{"line_number":1594,"context_line":"        return os.path.join(export_path, volume.name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf091321_5dac8b8b","line":1591,"range":{"start_line":1591,"start_character":56,"end_line":1591,"end_character":61},"updated":"2017-06-08 15:45:17.000000000","message":"volume.group","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"c8c79404884fcc0b707f7a0b7100b16ce5f91c94","unresolved":false,"context_lines":[{"line_number":1588,"context_line":"        export_path \u003d self.configuration.gpfs_mount_point_base"},{"line_number":1589,"context_line":"        if volume.group_id is not None:"},{"line_number":1590,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":1591,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1592,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":1593,"context_line":"                return os.path.join(export_path, cgname, volume.name)"},{"line_number":1594,"context_line":"        return os.path.join(export_path, volume.name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f231b9d_970d5f39","line":1591,"range":{"start_line":1591,"start_character":56,"end_line":1591,"end_character":61},"in_reply_to":"bf091321_5dac8b8b","updated":"2017-06-12 12:04:50.000000000","message":"Done","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"c6c0990dfe4fd94611b77fd68d43156f25b638a4","unresolved":false,"context_lines":[{"line_number":1601,"context_line":"        # Check if the volume is part of a consistency group and return"},{"line_number":1602,"context_line":"        # the local_path accordingly."},{"line_number":1603,"context_line":"        if volume.group_id is not None:"},{"line_number":1604,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":1605,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1606,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":1607,"context_line":"                return os.path.join(base_local_path, cgname, volume.name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf091321_7dc86f54","line":1604,"range":{"start_line":1604,"start_character":12,"end_line":1604,"end_character":65},"updated":"2017-06-08 15:45:17.000000000","message":"Not needed.","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"c8c79404884fcc0b707f7a0b7100b16ce5f91c94","unresolved":false,"context_lines":[{"line_number":1601,"context_line":"        # Check if the volume is part of a consistency group and return"},{"line_number":1602,"context_line":"        # the local_path accordingly."},{"line_number":1603,"context_line":"        if volume.group_id is not None:"},{"line_number":1604,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":1605,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1606,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":1607,"context_line":"                return os.path.join(base_local_path, cgname, volume.name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f231b9d_7712c35a","line":1604,"range":{"start_line":1604,"start_character":12,"end_line":1604,"end_character":65},"in_reply_to":"bf091321_7dc86f54","updated":"2017-06-12 12:04:50.000000000","message":"Done","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"c6c0990dfe4fd94611b77fd68d43156f25b638a4","unresolved":false,"context_lines":[{"line_number":1602,"context_line":"        # the local_path accordingly."},{"line_number":1603,"context_line":"        if volume.group_id is not None:"},{"line_number":1604,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":1605,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1606,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":1607,"context_line":"                return os.path.join(base_local_path, cgname, volume.name)"},{"line_number":1608,"context_line":"        return os.path.join(base_local_path, volume.name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"bf091321_1dddf313","line":1605,"range":{"start_line":1605,"start_character":56,"end_line":1605,"end_character":61},"updated":"2017-06-08 15:45:17.000000000","message":"volume.group","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"},{"author":{"_account_id":9207,"name":"Gaurang Tapase","email":"gtapase@ddn.com","username":"gaurang.tapase"},"change_message_id":"c8c79404884fcc0b707f7a0b7100b16ce5f91c94","unresolved":false,"context_lines":[{"line_number":1602,"context_line":"        # the local_path accordingly."},{"line_number":1603,"context_line":"        if volume.group_id is not None:"},{"line_number":1604,"context_line":"            group \u003d volume_utils.group_get_by_id(volume.group_id)"},{"line_number":1605,"context_line":"            if volume_utils.is_group_a_cg_snapshot_type(group):"},{"line_number":1606,"context_line":"                cgname \u003d \"consisgroup-%s\" % volume.group_id"},{"line_number":1607,"context_line":"                return os.path.join(base_local_path, cgname, volume.name)"},{"line_number":1608,"context_line":"        return os.path.join(base_local_path, volume.name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f231b9d_570fc72d","line":1605,"range":{"start_line":1605,"start_character":56,"end_line":1605,"end_character":61},"in_reply_to":"bf091321_1dddf313","updated":"2017-06-12 12:04:50.000000000","message":"Done","commit_id":"5ca62dd99e603a31f319fb260d123dbf4dbd0de6"}]}