galeracheck uses xinetd::service but xinetd is not declared

Bug #1348863 reported by Dmitry Borodaenko on 2014-07-26
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Critical
Dmitry Borodaenko

Bug Description

Deploying CentOS HA + nova-network + Ceph with 5.1 ISO #361 with /etc/puppet/modules updated to fuel-library commit d1c7f67b3cf51978d3178c8666ea398f2477dcb5 (patched with https://review.openstack.org/109739) I get the following error:

Sat Jul 26 00:07:57 +0000 2014 Puppet (err): Could not find dependent Service[xinetd] for File[/etc/xinetd.d/galeracheck] at /etc/puppet/modules/xinetd/manifests/service.pp:82

Looks like in https://review.openstack.org/106516 we've introduced new class galera::status that declares a xinetd::service resource, but definition of xinetd::service includes a dependency on Service['xinetd'], that is declared in xinetd class that we don't declare anywhere.

Dmitry Borodaenko (angdraug) wrote :
Changed in fuel:
assignee: nobody → Dmitry Borodaenko (dborodaenko)

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

Changed in fuel:
status: Triaged → In Progress

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

commit e4bfabadf887da5407305b80fc0bf3e963e8d208
Author: Dmitry Borodaenko <email address hidden>
Date: Fri Jul 25 18:54:54 2014 -0700

    include xinetd in galera::status

    Can't use xinetd::service without including xinetd class.

    Change-Id: I2ed1f5f08a42742ddc9fb71c3cc2e709b6df2b36
    Closes-Bug: #1348863

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

Other bug subscribers