From 6f510cc38149a1e5b98ac5a4f6f4a21429298e06 Mon Sep 17 00:00:00 2001 From: Matt Riben Date: Thu, 6 Jun 2024 00:43:37 -0500 Subject: [PATCH] chore: adjust debug logs for updating a dependent resource Signed-off-by: Matt Riben --- .../dependent/AbstractDependentResource.java | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/dependent/AbstractDependentResource.java b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/dependent/AbstractDependentResource.java index ea1e020bfb..566f0f7625 100644 --- a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/dependent/AbstractDependentResource.java +++ b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/dependent/AbstractDependentResource.java @@ -71,10 +71,17 @@ protected ReconcileResult reconcile(P primary, R actualResource, Context

c logForOperation("Updating", primary, desired); var updatedResource = handleUpdate(actualResource, desired, primary, context); return ReconcileResult.resourceUpdated(updatedResource); + } else { + log.debug("Update skipped for dependent {} as it matched the existing one", + actualResource instanceof HasMetadata + ? ResourceID.fromResource((HasMetadata) actualResource) + : getClass().getSimpleName()); } } else { - log.debug("Update skipped for dependent {} as it matched the existing one", - actualResource); + log.debug("Update skipped for dependent {} implement Updater interface to modify it", + actualResource instanceof HasMetadata + ? ResourceID.fromResource((HasMetadata) actualResource) + : getClass().getSimpleName()); } } } else {