openstack
/
puppet-tripleo
Code Issues Proposed changes

Add -s (silent) to curl command for CRL refresh 

Without it, it doesn't reload the services it should.

Change-Id: I43e6188700deb585f905ca700e69b6875f0ded45
Closes-Bug: #1712404
This commit is contained in:
Juan Antonio Osorio Robles 2017-08-22 21:06:14 +03:00
parent 92c04bbb77
commit ab48d3a438
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/certmonger/ca/crl.pp
View File

@ -129,7 +129,7 @@ class tripleo::certmonger::ca::crl (
if $ensure == 'present' {
# Fetch CRL in cron job and notify needed services
$cmd_list = concat(["${sleep}curl -L -o ${fetched_crl} ${crl_source}"], $process_cmd, $reload_cmds)
$cmd_list = concat(["${sleep}curl -s -L -o ${fetched_crl} ${crl_source}"], $process_cmd, $reload_cmds)
$cron_cmd = join($cmd_list, ' && ')
} else {
$cron_cmd = absent

4
spec/classes/tripleo_certmonger_ca_crl_spec.rb
View File

@ -47,7 +47,7 @@ describe 'tripleo::certmonger::ca::crl' do
end
let :cron_cmd do
"curl -L -o #{params[:crl_preprocessed]} #{params[:crl_source]} && #{process_cmd}"
"curl -s -L -o #{params[:crl_preprocessed]} #{params[:crl_source]} && #{process_cmd}"
end
it 'should create and process CRL file' do
@ -75,7 +75,7 @@ describe 'tripleo::certmonger::ca::crl' do
end
let :cron_cmd do
"curl -L -o #{params[:crl_dest]} #{params[:crl_source]}"
"curl -s -L -o #{params[:crl_dest]} #{params[:crl_source]}"
end
it 'should create and process CRL file' do