Background
The issue that we have is precisely cemented in the `electrical-elasticsearch` Puppet Module.
To make things work the way we need them to, we have a few options:
- Fork the Module and push forward
- This has the obvious issue of needing to re-integrate all the time. It's not a horrible approach but not super exciting either as time presses on.
- This does technically have the advantage of easiest to attempt to push changes upstream. We can't push SIMP-specific changes, but we can push bug fixes, etc...
- Create a module named simp_elasticsearch which is just a profile set of the elasticsearch module
- This is good for the community but rapidly grows into a lot of pain in terms of figuring out what's going on as well as massive amounts of module creep all over your Environments
- Shove everything into the simp module
- I hate this the most because, as Chris and Kendall both pointed out, you will then need to push out an update of the simp module for minor changes that don't actually affect everyone. This may hurt upgrade capabilities for downstream users.