summaryrefslogtreecommitdiffstats
path: root/container-disc
diff options
context:
space:
mode:
authorJon Bratseth <bratseth@gmail.com>2023-03-06 10:39:51 +0100
committerJon Bratseth <bratseth@gmail.com>2023-03-06 10:39:51 +0100
commit481d6b5a8e05d1016cb0d3da24fdf274bdc2f778 (patch)
treed9f606675daf5e8bea46c84596d40046dbc061f1 /container-disc
parente22467e6500b212a611827659d79b680f1b7f805 (diff)
Use the latest metric object
Diffstat (limited to 'container-disc')
-rw-r--r--container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java14
1 files changed, 8 insertions, 6 deletions
diff --git a/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java b/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java
index 0d18eb9aa8f..1611aea6af3 100644
--- a/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java
+++ b/container-disc/src/main/java/com/yahoo/container/jdisc/ConfiguredApplication.java
@@ -99,7 +99,6 @@ public final class ConfiguredApplication implements Application {
private final Thread portWatcher;
private HandlersConfigurerDi configurer;
private QrConfig qrConfig;
- private Metric metric; // Cannot be injected before the application is set up
private Register slobrokRegistrator = null;
private Supervisor supervisor = null;
@@ -164,7 +163,6 @@ public final class ConfiguredApplication implements Application {
ContainerBuilder builder = createBuilderWithGuiceBindings();
configurer = createConfigurer(builder.guiceModules().activate());
- metric = configurer.getComponent(Metric.class);
initializeAndActivateContainer(builder, () -> {});
reconfigurerThread.setDaemon(true);
reconfigurerThread.start();
@@ -301,10 +299,7 @@ public final class ConfiguredApplication implements Application {
startAndStopServers(currentServers);
startAndRemoveClients(Container.get().getClientProviderRegistry().allComponents());
-
- log.info("Switching to the latest deployed set of configurations and components. " +
- "Application config generation: " + configurer.generation());
- metric.set(APPLICATION_GENERATION.baseName(), configurer.generation(), metric.createContext(Map.of()));
+ signalActivation();
}
private void activateContainer(ContainerBuilder builder, Runnable onPreviousContainerTermination) {
@@ -321,6 +316,13 @@ public final class ConfiguredApplication implements Application {
}
}
+ private void signalActivation() {
+ log.info("Switching to the latest deployed set of configurations and components. " +
+ "Application config generation: " + configurer.generation());
+ var metric = configurer.getComponent(Metric.class);
+ metric.set(APPLICATION_GENERATION.baseName(), configurer.generation(), metric.createContext(Map.of()));
+ }
+
private ContainerBuilder createBuilderWithGuiceBindings() {
ContainerBuilder builder = activator.newContainerBuilder();
setupGuiceBindings(builder.guiceModules());