Enhance exception handling of ActionCommands

Fix #337 Catches exceptions in the Action excec method. Otherwise can the setup and teardown methods are interruppted and the kafka topics are not removed after an experiment.

fix NullPointer exception by FindAndFirst() by checking for failture first enhance normal logging in case of problematic commands

theodolite/src/main/kotlin/theodolite/benchmark/Action.kt

@@ -4,9 +4,12 @@ import com.fasterxml.jackson.annotation.JsonInclude
 import com.fasterxml.jackson.databind.annotation.JsonDeserialize
 import io.fabric8.kubernetes.client.NamespacedKubernetesClient
 import io.quarkus.runtime.annotations.RegisterForReflection
+import mu.KotlinLogging
 import theodolite.util.ActionCommandFailedException
 import theodolite.util.Configuration
 
+private val logger = KotlinLogging.logger {}
+
 @JsonDeserialize
 @RegisterForReflection
 @JsonInclude(JsonInclude.Include.NON_NULL)
@@ -16,15 +19,22 @@ class Action {
     lateinit var exec: Command
 
     fun exec(client: NamespacedKubernetesClient) {
-        val exitCode = ActionCommand(client = client)
-            .exec(
-                matchLabels = selector.pod.matchLabels,
-                container = selector.container,
-                timeout = exec.timeoutSeconds,
-                command = exec.command
-        )
+        try{
+            val exitCode = ActionCommand(client = client)
+                .exec(
+                    matchLabels = selector.pod.matchLabels,
+                    container = selector.container,
+                    timeout = exec.timeoutSeconds,
+                    command = exec.command
+                )
             if(exitCode != 0){
-            throw ActionCommandFailedException("Error while executing action, finished with exit code $exitCode")
+                logger.info("Actions exit code was not zero instead: $exitCode")
+                logger.info("Selector: ${selector.pod.matchLabels}, container: ${selector.container}")
+                logger.info("Command: ${exec.command.joinToString(separator = " ")}")
+            }
+        }
+        catch (e: Exception){
+            logger.info { e.stackTraceToString() }
         }
     }
 }