openstack/puppet-tripleo
Code Issues Proposed changes

Merge "Fixes incorrect ownership of ODL TLS cert/key"

Browse Source
This commit is contained in:
Zuul 2018-03-21 12:13:17 +00:00 committed by Gerrit Code Review
commit 4738c61b15
3 changed files with 5 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/certmonger/opendaylight.pp
View File

@ -62,13 +62,9 @@ class tripleo::certmonger::opendaylight (
require => Class['::certmonger'], require => Class['::certmonger'],
} }
file { $service_certificate : file { $service_certificate :
owner => 'odl',
group => 'odl',
require => Certmonger_certificate['opendaylight'] require => Certmonger_certificate['opendaylight']
} }
file { $service_key : file { $service_key :
owner => 'odl',
group => 'odl',
require => Certmonger_certificate['opendaylight'] require => Certmonger_certificate['opendaylight']
} }

5
releasenotes/notes/fix-odl-tls-owner-77d2d71fe39ea3e7.yaml Normal file
View File

@ -0,0 +1,5 @@
---
fixes:
- |
Fixes a bug where TLS certificates for ODL could not be generated correctly
for deployment due to wrong owner/group applied to the files.

4
spec/classes/tripleo_certmonger_opendaylight_spec.rb
View File

@ -47,13 +47,9 @@ describe 'tripleo::certmonger::opendaylight' do
:wait => true, :wait => true,
) )
is_expected.to contain_file(params[:service_certificate]).with( is_expected.to contain_file(params[:service_certificate]).with(
:owner => 'odl',
:group => 'odl',
:require => 'Certmonger_certificate[opendaylight]' :require => 'Certmonger_certificate[opendaylight]'
) )
is_expected.to contain_file(params[:service_key]).with( is_expected.to contain_file(params[:service_key]).with(
:owner => 'odl',
:group => 'odl',
:require => 'Certmonger_certificate[opendaylight]' :require => 'Certmonger_certificate[opendaylight]'
) )
end end