From: Abhishek Lekshmanan <abhishek@suse.com>
To: Pete Zaitcev <zaitcev@redhat.com>
Cc: Abhishek Lekshmanan <abhishek@suse.com>,
ceph-devel <ceph-devel@vger.kernel.org>,
Radoslaw Zarzynski <rzarzynski@mirantis.com>
Subject: Re: Queries on RGW Multitenancy
Date: Fri, 22 Apr 2016 18:32:01 +0200 [thread overview]
Message-ID: <87bn517gfy.fsf@suse.com> (raw)
In-Reply-To: <20160421155916.637a7e77@lembas.zaitcev.lan>
Pete Zaitcev writes:
> On Wed, 20 Apr 2016 18:14:36 +0200
> Abhishek Lekshmanan <abhishek@suse.com> wrote:
>
>> Are there any changes in the way bucket and object urls are used by
>> clients after creating a user with a tenant? My understanding so far has
>> been that the urls look the same for clients, and based on the
>> credentials passed in, we determine the tenant for requests. Is this
>> correct?
>
> Yes.
>
>> Also on making a bucket/object public the urls follow a format like:
>> `<host>/<tenant>:<bucket>`
>>
>> Is there an equivalent s3 vhost style url for the above? Trying any of
>> tenant.bucket.host combos mostly gave 404s
>
> No, sadly there is no such syntax. This is because a bucket name of
> "buc.ket" is valid, so there's no way to tell a cross-tenant reference
> in your proposal from a valid bucket name within a tenant. One has to
> continue using the old access method, like when using extended names
> of Virgina region buckets.
Thanks for clarifying, yeah I expected it would be difficult since `.`
was allowed to be used in bucket names and you can't construct domain
names with most other seperators, not a deal breaker :)
>
>> And similarly for swift making a bucket public, the only url that seemed
>> accessible was the above mentioned `<host>/<tenant>:<bucket>` format and
>> there is no swift url of the format
>> `<host>/swift/v1/<tenant>/<container>` etc? Is this also the expected
>> behaviour?
>
> Historically, we did not have the ability to do this, because, unlike
> traditional Swift, our URLs did not have a tenant in them. E.g. it was
> just /swift/v1/<container>. In Jewel, the tenant is included. It was
> made fully backwards compatible by saving the bit to signify the
> tenant-included URL into the token (the "rgwts" token).
> So now the cross-tenant access is done in the exactly the same way as in OpenStack
> Swift.
Ah ok, the information is encoded in the token, so on making a
bucket/container public we'll still access it at
<host>/tenant:container/... location, rather than a /swift/tenant sort
of url
>
> One problem remains, as you can guess: cross-tenant access while
> authenticated with Keystone. This is simply not implemented. We considered
> a couple of approaches, such as
> - a special syntax, such as backslash "tenant\bucket"
> - a new header "X-RGW-Tenant"
> - a user attribute in Keystone (requires cooperation from Keystone)
> - a separate endpoint in the Keystone catalog, possibly using regions
>
> Thanks for raising this issue. Honestly I gave up on finding an elegant
> solution for now, although tinkering with endpoints seems like a winner.
> I think Radoslaw liked it too. If you have ideas, by all means please
> propose them.
I might be understanding this wrong, but since endpoints are an admin only
thing (to create), are you suggesting something like a different region
per tenant or so?
Doesn't keystone returns the tenantid while authenticating (I could be
wrong), in which case we somehow enforce that the tenant-id for a user
created in rgw[1] if you're using keystone should match the ones used at
openstack and check for the value of tenant-id returned by keystone when
authenticating?
[1]: this is still tricky, as we create users at runtime after first
authentication with keystone iirc
>
> -- Pete
--
Abhishek Lekshmanan
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2016-04-22 16:32 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-20 16:14 Queries on RGW Multitenancy Abhishek Lekshmanan
2016-04-21 21:59 ` Pete Zaitcev
2016-04-21 23:30 ` Pete Zaitcev
2016-04-22 16:32 ` Abhishek Lekshmanan [this message]
2016-04-22 17:23 ` Pete Zaitcev
2016-04-22 17:36 ` Abhishek L
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87bn517gfy.fsf@suse.com \
--to=abhishek@suse.com \
--cc=ceph-devel@vger.kernel.org \
--cc=rzarzynski@mirantis.com \
--cc=zaitcev@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).