Missing impl of microversions 41, 42 and 43

Bug #1715759 reported by John Griffith
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-cinderclient
Fix Released
Undecided
Jay Bryant

Bug Description

While submitting the attachment-complete changes the MAX version in the client was at 40. I bumped that up to 44 thinking that the other changes must be "in flight" and just not merged yet. Turns out that isn't the case; and these were just skipped.

This patch implements the changes for 41 and 43; 42 is a noop for the client as it's a server side only change.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-cinderclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/501897

Changed in python-cinderclient:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/501902

Changed in python-cinderclient:
assignee: John Griffith (john-griffith) → Jay Bryant (jsbryant)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-cinderclient (master)

Reviewed: https://review.openstack.org/501902
Committed: https://git.openstack.org/cgit/openstack/python-cinderclient/commit/?id=2255fc99da9752737dcaa96ae4507b646074afb2
Submitter: Jenkins
Branch: master

commit 2255fc99da9752737dcaa96ae4507b646074afb2
Author: j-griffith <email address hidden>
Date: Thu Sep 7 23:51:37 2017 +0000

    Implement metadata for backup create/update

    We have a gap in the micro versions implemented in the cinderclient
    and the MAX version being reported by the client.

    This patch addresses the missing changes for micro version 3.43
    which adds support for metadata for backup create and update.

    Change-Id: I14c7205784c53ec3ab0985a4460a88f59de171fb
    Partial-Bug: #1715759

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/501897
Committed: https://git.openstack.org/cgit/openstack/python-cinderclient/commit/?id=3d1225cbb0435c9912ff1359f2db07ae4e6c81eb
Submitter: Jenkins
Branch: master

commit 3d1225cbb0435c9912ff1359f2db07ae4e6c81eb
Author: j-griffith <email address hidden>
Date: Thu Sep 7 23:29:55 2017 +0000

    Implement UserID in snapshot list response

    We have a gap in the microversions implemented in the cinderclient
    and the MAX version being reported by the client.

    One of those is the addition of "user_id" in the snapshot list
    response. This patch adds that with the appropriate version
    checking. Note that the Max version has already been updated,
    but there are a few implementations missing.

    Change-Id: Ia8bdb46b46091a89996e404825466430b8906d18
    Partial-Bug: #1715759

Revision history for this message
Sean McGinnis (sean-mcginnis) wrote :

I believe we got all of these now. If not, please reopen.

Changed in python-cinderclient:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.