openstack
/
puppet-cinder
Code Issues Proposed changes

Use service anchor to ensure API is up 

... instead of the individual service resource which can be named
differently according to service_name.

Closes-Bug: #1949658
Change-Id: I847b88710578818f6c792d1603e02759072a9d25
(cherry picked from commit 614d16329a)
This commit is contained in:
Takashi Kajinami 2021-11-04 11:31:00 +09:00
parent eb2c462cbc
commit 3241ff122a
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/api.pp
View File

@ -264,7 +264,7 @@ running as a standalone service, or httpd for being run by a httpd server")
# lint:endignore
}
}
create_resources('openstacklib::service_validation', $validation_cmd, {'subscribe' => 'Service[cinder-api]'})
create_resources('openstacklib::service_validation', $validation_cmd, {'subscribe' => 'Anchor[cinder::service::end]'})
}
}

2
spec/classes/cinder_api_spec.rb
View File

@ -126,7 +126,7 @@ describe 'cinder::api' do
end
it { is_expected.to contain_openstacklib__service_validation('cinder-api').with(
:command => 'cinder --os-auth-url http://localhost:5000 --os-project-name services --os-username cinder --os-password foo list',
:subscribe => 'Service[cinder-api]',
:subscribe => 'Anchor[cinder::service::end]',
)}
end