virsh secret-define set lowercase uuid

Bug #1671029 reported by Valery Tschopp
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
puppet-nova
Fix Released
High
Valery Tschopp

Bug Description

The 'virsh secret-define --file secret.xml' converts the uuid to lowercase.

This breaks the exec 'get-or-set virsh secret' unless condition, and it applies it forever if the nova::compute::rbd $libvirt_rbd_secret_uuid parameter is uppercase. i.e. 8D1F7CE2-EC5E-44C7-86BB-51BB951D05DD

Uppercase uuid in secret.xml:

<secret ephemeral='no' private='no'>
  <usage type='ceph'>
    <name>uppercase UUID secret</name>
  </usage>
  <uuid>8D1F7CE2-EC5E-44C7-86BB-51BB951D05DD</uuid>
</secret>

$ virsh secret-define --file secret.xml
Secret 8d1f7ce2-ec5e-44c7-86bb-51bb951d05dd created

$ virsh secret-list
 UUID Usage
---------------------------------------------------------------------
 8d1f7ce2-ec5e-44c7-86bb-51bb951d05dd ceph uppercase UUID secret

description: updated
description: updated
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-nova (master)

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

Changed in puppet-nova:
assignee: nobody → Saverio Proto (zioproto)
status: New → In Progress
Changed in puppet-nova:
importance: Undecided → High
Changed in puppet-nova:
assignee: Saverio Proto (zioproto) → Valery Tschopp (valery-tschopp)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (master)

Reviewed: https://review.openstack.org/443056
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=1003a8fd8618dec25ef4428034dfa7628dc1d74e
Submitter: Jenkins
Branch: master

commit 1003a8fd8618dec25ef4428034dfa7628dc1d74e
Author: Valery Tschopp <email address hidden>
Date: Wed Mar 8 10:10:26 2017 +0100

    virsh secret uuid are case insensitive

    Change-Id: I3d2f13e3b4a3c4b9ef8e49cd19d12d3e866ad69b
    Closes-bug: #1671029

Changed in puppet-nova:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-nova (stable/ocata)

Fix proposed to branch: stable/ocata
Review: https://review.openstack.org/444250

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-nova (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/444251

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-nova (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/444256

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (stable/ocata)

Reviewed: https://review.openstack.org/444250
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=dbff01f3dfa6aaa850d8c85cfbb0075928c10d53
Submitter: Jenkins
Branch: stable/ocata

commit dbff01f3dfa6aaa850d8c85cfbb0075928c10d53
Author: Valery Tschopp <email address hidden>
Date: Wed Mar 8 10:10:26 2017 +0100

    virsh secret uuid are case insensitive

    Change-Id: I3d2f13e3b4a3c4b9ef8e49cd19d12d3e866ad69b
    Closes-bug: #1671029
    (cherry picked from commit 1003a8fd8618dec25ef4428034dfa7628dc1d74e)

tags: added: in-stable-ocata
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (stable/mitaka)

Reviewed: https://review.openstack.org/444256
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=69126551fe72163d5f21f7c9676c989803a7851d
Submitter: Jenkins
Branch: stable/mitaka

commit 69126551fe72163d5f21f7c9676c989803a7851d
Author: Valery Tschopp <email address hidden>
Date: Wed Mar 8 10:10:26 2017 +0100

    virsh secret uuid are case insensitive

    Change-Id: Ic5a214d266f85d49bcafeed52a9c7c30ce7924ac
    Closes-bug: #1671029
    (cherry picked from commit 1003a8fd8618dec25ef4428034dfa7628dc1d74e)

tags: added: in-stable-mitaka
tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-nova (stable/newton)

Reviewed: https://review.openstack.org/444251
Committed: https://git.openstack.org/cgit/openstack/puppet-nova/commit/?id=a64ddc7f286dc52194dcbaeb052c8fb126e60f6a
Submitter: Jenkins
Branch: stable/newton

commit a64ddc7f286dc52194dcbaeb052c8fb126e60f6a
Author: Valery Tschopp <email address hidden>
Date: Wed Mar 8 10:10:26 2017 +0100

    virsh secret uuid are case insensitive

    Change-Id: I85b53dc2cc81a6a434da7722cdceb5733a99d2ce
    Closes-bug: #1671029
    (cherry picked from commit 1003a8fd8618dec25ef4428034dfa7628dc1d74e)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-nova 11.0.0

This issue was fixed in the openstack/puppet-nova 11.0.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-nova 10.4.1

This issue was fixed in the openstack/puppet-nova 10.4.1 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-nova 9.5.1

This issue was fixed in the openstack/puppet-nova 9.5.1 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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