elasticsearch.yml content overwritten about puppet-elasticsearch

[electrical]

Hi,

the config file is managed indeed by the module.
any reason for not managing it?

[ctrlaltdel]

Hi Electrical,

Sorry for the incomplete explanation. The trouble is that after this puppet run the file /etc/elasticsearch/elasticsearch.yml only contained " ### MANAGED BY PUPPET ###\n" whereas I expected it to contain the actual configuration.

From the puppet agent log, you can see that the resource named Stage[main]/Elasticsearch::Config/File[/etc/elasticsearch/elasticsearch.yml] has its content changed twice. My conclusion is that the bug comes from Puppet interpreting the ERB template content in two different ways, the first time with access to the settings variable but not the second time.

Here's what the applied recipe contains.

  class { 'elasticsearch':
    pkg_source  => "/tmp/elasticsearch-${es_version}.deb",
    require     => [ Package['openjdk-6-jre-headless'], Exec['download_elasticsearch'] ],
    autoupgrade => true,
    config      => {
      'network' => {
        'host'  => '127.0.0.1',
      }
    }
  }

I hope that it now makes sense?

[electrical]

strange that it would be an issue. i do test against 2.7.x ( 2.7.23 in this case ) and get no errors at all.
Also the config file resource is only managed once.
There is also only 1 event scheduling the refresh of the service.

but i accept the patch as it didn't break anything.