set sane defaults for monitoring interval. default to monitoring enabled

CDOT-CV · Oct 11, 2024 · c6c4cfa · c6c4cfa
1 parent ad91aeb
commit c6c4cfa
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/jpo-ode-svcs/src/main/java/us/dot/its/jpo/ode/traveler/TimDepositController.java b/jpo-ode-svcs/src/main/java/us/dot/its/jpo/ode/traveler/TimDepositController.java
@@ -115,7 +115,9 @@ public TimDepositController(OdeProperties odeProperties) {
          logger.info("TIM ingest monitoring enabled.");
 
          ScheduledExecutorService scheduledExecutorService = Executors.newSingleThreadScheduledExecutor();
-         Long monitoringInterval = Long.valueOf(odeProperties.getProperty(ConfigEnvironmentVariables.ODE_TIM_INGEST_MONITORING_INTERVAL));
+         // 3600 seconds, or one hour, was determined to be a sane default for the monitoring interval if monitoring is enabled
+         // but there was no interval set in the .env file
+         Long monitoringInterval = Long.valueOf(odeProperties.getProperty(ConfigEnvironmentVariables.ODE_TIM_INGEST_MONITORING_INTERVAL, "3600"));
 
          scheduledExecutorService.scheduleAtFixedRate(new TimIngestWatcher(monitoringInterval), monitoringInterval, monitoringInterval, java.util.concurrent.TimeUnit.SECONDS);
       } else {

diff --git a/sample.env b/sample.env
@@ -106,5 +106,5 @@ ADM_LOG_TO_CONSOLE=true
 ADM_LOG_LEVEL=INFO
 
 # ODE Monitoring
-ODE_TIM_INGEST_MONITORING_ENABLED=false
-TIM_INGEST_MONITORING_INTERVAL=
+ODE_TIM_INGEST_MONITORING_ENABLED=true
+TIM_INGEST_MONITORING_INTERVAL=60