'ceph' installs on the nodes with only 'cinder' role but it shouldn't

[Dennis Dmitriev]

Found on the following configuration (5.1.1):
- Ubuntu/HA
- Neutron/Vlan (with balance-slb bonding)
- Cinder/LVM for volumes
- Ceph for images
- RadosGW for objects

Nodes role:
- 3x Controller
- 3x Compute
- 4x Cinder
- 2x Ceph

Cluster is deployed successfully, but on all Cinder nodes 'ceph' is installed bu puppet:
======== puppet.log from one of cinder nodes:
Wed Dec 03 10:00:47 +0000 2014 Puppet (debug): Installing package ceph from purged to present
Wed Dec 03 10:00:47 +0000 2014 Puppet (debug): Executing '/usr/bin/apt-get -q -y -o DPkg::Options::=--force-confold --force-yes install ceph'
Wed Dec 03 10:00:52 +0000 2014 /Stage[main]/Ceph::Params/Package[ceph]/ensure (notice): ensure changed 'purged' to 'present'
...
Wed Dec 03 10:00:53 +0000 2014 /Stage[main]/Ceph::Conf/Exec[ceph-deploy config pull] (info): Starting to evaluate the resource
Wed Dec 03 10:00:53 +0000 2014 Exec[ceph-deploy config pull](provider=posix) (debug): Executing 'ceph-deploy --overwrite-conf config pull node-1'
Wed Dec 03 10:00:53 +0000 2014 Puppet (debug): Executing 'ceph-deploy --overwrite-conf config pull node-1'
...

[Bogdan Dobrelya (bogdando)]

A confirmation required if this one affects 6.x as well

[Dennis Dmitriev (ddmitriev)]

Checked on 6.0 :
- CentOS/simple
- Nova-network/vlan,
- Cinder/LVM for volumes,
- Ceph for images;

Node roles:
- controller + ceph;
- compute + ceph;
- cinder

Result:
Node with 'cinder' role contains ceph packages installed:
==============================================
[root@nailgun ~]# fuel node
id | status | name | cluster | ip | mac | roles | pending_roles | online | group_id
---|--------|------------------|---------|------------|-------------------|----------------------|---------------|--------|---------
5 | ready | Untitled (44:61) | 2 | 10.108.0.5 | 64:04:ab:5c:44:61 | ceph-osd, compute | | True | 2
4 | ready | Untitled (60:eb) | 2 | 10.108.0.4 | 64:bc:19:f7:60:eb | ceph-osd, controller | | True | 2
6 | ready | Untitled (84:ce) | 2 | 10.108.0.3 | 64:08:67:b1:84:ce | cinder | | True | 2

[root@node-6 ~]# ceph
ceph ceph-conf ceph-dencoder ceph-disk-udev ceph-osd ceph-syn
ceph-authtool ceph-coverage ceph-deploy cephfs ceph-post-file
ceph-brag ceph-create-keys ceph-disk ceph-mds ceph-rbdnamer
ceph-client-debug ceph-crush-location ceph-disk-activate ceph-mon ceph-rest-api
ceph-clsinfo ceph-debugpack ceph-disk-prepare ceph_mon_store_converter ceph-run

[root@node-6 ~]# /etc/init.d/ceph status
[root@node-6 ~]#

{"build_id": "2014-11-25_15-47-03", "ostf_sha": "a35f516f1606b0d03d51ff63bfe3fbe23de4b622", "build_number": "6", "auth_required": true, "api": "1.0", "nailgun_sha": "981f860963980ae056f3d25e98133d1eae9b035a", "production": "docker", "fuelmain_sha": "9d94650f9517aee2102aedcfa1816c4ee1880fe4", "astute_sha": "c15623d05ccdf7ac10873e7a90df954de8726280", "feature_groups": ["mirantis"], "release": "6.0", "release_versions": {"2014.2-6.0": {"VERSION": {"build_id": "2014-11-25_15-47-03", "ostf_sha": "a35f516f1606b0d03d51ff63bfe3fbe23de4b622", "build_number": "6", "api": "1.0", "nailgun_sha": "981f860963980ae056f3d25e98133d1eae9b035a", "production": "docker", "fuelmain_sha": "9d94650f9517aee2102aedcfa1816c4ee1880fe4", "astute_sha": "c15623d05ccdf7ac10873e7a90df954de8726280", "feature_groups": ["mirantis"], "release": "6.0", "fuellib_sha": "2b5d532265af40111fe82d9c73bc976d75905c86"}}}, "fuellib_sha": "2b5d532265af40111fe82d9c73bc976d75905c86"}

[OpenStack Infra (hudson-openstack)]

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

[Bogdan Dobrelya (bogdando)]

Medium priority bugs are not backported

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/140080
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=6ea5fcc87877d817ef4d0e9570fb547a0aa254d3
Submitter: Jenkins
Branch: master

commit 6ea5fcc87877d817ef4d0e9570fb547a0aa254d3
Author: Stanislaw Bogatkin <email address hidden>
Date: Mon Dec 8 20:05:23 2014 +0300

    Check need to install ceph on node.

    In configuration when used ceph for Glance, but for Cinder use something
    else (like a lvm or vmdk) we today installing ceph on Cinder nodes too,
    but not use it. I add check for that situation.

    Change-Id: I3dc327180d2bd7c9918b24ad2b9b867321a279d2
    Closes-Bug: 1398801