RGW/admin: add tenant to bucket struct field

[krafZLorG]

RGW/admin: add tenant to bucket struct field

This change adds a new bucket struct field, so we can use rgw.admin.API.GetBucketInfo() to get a bucket's tenant.

$ radosgw-admin bucket stats --bucket "tenant0/qb001"
{
    "bucket": "qb001",
    "num_shards": 11,
    "tenant": "tenant0",

Signed-off-by: Dmitry Kvashnin [email protected]

[anoopcs9]

Currently we don't verify the returned Bucket struct in corresponding tests? At least in this context shouldn't we add a check to verify newly added Tenant field?

[phlogistonjohn]

@Mergifyio rebase

[mergify]

rebase

✅ Branch has been successfully rebased

[phlogistonjohn]

@krafZLorG @anoopcs9 @thotz I'm not clear what the state of this PR is. Do we need changes, reviews, or some other form of attention? I just triggered a rebase but I don't know where to go next with this.

[anoopcs9]

@krafZLorG Can you consider the suggestion from #898 (comment)

[krafZLorG]

There is currently no test for checking the Bucket structure. Why is it a stop factor for adding a missing field? (#704)

[anoopcs9]

There is currently no test for checking the Bucket structure. Why is it a stop factor for adding a missing field? (#704)

Alright. I tend to agree. I'll wait for others to have a say before final approval.

[phlogistonjohn]

Changes look OK to me but I'd prefer a 2nd review from a subject matter expert on RGW like @thotz - I don't think we need to require a new test although it would be nice, it could help avoid future regressions.

[phlogistonjohn]

@Mergifyio rebase

[mergify]

rebase

✅ Branch has been successfully rebased

[anoopcs9]

@Mergifyio rebase

[mergify]

rebase

✅ Branch has been successfully rebased

[phlogistonjohn]

👍 👍 thanks everyone, for both the contribution and reviews!