Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-10 Thread Jim Rollenhagen
On Thu, May 10, 2018 at 11:19 AM, Matthew Thode wrote: > On 18-05-10 11:09:41, Jim Rollenhagen wrote: > > > > Awesome. Can someone submit a patch? It will need to remove the special > URL > > building for radosgw linked earlier in the thread, and add a reference to > > this config option in the d

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-10 Thread Matthew Thode
On 18-05-10 11:09:41, Jim Rollenhagen wrote: > On Thu, May 10, 2018 at 11:05 AM, Matthew Thode > wrote: > > > On 18-05-09 15:14:37, Matthew Thode wrote: > > > On 18-05-09 12:24:32, Clay Gerrard wrote: > > > > On Wed, May 9, 2018 at 12:08 PM, Matthew Thode < > > prometheanf...@gentoo.org> > > > >

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-10 Thread Jim Rollenhagen
On Thu, May 10, 2018 at 11:05 AM, Matthew Thode wrote: > On 18-05-09 15:14:37, Matthew Thode wrote: > > On 18-05-09 12:24:32, Clay Gerrard wrote: > > > On Wed, May 9, 2018 at 12:08 PM, Matthew Thode < > prometheanf...@gentoo.org> > > > wrote: > > > > > > > > > > > * Proper fix would be to make ce

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-10 Thread Matthew Thode
On 18-05-09 15:14:37, Matthew Thode wrote: > On 18-05-09 12:24:32, Clay Gerrard wrote: > > On Wed, May 9, 2018 at 12:08 PM, Matthew Thode > > wrote: > > > > > > > > * Proper fix would be to make ceph support the account field > > > > > > > Is the 'rgw_swift_account_in_url' option not correct/suf

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-09 Thread Julia Kreger
On Wed, May 9, 2018 at 5:10 PM, Casey Bodley wrote: > > On 05/09/2018 11:40 AM, Casey Bodley wrote: >> >> >> 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 s

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-09 Thread Casey Bodley
On 05/09/2018 11:40 AM, Casey Bodley wrote: 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

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-09 Thread Matthew Thode
On 18-05-09 12:24:32, Clay Gerrard wrote: > On Wed, May 9, 2018 at 12:08 PM, Matthew Thode > wrote: > > > > > * Proper fix would be to make ceph support the account field > > > > Is the 'rgw_swift_account_in_url' option not correct/sufficient? > I didn't see that option, I'll test and get back

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-09 Thread Clay Gerrard
On Wed, May 9, 2018 at 12:35 PM, Jim Rollenhagen wrote: > > It works with both, see the link from earlier in the thread: > https://github.com/openstack/ironic/blob/214b694f05d200ac1e2ce6db631546 > f2831c01f7/ironic/common/glance_service/v2/image_service.py#L152-L185 > > Ah! Perfect! Thanks for

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-09 Thread Jim Rollenhagen
On Wed, May 9, 2018 at 3:24 PM, Clay Gerrard wrote: > > > On Wed, May 9, 2018 at 12:08 PM, Matthew Thode > wrote: > >> >> * Proper fix would be to make ceph support the account field >> > > Is the 'rgw_swift_account_in_url' option not correct/sufficient? > I guess we could just document that pe

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-09 Thread Clay Gerrard
On Wed, May 9, 2018 at 12:08 PM, Matthew Thode wrote: > > * Proper fix would be to make ceph support the account field > Is the 'rgw_swift_account_in_url' option not correct/sufficient? > * Workaround would be to specify an old swiftclient to install (3.1.0, > pre-ocata) > Doesn't seem great

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-09 Thread Matthew Thode
On 18-05-09 13:42:02, Jim Rollenhagen wrote: > On Wed, May 9, 2018 at 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 link

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-09 Thread Jim Rollenhagen
On Wed, May 9, 2018 at 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/acco

Re: [openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-09 Thread Casey Bodley
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/obj

[openstack-dev] [swift][ironic][ceph][radosgw] radosgw "support" in python-swiftclient droped for ocata and above

2018-05-09 Thread Matthew Thode
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 expe