diff options
author | Gabriel Nagy <gabriel.nagy@puppet.com> | 2021-02-22 19:00:25 +0200 |
---|---|---|
committer | Gabriel Nagy <gabriel.nagy@puppet.com> | 2021-02-22 19:08:35 +0200 |
commit | 44be62fc43407528d5873cdee1559c55a373257b (patch) | |
tree | 010e0198e5fa38a6f888b45cb842d879094ada3d | |
parent | 209b03b13f67a76e53048c5f854af7d0bd0a6a87 (diff) | |
download | puppet-augeas_core-44be62fc43407528d5873cdee1559c55a373257b.tar.gz puppet-augeas_core-44be62fc43407528d5873cdee1559c55a373257b.tar.bz2 |
(MODULES-10950) Infer application name from run mode
Fix a regression introduced by MODULES-7397[1] which incorrectly assumed
that the application name can be queried through
`Puppet::Application.name`, causing lenses that are pluginsynced not to
be loaded.
Since we only need to find out whether or not we're running as part of
`puppet agent`, it should be enough to use `Puppet.run_mode.name`, which
returns `:agent` in this case, and `:user` otherwise.
[1] https://github.com/puppetlabs/puppetlabs-augeas_core/pull/27
-rw-r--r-- | lib/puppet/provider/augeas/augeas.rb | 2 | ||||
-rw-r--r-- | spec/unit/provider/augeas/augeas_spec.rb | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/lib/puppet/provider/augeas/augeas.rb b/lib/puppet/provider/augeas/augeas.rb index c8f9643..b0a3e4f 100644 --- a/lib/puppet/provider/augeas/augeas.rb +++ b/lib/puppet/provider/augeas/augeas.rb @@ -367,7 +367,7 @@ Puppet::Type.type(:augeas).provide(:augeas) do load_path.flatten! end - if Puppet::Application.name == :agent + if Puppet.run_mode.name == :agent if Puppet::FileSystem.exist?("#{Puppet[:libdir]}/augeas/lenses") load_path << "#{Puppet[:libdir]}/augeas/lenses" end diff --git a/spec/unit/provider/augeas/augeas_spec.rb b/spec/unit/provider/augeas/augeas_spec.rb index 2d53933..f98a0f4 100644 --- a/spec/unit/provider/augeas/augeas_spec.rb +++ b/spec/unit/provider/augeas/augeas_spec.rb @@ -1124,7 +1124,7 @@ describe Puppet::Type.type(:augeas).provider(:augeas) do context 'when running application is agent' do before(:each) do Puppet[:libdir] = my_fixture_dir - Puppet::Application.stubs(:name).returns(:agent) + Puppet.run_mode.stubs(:name).returns(:agent) end it 'offers pluginsync augeas/lenses subdir' do @@ -1139,7 +1139,7 @@ describe Puppet::Type.type(:augeas).provider(:augeas) do context 'when running application is not agent' do before(:each) do - Puppet::Application.stubs(:name).returns(:apply) + Puppet.run_mode.stubs(:name).returns(:user) env = Puppet::Node::Environment.create('root', ['/modules/foobar']) Puppet.stubs(:lookup).returns(env) |