(oops, resending without html) On 05/09/2018 11:22 AM, Matthew Thode wrote:
python-swiftclient prior to 3.2.0 seemed to incidentally support radosgw tempurls. That is, there was no official support, but it still worked. In 3.2.0 (specifically the linked commit(s)) tempurls were validated to require /v1/account/container/object, which does not work with radosgw as it expects /v1/container/object. This means that radosgw tempurls fail to work, which further means that radosgw will stop working for things like ironic. I can see the point that swiftclient should not care about ceph not fully implementing the swift spec and not supporting the radosgw url syntax, but it seems like a step back. If this is not fixed then things like ironic will not work with radosgw for Ocata and above (as that's when this change was made). We'd need to wait for either ceph to fix this and support the account part of the url (probably just dropping it) or have people fork python-swiftclient to 'fix' it. I'm not sure what the right answer is... https://github.com/openstack/python-swiftclient/commit/4c955751d340a8f71a2eebdb3c58d90b36874a66 https://github.com/openstack/ironic/blob/214b694f05d200ac1e2ce6db631546f2831c01f7/ironic/common/glance_service/v2/image_service.py#L152-L185 https://bugs.launchpad.net/ironic/+bug/1747384 __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: OpenStack-dev-request@xxxxxxxxxxxxxxxxxxx?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
Thanks for raising the issue. Radosgw does have a config option 'rgw_swift_account_in_url' to expect this url format, though it defaults to false and I'm not 100% sure that it applies correctly to tempurls.
-- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
