issue with REG_SZ and REG_EXPAND_SZ on Windows 2012R2 about puppetlabs-windows_env

[badgerious]

Try:

type => "REG_EXPAND_SZ",

Instead of

type => REG_EXPAND_SZ,

Seems like new puppet interprets unquoted strings differently. I'll have to update the docs.

[pngfei]

quote the REG_EXPAND_SZ, the issue is gone

[badgerious]

I haven't tested that particular combination, no. Could you show me a sample of the puppet code that results in that error?

[mikecurrancs]

Here is the code that works on PE3.8 / w2k8r2 -- we read in hiera to build the variables below.

class cs_win_java (
$jre_version,
$jre_installer,
$jre_pkgname,
$jre_directory,
$staging_path,
$puppet_file_mount
) {

file { $jre_installer:
ensure => file,
owner => 'Administrator',
group => 'Administrators',
mode => '0770',
path => "${staging_path}/${jre_installer}",
source => "puppet:///${puppet_file_mount}/${module_name}/${jre_installer}"
} ->
package { $jre_pkgname:
ensure => installed,
source => "${staging_path}/${jre_installer}",
install_options => ['/s'],
} ->
windows_env { "PATH=%ProgramFiles(x86)%\Java${jre_directory}\bin":
type => REG_EXPAND_SZ,
} ->
windows_env { "JAVA_HOME=%ProgramFiles(x86)%\Java${jre_directory}":
type => REG_EXPAND_SZ,
}
}

[mikecurrancs]

I realized my agent was 4.4.1 and server was 4.3.2 -- replace agent with 4.3.2 , hoping it was a 4.4.x issue -- but same error.

[mikecurrancs]

Well -- I found that I did not have win32/registry installed with puppet server 4.3.2 -- and that there is an open bug report, where your module is listed as affected by the way Ruby 2.1.x (pre-packaged with puppet 4.x) Win32::Registry is handled -- here is the link

https://tickets.puppetlabs.com/browse/PUP-3837

[badgerious]

PUP-3837 was handled by #8, so that shouldn't be an issue. I'm not sure what you mean that you didn't have win32/registry installed with puppet server; it should only be installed on the Windows client, never on the (presumably Linux, but certainly not Windows) server.

I need to spin up a VM to test and I haven't had a chance yet; hopefully this weekend.