Skip to content

Commit 344b000

Browse files
committed
Remove undefined $code_manager_auto_configure_nc_value variable
Prior to this commit, we referenced $code_manager_auto_configure_nc_value in a pick statement despite the fact that the variable has been commented out. After this commit, we remove the commented out code and remove the $code_manager_auto_configure_nc_value variable reference
1 parent 343d308 commit 344b000

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

manifests/init.pp

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,8 @@
33
Boolean $force_code_manager_webhook = false,
44
) {
55

6-
#Determine if code manager is enabled
7-
#Querying the NC requires the puppetclassify gem and this seems like undue setup burden for a small check at this time
8-
#$code_manager_auto_configure_nc_value = node_groups('PE Master')['PE Master']['classes']['puppet_enterprise::profile::master']['code_manager_auto_configure']
96
$code_manager_auto_configure_hiera_value = hiera('puppet_enterprise::profile::master::code_manager_auto_configure', undef)
10-
#The NC value will take precedence over the hiera value, if neither is set then it defaults to false
11-
$code_manager_auto_configure = pick($code_manager_auto_configure_nc_value, $code_manager_auto_configure_hiera_value, $force_code_manager_webhook)
7+
$code_manager_auto_configure = pick($code_manager_auto_configure_hiera_value, $force_code_manager_webhook)
128

139
if versioncmp( $::pe_server_version, '2015.2.99' ) <= 0 or $force_zack_r10k_webhook {
1410
include pe_code_manager_webhook::zack_r10k_webhook

0 commit comments

Comments
 (0)