Skip to content

Commit b55ec0e

Browse files
committed
feat: add logging
1 parent 62d4539 commit b55ec0e

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/dependent/AbstractDependentResource.java

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,12 @@
22

33
import io.fabric8.kubernetes.api.model.HasMetadata;
44
import io.javaoperatorsdk.operator.api.reconciler.Context;
5+
import org.slf4j.Logger;
6+
import org.slf4j.LoggerFactory;
57

68
public abstract class AbstractDependentResource<R, P extends HasMetadata>
79
implements DependentResource<R, P> {
10+
private static final Logger log = LoggerFactory.getLogger(AbstractDependentResource.class);
811

912
private final boolean creatable = this instanceof Creator;
1013
private final boolean updatable = this instanceof Updater;
@@ -33,14 +36,22 @@ public void reconcile(P primary, Context context) {
3336
var desired = desired(primary, context);
3437
if (maybeActual.isEmpty()) {
3538
if (creatable) {
39+
log.debug("Creating dependent {} for primary {}", desired, primary);
3640
creator.create(desired, primary, context);
3741
}
3842
} else {
3943
final var actual = maybeActual.get();
4044
if (updatable && !updater.match(actual, desired, context)) {
45+
log.debug("Updating dependent {} for primary {}", desired, primary);
4146
updater.update(actual, desired, primary, context);
47+
} else {
48+
log.debug("Update skipped for dependent {} as it matched the existing one", desired);
4249
}
4350
}
51+
} else {
52+
log.debug(
53+
"Dependent {} is read-only, implement Creator and/or Updater interfaces to modify it",
54+
getClass().getSimpleName());
4455
}
4556
}
4657

0 commit comments

Comments
 (0)