setup_test_volume: stop managing /var/lib/cinder
/var/lib/cinder is already managed by packaging. We should not manage it with Puppet, it can lead to permissions and orchestration issues. Instead, just require cinder packaging before managing lvm volumes. Change-Id: I14275a326c0e655dce5d2a27e45811bba66602b2 Closes-bug: #1531521
This commit is contained in:
parent
a52d4bce05
commit
4d3e0665c6
@ -23,13 +23,7 @@ class cinder::setup_test_volume(
|
||||
) {
|
||||
|
||||
package { 'lvm2':
|
||||
ensure => present,
|
||||
} ~>
|
||||
|
||||
file { $volume_path:
|
||||
ensure => directory,
|
||||
owner => 'cinder',
|
||||
group => 'cinder',
|
||||
ensure => present,
|
||||
require => Package['cinder'],
|
||||
} ~>
|
||||
|
||||
|
@ -3,11 +3,7 @@ require 'spec_helper'
|
||||
describe 'cinder::setup_test_volume' do
|
||||
|
||||
it { is_expected.to contain_package('lvm2').with(
|
||||
:ensure => 'present'
|
||||
) }
|
||||
|
||||
it { is_expected.to contain_file('/var/lib/cinder').with(
|
||||
:ensure => 'directory',
|
||||
:ensure => 'present',
|
||||
:require => 'Package[cinder]'
|
||||
) }
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user