From 4861b02ee79e6bc64706f8c4e37fb31bb0647652 Mon Sep 17 00:00:00 2001 From: "omid.tavakoli" Date: Mon, 24 Nov 2025 15:53:23 +0100 Subject: [PATCH 1/2] increase timeout and added some log lines --- .../plugins/commands/KubernetesGetDeploymentSpecCommand.java | 5 +++-- .../freenow/sauron/plugins/utils/KubernetesConstants.java | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/commands/KubernetesGetDeploymentSpecCommand.java b/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/commands/KubernetesGetDeploymentSpecCommand.java index 2340e2f8..8a44167d 100644 --- a/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/commands/KubernetesGetDeploymentSpecCommand.java +++ b/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/commands/KubernetesGetDeploymentSpecCommand.java @@ -29,7 +29,7 @@ public Optional getDeploymentSpec(String serviceLabel, Kuberne try { String labelSelector = String.format("%s=%s", serviceLabel, service); - log.debug("Filtering deployment {} using selector {}", resource, labelSelector); + log.debug("Filtering resource {} using selector {}", resource, labelSelector); var deployments = createAppsV1Api(client).listNamespacedDeployment( K8S_DEFAULT_NAMESPACE, K8S_PRETTY_OUTPUT, @@ -65,9 +65,10 @@ public Optional getDeploymentSpec(String serviceLabel, Kuberne } catch (ApiException ex) { - log.error(ex.getMessage(), ex); + log.error("getDeploymentSpec failed '{}'", ex.getMessage(), ex); } + log.warn("No deployment returned for service '{}'", service); return Optional.empty(); } } diff --git a/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/utils/KubernetesConstants.java b/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/utils/KubernetesConstants.java index 66fdcb47..3d738497 100644 --- a/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/utils/KubernetesConstants.java +++ b/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/utils/KubernetesConstants.java @@ -7,7 +7,7 @@ public final class KubernetesConstants { public static final String K8S_DEFAULT_NAMESPACE = "default"; - public static final int K8S_API_TIMEOUT_SECONDS = 5; + public static final int K8S_API_TIMEOUT_SECONDS = 30; public static final String K8S_PRETTY_OUTPUT = Boolean.FALSE.toString(); } From 8d3371e057ae7b59eb5bc811087aefa576f65502 Mon Sep 17 00:00:00 2001 From: "omid.tavakoli" Date: Tue, 25 Nov 2025 08:27:54 +0100 Subject: [PATCH 2/2] logging level changed --- .../plugins/commands/KubernetesGetDeploymentSpecCommand.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/commands/KubernetesGetDeploymentSpecCommand.java b/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/commands/KubernetesGetDeploymentSpecCommand.java index 8a44167d..900258e9 100644 --- a/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/commands/KubernetesGetDeploymentSpecCommand.java +++ b/plugins/kubernetesapi-report/src/main/java/com/freenow/sauron/plugins/commands/KubernetesGetDeploymentSpecCommand.java @@ -26,9 +26,9 @@ protected AppsV1Api createAppsV1Api(ApiClient client) public Optional getDeploymentSpec(String serviceLabel, KubernetesResources resource, String service, ApiClient client) { + String labelSelector = String.format("%s=%s", serviceLabel, service); try { - String labelSelector = String.format("%s=%s", serviceLabel, service); log.debug("Filtering resource {} using selector {}", resource, labelSelector); var deployments = createAppsV1Api(client).listNamespacedDeployment( K8S_DEFAULT_NAMESPACE, @@ -68,7 +68,7 @@ public Optional getDeploymentSpec(String serviceLabel, Kuberne log.error("getDeploymentSpec failed '{}'", ex.getMessage(), ex); } - log.warn("No deployment returned for service '{}'", service); + log.warn("No deployment returned for service '{}' using selector {}", service, labelSelector); return Optional.empty(); } }