Skip to content
Snippets Groups Projects
Commit 55df376b authored by Erik Oudsen's avatar Erik Oudsen
Browse files

Merge branch 'prometheus-metrics' into 'development'

Fixes bug where worker busy-ness would always be 100

See merge request eip/code-measures!41
parents 95b21225 9f84427e
Branches
Tags
2 merge requests!78Master merge,!41Fixes bug where worker busy-ness would always be 100
Pipeline #163354 passed
......@@ -97,13 +97,13 @@ public class JobRunner {
logger.debug("Worker running job {}", msg.getId());
try (var job = this.splitter.split(msg)) {
gauge.set(0.0);
gauge.set(100.0);
var message = this.checker.run(job);
this.currentJob.set(null);
replySender.send(message);
gauge.set(100.0);
gauge.set(0.0);
} catch (final Exception ex) {
gauge.set(100.0);
gauge.set(0.0);
logger.error("Exception while checking job {}", msg.getId(), ex);
this.currentJob.set(null);
try {
......
......@@ -107,7 +107,7 @@ public class Worker {
final var busyGauge = Gauge.build()
.name("busy")
.help("Current busy status.").register();
busyGauge.set(100.0);
busyGauge.set(0.0);
this.resolver.add(Gauge.class, busyGauge);
final var checker = this.resolver.get(Checker.class);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment