Skip to content

Commit f99d48d

Browse files
committed
init.pp: remove useless exec with default title
Per the docs, the special value `default` sets default attribute values for other resource bodies in the same expression. There are no resources in this expression so it doesn't do anything. This was probably intended to be a "legacy-style default, but the style guide says to avoid these anyway.
1 parent 063e230 commit f99d48d

File tree

2 files changed

+0
-10
lines changed

2 files changed

+0
-10
lines changed

REFERENCE.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -180,12 +180,8 @@ Default value: `true`
180180

181181
##### <a name="-python--umask"></a>`umask`
182182

183-
Data type: `Optional[Python::Umask]`
184-
185183
The default umask for invoked exec calls.
186184

187-
Default value: `undef`
188-
189185
##### <a name="-python--manage_gunicorn"></a>`manage_gunicorn`
190186

191187
manage the state for package gunicorn

manifests/init.pp

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@
6262
Stdlib::Httpurl $anaconda_installer_url = 'https://repo.anaconda.com/archive/Anaconda3-5.2.0-Linux-x86_64.sh',
6363
Stdlib::Absolutepath $anaconda_install_path = '/opt/python',
6464
Boolean $manage_scl = true,
65-
Optional[Python::Umask] $umask = undef,
6665
) inherits python::params {
6766
$exec_prefix = $provider ? {
6867
'scl' => "/usr/bin/scl enable ${version} -- ",
@@ -76,11 +75,6 @@
7675
Class['python::install']
7776
-> Class['python::config']
7877

79-
# Set default umask.
80-
exec { default:
81-
umask => $umask,
82-
}
83-
8478
# Allow hiera configuration of python resources
8579
create_resources('python::pip', $python_pips)
8680
create_resources('python::pyvenv', $python_pyvenvs)

0 commit comments

Comments
 (0)