openstack/puppet-octavia
Code Issues Proposed changes

Create certificates are package installs

Browse Source 
When setting certificate folders to /etc/octavia/certs
or similar that depends on /etc/octavia folder the resource
will fail because it doesn't create the whole folder path
recursively.

This changes the dependency chaining to make sure that
certificates are modified after package installs so that
the /etc/octavia folder for example is available.

Closes-Bug: 1804883
Change-Id: I551053b96fde110478945955f3d7dc42bf3e65ef
This commit is contained in:
Tobias Urdin 2018-11-23 21:05:56 +01:00
parent 0ea4e10dd3
commit 768d2900eb
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
manifests/deps.pp
View File

@ -16,6 +16,9 @@ class octavia::deps {
-> anchor { 'octavia::config::begin': }
-> Octavia_config<||>
~> anchor { 'octavia::config::end': }
-> anchor { 'octavia::certificate::begin': }
-> File<| tag == 'octavia-certificate' |>
~> anchor { 'octavia::certificate::end': }
-> anchor { 'octavia::db::begin': }
-> anchor { 'octavia::db::end': }
~> anchor { 'octavia::dbsync::begin': }
@ -38,5 +41,5 @@ class octavia::deps {
Anchor['octavia::config::end'] ~> Anchor['octavia::service::begin']
# Changes in certificate or folders will restart services.
File<| tag == 'octavia-certificate' |> ~> Anchor['octavia::service::begin']
Anchor['octavia::certificate::end'] ~> Anchor['octavia::service::begin']
}

5
releasenotes/notes/certificate-folder-after-pkg-802703c4a8d646b3.yaml Normal file
View File

@ -0,0 +1,5 @@
---
fixes:
- |
Fixed a bug where certificate folders that depended on paths provided
by packages failed.