Make HEAP and JMX OPTS configurable

[confluentinc]

Make HEAP and JMX OPTS configurable

[cotedm]

Looks like we need to add a way to pass the environment variables into the systemd services. This issue will be used to make that happen. The restart and uninstall roles I'll open new issues for, makes sense to me!

[OneCricketeer]

@cotedm

There was KAKFA-1566, not sure what happened to it, but it would enable putting EnvironmentFile as a systemd directive.

Otherwise, you would need to template out the systemd scripts to add something like

{% for k,v in service.environment.items() %}
Environment={{k}}={{v}}
{% endfor %}

[OneCricketeer]

The other option would just template the EnvironmentFile in this project, then use lineinfile, and replacing [Service] with [Service]\nEnvironmentFile={{service}}-env.sh. And make sure that is idempotent.

[cotedm]

@Cricket007 yes that issue appears abandoned, so probably not a good option near term. I'm exploring if we can do a drop in and continue using the Confluent provided systemd scripts or use the systemctl edit style with ansible in some way. It seems the ansible systemd module doesn't support the environment or environmentfile directive. If that doesn't work out, I think templating the environment file and then referencing it from the on systemd scripts on the fly is the only way to go. A bit uglier.