From bb79f7bc2c44f56ef04518e89441db47786c1a22 Mon Sep 17 00:00:00 2001 From: RAJAT SINGH Date: Thu, 23 Jan 2020 18:32:08 +0530 Subject: [PATCH] Ceph: PV provisioning fails upon upgrade CSI CephFS provisioner fails to provision a Persistent Volume after upgrading from Rook Ceph Operator 1.1.2 to 1.1.7. This issue can be fixed by removing givePoolAppTag function whenever there is an appName cephFs. Signed-off-by: RAJAT SINGH --- pkg/operator/ceph/file/filesystem.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/pkg/operator/ceph/file/filesystem.go b/pkg/operator/ceph/file/filesystem.go index 9f59c8e5f..27001b45f 100644 --- a/pkg/operator/ceph/file/filesystem.go +++ b/pkg/operator/ceph/file/filesystem.go @@ -34,7 +34,6 @@ import ( const ( dataPoolSuffix = "data" metaDataPoolSuffix = "metadata" - appName = "cephfs" ) // Filesystem represents an instance of a Ceph filesystem (CephFS) @@ -144,7 +143,7 @@ func newFS(name, namespace string) *Filesystem { func SetPoolSize(f *Filesystem, context *clusterd.Context, spec cephv1.FilesystemSpec) error { // generating the metadata pool's name metadataPoolName := generateMetaDataPoolName(f) - err := client.CreatePoolWithProfile(context, f.Namespace, metadataPoolName, spec.MetadataPool, appName) + err := client.CreatePoolWithProfile(context, f.Namespace, metadataPoolName, spec.MetadataPool, "") if err != nil { return errors.Wrapf(err, "failed to update metadata pool %q", metadataPoolName) } @@ -152,7 +151,7 @@ func SetPoolSize(f *Filesystem, context *clusterd.Context, spec cephv1.Filesyste dataPoolNames := generateDataPoolNames(f, spec) for i, pool := range spec.DataPools { poolName := dataPoolNames[i] - err := client.CreatePoolWithProfile(context, f.Namespace, poolName, pool, appName) + err := client.CreatePoolWithProfile(context, f.Namespace, poolName, pool, "") if err != nil { return errors.Wrapf(err, "failed to update datapool %q", poolName) }