fix service handling (should be compatible with most Linux distributi…

[hoonetorg]

…ons nowadays)

[hoonetorg]

This works also with systemd on el7 distries. service mysql start/stop will be redirected to systemctl start/stop mysql. should be the same for Debian, Ubuntu 16.04+

[deviantony]

Thanks for the PR but I'm not sure if I should merge it because I think it can cause issues on older distributions.

[hoonetorg]

I understand your concerns about backward compatibility. On the other hand all major distributions switched to systemd now and there restoring backups with pyxtrabackup must be broken, am I wrong? The "service" command can be used on sysv and systemd as I said. A colleague told me a Debian Lenny would not have a service command (I did not verify :) ).

Probably it's better to

• check, which init system is used and create the appropriate command then or
• fail restoration if a mysql server process runs(then the user must manually stop mysql process before restoration) -> this is then also not backward compatible, hmm

For me it is fixed (I compiled an rpm for el7 with the fix and put it in a repo :) )

[deviantony]

I've started to debate about this in #32

And from that comment https://github.com/deviantony/xtrabackup-scripts/issues/32#issuecomment-159010689

I do not want to add more code to manage service handling depending on the system.

In order to work with the majority of systems, I think that it would be better to let the user manage the service by himself. I'd accept a PR to remove the service management from the tool and let the user manage the service with an updated procedure.

In the meantime, I'll merge your PR as, as you said, most new distros should support it.

[deviantony]

Released and pushed to pypi.

[hoonetorg]

:+1: