Skip to content

Commit e82d6bd

Browse files
Use ensure_resource to manage the docker-compose-plugin
This is likely the expected behavior for this module
1 parent 21d48a1 commit e82d6bd

File tree

1 file changed

+14
-2
lines changed

1 file changed

+14
-2
lines changed

Diff for: manifests/compose.pp

+14-2
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,22 @@
2020

2121
case $facts['os']['family'] {
2222
'Debian': {
23-
ensure_packages('docker-compose-plugin', { ensure => $package_ensure, require => defined(bool2str($docker::use_upstream_package_source)) ? { true => Apt::Source['docker'], false => undef } }) #lint:ignore:140chars
23+
ensure_resource('package', 'docker-compose-plugin', {
24+
ensure => $package_ensure,
25+
require => defined(bool2str($docker::use_upstream_package_source)) ? {
26+
true => Apt::Source['docker'],
27+
false => undef,
28+
},
29+
})
2430
}
2531
'RedHat': {
26-
ensure_packages('docker-compose-plugin', { ensure => $package_ensure, require => defined(bool2str($docker::use_upstream_package_source)) ? { true => Yumrepo['docker'], false => undef } }) #lint:ignore:140chars lint:ignore:unquoted_string_in_selector
32+
ensure_resource('package', 'docker-compose-plugin', {
33+
ensure => $package_ensure,
34+
require => defined(bool2str($docker::use_upstream_package_source)) ? {
35+
true => Yumrepo['docker'],
36+
false => undef,
37+
},
38+
})
2739
}
2840
'Windows': {
2941
fail('Docker compose is installed with docker machine on Windows')

0 commit comments

Comments
 (0)