Comment 10 for bug 1766692

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

Reviewed: https://review.openstack.org/567599
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=2386e015ef702ae8db29e396e436a9e293df8f3b
Submitter: Zuul
Branch: stable/queens

commit 2386e015ef702ae8db29e396e436a9e293df8f3b
Author: esberglu <email address hidden>
Date: Thu May 10 09:08:14 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

    Change-Id: I1baef962b4b8074f3f9b9ad5402970edc9a3a776
    Related-Bug: #1766692