Deploying Cinder VMDK doesn't set up cinder-volume

[Chris Clason]

in 6.0, selecting cinder VMDK results in cinder-volume not being deployed on the controllers, instead it would require a node with the cinder role which would end up with an un-used storage volume.

affects: 6.1, 6.0, 5.1.x

[Igor Gajsin (igajsin)]

Andrew, just assign cinder-role to controller node. In this case no useless storage volume will be used.

[Chris Clason (cclason)]

Igor, this is not intuitive at all. Andrew and myself both did not know how to do this, it's not realistic to expect a customer to know to do this as well. Please have Fuel automatically do what is needed when VMDK is selected for the cinder back end and don't let the user mess it up.

[Bogdan Dobrelya (bogdando)]

I'm returning this issue to the Fuel library as it seems had no activity for a while

[OpenStack Infra (hudson-openstack)]

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

[Andrey Danin (gcon-monolake)]

Here is the blueprint [0] we are going to implement in Fuel 6.1. Briefly, there will be a separate role for Cinder-VMDK service. It'll have a better disk allocation than simple 'cinder' role. A user can combine the new role with any (or almost any) other roles, for instance with Ceph. So, we are going to improve an experience with Cinder+VMDK in Fuel 6.1. I personally like the idea proposed by Andrew in his patch-set, but I'm pretty sure we'll refactor cluster_ha.pp. So, maybe it's better to wait two more weeks until we'll manage to merge first parts of a refactored code. Anyway, I like the idea to have the 'host' parameter modified in upstream manifests.

About Fuel 5.1 and 6.0.
Andrew's patch-set can be applied to these releases. Anyway, we need to document such tricky behavior (a separate role must be assigned somewhere and some space can be wasted) in UserGuide and maybe add some warnings on UI. I think, it should be enough for 5.1 and 6.0.

[0] https://blueprints.launchpad.net/fuel/+spec/cinder-vmdk-role/

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/5.1
Review: https://review.openstack.org/157021

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/6.0
Review: https://review.openstack.org/157022

[Igor Gajsin (igajsin)]

Won't Fix for Fuel 6.1 because it will be fixed by https://blueprints.launchpad.net/fuel/+spec/cinder-vmdk-role

[OpenStack Infra (hudson-openstack)]

Change abandoned by Bogdan Dobrelya (<email address hidden>) on branch: master
Review: https://review.openstack.org/152279

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/157022
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=a3d64db41bfcc95146579be17a5da6fee72fd98a
Submitter: Jenkins
Branch: stable/6.0

commit a3d64db41bfcc95146579be17a5da6fee72fd98a
Author: Andrew Woodward <email address hidden>
Date: Tue Jan 13 13:33:56 2015 -0800

    Update cinder $manage_volume condition for cinder VMDK

    $manage_volume condition was improperly checking for the cinder role to
     deploy cinder-volume to. We need to deploy cinder-volume the same way
     as we do for ceph

    add host param to cinder backend vmkd as it happens is
    https://review.openstack.org/#/c/157004/

    Change-Id: I9f615b88f6d22a902ed93f7a6e1eb84803991b34
    Closes-Bug: 1410517

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/157021
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=fe5039ca5e2ee894bddc22b7ec7020745195170c
Submitter: Jenkins
Branch: stable/5.1

commit fe5039ca5e2ee894bddc22b7ec7020745195170c
Author: Andrew Woodward <email address hidden>
Date: Tue Jan 13 13:33:56 2015 -0800

    Update cinder $manage_volume condition for cinder VMDK

    $manage_volume condition was improperly checking for the cinder role to
     deploy cinder-volume to. We need to deploy cinder-volume the same way
     as we do for ceph

    add host param to cinder backend vmkd as it happens is
    https://review.openstack.org/#/c/157004/

    Change-Id: I9f615b88f6d22a902ed93f7a6e1eb84803991b34
    Closes-Bug: 1410517

[okosse (okosse)]

I check it on fuel-6.0.1-102-2015-02-22_20-49-44.iso

[Irina Povolotskaya (ipovolotskaya)]

This issue will be added to 6.0.1 RN as Resolved Issue.

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/182996

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: stable/6.1
Review: https://review.openstack.org/194961

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/194961
Committed: https://git.openstack.org/cgit/stackforge/fuel-docs/commit/?id=0e26e7d7cc153d179ec34985645dd23cdd239ddb
Submitter: Jenkins
Branch: stable/6.1

commit 5cc5f0c643aebecaf3bf4580535a3ea7c3334a6c
Author: Mike Scherbakov <email address hidden>
Date: Tue Jun 23 13:43:35 2015 -0700

    Removed streamlined patching backend pieces

    Change-Id: I955e76ccdbd12a9145f4e9b689f80bdf9fcaf929

commit 563c4b5c78ebfcb1f4f91047c2919f6270f9a1d4
Author: Mike Scherbakov <email address hidden>
Date: Tue Jun 23 13:30:30 2015 -0700

    Removed outdated patching guide

    Change-Id: I76180c277789ade9c5ebedd19fe2092847c0b7d9

commit 8d120c14bec1ab41d448683ad146a3053a57c4ee
Author: Irina Povolotskaya <email address hidden>
Date: Tue Jun 23 19:59:11 2015 +0300

    Add dual hypervisor ref arch into 6.1 docs

    Change-Id: I900c24c9de878eafadbfc995aa879b7f55737fac

commit feebd1592d3305b64bbdfd0bc5fe108190aef120
Author: OlgaGusarenko <email address hidden>
Date: Tue Jun 23 18:38:17 2015 +0300

    [OPs guide] Running Ceilometer section edits

    1. conf file extract is updated
    2. note is updated

    Closes-bug: 1467817
    Change-Id: I0217e164108e0ba6c1397045a5e57d13ff429223

commit 44a93f9dead7511a3461ec35248dbb689c81eafd
Author: OlgaGusarenko <email address hidden>
Date: Tue Jun 23 18:04:40 2015 +0300

    [RN6_1] Final changes

    1. capitalization
    2. 2014.2 to 2014.2.2
    3. general improvements

    Change-Id: I45057e90c90550559f66bc67ccdf97a559fd9000

commit bb41389cae58084285688853281516b659686422
Author: evkonstantinov <email address hidden>
Date: Tue Jun 23 16:45:35 2015 +0300

    Update patching decription

    Update patching description with
    the standard Linux commands.

    Change-Id: Ia1a8346639c468fdfce15a11d2430bf3a4731244

commit bf3018fae3f2e564413d33aba6cdebf8868f0b4e
Author: OlgaGusarenko <email address hidden>
Date: Tue Jun 23 15:55:49 2015 +0300

    [RN6_1] Clean up

    1. Rearranges sections
    2. Improves RST
    3. Changes titles order

    Change-Id: I6110bf515667d3d6ba08ad35ff5d593dbc96641e

commit 1c7e4457808e8f2d6c56fdf31252170972e444b9
Author: Maria Zlatkova <email address hidden>
Date: Tue Jun 23 15:26:28 2015 +0300

    Replaces VBOX screenshots

    This patch:
    - replaces VBOX screenshots
    - changes the link for Download Mirantis VirtualBox scripts
     to https://docs.mirantis.com/openstack/fuel/fuel-master/#downloads

    Change-Id: I58dede960c5c3355d39b07ff44b757403f6af02c
    Closes-Bug: #1467872

commit 0a568bf53fc0e25d1d692d5d74b4a7b4d983bbcc
Author: evkonstantinov <email address hidden>
Date: Tue Jun 23 14:01:55 2015 +0300

    6.1 --separate repos

    change wording and add links to the
    separate repos feature.

    Change-Id: Ib5d0778a0d8f1534f79ed2f553574cb69a3150b0

commit 95a188b21cbdd064d92696b7920e6a0105fe0c56
Author: Maria Zlatkova <email address hidden>
Date: Tue Jun 23 12:07:28 2015 +0300

    Corrects the output 'pcs status'

    Changes the example outputs to appropriate ones.

    Change-Id: Ib6d83...

[Vitaly Sedelnik (vsedelnik)]

Per comment #8 the status for 6.1 should be Fix Released, not Won't Fix