Merge lp://staging/~cbjchen/cinder/lp1514325 into lp://staging/~ubuntu-server-dev/cinder/kilo

Proposed by Liang Chen
Status: Rejected
Rejected by: Marc Deslauriers
Proposed branch: lp://staging/~cbjchen/cinder/lp1514325
Merge into: lp://staging/~ubuntu-server-dev/cinder/kilo
Diff against target: 75 lines (+55/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/encode-repr-return.patch (+47/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp://staging/~cbjchen/cinder/lp1514325
Reviewer Review Type Date Requested Status
Marc Deslauriers Disapprove
Review via email: mp+281351@code.staging.launchpad.net

Description of the change

SRU LP#1514325

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank you for the work. The bug doesn't mention if the issue is fixed in wily and xenial, could you check and update the report? Is there also a point doing a SRU for vivid now, knowing it's neither a LTS nor a current stable version...

Revision history for this message
Liang Chen (cbjchen) wrote :

Hi Sebastien,

Thank you for considering the merge proposal.
The code that introduces this bug was moved to an independent project(python-oslo.versionedobjects) for wily and later. So fix will need to be done there. I have backported the fix into stable/liberty branch in OpenStack upstream repository. And I can propose a SRU for ubuntu python-oslo.versionedobjects package if the fix has not been brought in. But for cinder package, nothing needs to be done for wily and later.

Thanks.
Liang

Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Thanks for the merge request, but since vivid is now End of Life, I am rejecting it.

review: Disapprove

Unmerged revisions

238. By lchen <<email address hidden>@canonical.com>

SRU LP: #1514325

* Fix unicode volue name error. (LP:#1514325):
  - d/p encode-repr-return.patch: ensure __repr__ return value encoded

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