Code review comment for lp:~james-page/charm-helpers/refactor-service-control

Revision history for this message
Matthew Wedgwood (mew) wrote :

For service_reload, I don't think it's safe to assume that a service should be restarted if reload fails. I'm at a loss to find an example, but services whose "reload" commands handle invalid configuration gracefully might be brought down on a "restart."

« Back to merge proposal