openstack/puppet-tripleo
Code Issues Proposed changes

Merge "Fix docker debug/mirrors JSON augeas changes"

Browse Source
This commit is contained in:
Zuul 2018-04-11 00:31:49 +00:00 committed by Gerrit Code Review
commit e2220b6d7f
2 changed files with 12 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
manifests/profile/base/docker.pp
View File

@ -173,19 +173,21 @@ class tripleo::profile::base::docker (
if $registry_mirror {
$mirror_changes = [
'set dict/entry[. = "registry-mirrors"] "registry-mirrors',
"set dict/entry[. = \"registry-mirrors\"]/array/string \"${registry_mirror}\""
"set dict/entry[. = 'registry-mirrors'] registry-mirrors",
"set dict/entry[. = 'registry-mirrors']/array/string \"${registry_mirror}\""
]
} else {
$mirror_changes = [ 'rm dict/entry[. = "registry-mirrors"]', ]
$mirror_changes = [ "rm dict/entry[. = 'registry-mirrors']", ]
}
if $debug {
$debug_changes = [
'set dict/entry[. = "debug"] "debug"',
"set dict/entry[. = \"debug\"]/const \"${debug}\"",]
"set dict/entry[. = 'debug'] debug",
"set dict/entry[. = 'debug']/const true", ]
} else {
$debug_changes = [ 'rm dict/entry[. = "debug"]', ]
$debug_changes = [
"set dict/entry[. = 'debug'] debug",
"set dict/entry[. = 'debug']/const false", ]
}
file { '/etc/docker/daemon.json':

8
spec/classes/tripleo_profile_base_docker_spec.rb
View File

@ -86,8 +86,8 @@ describe 'tripleo::profile::base::docker' do
it { is_expected.to contain_file('/etc/systemd/system/docker.service.d/99-unset-mountflags.conf') }
it {
is_expected.to contain_augeas('docker-daemon.json-mirror').with_changes(
['set dict/entry[. = "registry-mirrors"] "registry-mirrors',
"set dict/entry[. = \"registry-mirrors\"]/array/string \"http://foo/bar\""])
["set dict/entry[. = 'registry-mirrors'] registry-mirrors",
"set dict/entry[. = 'registry-mirrors']/array/string \"http://foo/bar\""])
}
end
@ -103,8 +103,8 @@ describe 'tripleo::profile::base::docker' do
it { is_expected.to contain_file('/etc/systemd/system/docker.service.d/99-unset-mountflags.conf') }
it {
is_expected.to contain_augeas('docker-daemon.json-debug').with_changes(
['set dict/entry[. = "debug"] "debug"',
"set dict/entry[. = \"debug\"]/const \"true\""])
["set dict/entry[. = 'debug'] debug",
"set dict/entry[. = 'debug']/const true"])
}
end