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

Bug #1324954 reported by Andrey Grebennikov on 2014-05-30
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Medium
Dmitry Borodaenko
4.1.x
Medium
Dmitry Borodaenko
5.0.x
Medium
Dmitry Borodaenko

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)

Changed in fuel:
status: New → Confirmed
milestone: none → 5.1
assignee: nobody → Dmitry Borodaenko (dborodaenko)
importance: Undecided → Medium
tags: added: customer-found
Dmitry Borodaenko (angdraug) wrote :

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.

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

Changed in fuel:
status: Confirmed → In Progress

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

Changed in fuel:
status: In Progress → Fix Committed

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

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

    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

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

    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

tags: added: in progress

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"

}

tags: removed: in progress
Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers