Skip to content

Commit 1fbe012

Browse files
committed
Allow bypassing curl package ensure if needed
1 parent 813dcbd commit 1fbe012

File tree

5 files changed

+31
-4
lines changed

5 files changed

+31
-4
lines changed

README.md

+8
Original file line numberDiff line numberDiff line change
@@ -240,6 +240,14 @@ class { 'docker':
240240
}
241241
```
242242

243+
If the curl package is being managed elsewhere and the curl ensure in this module is conflicting,
244+
it can be disabled by setting the following parameter globally or in compose / machine resources:
245+
```puppet
246+
class { 'docker':
247+
curl_ensure => false
248+
}
249+
```
250+
243251
### Proxy on Windows
244252

245253
To use docker through a proxy on Windows, a System Environment Variable HTTP_PROXY/HTTPS_PROXY must be set. See [Docker Engine on Windows](https://docs.microsoft.com/en-us/virtualization/windowscontainers/manage-docker/configure-docker-daemon#proxy-configuration)

manifests/compose.pp

+9-2
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,16 @@
2020
# [*proxy*]
2121
# Proxy to use for downloading Docker Compose.
2222
#
23+
# [*curl_ensure*]
24+
# Whether or not the curl package is ensured by this module.
25+
#
26+
#
2327
class docker::compose(
2428
Optional[Pattern[/^present$|^absent$/]] $ensure = 'present',
2529
Optional[String] $version = $docker::params::compose_version,
2630
Optional[String] $install_path = $docker::params::compose_install_path,
27-
Optional[String] $proxy = undef
31+
Optional[String] $proxy = undef,
32+
Optional[Boolean] $curl_ensure = $docker::curl_ensure,
2833
) inherits docker::params {
2934

3035
if $proxy != undef {
@@ -68,7 +73,9 @@
6873
require => Exec["Install Docker Compose ${version}"]
6974
}
7075
} else {
71-
ensure_packages(['curl'])
76+
if $curl_ensure {
77+
ensure_packages(['curl'])
78+
}
7279
exec { "Install Docker Compose ${version}":
7380
path => '/usr/bin/',
7481
cwd => '/tmp',

manifests/init.pp

+5
Original file line numberDiff line numberDiff line change
@@ -379,6 +379,10 @@
379379
# Sets the prefered container registry mirror.
380380
# Default: undef
381381
#
382+
# [*curl_ensure*]
383+
# Whether or not the curl package is ensured by this module.
384+
# Default: true
385+
#
382386
# [*nuget_package_provider_version*]
383387
# The version of the NuGet Package provider
384388
# Default: undef
@@ -502,6 +506,7 @@
502506
Optional[Boolean] $service_hasstatus = $docker::params::service_hasstatus,
503507
Optional[Boolean] $service_hasrestart = $docker::params::service_hasrestart,
504508
Optional[String] $registry_mirror = $docker::params::registry_mirror,
509+
Optional[Boolean] $curl_ensure = $docker::params::curl_ensure,
505510
# Windows specific parameters
506511
Optional[String] $docker_msft_provider_version = $docker::params::docker_msft_provider_version,
507512
Optional[String] $nuget_package_provider_version = $docker::params::nuget_package_provider_version,

manifests/machine.pp

+8-2
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,15 @@
2020
# [*proxy*]
2121
# Proxy to use for downloading Docker Machine.
2222
#
23+
# [*curl_ensure*]
24+
# Whether or not the curl package is ensured by this module.
25+
#
2326
class docker::machine(
2427
Optional[Pattern[/^present$|^absent$/]] $ensure = 'present',
2528
Optional[String] $version = $docker::params::machine_version,
2629
Optional[String] $install_path = $docker::params::machine_install_path,
27-
Optional[String] $proxy = undef
30+
Optional[String] $proxy = undef,
31+
Optional[Boolean] $curl_ensure = $docker::curl_ensure,
2832
) inherits docker::params {
2933

3034
if $proxy != undef {
@@ -68,7 +72,9 @@
6872
require => Exec["Install Docker Machine ${version}"]
6973
}
7074
} else {
71-
ensure_packages(['curl'])
75+
if $curl_ensure {
76+
ensure_packages(['curl'])
77+
}
7278
exec { "Install Docker Machine ${version}":
7379
path => '/usr/bin/',
7480
cwd => '/tmp',

manifests/params.pp

+1
Original file line numberDiff line numberDiff line change
@@ -107,6 +107,7 @@
107107
$storage_pool_autoextend_percent = undef
108108
$storage_config_template = 'docker/etc/sysconfig/docker-storage.erb'
109109
$registry_mirror = undef
110+
$curl_ensure = true
110111
$os_lc = downcase($::operatingsystem)
111112
$docker_msft_provider_version = undef
112113
$nuget_package_provider_version = undef

0 commit comments

Comments
 (0)