openstack/puppet-ceph
Code Issues Proposed changes

ceph::repo : Ensure boolean values

Browse Source 
This ensures some parameters used for if-else logic take only boolean
values to avoid unexpected behavior caused by non-boolean value (eg.
'false'). This also makes sure that the release parameter takes a valid
string because it is used to construct the urls.

Change-Id: I49160cff2d35e8f2223431a7d3cd91b652252b9c
This commit is contained in:
Takashi Kajinami 2023-07-18 00:05:47 +09:00
parent b59f7024cc
commit 4dbd23f635
1 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
manifests/repo.pp
View File

@ -63,16 +63,16 @@
# Optional. Defaults to undef.
#
class ceph::repo (
$ensure = present,
$release = $ceph::params::release,
$fastcgi = false,
$proxy = undef,
$proxy_username = undef,
$proxy_password = undef,
$enable_epel = true,
$enable_sig = $ceph::params::enable_sig,
$stream = false,
$ceph_mirror = undef,
$ensure = present,
String[1] $release = $ceph::params::release,
Boolean $fastcgi = false,
$proxy = undef,
$proxy_username = undef,
$proxy_password = undef,
Boolean $enable_epel = true,
Boolean $enable_sig = $ceph::params::enable_sig,
Boolean $stream = false,
$ceph_mirror = undef,
) inherits ceph::params {
case $facts['os']['family'] {
'Debian': {