Ceph deployment fails when Ceph is not enabled for storage and is enabled for ephemeral

[Sergey Nikitin]

  release: "6.0"
  api: "1.0"
  build_number: "56"
  build_id: "2014-12-18_01-32-01"
  astute_sha: "16b252d93be6aaa73030b8100cf8c5ca6a970a91"
  fuellib_sha: "73332192a257ea02c40a39885c502ad1ebdf3eda"
  ostf_sha: "a9afb68710d809570460c29d6c3293219d3624d4"
  nailgun_sha: "5f91157daa6798ff522ca9f6d34e7e135f150a90"
  fuelmain_sha: "45caacadb878abfbd9d60e134d72229698b469c9"

Ubuntu. HA (2 controllers), Neutron+VLAN, Ceph for ephemeral volumes
Ceph roles are assigned only to controllers.

Deployment fails on compute nodes due to puppet error:
Duplicate declaration: Nova_config[libvirt/inject_partition] is already declared in file /etc/puppet/modules/ceph/manifests/ephemeral.pp:13; cannot redeclare at /etc/puppet/modules/openstack/manifests/compute.pp:315 on node node-7.domain.tld

I was able to bypass the issue and redeploy cluster successfully by commenting
'libvirt/inject_partition': value => '-2';
in
/etc/puppet/modules/ceph/manifests/ephemeral.pp

[Stanislav Makar (smakar)]

it's a bit strange, ceph on controllers only but compute.pp is executed

[Stanislav Makar (smakar)]

sorry
 errors from compute nodes

[Vladimir Kuklin (vkuklin)]

Cluster configuration is using ceph only for ephemeral. This is a rare use case, but we need to fix this issue in the next releases.

[Dmitry Borodaenko (angdraug)]

The problem was introduced in https://review.openstack.org/137606 -- on line 310, the conditional should use ephemeral_ceph instead of image_ceph. Also see how $use_ceph is determined in osnailyfacter::cluster_ha, it would help understand relation between all ceph related options.

[Ryan Moe (rmoe)]

Stanislav, any update on this issue?

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

commit ac43b4848f37c14522a35ec2887fe9a04f37e656
Author: Ryan Moe <email address hidden>
Date: Wed Feb 18 14:31:38 2015 -0800

    Fix configuration of Ceph-related compute options

    Previously we would only configure certain Ceph-related
    compute options when Ceph was used as a backend for Glance.
    Using Ceph for ephemeral volumes but not for images would
    result in a Puppet error. Using Ceph for volumes but not
    for images would result in a sub-optimal configuration.

    Change-Id: I49e83b056380b2591ade0cdb0af668f3508745d1
    Closes-bug: #1405407

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[Anastasia Palkina (apalkina)]

Verified on ISO #143

"build_id": "2015-02-24_22-54-44", "ostf_sha": "1a0b2c6618fac098473c2ed5a9af11d3a886a3bb", "build_number": "143", "release_versions": {"2014.2-6.1": {"VERSION": {"build_id": "2015-02-24_22-54-44", "ostf_sha": "1a0b2c6618fac098473c2ed5a9af11d3a886a3bb", "build_number": "143", "api": "1.0", "nailgun_sha": "f4487769f5526bd50ecd42af296f5587b7fc231f", "production": "docker", "python-fuelclient_sha": "5657dbf06fddb74adb61e9668eb579a1c57d8af8", "astute_sha": "6d6ad68e0cde286d74ac7d52e21da4fc8dcbe9ab", "feature_groups": ["mirantis"], "release": "6.1", "fuelmain_sha": "b975019fabdb429c1869047df18dd792d2163ecc", "fuellib_sha": "8384b8ca4db84794fb21e287202f05e31f78841c"}}}, "auth_required": true, "api": "1.0", "nailgun_sha": "f4487769f5526bd50ecd42af296f5587b7fc231f", "production": "docker", "python-fuelclient_sha": "5657dbf06fddb74adb61e9668eb579a1c57d8af8", "astute_sha": "6d6ad68e0cde286d74ac7d52e21da4fc8dcbe9ab", "feature_groups": ["mirantis"], "release": "6.1", "fuelmain_sha": "b975019fabdb429c1869047df18dd792d2163ecc", "fuellib_sha": "8384b8ca4db84794fb21e287202f05e31f78841c"

[OpenStack Infra (hudson-openstack)]

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

commit f1e1fe1609b8a1c969ceb119d94387e41912e14c
Author: Ryan Moe <email address hidden>
Date: Wed Feb 18 14:31:38 2015 -0800

    Fix configuration of Ceph-related compute options

    Previously we would only configure certain Ceph-related
    compute options when Ceph was used as a backend for Glance.
    Using Ceph for ephemeral volumes but not for images would
    result in a Puppet error. Using Ceph for volumes but not
    for images would result in a sub-optimal configuration.

    Change-Id: I49e83b056380b2591ade0cdb0af668f3508745d1
    Closes-bug: #1405407

[OpenStack Infra (hudson-openstack)]

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

commit 3fe49e227d754336ed0920ff18779c33c8d10986
Author: Ryan Moe <email address hidden>
Date: Wed Feb 18 14:31:38 2015 -0800

    Fix configuration of Ceph-related compute options

    Previously we would only configure certain Ceph-related
    compute options when Ceph was used as a backend for Glance.
    Using Ceph for ephemeral volumes but not for images would
    result in a Puppet error. Using Ceph for volumes but not
    for images would result in a sub-optimal configuration.

    Change-Id: I49e83b056380b2591ade0cdb0af668f3508745d1
    Closes-bug: #1405407

[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...