Jira (HI-6) Hiera needs to validate the structure of its config file

[Charlie Sharpsteen (JIRA)]

Title: Message Title

Charlie Sharpsteen commented on an issue   Re: Hiera needs to validate the structure of its config file Level of impact (not sure whether to make this public, so going with private): YAML is prone to very subtle errors that can completely change the structure of the loaded config. This is also compounded by the whitespace-sensitive nature of the format. In this case, it took two support engineers two days of comparing notes and test environments to notice a misplaced hyphen as Hiera wasn't visibly complaining about loading its config — it was just failing to provide a customer with correct data for their manifests. Much debugging time was spent probing the data binding system in Puppet for problems. In the case of the `datadir` setting, some very cursory structural validation would have saved the day: @config[:backends].each do |backend| raise Error unless @config[backend][:datadir].is_a? String end Add Comment   Hiera / HI-6 Hiera needs to validate the structure of its config file Hiera does have a [validate method in config.rb|https://github.com/puppetlabs/hiera/blob/1.2.1/lib/hiera/config.rb#L51-L62] that is applied to the data structure loaded from `hiera.yaml`. Unfortunately, this validator currently only traps errors arising from missing the `deep_merge` gem. Hiera assumes that the config file will follow a certain structur... This message was sent by Atlassian JIRA (v6.1.4#6159-sha1:44eaede) -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to puppet-bugs+unsubscr...@googlegroups.com. To post to this group, send email to puppet-bugs@googlegroups.com. Visit this group at http://groups.google.com/group/puppet-bugs. For more options, visit https://groups.google.com/groups/opt_out.