OpenStack-Ansible

os_cinder may create only the first volume type

Bug #1649914 reported by Adrien Cunin
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
High
Andy McCrae

Bug Description

Following https://git.openstack.org/cgit/openstack/openstack-ansible-os_cinder/commit/?id=27a895cbd5602a35e797adbb619520553e0e7eb8 (see bug #1588777),

Consider you have two (or more) cinder-volume hosts, each one has its own cinder_backends definition (through container_vars) containing a volume backend/type (different for each host); cinder_backends.yml will be called only for the first cinder-volume host, and as such only the volume type associated with it will get created.

(Also, `inventory_hostname in groups['cinder_volume'][0]` should probably be written with == instead of in)

Jean-Philippe Evrard (jean-philippe-evrard)
Changed in openstack-ansible:
status: New → Confirmed
importance: Undecided → High
Andy McCrae (andrew-mccrae)
Changed in openstack-ansible:
assignee: nobody → Andy McCrae (andrew-mccrae)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible-os_cinder (master)

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

Changed in openstack-ansible:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible-os_cinder (master)

Reviewed: https://review.openstack.org/413743
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-os_cinder/commit/?id=879082f204cd848f1423714a3d93943c4c7d9fa6
Submitter: Jenkins
Branch: master

commit 879082f204cd848f1423714a3d93943c4c7d9fa6
Author: Andy McCrae <email address hidden>
Date: Wed Dec 21 18:02:07 2016 +0000

    Ensure all cinder_backends are created

    Combine all cinder_volume host's "cinder_backends" var to ensure all
    cinder_backends are created, and there is no race condition setting up
    the same backend for multiple hosts (e.g. rbd backends which exist on
    multiple volume servers).

    This adds a second cinder_volume node for testing, with a new backend,
    as well as the old backend - this should ensure duplicate backends are
    not created whilst backends are created for each cinder_volumes host.

    Change-Id: I1c2fb1ef07423fc415622767ff09f058c773c1ed
    Closes-Bug: 1649914

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible-os_cinder (stable/newton)

Reviewed: https://review.openstack.org/418372
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-os_cinder/commit/?id=e34ba7b5aa1fc255d572c5c00cb8f816a91123c1
Submitter: Jenkins
Branch: stable/newton

commit e34ba7b5aa1fc255d572c5c00cb8f816a91123c1
Author: Andy McCrae <email address hidden>
Date: Wed Dec 21 18:02:07 2016 +0000

    Ensure all cinder_backends are created

    Combine all cinder_volume host's "cinder_backends" var to ensure all
    cinder_backends are created, and there is no race condition setting up
    the same backend for multiple hosts (e.g. rbd backends which exist on
    multiple volume servers).

    This adds a second cinder_volume node for testing, with a new backend,
    as well as the old backend - this should ensure duplicate backends are
    not created whilst backends are created for each cinder_volumes host.

    Change-Id: I1c2fb1ef07423fc415622767ff09f058c773c1ed
    Closes-Bug: 1649914
    (cherry picked from commit 879082f204cd848f1423714a3d93943c4c7d9fa6)

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible-os_cinder 15.0.0.0b3

This issue was fixed in the openstack/openstack-ansible-os_cinder 15.0.0.0b3 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/openstack-ansible-os_cinder 14.0.7

This issue was fixed in the openstack/openstack-ansible-os_cinder 14.0.7 release.

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

Other bug subscribers

Remote bug watches

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