Use systemd on RHEL/CentOS 7

[joerocklin]

On a minimal CentOS 7 system using just systemd (without initscripts installed) the service setup fails because /sbin/service isn't installed. The systemd service definitions on CentOS 7 correctly use the graceful command for the reload process.

[svanzoest]

This looks good to me, can you update the serverspec and chefspec to match?
Do we need the provider Systemd line? if chef already does this correctly, we would not need to call it out explicitly.

[drpebcak]

This will break Amazon Linux support #302

Amazon Linux does not use systemd, but it does return rhel for platform_family

[svanzoest]

@drpebcak yes leaving that to chef is better rather than hard coding it.

[joerocklin]

Does RHEL/etc < 7 or Amazon need the reload command defined this way external to the package-provided init scripts? Should all of the case statements just be removed?

[drpebcak]

I believe the Chef::Provider::Service::Init::Redhat class still looks for /sbin/service... not 100% sure though.

[svanzoest]

@joerocklin this is because redhat (and fedora) do not use graceful, but rather use restart. See #160 among others. It seems like they fixed it in RHEL7, which means we can guard it and only use it on lower versions.

We can not decide upon a server provider and just leave that decision to chef itself. It is much better at that and it is important for that to be consistent across cookbooks.

[svanzoest]

where are with this one @drpebcak ?

[drpebcak]

Not sure what we wanted to do for this. We could say:

if node['platform_version'].to_f < 7.0 || node['platform'] == 'amazon' 
  reload_command '/sbin/service httpd graceful'
else
  provider Chef::Provider::Service::Systemd
end

[svanzoest]

@drpebcak I wouldn't even specify the provider, just let chef handle that.

if node['platform_version'].to_f < 7.0 || node['platform'] == 'amazon' 
  reload_command '/sbin/service httpd graceful'
end

[drpebcak]

@svanzoest would that work? I thought chef only recognizes one provider per platform (chef/chef#1717). I think we would still have to override it if we wanted to use systemd.

[drpebcak]

I'm not sure if this was implemented yet either, but this could also potentially fix this problem.

[svanzoest]

@drpebcak It was added in chef 12 and chef-client 3.8.2 (via chef-boneyard/chef-client#190)

[drpebcak]

Do you think we could let newer chef releases pick the provider automatically, but hard code it in some places for backwards compatibility?

I think that'd be the best way to go.

[svanzoest]

I guess we can but I do not know what we would hardcode and what it would buy us. This really not a cookbook issue as each cookbook would have the same issue.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.