puppet 4.8.2-universal-darwin → 4.9.0-universal-darwin

checksums.yaml

@@ -0,0 +1,15 @@
+---
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    YmU1Y2MyNjEzNjkzNTJkZWVjNWJkZTQ2OTIzMmYzMWQ1YTY1MGIxOQ==
+  data.tar.gz: !binary |-
+    OWRkMGE5Zjc4NWI3Y2VjOGI3ODBiN2JkZTA5NzZhYjI2MDdlYWViOQ==
+SHA512:
+  metadata.gz: !binary |-
+    MTJiODQzOTZiYzU1Y2Q3NThjYzAyYTkyZGFhMWIyNDI3Njg5MjU1M2JiNzQ3
+    MjVmNDg5OTlhNDVjMmJiOWYxMDM1MmM2NWZmZjU2NjU4OWU3MzQxZDc0Yzdi
+    MTljNWU0YjI2ODljMzIwODVhMjUxNjllNzYxMzY5YjRlMzhkNDU=
+  data.tar.gz: !binary |-
+    ZWVlYTU3M2I5OTI0YWJjYWRiNGU4NWEzMjBiMzk1NjcwZmZiYTQzZTVhMDUz
+    ZmMxYjY4ODNhNWE1MWM0NGY5M2RjNWMzMmMzMTlmNjExODhkODdjNjA4Yjg2
+    NDA4MzZmNTM3NmVlYjhhODU2N2Y4ZTYxNmNkMTA4NWUzZjI1MmI=

data/CONTRIBUTING.md

@@ -104,13 +104,37 @@ a ticket number.
 * After feedback has been given we expect responses within two weeks. After two
   weeks we may close the pull request if it isn't showing any activity.
 
+## Revert Policy
+By running tests in advance and by engaging with peer review for prospective
+changes, your contributions have a high probability of becoming long lived
+parts of the the project. After being merged, the code will run through a
+series of testing pipelines on a large number of operating system
+environments. These pipelines can reveal incompatibilities that are difficult
+to detect in advance.
+
+If the code change results in a test failure, we will make our best effort to
+correct the error. If a fix cannot be determined and committed within 24 hours
+of its discovery, the commit(s) responsible _may_ be reverted, at the
+discretion of the committer and Puppet maintainers. This action would be taken
+to help maintain passing states in our testing pipelines.
+
+The original contributor will be notified of the revert in the Jira ticket
+associated with the change. A reference to the test(s) and operating system(s)
+that failed as a result of the code change will also be added to the Jira
+ticket. This test(s) should be used to check future submissions of the code to
+ensure the issue has been resolved.
+
+### Summary
+* Changes resulting in test pipeline failures will be reverted if they cannot
+  be resolved within one business day.
+
 # Additional Resources
 
 * [Puppet community guidelines](https://docs.puppet.com/community/community_guidelines.html)
 * [Bug tracker (Jira)](https://tickets.puppetlabs.com)
 * [Contributor License Agreement](http://links.puppet.com/cla)
 * [General GitHub documentation](https://help.github.com/)
-* [GitHub pull request documentation](https://help.github.com/send-pull-requests/)
+* [GitHub pull request documentation](https://help.github.com/articles/creating-a-pull-request/)
 * #puppet-dev IRC channel on freenode.org ([Archive](https://botbot.me/freenode/puppet-dev/))
 * [puppet-dev mailing list](https://groups.google.com/forum/#!forum/puppet-dev)
 * [Community PR Triage notes](https://github.com/puppet-community/community-triage/tree/master/core/notes)

data/Gemfile

@@ -26,10 +26,15 @@ end
 gem "puppet", :path => File.dirname(__FILE__), :require => false
 gem "facter", *location_for(ENV['FACTER_LOCATION'] || ['> 2.0', '< 4'])
 gem "hiera", *location_for(ENV['HIERA_LOCATION'] || ['>= 2.0', '< 4'])
+# PUP-7115 - return to a gem dependency in Puppet 5
+# gem "semantic_puppet", *location_for(ENV['SEMANTIC_PUPPET_LOCATION'] || ['>= 0.1.3', '< 2'])
 gem "rake", "10.1.1", :require => false
 # Hiera has an unbound dependency on json_pure
 # json_pure 2.0.2+ officially requires Ruby >= 2.0, but should have specified that in 2.0
 gem 'json_pure', '~> 1.8', :require => false
+# i18n support (gettext-setup and dependencies)
+gem 'gettext-setup', '>= 0.10', '< 1.0', :require => false
+gem 'locale', '~> 2.1', :require => false
 
 group(:development, :test) do
   gem "rspec", "~> 3.1", :require => false
@@ -58,6 +63,7 @@ group(:development, :test) do
   gem 'addressable', '< 2.5.0'
   gem 'webmock', '~> 1.24'
   gem 'vcr', '~> 2.9'
+  gem "hocon", :require => false
 end
 
 group(:development) do

data/Rakefile

@@ -103,3 +103,4 @@ task(:commits) do
     puts "...passed"
   end
 end
+

data/ext/project_data.yaml

@@ -9,7 +9,7 @@ version_file: 'lib/puppet/version.rb'
 # files and gem_files are space separated lists
 files: '[A-Z]* install.rb bin lib conf man examples ext tasks spec'
 # The gem specification bits only work on Puppet >= 3.0rc, NOT 2.7.x and earlier
-gem_files: '[A-Z]* install.rb bin lib conf man examples ext tasks spec'
+gem_files: '[A-Z]* install.rb bin lib conf man examples ext tasks spec locales'
 gem_test_files: 'spec/**/*'
 gem_executables: 'puppet'
 gem_default_executables: 'puppet'
@@ -20,6 +20,10 @@ gem_runtime_dependencies:
   facter: ['> 2.0', '< 4']
   hiera: ['>= 2.0', '< 4']
   json_pure: '~> 1.8'
+  # PUP-7115 - return to a gem dependency in Puppet 5
+  # semantic_puppet: ['>= 0.1.3', '< 2']
+  gettext-setup: ['>= 0.10', '< 1']
+  locale: '~> 2.1'
 gem_rdoc_options:
   - --title
   - "Puppet - Configuration Management"

data/ext/windows/service/daemon.rb

@@ -123,7 +123,8 @@ class WindowsDaemon < Win32::Daemon
   def log(msg, level)
     if LEVELS.index(level) >= @loglevel
       if (@LOG_TO_FILE)
-        File.open(LOG_FILE, 'a') { |f| f.puts("#{Time.now} Puppet (#{level}): #{msg}") }
+        # without this change its possible that we get Encoding errors trying to write UTF-8 messages in current codepage
+        File.open(LOG_FILE, 'a:UTF-8') { |f| f.puts("#{Time.now} Puppet (#{level}): #{msg}") }
       end
 
       case level

data/install.rb

@@ -108,9 +108,24 @@ def do_man(man, strip = 'man/')
     FileUtils.makedirs(om, {:mode => 0755, :verbose => true})
     FileUtils.chmod(0755, om)
     FileUtils.install(mf, omf, {:mode => 0644, :preserve => true, :verbose => true})
-    gzip = %x{which gzip}
-    gzip.chomp!
-    %x{#{gzip} -f #{omf}}
+    # Solaris does not support gzipped man pages. When called with
+    # --no-check-prereqs/without facter the default gzip behavior still applies
+    unless $operatingsystem == "Solaris"
+      gzip = %x{which gzip}
+      gzip.chomp!
+      %x{#{gzip} -f #{omf}}
+    end
+  end
+end
+
+def do_locales(locale, strip = 'locales/')
+  locale.each do |lf|
+    next if File.directory? lf
+    olf = File.join(InstallOptions.locale_dir, lf.sub(/^#{strip}/, ''))
+    op = File.dirname(olf)
+    FileUtils.makedirs(op, {:mode => 0755, :verbose => true})
+    FileUtils.chmod(0755, op)
+    FileUtils.install(lf, olf, {:mode => 0644, :preserve => true, :verbose => true})
   end
 end
 
@@ -190,6 +205,9 @@ def prepare_installation
     opts.on('--bindir[=OPTIONAL]', 'Installation directory for binaries', 'overrides RbConfig::CONFIG["bindir"]') do |bindir|
       InstallOptions.bindir = bindir
     end
+    opts.on('--localedir[=OPTIONAL]', 'Installation directory for locale information', 'Default /opt/puppetlabs/puppet/share/locale') do |localedir|
+      InstallOptions.localedir = localedir
+    end
     opts.on('--ruby[=OPTIONAL]', 'Ruby interpreter to use with installation', 'overrides ruby used to call install.rb') do |ruby|
       InstallOptions.ruby = ruby
     end
@@ -242,15 +260,19 @@ def prepare_installation
     $operatingsystem = Facter.value :operatingsystem
   end
 
-  if not InstallOptions.configdir.nil?
-    configdir = InstallOptions.configdir
-  elsif $operatingsystem == "windows"
+  if $operatingsystem == "windows"
     begin
+      # populates constants used to specify default Windows directories
       require 'win32/dir'
     rescue LoadError => e
       puts "Cannot run on Microsoft Windows without the win32-process, win32-dir & win32-service gems: #{e}"
       exit -1
     end
+  end
+
+  if not InstallOptions.configdir.nil?
+    configdir = InstallOptions.configdir
+  elsif $operatingsystem == "windows"
     configdir = File.join(Dir::COMMON_APPDATA, "PuppetLabs", "puppet", "etc")
   else
     configdir = "/etc/puppetlabs/puppet"
@@ -294,6 +316,16 @@ def prepare_installation
     bindir = RbConfig::CONFIG['bindir']
   end
 
+  if not InstallOptions.localedir.nil?
+    localedir = InstallOptions.localedir
+  else
+    if $operatingsystem == "windows"
+      localedir = File.join(Dir::PROGRAM_FILES, "Puppet Labs", "Puppet", "puppet", "share", "locale")
+    else
+      localedir = "/opt/puppetlabs/puppet/share/locale"
+    end
+  end
+
   if not InstallOptions.sitelibdir.nil?
     sitelibdir = InstallOptions.sitelibdir
   else
@@ -331,6 +363,7 @@ def prepare_installation
   rundir = join(destdir, rundir)
   logdir = join(destdir, logdir)
   bindir = join(destdir, bindir)
+  localedir = join(destdir, localedir)
   mandir = join(destdir, mandir)
   sitelibdir = join(destdir, sitelibdir)
 
@@ -342,6 +375,7 @@ def prepare_installation
   FileUtils.makedirs(vardir)
   FileUtils.makedirs(rundir)
   FileUtils.makedirs(logdir)
+  FileUtils.makedirs(localedir)
 
   InstallOptions.site_dir = sitelibdir
   InstallOptions.codedir = codedir
@@ -352,6 +386,7 @@ def prepare_installation
   InstallOptions.var_dir = vardir
   InstallOptions.run_dir = rundir
   InstallOptions.log_dir = logdir
+  InstallOptions.locale_dir = localedir
 end
 
 ##
@@ -464,6 +499,7 @@ FileUtils.cd File.dirname(__FILE__) do
   ri    = glob(%w{bin/*.rb lib/**/*.rb}).reject { |e| e=~ /\.(bat|cmd)$/ }
   man   = glob(%w{man/man[0-9]/*})
   libs  = glob(%w{lib/**/*})
+  locales = glob(%w{locales/**/*})
 
   prepare_installation
 
@@ -477,5 +513,6 @@ FileUtils.cd File.dirname(__FILE__) do
   do_bins(bins, InstallOptions.bin_dir)
   do_bins(windows_bins, InstallOptions.bin_dir, 'ext/windows/') if $operatingsystem == "windows" && InstallOptions.batch_files
   do_libs(libs)
+  do_locales(locales)
   do_man(man) unless $operatingsystem == "windows"
 end

data/lib/hiera/puppet_function.rb

@@ -43,37 +43,35 @@ class Hiera::PuppetFunction < Puppet::Functions::InternalFunction
   end
 
   def hiera_no_default(scope, key)
-    post_lookup(scope, key, lookup(scope, key, nil, nil))
+    post_lookup(scope, key, lookup(scope, key, nil, false, nil))
   end
 
   def hiera_with_default(scope, key, default, override = nil)
-    undefined = (@@undefined_value ||= Object.new)
-    result = lookup(scope, key, undefined, override)
-    post_lookup(scope, key, result.equal?(undefined) ? default : result)
+    post_lookup(scope, key, lookup(scope, key, default, true, override))
   end
 
   def hiera_block1(scope, key, &default_block)
-    common(scope, key, nil, default_block)
+    post_lookup(scope, key, lookup(scope, key, nil, false, nil, &default_block))
   end
 
   def hiera_block2(scope, key, override, &default_block)
-    common(scope, key, override, default_block)
+    post_lookup(scope, key, lookup(scope, key, nil, false, override, &default_block))
   end
 
-  def common(scope, key, override, default_block)
-    undefined = (@@undefined_value ||= Object.new)
-    result = lookup(scope, key, undefined, override)
-    post_lookup(scope, key, result.equal?(undefined) ? default_block.call(key) : result)
-  end
-
-  private :common
-
-  def lookup(scope, key, default, override)
-    HieraPuppet.lookup(key, default, scope, override, merge_type)
+  def lookup(scope, key, default, has_default, override, &default_block)
+    unless Puppet[:strict] == :off
+      Puppet.warn_once(:deprecation, self.class.name,
+        "The function '#{self.class.name}' is deprecated in favor of using 'lookup'. See https://docs.puppet.com/puppet/#{Puppet.version}/reference/deprecated_language.html")
+    end
+    lookup_invocation = Puppet::Pops::Lookup::Invocation.new(scope, {}, {})
+    adapter = lookup_invocation.lookup_adapter
+    lookup_invocation.set_global_only unless adapter.global_only? || adapter.has_environment_data_provider?(lookup_invocation)
+    lookup_invocation.set_hiera_v3_location_overrides(override) unless override.nil? || override.is_a?(Array) && override.empty?
+    Puppet::Pops::Lookup.lookup(key, nil, default, has_default, merge_type, lookup_invocation, &default_block)
   end
 
   def merge_type
-    :priority
+    :first
   end
 
   def post_lookup(scope, key, result)

data/lib/hiera/scope.rb

@@ -1,9 +1,12 @@
 class Hiera
   class Scope
-    CALLING_CLASS = "calling_class"
-    CALLING_CLASS_PATH = "calling_class_path"
-    CALLING_MODULE = "calling_module"
-    MODULE_NAME = "module_name"
+    CALLING_CLASS = 'calling_class'.freeze
+    CALLING_CLASS_PATH = 'calling_class_path'.freeze
+    CALLING_MODULE = 'calling_module'.freeze
+    MODULE_NAME = 'module_name'.freeze
+
+    CALLING_KEYS = [CALLING_CLASS, CALLING_CLASS_PATH, CALLING_MODULE].freeze
+    EMPTY_STRING = ''.freeze
 
     attr_reader :real
 
@@ -21,20 +24,15 @@ class Hiera
       else
         ans = @real.lookupvar(key)
       end
+      ans == EMPTY_STRING ? nil : ans
+    end
 
-      if ans.nil? or ans == ""
-        nil
-      else
-        ans
-      end
+    def exist?(key)
+      CALLING_KEYS.include?(key) || @real.exist?(key)
     end
 
     def include?(key)
-      if [CALLING_CLASS, CALLING_CLASS_PATH, CALLING_MODULE].include? key
-        true
-      else
-        @real.exist?(key)
-      end
+      CALLING_KEYS.include?(key) || @real.include?(key)
     end
 
     def catalog

data/lib/puppet/application/face_base.rb

@@ -245,6 +245,10 @@ class Puppet::Application::FaceBase < Puppet::Application
       end
     end
 
+    if @face.deprecated?
+      Puppet.deprecation_warning("'puppet #{@face.name}' is deprecated and will be removed in a future release")
+    end
+
     result = @face.send(@action.name, *arguments)
     puts render(result, arguments) unless result.nil?
     status = true

data/lib/puppet/application/lookup.rb

@@ -297,11 +297,13 @@ Copyright (c) 2015 Puppet Labs, LLC Licensed under the Apache 2.0 License
         type = options.include?(:type) ? Puppet::Pops::Types::TypeParser.singleton.parse(options[:type], scope) : nil
         result = Puppet::Pops::Lookup.lookup(keys, type, options[:default_value], use_default_value, merge_options, lookup_invocation)
         puts renderer.render(result) unless explain
-      rescue Puppet::DataBinding::LookupError
+      rescue Puppet::DataBinding::LookupError => e
+        lookup_invocation.report_text { e.message }
         exit(1) unless explain
       end
-      puts format == :s ? lookup_invocation.explainer.to_s : renderer.render(lookup_invocation.explainer.to_hash) if explain
+      puts format == :s ? lookup_invocation.explainer.explain : renderer.render(lookup_invocation.explainer.to_hash) if explain
     end
+    exit(0)
   end
 
   def generate_scope

data/lib/puppet/application/resource.rb

@@ -166,7 +166,7 @@ Copyright (c) 2011 Puppet Labs, LLC Licensed under the Apache 2.0 License
     require 'tempfile'
     # Prefer the current directory, which is more likely to be secure
     # and, in the case of interactive use, accessible to the user.
-    tmpfile = Tempfile.new('x2puppet', Dir.pwd)
+    tmpfile = Tempfile.new('x2puppet', Dir.pwd, :encoding => Encoding::UTF_8)
     begin
       # sync write, so nothing buffers before we invoke the editor.
       tmpfile.sync = true

data/lib/puppet/data_providers/data_adapter.rb

@@ -1,6 +1,8 @@
 # A DataAdapter adapts an object with a Hash of data
 #
+# TODO: API 5.0, remove this class
 # @api private
+# @deprecated
 class Puppet::DataProviders::DataAdapter < Puppet::Pops::Adaptable::Adapter
   include Puppet::Plugins::DataProviders
 
@@ -11,6 +13,10 @@ class Puppet::DataProviders::DataAdapter < Puppet::Pops::Adaptable::Adapter
   end
 
   def initialize
+    unless Puppet[:strict] == :off
+      Puppet.warn_once(:deprecation, 'Puppet::DataProviders::DataAdapter',
+        'Puppet::DataProviders::DataAdapter is deprecated and will be removed in the next major version of Puppet')
+    end
     @data = {}
   end
 

data/lib/puppet/data_providers/data_function_support.rb

@@ -1,9 +1,16 @@
+# TODO: API 5.0, remove this module
 # @api private
+# @deprecated
 module Puppet::DataProviders::DataFunctionSupport
   def initialize_data(data_key, lookup_invocation)
     name = "#{data_key}::data"
     scope = lookup_invocation.scope
     Puppet::Util::Profiler.profile("Called #{name}", [ :functions, name ]) do
+      unless Puppet[:strict] == :off
+        Puppet.warn_once(:deprecation, 'Puppet::DataProviders::DataFunctionSupport',
+        'Puppet::DataProviders::DataFunctionSupport is deprecated and will be removed in the next major version of Puppet')
+      end
+
       loader = loader(data_key, scope)
       if loader && func = loader.load(:function, name)
         # function found, call without arguments, must return a Hash

data/lib/puppet/data_providers/function_env_data_provider.rb

@@ -8,11 +8,18 @@ module Puppet::DataProviders; end
 # The function is called on demand, and is associated with the compiler via an Adapter. This ensures that the data
 # is only produced once per compilation.
 #
+# TODO: API 5.0, remove this class
 # @api private
+# @deprecated
 class Puppet::DataProviders::FunctionEnvDataProvider < Puppet::Plugins::DataProviders::EnvironmentDataProvider
   include Puppet::DataProviders::DataFunctionSupport
 
   def loader(key, scope)
+    unless Puppet[:strict] == :off
+      Puppet.warn_once(:deprecation, 'Puppet::DataProviders::FunctionEnvDataProvider',
+        'Puppet::DataProviders::FunctionEnvDataProvider is deprecated and will be removed in the next major version of Puppet')
+    end
+
     # This loader allows the data function to be private or public in the environment
     scope.compiler.loaders.private_environment_loader
   end

data/lib/puppet/data_providers/function_module_data_provider.rb

@@ -8,11 +8,17 @@ module Puppet::DataProviders; end
 # The function is called on demand, and is associated with the compiler via an Adapter. This ensures that the data
 # is only produced once per compilation.
 #
+# TODO: API 5.0, remove this class
 # @api private
+# @deprecated
 class Puppet::DataProviders::FunctionModuleDataProvider < Puppet::Plugins::DataProviders::ModuleDataProvider
   include Puppet::DataProviders::DataFunctionSupport
 
   def loader(key, scope)
+    unless Puppet[:strict] == :off
+      Puppet.warn_once(:deprecation, 'Puppet::DataProviders::FunctionModuleDataProvider',
+      'Puppet::DataProviders::FunctionModuleDataProvider is deprecated and will be removed in the next major version of Puppet')
+    end
     scope.compiler.loaders.private_loader_for_module(key)
   end
 end

data/lib/puppet/data_providers/hiera_config.rb

@@ -1,11 +1,13 @@
-require 'pathname'
-require 'puppet/pops/lookup/interpolation'
+require 'puppet/plugins/data_providers'
 
+# @api private
 module Puppet::DataProviders
+  # TODO: API 5.0, remove this class
   # @api private
+  # @deprecated
   class HieraConfig
     include Puppet::Plugins::DataProviders
-    include Puppet::Pops::Lookup::Interpolation
+    include Puppet::Pops::Lookup::LocationResolver
 
     DEFAULT_CONFIG = {
       'version' => 4,
@@ -33,6 +35,11 @@ module Puppet::DataProviders
       end
     end
 
+    def self.config_exist?(config_root)
+      config_path = config_root + 'hiera.yaml'
+      config_path.exist?
+    end
+
     def self.create_config_type
       hierarchy_elem_type_base = 'Struct[{'\
         'backend=>String[1],'\
@@ -58,6 +65,10 @@ module Puppet::DataProviders
     # @param config_root [Pathname] Absolute path to the configuration root
     # @api public
     def initialize(config_root)
+      unless Puppet[:strict] == :off
+        Puppet.warn_once(:deprecation, 'Puppet::DataProviders::HieraConfig',
+          "Use of class Puppet::DataProviders::HieraConfig' is deprecated. Puppet::Pops::Lookup::HieraConfig should be used instead", config_path.to_s)
+      end
       @config_root = config_root
       @config_path = config_root + 'hiera.yaml'
       if @config_path.exist?
@@ -92,7 +103,7 @@ module Puppet::DataProviders
       data_providers = {}
       @config['hierarchy'].each do |he|
         name = he['name']
-        raise Puppet::DataBinding::LookupError, "#{path}: Name '#{name}' defined more than once" if data_providers.include?(name)
+        raise Puppet::DataBinding::LookupError, "#{@config_path}: Name '#{name}' defined more than once" if data_providers.include?(name)
         original_paths = he['paths']
         if original_paths.nil?
           single_path = he['path']
@@ -100,17 +111,27 @@ module Puppet::DataProviders
           original_paths = [single_path]
         end
         paths = original_paths.map { |path| interpolate(path, lookup_invocation, false)}
-        provider_name = he['backend']
-        provider_factory = injector.lookup(nil, service_type, PATH_BASED_DATA_PROVIDER_FACTORIES_KEY)[provider_name]
-        raise Puppet::DataBinding::LookupError, "#{@config_path}: No data provider is registered for backend '#{provider_name}' " unless provider_factory
-
         datadir = @config_root + (he['datadir'] || default_datadir)
-        data_providers[name] = create_data_provider(parent_data_provider, provider_factory, name,
-          provider_factory.resolve_paths(datadir, original_paths, paths, lookup_invocation))
+        provider_name = he['backend']
+        data_providers[name] = case provider_name
+        when 'json', 'yaml'
+          Puppet::Pops::Lookup::DataHashFunctionProvider.new(name, parent_data_provider, "#{provider_name}_data", {},
+            resolve_paths(datadir, original_paths, lookup_invocation, ".#{provider_name}"))
+        else
+          # TODO: Remove support for injected provider factories
+          paths = original_paths.map { |path| interpolate(path, lookup_invocation, false)}
+          provider_factory = injector.lookup(nil, service_type, PATH_BASED_DATA_PROVIDER_FACTORIES_KEY)[provider_name]
+          raise Puppet::DataBinding::LookupError, "#{@config_path}: No data provider is registered for backend '#{provider_name}' " unless provider_factory
+          create_data_provider(parent_data_provider, provider_factory, name, provider_factory.resolve_paths(datadir, original_paths, paths, lookup_invocation))
+        end
       end
       data_providers.values
     end
 
+    def name
+      "hiera version #{version}"
+    end
+
     def create_data_provider(parent_data_provider, provider_factory, name, resolved_paths)
       provider_factory_version = provider_factory.respond_to?(:version) ? provider_factory.version : 1
       if provider_factory_version == 1

data/lib/puppet/data_providers/hiera_env_data_provider.rb

@@ -2,7 +2,9 @@
 require_relative 'hiera_support'
 
 module Puppet::DataProviders
+  # TODO: API 5.0, remove this class
   # @api private
+  # @deprecated
   class HieraEnvDataProvider < Puppet::Plugins::DataProviders::EnvironmentDataProvider
     include HieraSupport
 
@@ -12,6 +14,10 @@ module Puppet::DataProviders
     # @param scope [Puppet::Parser::Scope] the parser scope where the environment is found
     # @return [Pathname] Path to root of the environment
     def provider_root(_, scope)
+      unless Puppet[:strict] == :off
+        Puppet.warn_once(:deprecation, 'Puppet::DataProviders::HieraEnvDataProvider',
+        'Puppet::DataProviders::HieraEnvDataProvider is deprecated and will be removed in the next major version of Puppet')
+      end
       Pathname.new(scope.environment.configuration.path_to_env)
     end
     protected :provider_root

data/lib/puppet/data_providers/hiera_interpolate.rb

@@ -1,3 +1,4 @@
+# TODO: API 5.0, remove this module
 # @deprecated Moved to Puppet::Pops::Lookup::Interpolation
 module Puppet::DataProviders::HieraInterpolate
   include Puppet::Pops::Lookup::Interpolation
@@ -15,6 +16,6 @@ module Puppet::DataProviders::HieraInterpolate
         Puppet.warn_once(:deprecation, 'HieraInterpolate#qualified_lookup', msg)
       end
     end
-    sub_lookup('<unknown key>', Puppet::Pops::Lookup::Invocation.new(nil), segments, value)
+    sub_lookup('<unknown key>', Puppet::Pops::Lookup::Invocation.current, segments, value)
   end
 end

data/lib/puppet/data_providers/hiera_module_data_provider.rb

@@ -2,7 +2,9 @@
 require_relative 'hiera_support'
 
 module Puppet::DataProviders
+  # TODO: API 5.0, remove this class
   # @api private
+  # @deprecated
   class HieraModuleDataProvider < Puppet::Plugins::DataProviders::ModuleDataProvider
     include HieraSupport
 
@@ -14,6 +16,10 @@ module Puppet::DataProviders
     # @raise [Puppet::DataBinder::LookupError] if the given module is can not be found
     #
     def provider_root(module_name, scope)
+      unless Puppet[:strict] == :off
+        Puppet.warn_once(:deprecation, 'Puppet::DataProviders::HieraModuleDataProvider',
+        'Puppet::DataProviders::HieraModuleDataProvider is deprecated and will be removed in the next major version of Puppet')
+      end
       env = scope.environment
       mod = env.modules.find { |m| m.name == module_name }
       raise Puppet::DataBinder::LookupError, "Environment '#{env.name}', cannot find module '#{module_name}'" unless mod

data/lib/puppet/data_providers/hiera_support.rb

@@ -1,6 +1,8 @@
 require_relative 'hiera_config'
 
+# TODO: API 5.0, remove this module
 # @api private
+# @deprecated
 module Puppet::DataProviders::HieraSupport
   def config_path
     @hiera_config.nil? ? 'not yet configured' : @hiera_config.config_path
@@ -17,6 +19,10 @@ module Puppet::DataProviders::HieraSupport
   # @param lookup_invocation [Puppet::Pops::Lookup::Invocation] The current lookup invocation
   # @param merge [Puppet::Pops::MergeStrategy,String,Hash<String,Object>,nil] Merge strategy or hash with strategy and options
   def unchecked_lookup(key, lookup_invocation, merge)
+    unless Puppet[:strict] == :off
+      Puppet.warn_once(:deprecation, 'Puppet::DataProviders::HieraSupport',
+      'Puppet::DataProviders::HieraSupport is deprecated and will be removed in the next major version of Puppet')
+    end
     lookup_invocation.with(:data_provider, self) do
       merge_strategy = Puppet::Pops::MergeStrategy.strategy(merge)
       lookup_invocation.with(:merge, merge_strategy) do

data/lib/puppet/data_providers/json_data_provider_factory.rb

@@ -3,9 +3,15 @@
 require 'json'
 
 module Puppet::DataProviders
+  # TODO: API 5.0, remove this class
   # @api private
+  # @deprecated
   class JsonDataProviderFactory < Puppet::Plugins::DataProviders::FileBasedDataProviderFactory
     def create(name, paths, parent_data_provider)
+      unless Puppet[:strict] == :off
+        Puppet.warn_once(:deprecation, 'Puppet::DataProviders::JsonDataProviderFactory',
+          'Puppet::DataProviders::JsonDataProviderFactory is deprecated and will be removed in the next major version of Puppet')
+      end
       JsonDataProvider.new(name, paths, parent_data_provider)
     end
 
@@ -14,9 +20,15 @@ module Puppet::DataProviders
     end
   end
 
+  # TODO: API 5.0, remove this class
   # @api private
+  # @deprecated
   class JsonDataProvider < Puppet::Plugins::DataProviders::PathBasedDataProvider
     def initialize_data(path, lookup_invocation)
+      unless Puppet[:strict] == :off
+        Puppet.warn_once(:deprecation, 'Puppet::DataProviders::JsonDataProvider',
+          'Puppet::DataProviders::JsonDataProvider is deprecated and will be removed in the next major version of Puppet')
+      end
       JSON.parse(Puppet::FileSystem.read(path, :encoding => 'utf-8'))
     rescue JSON::ParserError => ex
       # Filename not included in message, so we add it here.