From: Matthew Oliver <moliver@suse.com>
Date: Wed, 21 Oct 2020 06:14:40 +0000 (+0000)
Subject: RGW: Check all shards for user manifest parts
X-Git-Tag: v17.0.0~555^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=e0d67a1835d55ab0a24b420a662be7c31c44da04;p=ceph-ci.git

RGW: Check all shards for user manifest parts

When we go and read an user manifest LO (or a DLO in the swift api)
there is a bug where we supply a defaulted params object that defaults
shard_id to 0. This means that when issueing a GET to the LO it'll only
check shard 0 for parts.

This is causing errors in tempest tests, and is obviously wrong.

This patch defaults the ListParams.shard_id to -1, so by default
all shards are checked for parts and therefore complete LO's
are returned.

Fixes: https://tracker.ceph.com/issues/47801
Fixes: https://tracker.ceph.com/issues/47800
Signed-off-by: Matthew Oliver <moliver@suse.com>
---

diff --git a/src/rgw/rgw_sal.h b/src/rgw/rgw_sal.h
index fc7dd911fe9..882d562e12c 100644
--- a/src/rgw/rgw_sal.h
+++ b/src/rgw/rgw_sal.h
@@ -154,7 +154,7 @@ class RGWBucket {
       RGWAccessListFilter *filter{nullptr};
       bool list_versions{false};
       bool allow_unordered{false};
-      int shard_id{0};
+      int shard_id{-1};
     };
     struct ListResults {
       vector<rgw_bucket_dir_entry> objs;