Skip to content

Commit 6396b48

Browse files
authored
improve: informer health check should not rely on isWatching (#3209)
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
1 parent 8168ebf commit 6396b48

File tree

1 file changed

+3
-5
lines changed
  • operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer

1 file changed

+3
-5
lines changed

operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/event/source/informer/InformerWrapper.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -221,16 +221,14 @@ public boolean isRunning() {
221221

222222
@Override
223223
public Status getStatus() {
224-
var status = isRunning() && hasSynced() && isWatching() ? Status.HEALTHY : Status.UNHEALTHY;
224+
var status = isRunning() && hasSynced() ? Status.HEALTHY : Status.UNHEALTHY;
225225
log.debug(
226-
"Informer status: {} for type: {}, namespace: {}, details [is running: {}, has synced: {},"
227-
+ " is watching: {}]",
226+
"Informer status: {} for type: {}, namespace: {}, details [is running: {}, has synced: {}]",
228227
status,
229228
informer.getApiTypeClass().getSimpleName(),
230229
namespaceIdentifier,
231230
isRunning(),
232-
hasSynced(),
233-
isWatching());
231+
hasSynced());
234232
return status;
235233
}
236234

0 commit comments

Comments
 (0)