Skip to content

Commit 34f9d63

Browse files
laxmikantbpandharemetacosm
authored andcommitted
feature: modified events logic
1 parent cc6c37f commit 34f9d63

File tree

2 files changed

+13
-12
lines changed

2 files changed

+13
-12
lines changed

Diff for: operator-framework-core/src/main/java/io/javaoperatorsdk/operator/Metrics.java

+12-12
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ public <R extends CustomResource> UpdateControl<R> timeControllerCreateOrUpdate(
3636
.register(registry);
3737
try {
3838
final var result = timer.record(() -> controller.createOrUpdateResource(resource, context));
39+
System.out.println("I am here in create update");
3940
String successType = "cr";
4041
if (result.isUpdateStatusSubResource()) {
4142
successType = "status";
@@ -47,9 +48,6 @@ public <R extends CustomResource> UpdateControl<R> timeControllerCreateOrUpdate(
4748
.counter(
4849
"operator.sdk.controllers.execution.success", "controller", name, "type", successType)
4950
.increment();
50-
registry
51-
.counter("operator.sdk.events.received", "events", name, "type", "totalEvents")
52-
.increment();
5351
return result;
5452
} catch (Exception e) {
5553
registry
@@ -60,9 +58,6 @@ public <R extends CustomResource> UpdateControl<R> timeControllerCreateOrUpdate(
6058
"exception",
6159
e.getClass().getSimpleName())
6260
.increment();
63-
registry
64-
.counter("operator.sdk.events.received", "events", name, "exception", "totalEvents")
65-
.increment();
6661
throw e;
6762
}
6863
}
@@ -81,6 +76,7 @@ public DeleteControl timeControllerDelete(
8176
.register(registry);
8277
try {
8378
final var result = timer.record(() -> controller.deleteResource(resource, context));
79+
System.out.println("I am here in delete");
8480
String successType = "notDelete";
8581
if (result == DeleteControl.DEFAULT_DELETE) {
8682
successType = "delete";
@@ -89,9 +85,6 @@ public DeleteControl timeControllerDelete(
8985
.counter(
9086
"operator.sdk.controllers.execution.success", "controller", name, "type", successType)
9187
.increment();
92-
registry
93-
.counter("operator.sdk.events.received", "events", name, "type", "totalEvents")
94-
.increment();
9588
return result;
9689
} catch (Exception e) {
9790
registry
@@ -102,9 +95,6 @@ public DeleteControl timeControllerDelete(
10295
"exception",
10396
e.getClass().getSimpleName())
10497
.increment();
105-
registry
106-
.counter("operator.sdk.events.received", "events", name, "exception", "totalEvents")
107-
.increment();
10898
throw e;
10999
}
110100
}
@@ -119,6 +109,16 @@ public void timeControllerRetry() {
119109

120110
}
121111

112+
public void timeControllerEvents() {
113+
114+
registry
115+
.counter(
116+
"operator.sdk.total.events.received", "events", "totalEvents", "type",
117+
"eventsReceived")
118+
.increment();
119+
120+
}
121+
122122
public static class NoopMeterRegistry extends MeterRegistry {
123123
public NoopMeterRegistry(Clock clock) {
124124
super(clock);

Diff for: operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/DefaultEventHandler.java

+1
Original file line numberDiff line numberDiff line change
@@ -116,6 +116,7 @@ public void handleEvent(Event event) {
116116
final Predicate<CustomResource> selector = event.getCustomResourcesSelector();
117117
for (String uid : eventSourceManager.getLatestResourceUids(selector)) {
118118
eventBuffer.addEvent(uid, event);
119+
metrics.timeControllerEvents();
119120
executeBufferedEvents(uid);
120121
}
121122
} finally {

0 commit comments

Comments
 (0)