Workaround CI failure with explicit lookup

smortex · smortex · commit 3a3ff1d2524d · 2023-03-20T16:40:57.000-10:00
I would expect Automatic Parameter Lookup to do exactly this, but it
does not.
diff --git a/REFERENCE.md b/REFERENCE.md
@@ -753,6 +753,8 @@ Data type: `String[1]`
 
 The group of the virtualenv being manipulated.
 
+Default value: `lookup('python::pip::group')`
+
 ##### <a name="-python--pip--index"></a>`index`
 
 Data type: `Variant[Boolean,String[1]]`
diff --git a/manifests/pip.pp b/manifests/pip.pp
@@ -49,7 +49,7 @@
 #   }
 #
 define python::pip (
-  String[1]                                         $group,
+  String[1]                                         $group          = lookup('python::pip::group'), # lint:ignore:lookup_in_parameter
   String[1]                                         $pkgname        = $name,
   Variant[Enum[present, absent, latest], String[1]] $ensure         = present,
   Variant[Enum['system'], Stdlib::Absolutepath]     $virtualenv     = 'system',

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@`
`49`	`49`	`# }`
`50`	`50`	`#`
`51`	`51`	`define python::pip (`
`52`		`- String[1] $group,`
	`52`	`+ String[1] $group = lookup('python::pip::group'), # lint:ignore:lookup_in_parameter`
`53`	`53`	`String[1] $pkgname = $name,`
`54`	`54`	`Variant[Enum[present, absent, latest], String[1]] $ensure = present,`
`55`	`55`	`Variant[Enum['system'], Stdlib::Absolutepath] $virtualenv = 'system',`