(EZ-31) Add pre_start_action to start scripts

[dankreek]

In order for the install command to work from a Puppet server build I needed to add the PermissionsStartOnly=true line which instructs systemd to run the ExecStartPre commands as root before switching to the service user

[haus]

[haus]

[dankreek]

This the most exciting code review I've ever had.

[haus]

@dankreek have you tried using it for the rundir thingy thing?

[shrug]

Do we have any post_start_actions that we need to account for that want to run as the service user?

[haus]

@shrug you mean because of PermissionsStartOnly=true ?

[shrug]

Yes

[dankreek]

I've functionally reviewed this on Cent7 and it went well, I'm going to try it on Ubuntu 14.04 and Cent6 today.

[haus]

I'm 👍 and @shrug i'm not away of any post_start_actions that need to run as the service user. For sysv they are all running as root already anyway.

[dankreek]

Yeah, I was kind of wondering if we should maybe need a flag for "run post/pre stuff as root." It could easily be added later if needed.

[haus]

@dankreek because sysv pre and post stuff will always run as root, I'm not going to worry about it for a while.

[shrug]

👍

[dankreek]

@haus you want to push that button?