Felipe,
  A couple comments on your debdiffs.
a.) Please add dep8 comments, and point to the upstream commits if you can.
b.) in trunk, we dropped use of 'strip_version' everywhere in revno 362 [1] , 
then a pep8 cleanup in revno 364 [2], but then added the strip_version *back* 
for glance at revno 365 [3].  revno 365 was 2014-12-12, only 2 days after 362.  
 I have since lost context, but I suspect that doing so broke an older cloud 
(quite possibly canonistack).

It seems that you have incorporated upstream commits 362 and 364 but not
365.

As I said in commit message at 365, strip_version is certainly
nefarious, but seems to be necessary.

--
[1] http://bazaar.launchpad.net/~smoser/simplestreams/trunk/revision/362
[2] http://bazaar.launchpad.net/~smoser/simplestreams/trunk/revision/364
[3] http://bazaar.launchpad.net/~smoser/simplestreams/trunk/revision/365

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1346935

Title:
  [SRU] Failing to create simplestreams bucket on ceph-radosgw

To manage notifications about this bug go to:
https://bugs.launchpad.net/simplestreams/+bug/1346935/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to