The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Bug #11787

Hiera_hash should correctly check for scope

Added by Hunter Haugen over 2 years ago. Updated almost 2 years ago.

Status:Merged - Pending ReleaseStart date:01/05/2012
Priority:NormalDue date:
Assignee:Kelsey Hightower% Done:

0%

Category:hiera-puppet
Target version:1.0.0
Keywords: Affected Hiera Version:
Branch:https://github.com/puppetlabs/hiera-puppet/pull/33

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com

This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.


Description

hiera_hash() line 24 if self.respond_to?("{}") should actually be if self.respond_to?("[]") because this is checking for a puppet scope, and thus able to access facts and variables via []

History

#1 Updated by Kelsey Hightower almost 2 years ago

  • Category set to hiera-puppet
  • Status changed from Unreviewed to Accepted
  • Assignee set to Kelsey Hightower
  • Target version set to 1.0.0

#2 Updated by Kelsey Hightower almost 2 years ago

  • Status changed from Accepted to In Topic Branch Pending Review
  • Branch set to https://github.com/puppetlabs/hiera-puppet/pull/33

I’ve updated all the hiera parser functions not to use Hiera::Scope. Starting with Puppet ~2.7.10, the scope object allow variable lookups via [].

This specific issue should be fixed here: https://github.com/puppetlabs/hiera-puppet/pull/33/files#diff-21

#3 Updated by Kelsey Hightower almost 2 years ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release

Also available in: Atom PDF