yahoo-eng-team team mailing list archive
-
yahoo-eng-team team
-
Mailing list archive
-
Message #72502
[Bug 1309597] Re: wrong swfit acl format when setting swift_store_admin_tenant
** Changed in: glance
Status: In Progress => Invalid
--
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Glance.
https://bugs.launchpad.net/bugs/1309597
Title:
wrong swfit acl format when setting swift_store_admin_tenant
Status in Glance:
Invalid
Bug description:
Hi,
I'm testing on icehouse (git >>
2c4bd695652a628758eb56cb36394940a855d696)
Here how to reproduce:
http://paste.openstack.org/show/61570/
I have Three problems here
* The admin tenant list seems to be applied only on the write ACL.
* Whatever the value of the admin_tenants list, the string ":*" is added to the list, which will result in having a bad format or the ACL string, Swift accept this string format but will end up with an error when container is accessed by glance. Making the image unusable in some cases
* Also write acls value are not correct when the set acl fucntion is called several times, because write_tenant has [] as a default parameter (know python problem when a mutable object is passed as default parameter).
I checked the code and we can easily see the problem here.
in glance.store.swift
http://paste.openstack.org/show/61574/
To manage notifications about this bug go to:
https://bugs.launchpad.net/glance/+bug/1309597/+subscriptions
References