Skip to content

Commit 8022d1c

Browse files
committed
wip
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
1 parent 51a6af3 commit 8022d1c

File tree

3 files changed

+14
-4
lines changed

3 files changed

+14
-4
lines changed

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,4 +49,8 @@ public Optional<ResourceEvent> getLatestEventAfterLastUpdateEvent(String updated
4949
}
5050
return Optional.empty();
5151
}
52+
53+
public int getActiveUpdates() {
54+
return activeUpdates;
55+
}
5256
}

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

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -93,9 +93,7 @@ public void changeNamespaces(Set<String> namespaces) {
9393
@SuppressWarnings("unchecked")
9494
public R eventFilteringUpdateAndCacheResource(R resourceToUpdate, UnaryOperator<R> updateMethod) {
9595
ResourceID id = ResourceID.fromResource(resourceToUpdate);
96-
if (log.isDebugEnabled()) {
97-
log.debug("Update and cache: {}", id);
98-
}
96+
log.debug("Update and cache: {}", id);
9997
R updatedResource = null;
10098
try {
10199
temporaryResourceCache.startEventFilteringModify(id);

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

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,8 +87,16 @@ public synchronized Optional<ResourceEvent> doneEventFilterModify(
8787
var ed = activeUpdates.get(resourceID);
8888
if (ed.decreaseActiveUpdates()) {
8989
activeUpdates.remove(resourceID);
90-
return ed.getLatestEventAfterLastUpdateEvent(updatedResourceVersion);
90+
var res = ed.getLatestEventAfterLastUpdateEvent(updatedResourceVersion);
91+
log.debug(
92+
"Zero active updates for resource id: {}; event after update event: {}; updated resource"
93+
+ " version: {}",
94+
resourceID,
95+
res.isPresent(),
96+
updatedResourceVersion);
97+
return res;
9198
} else {
99+
log.debug("Active updates {} for resource id: {}", ed.getActiveUpdates(), resourceID);
92100
return Optional.empty();
93101
}
94102
}

0 commit comments

Comments
 (0)