Prohibit possibility of adding cinder nodes to an environment with Ceph RBD

Bug #1372914 reported by Anastasia Palkina
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Alexandra Morozova

Bug Description

"build_id": "2014-09-17_21-40-34",
"ostf_sha": "64cb59c681658a7a55cc2c09d079072a41beb346",
"build_number": "11",
"auth_required": true,
"api": "1.0",
"nailgun_sha": "eb8f2b358ea4bb7eb0b2a0075e7ad3d3a905db0d",
"production": "docker",
"fuelmain_sha": "8ef433e939425eabd1034c0b70e90bdf888b69fd",
"astute_sha": "f5fbd89d1e0e1f22ef9ab2af26da5ffbfbf24b13",
"feature_groups": ["mirantis"],
"release": "5.1",
"release_versions": {"2014.1.1-5.1": {"VERSION": {"build_id": "2014-09-17_21-40-34", "ostf_sha": "64cb59c681658a7a55cc2c09d079072a41beb346", "build_number": "11", "api": "1.0", "nailgun_sha": "eb8f2b358ea4bb7eb0b2a0075e7ad3d3a905db0d", "production": "docker", "fuelmain_sha": "8ef433e939425eabd1034c0b70e90bdf888b69fd", "astute_sha": "f5fbd89d1e0e1f22ef9ab2af26da5ffbfbf24b13", "feature_groups": ["mirantis"], "release": "5.1", "fuellib_sha": "d9b16846e54f76c8ebe7764d2b5b8231d6b25079"}}},
"fuellib_sha": "d9b16846e54f76c8ebe7764d2b5b8231d6b25079"

I did that test case:

1. Create new environment (CentOS, HA mode)
2. Choose nova-network, vlan manager
3. Choose both Ceph
4. Choose Sahara
5. Add 3 controller+ceph+cinder, 2 compute+ceph, 1 cinder
6. Configure interfaces (see screen) and untag management network
7. Start deployment. It was successful
8. But there were many errors in cinder logs (see this bug https://bugs.launchpad.net/fuel/+bug/1352335 )

This errors appears because adding cinder nodes to an environment with Ceph RBD used as backend for volumes (instead of Cinder LVM backend) is not a valid configuration.
So we should to prohibit it on UI.

Changed in fuel:
status: New → Confirmed
Changed in fuel:
assignee: Fuel UI Team (fuel-ui) → Alexandra Morozova (astepanchuk)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

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

Reviewed: https://review.openstack.org/131479
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=a6cb43038ce33132dd906a02c925b426b102b97f
Submitter: Jenkins
Branch: master

commit a6cb43038ce33132dd906a02c925b426b102b97f
Author: Alexandra Morozova <email address hidden>
Date: Tue Oct 28 16:47:48 2014 +0100

    Not allow Cinder nodes for Ceph RBD

     - added restriction in openstack.yaml

    Change-Id: I70c0a15a0999df095f5ca249ef888717be87035e
    Closes-bug: #1372914

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Verified on ISO #63:

"build_id": "2014-10-30_04-21-22", "ostf_sha": "f47fd1d66a7255213ee075d5c11b8f111f922000", "build_number": "63", "auth_required": true, "api": "1.0", "nailgun_sha": "02c6bb2e54bbec76da33167eaf5f2e0b3e2e50a7", "production": "docker", "fuelmain_sha": "2ade7c571380a091048d103a6affff634b5b2520", "astute_sha": "97eea90efe0a1f17b4934919d6e459d270c10372", "feature_groups": ["mirantis", "techpreview"], "release": "6.0-techpreview", "release_versions": {"2014.2-6.0-techpreview": {"VERSION": {"build_id": "2014-10-30_04-21-22", "ostf_sha": "f47fd1d66a7255213ee075d5c11b8f111f922000", "build_number": "63", "api": "1.0", "nailgun_sha": "02c6bb2e54bbec76da33167eaf5f2e0b3e2e50a7", "production": "docker", "fuelmain_sha": "2ade7c571380a091048d103a6affff634b5b2520", "astute_sha": "97eea90efe0a1f17b4934919d6e459d270c10372", "feature_groups": ["mirantis", "techpreview"], "release": "6.0-techpreview", "fuellib_sha": "45b6fc42091a0a33d3e48fbe78b782ce743aedc1"}}}, "fuellib_sha": "45b6fc42091a0a33d3e48fbe78b782ce743aedc1"

Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/126930
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=4a108c7fc29a9526149c3a65f011fe9e8cec6f5f
Submitter: Jenkins
Branch: master

commit 4a108c7fc29a9526149c3a65f011fe9e8cec6f5f
Author: Przemyslaw Kaminski <email address hidden>
Date: Wed Dec 10 09:45:44 2014 +0100

    Fixes for YAML role restrictions definitions

    Plaintext description of rules is provided here:

    https://etherpad.openstack.org/p/openstack.yaml_improvements

    - new implementation of 'limits' checker on the JS side
    - alembic migration added to update roles_metadata and
      attributes_metadata

    DocImpact
    Related-Bug: #1381613
    Related-Bug: #1372914
    Closes-Bug: #1253115
    Related-Bug: #1401111
    Change-Id: Id561735c9ff0d501ee83bb7190fe614dac5d5397
    Implements: blueprint refactor-role-restrictions-check

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

Other bug subscribers

Remote bug watches

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