Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deep_merge_options are not set when merge_behavior => deeper #174

Closed
antaflos opened this issue Dec 5, 2016 · 1 comment
Closed

deep_merge_options are not set when merge_behavior => deeper #174

antaflos opened this issue Dec 5, 2016 · 1 comment

Comments

@antaflos
Copy link
Contributor

antaflos commented Dec 5, 2016

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 4.8.1
  • Ruby: 2.1.9p490
  • Distribution: Ubuntu 16.04
  • Module version: 2.3.0

How to reproduce (e.g Puppet code you use)

  class { '::hiera':
    create_keys        => false,
    master_service     => 'puppetserver',
    manage_package     => true,
    provider           => 'puppet_gem',
    eyaml              => true,
    eyaml_version      => 'latest',
    eyaml_datadir      => '/etc/puppetlabs/code/environments/%{::environment}/hieradata',
    eyaml_extension    => 'yaml',
    eyaml_gpg          => true,
    ruby_gpg_version   => 'latest',
    merge_behavior     => 'deeper',
    deep_merge_version => 'latest',
    backends           => [
      'eyaml',
    ],
    backend_options    => {
      'eyaml'      => {
        'gpg_gnupghome'    => '/etc/puppetlabs/puppet/keys/gpg',
      },
    },
    hierarchy          => [
      'nodes/%{::trusted.certname}',
      'common',
    ],
    deep_merge_options => {
      'knockout_prefix' => '--',
    },
  }

What are you seeing

hiera.yaml does not contain

:deep_merge_options:
  :knockout_prefix: --

What behaviour did you expect instead

hiera.yaml should contain

:deep_merge_options:
  :knockout_prefix: --

Any additional information you'd like to impart

I have submitted PR #172 to fix this.

@antaflos
Copy link
Contributor Author

antaflos commented Dec 6, 2016

With PR #172 merged (thanks @igalic!) this can be closed.

@antaflos antaflos closed this as completed Dec 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant