Cinder volume service is not managed by Pacemaker on BlockStorage

Bug #1628912 reported by Raoul Scarazzini
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Michele Baldessari

Bug Description

Last TripleO deployments failed at this step:

2016-09-29 00:42:57Z [AllNodesDeploySteps]: CREATE_FAILED Error: resources.AllNodesDeploySteps.resources.BlockStorageDeployment_Step5.resources[0]: Deployment to server failed: deploy_status_code: Deployment exited with non-zero status code: 1

And specifically with this message:

exception: connect failed Error: Could not find dependency Exec[wait-for-settle] for Pacemaker::Resource::Systemd[openstack-cinder-volume]

Making some verifications the broke element is openstack-cinder-volume, and specifically its inclusion in overcloud_volume.pp. The faulty code/assumption seems to be ./roles_data.yaml:

- name: BlockStorage
  ServicesDefault:
    - OS::TripleO::Services::CACerts
    - OS::TripleO::Services::CinderVolume
    - OS::TripleO::Services::Kernel
    - OS::TripleO::Services::Ntp
    - OS::TripleO::Services::Timezone
    - OS::TripleO::Services::Snmp
    - OS::TripleO::Services::TripleoPackages
    - OS::TripleO::Services::TripleoFirewall
    - OS::TripleO::Services::SensuClient
    - OS::TripleO::Services::FluentdClient
    - OS::TripleO::Services::VipHosts

By default we have in environments/puppet-pacemaker.yaml

  OS::TripleO::Services::CinderVolume: ../puppet/services/pacemaker/cinder-volume.yaml

So the above BlockStorage won't work with pacemaker

Changed in tripleo:
assignee: nobody → Michele Baldessari (michele)
status: New → In Progress
Raoul Scarazzini (rasca)
description: updated
Revision history for this message
Michele Baldessari (michele) wrote :

Not sure why LP is not picking this up, but Giulio's fix is tracked here: https://review.openstack.org/379410

Changed in tripleo:
importance: Undecided → High
milestone: none → ocata-1
tags: added: newton-backport-potential newton-rc-potential
Changed in tripleo:
milestone: ocata-1 → newton-rc3
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (master)

Reviewed: https://review.openstack.org/379410
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=7822c9756a80f0d49a25c668e70628fa051fbdf9
Submitter: Jenkins
Branch: master

commit 7822c9756a80f0d49a25c668e70628fa051fbdf9
Author: Giulio Fidente <email address hidden>
Date: Thu Sep 29 14:05:46 2016 +0200

    Cinder volume service is not managed by Pacemaker on BlockStorage

    We do not want cinder-volume to be managed by Pacemaker on
    BlockStorage nodes, where Pacemaker is not running at all.

    This change adds a new BlockStorageCinderVolume service name
    which can (and is, by default) mapped to the non Pacemaker
    implementation of the service.

    The error was:
    Could not find dependency Exec[wait-for-settle] for
    Pacemaker::Resource::Systemd[openstack-cinder-volume]

    Also moves cinder::host setting into the Pacemaker specific service
    definition because we only want to set a shared host= string when
    the service is managed by Pacemaker.

    Closes-Bug: #1628912
    Change-Id: I2f7e82db4fdfd5f161e44d65d17893c3e19a89c9

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/381240

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (stable/newton)

Reviewed: https://review.openstack.org/381240
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=582bb8af050bde8b86947f03ab7ebed935d0b22f
Submitter: Jenkins
Branch: stable/newton

commit 582bb8af050bde8b86947f03ab7ebed935d0b22f
Author: Giulio Fidente <email address hidden>
Date: Thu Sep 29 14:05:46 2016 +0200

    Cinder volume service is not managed by Pacemaker on BlockStorage

    We do not want cinder-volume to be managed by Pacemaker on
    BlockStorage nodes, where Pacemaker is not running at all.

    This change adds a new BlockStorageCinderVolume service name
    which can (and is, by default) mapped to the non Pacemaker
    implementation of the service.

    The error was:
    Could not find dependency Exec[wait-for-settle] for
    Pacemaker::Resource::Systemd[openstack-cinder-volume]

    Also moves cinder::host setting into the Pacemaker specific service
    definition because we only want to set a shared host= string when
    the service is managed by Pacemaker.

    Closes-Bug: #1628912
    Change-Id: I2f7e82db4fdfd5f161e44d65d17893c3e19a89c9
    (cherry picked from commit 7822c9756a80f0d49a25c668e70628fa051fbdf9)

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 5.0.0.0rc3

This issue was fixed in the openstack/tripleo-heat-templates 5.0.0.0rc3 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 5.0.0

This issue was fixed in the openstack/tripleo-heat-templates 5.0.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 6.0.0.0b1

This issue was fixed in the openstack/tripleo-heat-templates 6.0.0.0b1 development milestone.

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

Other bug subscribers