Comment 7 for bug 1766692

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to nova-powervm (stable/queens)

Reviewed: https://review.openstack.org/567434
Committed: https://git.openstack.org/cgit/openstack/nova-powervm/commit/?id=49330fcf28425cebfb281d0721a2d1d0bcf4d801
Submitter: Zuul
Branch: stable/queens

commit 49330fcf28425cebfb281d0721a2d1d0bcf4d801
Author: Eric Fried <email address hidden>
Date: Fri Apr 20 16:09:02 2018 -0500

    Stringify instance UUID

    oslo.versionedobjects merged a patch [1] resulting in the referenced
    bug. A second patch [2] was merged that was supposed to fix the issue
    If coerce's value arg is a unicode string, then the method will return
    unicode [3]. Subsequent checks for isinstance(str) [4] will fail and a
    TypeError will be raised trying to handle the uuid as an int in the
    else block. This change ensures that the uuid is an instance of str so
    that it is handled properly. This problem was fixed in pypowervm 1.1.15
    [5] and the minimum version was bumped for master. However, we can't
    backport the pypowervm requirements bump to the stable branches so we
    have to fix it here.

    [1] https://review.openstack.org/#/c/559815/
    [2] https://review.openstack.org/#/c/561674/
    [3] https://review.openstack.org/#/c/561674/2/oslo_versionedobjects/fields.py@367
    [4] https://github.com/powervm/pypowervm/blob/1.1.10/pypowervm/utils/uuid.py#L50-L56
    [5] https://github.com/powervm/pypowervm/commit/d55b4c84

    Related-Bug: #1766692
    Change-Id: Ic3b11b071a055177bbcfb0555e2a391ba419d191