From e82d6bd04fa4a81f90d5b0f92982ca7709518530 Mon Sep 17 00:00:00 2001 From: Nathan Carlson Date: Tue, 16 Apr 2024 15:01:59 -0500 Subject: [PATCH] Use ensure_resource to manage the docker-compose-plugin This is likely the expected behavior for this module --- manifests/compose.pp | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/manifests/compose.pp b/manifests/compose.pp index f8e56ca3..5174d813 100644 --- a/manifests/compose.pp +++ b/manifests/compose.pp @@ -20,10 +20,22 @@ case $facts['os']['family'] { 'Debian': { - 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 + ensure_resource('package', 'docker-compose-plugin', { + ensure => $package_ensure, + require => defined(bool2str($docker::use_upstream_package_source)) ? { + true => Apt::Source['docker'], + false => undef, + }, + }) } 'RedHat': { - 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 + ensure_resource('package', 'docker-compose-plugin', { + ensure => $package_ensure, + require => defined(bool2str($docker::use_upstream_package_source)) ? { + true => Yumrepo['docker'], + false => undef, + }, + }) } 'Windows': { fail('Docker compose is installed with docker machine on Windows')