From c3a9f5fdd9b26338445cd81b636761d22ac893f4 Mon Sep 17 00:00:00 2001 From: immanuwell Date: Sat, 9 May 2026 12:50:31 +0400 Subject: [PATCH] docs: fix duplicated words in docs and comments Clean up duplicated words and a typo in object storage docs, comments, and user-facing messages. Signed-off-by: immanuwell --- .../Object-Storage-RGW/ceph-object-bucket-claim.md | 2 +- .../Storage-Configuration/Object-Storage-RGW/object-storage.md | 2 +- cmd/rook/userfacing/multus/validation/validation.go | 2 +- pkg/daemon/ceph/client/filesystem.go | 2 +- pkg/daemon/ceph/osd/device.go | 2 +- pkg/operator/ceph/cluster/mgr/mgr_test.go | 2 +- pkg/operator/ceph/file/dependent.go | 2 +- 7 files changed, 7 insertions(+), 7 deletions(-) diff --git a/Documentation/Storage-Configuration/Object-Storage-RGW/ceph-object-bucket-claim.md b/Documentation/Storage-Configuration/Object-Storage-RGW/ceph-object-bucket-claim.md index b19d38602..0bc008c3b 100644 --- a/Documentation/Storage-Configuration/Object-Storage-RGW/ceph-object-bucket-claim.md +++ b/Documentation/Storage-Configuration/Object-Storage-RGW/ceph-object-bucket-claim.md @@ -108,7 +108,7 @@ OBCs in unexpected ways. At worst, users may brick the whole S3 object store for (`bucketPolicy` in particular). Administrators should take care to enable features only when they are personally willing to take on the risks. -OBC `additionalConfig` fields can be enabled and disabled using the the `rook-ceph-operator-config` +OBC `additionalConfig` fields can be enabled and disabled using the `rook-ceph-operator-config` configmap value `ROOK_OBC_ALLOW_ADDITIONAL_CONFIG_FIELDS`. ### OBC Custom Resource after Bucket Provisioning diff --git a/Documentation/Storage-Configuration/Object-Storage-RGW/object-storage.md b/Documentation/Storage-Configuration/Object-Storage-RGW/object-storage.md index 04097381f..9435128bc 100644 --- a/Documentation/Storage-Configuration/Object-Storage-RGW/object-storage.md +++ b/Documentation/Storage-Configuration/Object-Storage-RGW/object-storage.md @@ -684,7 +684,7 @@ The Ceph Object Gateway supports accessing buckets using [virtual host-style](https://docs.aws.amazon.com/AmazonS3/latest/userguide/VirtualHosting.html) addressing, which allows addressing buckets using the bucket name as a subdomain in the endpoint. -AWS has deprecated the the alternative path-style addressing mode which is Rook and Ceph's default. +AWS has deprecated the alternative path-style addressing mode which is Rook and Ceph's default. As a result, many end-user applications have begun to remove path-style support entirely. Many production clusters will have to enable virtual host-style address. diff --git a/cmd/rook/userfacing/multus/validation/validation.go b/cmd/rook/userfacing/multus/validation/validation.go index 23127ab65..9593a4ded 100644 --- a/cmd/rook/userfacing/multus/validation/validation.go +++ b/cmd/rook/userfacing/multus/validation/validation.go @@ -109,7 +109,7 @@ func init() { "The namespace for validation test resources. "+ "It is recommended to set this to the namespace in which Rook's Ceph cluster will be installed.") - // VarPF() keeps the the specific var passed to it for setting at runtime, and the current + // VarPF() keeps the specific var passed to it for setting at runtime, and the current // val of that var when VarPF() is called is used as the default validationConfig.ResourceTimeout = defaultConfig.ResourceTimeout t := (*timeoutMinutes)(&validationConfig.ResourceTimeout) diff --git a/pkg/daemon/ceph/client/filesystem.go b/pkg/daemon/ceph/client/filesystem.go index db501563a..733f4e181 100644 --- a/pkg/daemon/ceph/client/filesystem.go +++ b/pkg/daemon/ceph/client/filesystem.go @@ -525,7 +525,7 @@ func listSubVolumeSnapshotPendingClones(context *clusterd.Context, clusterInfo * } if err := json.Unmarshal(buf, &pendingClones); err != nil { - return pendingClones, errors.Wrapf(err, "failed to unmarshal pending clones list for for snapshot %q in filesystem %q subvolume group %q", snap, fsName, groupName) + return pendingClones, errors.Wrapf(err, "failed to unmarshal pending clones list for snapshot %q in filesystem %q subvolume group %q", snap, fsName, groupName) } return pendingClones, nil diff --git a/pkg/daemon/ceph/osd/device.go b/pkg/daemon/ceph/osd/device.go index 0d2385748..0ae1a9aae 100644 --- a/pkg/daemon/ceph/osd/device.go +++ b/pkg/daemon/ceph/osd/device.go @@ -63,7 +63,7 @@ type DeviceOsdIDEntry struct { Config DesiredDevice // Device specific config options PersistentDevicePaths []string DeviceInfo *sys.LocalDisk // low-level info about the device - RestoreOSD bool // Restore OSD by reparing it with with OSD ID + RestoreOSD bool // Restore OSD by repairing it with OSD ID } func (m *DeviceOsdMapping) String() string { diff --git a/pkg/operator/ceph/cluster/mgr/mgr_test.go b/pkg/operator/ceph/cluster/mgr/mgr_test.go index 77b957a85..2650e8dc4 100644 --- a/pkg/operator/ceph/cluster/mgr/mgr_test.go +++ b/pkg/operator/ceph/cluster/mgr/mgr_test.go @@ -348,7 +348,7 @@ func TestUpdateServiceSelectors(t *testing.T) { _, err2 := c.context.Clientset.CoreV1().Services(c.clusterInfo.Namespace).Create(clusterInfo.Context, &svc2, metav1.CreateOptions{}) assert.NoError(t, err2) - // Check the the label has been only removed from svc1 + // Check the label has been only removed from svc1 c.updateServiceSelectors() // Make sure we remove controller.DaemonIDLabel label from all services tagged as 'app: rook-ceph-mgr' diff --git a/pkg/operator/ceph/file/dependent.go b/pkg/operator/ceph/file/dependent.go index 3cda690c2..14ad913b6 100644 --- a/pkg/operator/ceph/file/dependent.go +++ b/pkg/operator/ceph/file/dependent.go @@ -60,7 +60,7 @@ func filesystemExists(clusterdCtx *clusterd.Context, clusterInfo *cephclient.Clu _, err := cephclient.GetFilesystem(clusterdCtx, clusterInfo, name) if err != nil { if code, ok := kexec.ExitStatus(err); ok && code == int(syscall.ENOENT) { - log.NamedInfo(nsName, logger, "filesystem deletion will continue without checking for dependencies since the the filesystem does not exist within Ceph") + log.NamedInfo(nsName, logger, "filesystem deletion will continue without checking for dependencies since the filesystem does not exist within Ceph") return false, nil } return false, errors.Wrapf(err, "failed to check for existence of CephFilesystem %q", nsName)