Arch: systemd service file does not start salt-master (2014.7.0rc6)

[cro]

While experimenting with Arch this evening I discovered that for RC6, systemctl start salt-master will stick. Changing the Type in the .service file from notify to simple will resolve this, but I am pretty sure this is not the 'right' solution.

[techhat]

I've seen this value flip-flop a lot on Arch. @terminalmage and @gtmanfred, what is the official word on this?

[gtmanfred]

This has to be solved in the process management, it it changed again since I fixed it, then it probably broke. I will take a look tomorrow

[gtmanfred]

It should absolutely be notify, please make sure you have python2-systemd installed

[s0undt3ch]

On October 26, 2014 3:21:23 AM GMT, Daniel Wallace notifications@github.com wrote:

It should absolutely be notify, please make sure you have
python2-systemd installed

---

Reply to this email directly or view it on GitHub:
#16912 (comment)

So python2-systemd should probably also be installed by the bootstrap script right?
Pedro Algarvio @ Phone

[terminalmage]

Yeah, if installed via git then this dependency would not be installed automatically, so the bootstrap script would need to handle this.

Is that how @cro installed it?

[cro]

Yes, I installed via bootstrap.

C. R. Oldham, Engineer, SaltStack
cr@saltstack.com

On Oct 26, 2014, at 07:24, Erik Johnson notifications@github.com wrote:

Yeah, if installed via git then this dependency would not be installed automatically, so the bootstrap script would need to handle this.

Is that now @cro installed it?

—
Reply to this email directly or view it on GitHub.

[s0undt3ch]

OK. I'll get a fix fix this today

[rallytime]

Thanks for the the fix @s0undt3ch! I just tested this and it is fixed. Do we need to get this change/update the bootstrap script for the next RC?

[rallytime]

@s0undt3ch I was asked by @terminalmage to check this in Fedora 20 as well, since it also uses systemd and the salt-master service is also hanging for Fedora 20.

[terminalmage]

Yeah, the idea being that there is a separate function to install fedora deps. Though whether or not to test this could be obviated by looking at the code.

/me looks...

[terminalmage]

I don't see it there: https://github.com/saltstack/salt-bootstrap/blob/develop/bootstrap-salt.sh#L2429

[s0undt3ch]

I'll have to fix Fedora and a any other systemd distro...

[s0undt3ch]

The new bootstrap script (already submitted PR's) now install the python bindings of systemd on git installs(Arch and Fedora)

[rallytime]

Thanks @s0undt3ch! :)

[terminalmage]

@s0undt3ch FTW!

[cachedout]

@s0undt3ch Could you link the PRs so we can close this when they're merged. Thanks!

[s0undt3ch]

Nice!

[rallytime]

I've confirmed that this has been fixed on Arch and Fedora now.