Change also service_name for Fedora

[paramite]

On Fedora-20+ mysqld service does not work any more and has been
replaced by mariadb service.

This patch has been tested both on Fedora 19 and Fedora 20.

[igalic]

Is this the same as #420 only with a sensible commit message?

[igalic]

It appears one way or another you'll have to rebase this.

[igalic]

Is this also true for RHEL 7?

[darth30joker]

I don't have RHEL 7

[igalic]

The answer is: Yes. https://access.redhat.com/site/documentation/en-US/Red_Hat_Enterprise_Linux/7-Beta/html/7.0_Release_Notes/chap-web-servers-and-servces.html - MariaDB is the default MySQL on RHEL7.

[paramite]

@igalic, basicaly you are right, but note that #420 does not fix the service_name issue at all (without the fix this module is useless on Fedora-20+). It only changes log file accordingly. I did not have problems with running mariadb with /var/log/mysqld.log, but I can add this to my patch, if you want me to do it.

Do you want me to implement fix for RHEL-7 too in this patch?

[igalic]

So, neither this, nor #420 are complete then, or compatible between each other?

@paramite yes, please. And also, rebase, please.

[darth30joker]

@paramite @igalic As I can see, default log file is located in /var/log/mariadb/mariadb.log. We'd better keep it there.
What about it?

[igalic]

That's exactly what I meant with my previous comment, "both are incomplete"

[darth30joker]

I just got a RHEL-7 ISO, and I will try my fix on RHEL-7 then.

[paramite]

@igalic, so I rebased and implemented also RHEL-7+ changes. I don't think that changing a log file path is necessary. Using the log file works and don't need any additional changes.

[igalic]

I have no opinion on that, because I don't use Fedora/RHEL/CentOS. That's why I asked you / @kingheaven - and you two can't agree.

I may have to ask higher powers for their opinion: pinging @apenney. @apenney to the MySQL room please!

[paramite]

@igalic: So after playing a bit more with mariadb on F20 I realized that log file is issue after all, so I added @kingheaven's patch to my patch.

[paramite]

Tested on Fedora-20, RHEL-6.5, RHEL-7

[paramite]

Not sure why Travis CI is failing on rake installation :/.

[igalic]

it's failing due to an issue I already fixed in #493
Unfortunately, I have no permissions to kick the build off again, but we can be pretty sure it'll work on the other platforms just as well as it does on ruby 1.9

[paramite]

Ok great, so can we move forward with this before yet another RHEL-7/Fedora-20 pull request will be created by another developer? This pull request contains code which is contained in both #494 and #420.

[igalic]

VERY WELL THEN! Merged,.

[paramite]

Thanks a lot :).