From 89a3eb5542e6af0c88e598e2f694c43a0a2f1be9 Mon Sep 17 00:00:00 2001 From: Joshua Hoblitt Date: Wed, 25 Sep 2024 14:21:53 -0700 Subject: [PATCH] rgw: fix cephObjectStore failing with pre-existing pools When attempting to use pre-existing (or cephBlockPool managed) pools and the cephObjectStore.spec.{metadataPool,dataPool} fields are not specified, rgw creation will fail with this error: 2024-09-25 21:00:17.744358 E | ceph-object-controller: failed to reconcile CephObjectStore "rook-ceph/test1". failed to create object store deployments: failed to create object pools: failed to create metadata pools: failed to create pool "test1.rgw.control": pool "test1.rgw.control" type is not defined as replicated or erasure coded Signed-off-by: Joshua Hoblitt (cherry picked from commit 9b952e0a553d06be42f907cfb8a0d0f3b4102483) --- pkg/operator/ceph/object/objectstore.go | 3 +++ 1 file changed, 3 insertions(+) diff --git a/pkg/operator/ceph/object/objectstore.go b/pkg/operator/ceph/object/objectstore.go index 84c129000..328d2a0bf 100644 --- a/pkg/operator/ceph/object/objectstore.go +++ b/pkg/operator/ceph/object/objectstore.go @@ -767,6 +767,9 @@ func ConfigurePools(context *Context, cluster *cephv1.ClusterSpec, metadataPool, if len(missingPools) > 0 { return fmt.Errorf("CR store pools are missing: %v", missingPools) } + + // pools exist, nothing to do + return nil } // get the default PG count for rgw metadata pools