IntenseWebs/freeipa
3
0
Fork 0
mirror of https://salsa.debian.org/freeipa-team/freeipa.git synced 2024-12-24 08:00:02 -06:00
Code Issues Packages Projects Releases Wiki Activity

Log service start/stop/restart message

Browse Source 
It wasn't apparent in the logs if a service stop or restart
was complete so in the case of a hang it wasn't obvious which
service was responsible. Including start here for completeness.

https://pagure.io/freeipa/issue/7436

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
Reviewed-By: Christian Heimes <cheimes@redhat.com>
This commit is contained in:
Rob Crittenden 2018-04-27 16:47:30 -04:00 committed by Christian Heimes
parent 3d30cf6034
commit 6856a9f46c
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
ipaplatform/base/services.py
View File

@ -29,6 +29,7 @@ import os
import json import json
import time import time
import collections import collections
import logging
import warnings import warnings
import six import six
@ -36,6 +37,9 @@ import six
from ipapython import ipautil from ipapython import ipautil
from ipaplatform.paths import paths from ipaplatform.paths import paths
logger = logging.getLogger(__name__)
# Canonical names of services as IPA wants to see them. As we need to have # Canonical names of services as IPA wants to see them. As we need to have
# *some* naming, set them as in Red Hat distributions. Actual implementation # *some* naming, set them as in Red Hat distributions. Actual implementation
# should make them available through knownservices.<name> and take care of # should make them available through knownservices.<name> and take care of
@ -289,6 +293,7 @@ class SystemdService(PlatformService):
super(SystemdService, self).stop( super(SystemdService, self).stop(
instance_name, instance_name,
update_service_list=update_service_list) update_service_list=update_service_list)
logger.debug('Stop of %s complete', instance)
def start(self, instance_name="", capture_output=True, wait=True): def start(self, instance_name="", capture_output=True, wait=True):
ipautil.run([paths.SYSTEMCTL, "start", ipautil.run([paths.SYSTEMCTL, "start",
@ -303,6 +308,8 @@ class SystemdService(PlatformService):
super(SystemdService, self).start( super(SystemdService, self).start(
instance_name, instance_name,
update_service_list=update_service_list) update_service_list=update_service_list)
logger.debug('Start of %s complete',
self.service_instance(instance_name))
def _restart_base(self, instance_name, operation, capture_output=True, def _restart_base(self, instance_name, operation, capture_output=True,
wait=False): wait=False):
@ -313,6 +320,8 @@ class SystemdService(PlatformService):
if wait and self.is_running(instance_name): if wait and self.is_running(instance_name):
self.wait_for_open_ports(self.service_instance(instance_name)) self.wait_for_open_ports(self.service_instance(instance_name))
logger.debug('Restart of %s complete',
self.service_instance(instance_name))
def reload_or_restart(self, instance_name="", capture_output=True, def reload_or_restart(self, instance_name="", capture_output=True,
wait=True): wait=True):