Skip to content

Commit

Permalink
(PA-5826) Only read Windows VERSION file during puppet apply
Browse files Browse the repository at this point in the history
Previously, puppetserver attempted to load the wrong VERSION file when managing
`package_version => auto` and compiling catalogs for Windows nodes as can be
seen by the new test:

    Evaluation Error: Error while evaluating a Function Call, Could not find any files from C:\Program Files\Puppet Labs\Puppet\VERSION (file: path/to/init.pp, line: 170, column: 42)

This adds a `puppet_runmode` fact. When running as puppet agent, the run mode will
be `agent`. When running as puppet apply, it will be `user`. If it's the latter
and we're a Windows host, then look for the VERSION file locally.
  • Loading branch information
joshcooper committed Oct 25, 2023
1 parent 560c1e8 commit ba7758d
Show file tree
Hide file tree
Showing 3 changed files with 37 additions and 1 deletion.
3 changes: 3 additions & 0 deletions lib/facter/puppet_runmode.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
Facter.add(:puppet_runmode) do
setcode { Puppet.run_mode.name.to_s }
end
5 changes: 4 additions & 1 deletion manifests/init.pp
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@
Optional $wait_for_pxp_agent_exit = undef,
Optional $wait_for_puppet_run = undef,
Array[Puppet_agent::Config] $config = [],
String $version_file_path = $facts['os']['family'] ? { 'windows' => "${facts['env_windows_installdir']}\\VERSION", default => '/opt/puppetlabs/puppet/VERSION' }
String $version_file_path = '/opt/puppetlabs/puppet/VERSION'
) inherits puppet_agent::params {
# The configure class uses $puppet_agent::config to manage settings in
# puppet.conf, and will always be present. It does not require management of
Expand Down Expand Up @@ -167,6 +167,9 @@
# The AIO package version and Puppet version can, on rare occasion, diverge.
# This logic checks for the AIO version of the server, since that's what the package manager cares about.
if $package_version == 'auto' {
if $facts['os']['family'] == 'windows' and $facts['puppet_runmode'] == 'user' {
$version_file_path = "${facts['env_windows_installdir']}\\VERSION"
}
$master_or_package_version = chomp(file($version_file_path))
} else {
$master_or_package_version = $package_version
Expand Down
30 changes: 30 additions & 0 deletions spec/classes/puppet_agent_osfamily_redhat_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -274,6 +274,36 @@
it { is_expected.to contain_package('puppet-agent').with_ensure(params[:package_version].to_s) }
end

context 'with auto package version' do
let(:params) do
{
manage_repo: false,
package_version: 'auto'
}
end

context 'with Windows agent' do
let(:facts) do
override_facts(super(),
os: { family: 'windows', 'windows' => { 'system32' => 'C:\Windows\System32' } },
'env_windows_installdir' => 'C:\Program Files\Puppet Labs\Puppet',
'puppet_agent_appdata' => 'C:\ProgramData',
'puppet_confdir' => 'C:\ProgramData\PuppetLabs\puppet',
'puppet_runmode' => 'agent'
)
end

before :each do
allow(Puppet::FileSystem).to receive(:exist?).and_call_original
allow(Puppet::FileSystem).to receive(:read_preserve_line_endings).and_call_original
allow(Puppet::FileSystem).to receive(:exist?).with('/opt/puppetlabs/puppet/VERSION').and_return true
allow(Puppet::FileSystem).to receive(:read_preserve_line_endings).with('/opt/puppetlabs/puppet/VERSION').and_return "7.6.5\n"
end

it { is_expected.to contain_class('puppet_agent::install::windows') }
end
end

it { is_expected.to contain_class('puppet_agent::osfamily::redhat') }
end

Expand Down

0 comments on commit ba7758d

Please sign in to comment.