Blame SOURCES/bz2015789-gcp-ilb-2-only-check-log_cmd-if-log-enabled.patch

a55b34
From 14576f7ca02fb0abff188238ac019e88ab06e878 Mon Sep 17 00:00:00 2001
a55b34
From: Oyvind Albrigtsen <oalbrigt@redhat.com>
a55b34
Date: Tue, 9 Nov 2021 11:49:36 +0100
a55b34
Subject: [PATCH] gcp-ilb: only check if log_cmd binary is available if
a55b34
 log_enable is true
a55b34
a55b34
---
a55b34
 heartbeat/gcp-ilb | 9 +++++----
a55b34
 1 file changed, 5 insertions(+), 4 deletions(-)
a55b34
a55b34
diff --git a/heartbeat/gcp-ilb b/heartbeat/gcp-ilb
a55b34
index 48dc3ac4e..f84f373b7 100755
a55b34
--- a/heartbeat/gcp-ilb
a55b34
+++ b/heartbeat/gcp-ilb
a55b34
@@ -37,7 +37,7 @@ if  type "socat" > /dev/null 2>&1; then
a55b34
 	OCF_RESKEY_cat_default="socat"
a55b34
 else 
a55b34
 	OCF_RESKEY_cat_default="nc"
a55b34
-fi;
a55b34
+fi
a55b34
 
a55b34
 
a55b34
 : ${OCF_RESKEY_cat=${OCF_RESKEY_cat_default}}
a55b34
@@ -53,7 +53,7 @@ pidfile="/var/run/$OCF_RESOURCE_INSTANCE.pid"
a55b34
 
a55b34
 
a55b34
 #Validate command for logging
a55b34
-if [ $OCF_RESKEY_log_enable = "true" ]; then
a55b34
+if ocf_is_true "$OCF_RESKEY_log_enable"; then
a55b34
     if  type $OCF_RESKEY_log_cmd > /dev/null 2>&1; then
a55b34
 	logging_cmd="$OCF_RESKEY_log_cmd $OCF_RESKEY_log_params"
a55b34
 	ocf_log debug "Logging command is: \'$logging_cmd\' "
a55b34
@@ -61,7 +61,7 @@ if [ $OCF_RESKEY_log_enable = "true" ]; then
a55b34
 	OCF_RESKEY_log_enable="false"
a55b34
 	ocf_log err "\'$logging_cmd\' is invalid. External logging disabled."
a55b34
 		
a55b34
-    fi;
a55b34
+    fi
a55b34
 fi
a55b34
 
a55b34
 
a55b34
@@ -285,7 +285,8 @@ ilb_stop() {
a55b34
 
a55b34
 ilb_validate() {
a55b34
 	check_binary "$OCF_RESKEY_cat"
a55b34
-	check_binary "$OCF_RESKEY_log_cmd"
a55b34
+
a55b34
+	ocf_is_true "$OCF_RESKEY_log_enable" && check_binary "$OCF_RESKEY_log_cmd"
a55b34
 
a55b34
 	if ! ocf_is_decimal "$OCF_RESKEY_port"; then
a55b34
 		ocf_exit_reason "$OCF_RESKEY_port is not a valid port"