From a8af6f9a36a2075df59615ab978124010331b55a Mon Sep 17 00:00:00 2001 From: Jiffin Tony Thottan Date: Fri, 5 Apr 2024 12:43:36 +0530 Subject: [PATCH] [BZ 2283643] object: virtulhostnames is not required in the endpoint for rgw The virtualhostnames for rgw is added to possible endpoint list, this endpoint is mainly used by rook operator for communicating with RGW server. If enable the virtualhostname feature for rgw, by default the service endpoint is added to the list. So the service endpoint can accessed even if the feature is enabled. Signed-off-by: Jiffin Tony Thottan (cherry picked from commit b5d79d1efa4ef2b859af4dd2cf3aa72230105b4a) Signed-off-by: Jiffin Tony Thottan --- pkg/operator/ceph/object/rgw.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/pkg/operator/ceph/object/rgw.go b/pkg/operator/ceph/object/rgw.go index f41ecff1f012..6743bb92a465 100644 --- a/pkg/operator/ceph/object/rgw.go +++ b/pkg/operator/ceph/object/rgw.go @@ -348,9 +348,6 @@ func getDomainName(s *cephv1.CephObjectStore, returnRandomDomainIfMultiple bool) for _, e := range s.Spec.Gateway.ExternalRgwEndpoints { endpoints = append(endpoints, e.String()) } - } else if s.Spec.Hosting != nil && len(s.Spec.Hosting.DNSNames) > 0 { - // if the store is internal and has DNS names, pick a random DNS name to use - endpoints = s.Spec.Hosting.DNSNames } else { return domainNameOfService(s) }