Merge lp://qastaging/~thedac/simplestreams/lp1719879 into lp://qastaging/simplestreams

Proposed by David Ames
Status: Merged
Merged at revision: 454
Proposed branch: lp://qastaging/~thedac/simplestreams/lp1719879
Merge into: lp://qastaging/simplestreams
Diff against target: 38 lines (+12/-4)
2 files modified
simplestreams/objectstores/swift.py (+8/-3)
simplestreams/openstack.py (+4/-1)
To merge this branch: bzr merge lp://qastaging/~thedac/simplestreams/lp1719879
Reviewer Review Type Date Requested Status
Server Team CI bot continuous-integration Approve
simplestreams-dev Pending
Review via email: mp+333011@code.qastaging.launchpad.net

Description of the change

Fix KSv3 Bugs

This change fixes two bugs.

One, when using keystoneauth1 sessions with swiftclient less than
3.3.0, catch the TypeError and use the old Connection method.

Two, the glance mirror would check if tenant_id matched before
recognizing an existing image. This cause duplicate images to be
uploaded. With keystoneauth1 sessions use a method to get the
project id.

LP: #1719879

To post a comment you must log in.
Revision history for this message
Server Team CI bot (server-team-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Scott Moser (smoser) wrote :

David,
Thanks. this looks good.

I opened bug 1728982 for the 'tenant_id' issue, and will merge this under two separate commits.

Revision history for this message
Scott Moser (smoser) wrote :

This went in in 2 pieces, upstream commit 454 and 453.
thanks David.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: