[18:17:55] <Eury> Hey guys, I'm using the puppet module for Elasticsearch, and I had a question : I get a dependency cycle when I try to set up a template.  Elasticsearch_template[fubar] => Elasticsearch::Template[fubar] => Class[Elasticsearch] => Elasticsearch::Template[fubar] => Elasticsearch_template[fubar]. I tried with hiera, and without, but in both cases I get the same error. Any ideas of why ?
[18:18:59] <leothrix> Eury: what version of Puppet, and do you have an example manifest in a gist/etc.?
[18:20:14] <leothrix> Eury: also the version of the module would be helpful to know
[18:22:43] <Eury> I'm using the last version of the module, I pulled it a week ago I believe. The version of puppet is the 4.8.0. I also found a similar case opened back then regarding instance.pp : https://github.com/elastic/puppet-elasticsearch/issues/150#issuecomment-53496327
[18:22:43] <Eury> And I see the same kind of line in template.pp : https://github.com/elastic/puppet-elasticsearch/blob/master/manifests/template.pp#L99
[18:23:44] <Eury> it has been changed since in instance.pp for : https://github.com/elastic/puppet-elasticsearch/blob/master/manifests/instance.pp#L158
[18:31:04] <leothrix> Eury: I'm gonna kick off an acceptance test for the template tests manually, but using templates with instances runs regularly and passes on most distros, and the issue you mentioned references some pretty old behavior, so it seems like it shouldn't be happening - can you perhaps post a cleaned-up manifest as an example somewhere I could try running?
[18:32:20] <leothrix> Eury: I just ran the template tests (https://github.com/elastic/puppet-elasticsearch/blob/a3f69ccaa4fcb2398c243e1937155186905f1bcb/spec/acceptance/003_template_spec.rb) against Puppet 4.10 and things work without dependency cycles, so I don't see anything glaring
[18:32:45] <Eury> leothrix: Ok trying to get you that
[18:36:32] <leothrix> Eury: Thanks! For context, the Puppet module has _lots_ of unit and acceptance tests to verify things work, so really big bugs like dependency cycle bugs should 100% be caught in testing, and I haven't seen any lately
[18:36:51] <Eury> leothrix: https://pastebin.com/RraLHT1e is that enough ?
[18:38:15] <leothrix> Eury: should be, let me test some things out
[18:57:30] <leothrix> Eury: yep, it's a bug. I created an issue for it and a workaround if you need to get around it in the meantime: https://github.com/elastic/puppet-elasticsearch/issues/855
[18:59:29] <Eury> leothrix: Alright ! Thanks !