Skip to content

Commit

Permalink
[BZ 2283643] object: virtulhostnames is not required in the endpoint …
Browse files Browse the repository at this point in the history
…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 <[email protected]>
(cherry picked from commit b5d79d1efa4ef2b859af4dd2cf3aa72230105b4a)
Signed-off-by: Jiffin Tony Thottan <[email protected]>
  • Loading branch information
thotz committed Jun 4, 2024
1 parent a2396a5 commit a8af6f9
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions pkg/operator/ceph/object/rgw.go
Original file line number Diff line number Diff line change
Expand Up @@ -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)
}
Expand Down

0 comments on commit a8af6f9

Please sign in to comment.