nova cell requres cinderclient

[meizhifang]

nova cell requres cinderclient.
I suggest In cells.pp,
nova::generic_service { 'cells':
    enabled => $enabled,
    manage_service => $manage_service,
    package_name => $::nova::params::cells_package_name,
    service_name => $::nova::params::cells_service_name,
    ensure_package => $ensure_package,
  }
add conditions "subscribe => Class['cinder::client']"

[Emilien Macchi (emilienm)]

It seems like we're trying to fix a packaging dependency issue.
Which distro are you using?

[meizhifang (mei-zhifang97)]

distro is openstack-kilo . el7 package is openstack-puppet-modules-2015.1.0-2.el7.noarch.rpm

[meizhifang (mei-zhifang97)]

Correct the above comments，el7 package is openstack-puppet-modules-2015.1.2-1.el7.noarch.rpm .

[Cody Herriges (ody-cat)]

This should be true is the same way that it is try for "api", https://github.com/openstack/puppet-nova/blob/master/manifests/api.pp#L295

[Matt Fischer (mfisch)]

Would there be a case where you'd have the cells class included but not the api class? You get this with the API class.

[Takashi Kajinami (kajinamit)]

Cell v1 was removed a long ago. I'll close this now.