From 22ee67813618351ca8a983eee65f6460af7d9afe Mon Sep 17 00:00:00 2001 From: Anis Eleuch Date: Tue, 26 Sep 2023 00:14:56 -0700 Subject: [PATCH] tier: Avoid doing versioned operations since not required anymore (#18108) Currently, setting a new tiering target returns an error when a bucket is versioned and the tiering credentials does not have authorization to specify a version-id when reading or removing a specific version; Since tiering does not require versioning anymore; avoid doing versioned operations when performing checklist ops while adding a new tiering configuration. --- cmd/warm-backend.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/cmd/warm-backend.go b/cmd/warm-backend.go index d04c6067f..2a95bf9c1 100644 --- a/cmd/warm-backend.go +++ b/cmd/warm-backend.go @@ -49,7 +49,7 @@ const probeObject = "probeobject" // to perform all operations defined in the WarmBackend interface. func checkWarmBackend(ctx context.Context, w WarmBackend) error { var empty bytes.Reader - rv, err := w.Put(ctx, probeObject, &empty, 0) + _, err := w.Put(ctx, probeObject, &empty, 0) if err != nil { if _, ok := err.(BackendDown); ok { return err @@ -60,7 +60,7 @@ func checkWarmBackend(ctx context.Context, w WarmBackend) error { } } - r, err := w.Get(ctx, probeObject, rv, WarmBackendGetOpts{}) + r, err := w.Get(ctx, probeObject, "", WarmBackendGetOpts{}) xhttp.DrainBody(r) if err != nil { if _, ok := err.(BackendDown); ok { @@ -78,7 +78,7 @@ func checkWarmBackend(ctx context.Context, w WarmBackend) error { } } } - if err = w.Remove(ctx, probeObject, rv); err != nil { + if err = w.Remove(ctx, probeObject, ""); err != nil { if _, ok := err.(BackendDown); ok { return err }