openstack/puppet-tempest
Code Issues Proposed changes

Merge "Simplify redundant resource dependency"

Browse Source
This commit is contained in:
Zuul 2024-04-16 08:10:24 +00:00 committed by Gerrit Code Review
commit 24853f714e
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
manifests/init.pp
View File

@ -649,10 +649,9 @@ class tempest(
command => ['tempest', 'init', $tempest_workspace], command => ['tempest', 'init', $tempest_workspace],
path => ['/bin', '/usr/bin'], path => ['/bin', '/usr/bin'],
refreshonly => true, refreshonly => true,
require => Package['tempest'], subscribe => Package['tempest'],
} }
Package<| tag == 'tempest-package' |> -> Exec['tempest-workspace'] Package<| tag == 'tempest-package' |> -> Exec['tempest-workspace']
Package['tempest'] ~> Exec['tempest-workspace']
Exec['tempest-workspace'] -> Tempest_config<||> Exec['tempest-workspace'] -> Tempest_config<||>
$tempest_conf = "${tempest_workspace}/etc/tempest.conf" $tempest_conf = "${tempest_workspace}/etc/tempest.conf"

2
spec/classes/tempest_init_spec.rb
View File

@ -445,7 +445,7 @@ describe 'tempest' do
:command => ['tempest', 'init', '/var/lib/tempest'], :command => ['tempest', 'init', '/var/lib/tempest'],
:path => ['/bin', '/usr/bin'], :path => ['/bin', '/usr/bin'],
:refreshonly => true, :refreshonly => true,
:require => 'Package[tempest]' :subscribe => 'Package[tempest]'
) )
end end
end end