Skip to content

Commit 4d477c5

Browse files
committed
log sanitization
1 parent cf84be7 commit 4d477c5

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

operator-framework/src/main/java/com/github/containersolutions/operator/processing/EventConsumer.java

+5-4
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,11 @@
55
import org.slf4j.Logger;
66
import org.slf4j.LoggerFactory;
77

8+
@SuppressWarnings("rawtypes")
89
class EventConsumer implements Runnable {
910

1011
private final static Logger log = LoggerFactory.getLogger(EventConsumer.class);
12+
1113
private final CustomResourceEvent event;
1214
private final EventDispatcher eventDispatcher;
1315
private final EventScheduler eventScheduler;
@@ -20,27 +22,26 @@ class EventConsumer implements Runnable {
2022

2123
@Override
2224
public void run() {
25+
log.debug("Processing event started: {}", event);
2326
if (processEvent()) {
2427
eventScheduler.eventProcessingFinishedSuccessfully(event);
28+
log.debug("Event processed successfully: {}", event);
2529
} else {
2630
this.eventScheduler.eventProcessingFailed(event);
31+
log.debug("Event processed failed: {}", event);
2732
}
2833
}
2934

3035
@SuppressWarnings("unchecked")
3136
private boolean processEvent() {
32-
3337
Watcher.Action action = event.getAction();
3438
CustomResource resource = event.getResource();
35-
log.info("Processing event {}", event);
3639
try {
3740
eventDispatcher.handleEvent(action, resource);
3841
} catch (RuntimeException e) {
3942
log.error("Processing event {} failed.", event, e);
40-
log.debug("Failed object: {}", resource);
4143
return false;
4244
}
43-
4445
return true;
4546
}
4647
}

operator-framework/src/main/java/com/github/containersolutions/operator/processing/EventScheduler.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -74,8 +74,8 @@ public void eventReceived(Watcher.Action action, R resource) {
7474
}
7575

7676
void scheduleEvent(CustomResourceEvent event) {
77-
log.debug("Current queue size {}", executor.getQueue().size());
78-
log.info("Scheduling event: {}", event);
77+
log.trace("Current queue size {}", executor.getQueue().size());
78+
log.debug("Scheduling event: {}", event);
7979
try {
8080
lock.lock();
8181
if (event.getAction() == Action.DELETED) {
@@ -109,7 +109,7 @@ void scheduleEvent(CustomResourceEvent event) {
109109
nextBackOff.get(), TimeUnit.MILLISECONDS);
110110
eventStore.addEventUnderProcessing(event);
111111
} finally {
112-
log.info("Scheduling event finished: {}", event);
112+
log.debug("Scheduling event finished: {}", event);
113113
lock.unlock();
114114
}
115115
}

0 commit comments

Comments
 (0)