Fuel for OpenStack

Ceph volume cannot be attached/created in case of different cinder rbd_user and pool name

Series 4.1.x
Bug #1324954

Bug #1324954 reported by Andrey Grebennikov on 2014-05-30

This bug affects 1 person

	Status	Importance	Assigned to	Milestone
Fuel for OpenStack	Fix Released	Medium	Dmitry Borodaenko	Fuel for OpenStack 5.1
4.1.x	Fix Committed	Medium	Dmitry Borodaenko	Fuel for OpenStack 4.1.1
5.0.x	Fix Committed	Medium	Dmitry Borodaenko	Fuel for OpenStack 5.0.1

Bug Description

If "rbd_user" options in the config files "nova.conf" on the compute node and "cinder.conf" on the controller is not equal to "rbd_pool", ceph volumes cannot be created or attached to the instance.
The reason is that env variable "CEPH_ARGS" takes the priority in case it is defined, but there in the manifests
cinder/manifests/volume/ceph.pp
ceph/manifests/nova_compute.pp
it is defined as "--id ${rbd_pool}'", but it has to be "--id ${user}'" (for nova_compute.pp) and "--id ${rbd_user}'" (for ceph.pp)
Or it can be just excluded from the manifests at all (probably)

Tags:

Revision history for this message

Andrey Grebennikov (agrebennikov) wrote on 2014-05-30:

Patch for fixing this bug Edit (1.4 KiB, text/plain)

Dmitry Borodaenko (angdraug) on 2014-05-30

Changed in fuel:
status:	New → Confirmed
milestone:	none → 5.1
assignee:	nobody → Dmitry Borodaenko (dborodaenko)
importance:	Undecided → Medium

Dmitry Borodaenko (angdraug) on 2014-05-30

tags:

added: customer-found

Revision history for this message

Dmitry Borodaenko (angdraug) wrote on 2014-05-30:

You're right, CEPH_ARGS was only necessary with Grizzly. Starting with Havana, Cinder and Nova consistently use ceph_args() method to add "--id <rbd_user>" parameter to all invokations of rbd and ceph commands, and Glance doesn't use these commands at all.

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-05-30: Fix proposed to fuel-library (master)

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

Changed in fuel:
status:	Confirmed → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-06-02: Fix proposed to fuel-library (stable/5.0)

Fix proposed to branch: stable/5.0
Review: https://review.openstack.org/97374

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-06-02: Fix proposed to fuel-library (stable/4.1)

Fix proposed to branch: stable/4.1
Review: https://review.openstack.org/97375

Changed in fuel:
status:	In Progress → Fix Committed

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-06-02: Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/96942
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=007f0043361e5ec71c74b7e8429f0fa775b3fe2f
Submitter: Jenkins
Branch: master

commit 007f0043361e5ec71c74b7e8429f0fa775b3fe2f
Author: Dmitry Borodaenko <email address hidden>
Date: Fri May 30 14:02:05 2014 -0700

remove CEPH_ARGS from Nova and Cinder

    Now that Nova and Cinder consistently pass --id option when executing rbd and
    ceph commands, it's no longer necessary to set CEPH_ARGS environment variable
    in the init scripts for Nova and Cinder services.

Change-Id: I29134942b1e7d5e2a276aecc00a9c5747cfa5a38
Closes-Bug: #1324954

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-06-03: Fix merged to fuel-library (stable/5.0)

Reviewed: https://review.openstack.org/97374
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=e94fe0d071030aa7feca32ceeaf244c2b73f955d
Submitter: Jenkins
Branch: stable/5.0

commit e94fe0d071030aa7feca32ceeaf244c2b73f955d
Author: Dmitry Borodaenko <email address hidden>
Date: Fri May 30 14:02:05 2014 -0700

remove CEPH_ARGS from Nova and Cinder

Change-Id: I29134942b1e7d5e2a276aecc00a9c5747cfa5a38
Closes-Bug: #1324954

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-06-03: Fix merged to fuel-library (stable/4.1)

Reviewed: https://review.openstack.org/97375
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=7ad6f2ea0226c4c879ab67d6f751d9b754f67c8d
Submitter: Jenkins
Branch: stable/4.1

commit 7ad6f2ea0226c4c879ab67d6f751d9b754f67c8d
Author: Dmitry Borodaenko <email address hidden>
Date: Fri May 30 14:02:05 2014 -0700

remove CEPH_ARGS from Nova and Cinder

Change-Id: I29134942b1e7d5e2a276aecc00a9c5747cfa5a38
Closes-Bug: #1324954

Andrey Sledzinskiy (asledzinskiy) on 2014-09-29

tags:

added: in progress

Revision history for this message

Andrey Sledzinskiy (asledzinskiy) wrote on 2014-09-30:

verified on
{

    "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"