From e0d67a1835d55ab0a24b420a662be7c31c44da04 Mon Sep 17 00:00:00 2001 From: Matthew Oliver Date: Wed, 21 Oct 2020 06:14:40 +0000 Subject: [PATCH] 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 --- src/rgw/rgw_sal.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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 objs; -- 2.39.5