[PATCH] Print better info mesages when we change a service

[PATCH] Print better info mesages when we change a service

[Jonatan Schlag]

[-- Attachment #1: Type: text/plain, Size: 1124 bytes --]

Signed-off-by: Jonatan Schlag <jonatan.schlag(a)ipfire.org>
---
 src/functions/functions.service | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/functions/functions.service b/src/functions/functions.service
index 7a12ffd..07bd2cd 100644
--- a/src/functions/functions.service
+++ b/src/functions/functions.service
@@ -45,6 +45,11 @@ service_stop() {
 	assert isset name
 
 	systemctl stop "${name}"
+	local ret=$?
+
+	log INFO "Stopped service '${name}', code=${ret}"
+
+	return ${ret}
 }
 
 service_restart() {
@@ -52,6 +57,11 @@ service_restart() {
 	assert isset name
 
 	systemctl restart "${name}"
+	local ret=$?
+
+	log INFO "Restarted service '${name}', code=${ret}"
+
+	return ${ret}
 }
 
 service_reload() {
@@ -60,7 +70,12 @@ service_reload() {
 
 	if service_status "${name}"; then
 		systemctl reload "${name}"
-		return $?
+		local ret=$?
+
+		log INFO "Reloaded service '${name}', code=${ret}"
+
+		return ${ret}
+
 	else
 		log WARNING "Cannot reload service '${name}' which is currently not running."
 	fi
-- 
2.6.3

Re: [PATCH] Print better info mesages when we change a service

[Michael Tremer]

[-- Attachment #1: Type: text/plain, Size: 1308 bytes --]

Why do we need this?

Best,
-Michael

On Sat, 2018-02-10 at 12:14 +0000, Jonatan Schlag wrote:
> Signed-off-by: Jonatan Schlag <jonatan.schlag(a)ipfire.org>
> ---
>  src/functions/functions.service | 17 ++++++++++++++++-
>  1 file changed, 16 insertions(+), 1 deletion(-)
> 
> diff --git a/src/functions/functions.service b/src/functions/functions.service
> index 7a12ffd..07bd2cd 100644
> --- a/src/functions/functions.service
> +++ b/src/functions/functions.service
> @@ -45,6 +45,11 @@ service_stop() {
>  	assert isset name
>  
>  	systemctl stop "${name}"
> +	local ret=$?
> +
> +	log INFO "Stopped service '${name}', code=${ret}"
> +
> +	return ${ret}
>  }
>  
>  service_restart() {
> @@ -52,6 +57,11 @@ service_restart() {
>  	assert isset name
>  
>  	systemctl restart "${name}"
> +	local ret=$?
> +
> +	log INFO "Restarted service '${name}', code=${ret}"
> +
> +	return ${ret}
>  }
>  
>  service_reload() {
> @@ -60,7 +70,12 @@ service_reload() {
>  
>  	if service_status "${name}"; then
>  		systemctl reload "${name}"
> -		return $?
> +		local ret=$?
> +
> +		log INFO "Reloaded service '${name}', code=${ret}"
> +
> +		return ${ret}
> +
>  	else
>  		log WARNING "Cannot reload service '${name}' which is
> currently not running."
>  	fi

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]