supervisor: reduce verbosity (#179)

worldcoin · Aug 8, 2024 · 6555ad5 · 6555ad5
1 parent 83dd83d
commit 6555ad5
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 8 deletions.
diff --git a/orb-supervisor/debian/orb-supervisor.worldcoin-supervisor.service b/orb-supervisor/debian/orb-supervisor.worldcoin-supervisor.service
@@ -6,7 +6,6 @@ Type=dbus
 BusName=org.worldcoin.OrbSupervisor1
 SyslogIdentifier=worldcoin-supervisor
 Environment=DBUS_SESSION_BUS_ADDRESS=unix:path=/tmp/worldcoin_bus_socket
-Environment="RUST_LOG=debug"
 ExecStart=/usr/local/bin/orb-supervisor
 
 [Install]

diff --git a/orb-supervisor/src/shutdown.rs b/orb-supervisor/src/shutdown.rs
@@ -1,6 +1,6 @@
 use std::{cmp::Ordering, fmt::Display, str::FromStr};
 
-use tracing::info;
+use tracing::debug;
 use zbus_systemd::login1::{self};
 use Kind::{DryHalt, DryPoweroff, DryReboot, Halt, Poweroff, Reboot};
 
@@ -128,7 +128,7 @@ pub async fn schedule_shutdown(
     shutdown_req: ScheduledShutdown,
 ) -> zbus::Result<PreemptionInfo> {
     let already_scheduled: Option<ScheduledShutdown> = {
-        info!("getting property `org.freedesktop.login1.Manager.ScheduledShutdown`");
+        debug!("getting property `org.freedesktop.login1.Manager.ScheduledShutdown`");
         let raw_tuple = proxy.scheduled_shutdown().await?;
         ScheduledShutdown::try_from_dbus(raw_tuple)
             .expect("infallible, the result should always parse")
@@ -143,7 +143,7 @@ pub async fn schedule_shutdown(
         PreemptionInfo::NoExistingShutdown
     };
 
-    info!(
+    debug!(
         "calling `org.freedesktop.login1.Manager.ScheduleShutdown` to shutdown system"
     );
     proxy

diff --git a/orb-supervisor/src/tasks/update.rs b/orb-supervisor/src/tasks/update.rs
@@ -5,7 +5,7 @@ use tokio::{
     task::JoinHandle,
     time::{self, error::Elapsed, Instant},
 };
-use tracing::{debug, info, instrument, warn};
+use tracing::{debug, info, instrument, trace, warn};
 use zbus_systemd::systemd1::{self, ManagerProxy};
 
 use crate::consts::{
@@ -45,7 +45,7 @@ pub fn spawn_shutdown_worldcoin_core_timer(
 
                 // reset the trigger if a new signup has started
                 _ = last_signup_started_event.changed() => {
-                    info!(
+                    debug!(
                         duration_s = DURATION_TO_STOP_CORE_AFTER_LAST_SIGNUP.as_secs(),
                         "new signup started, resetting timer",
                     );
@@ -149,12 +149,11 @@ async fn has_worldcoin_core_stopped(proxy: ManagerProxy<'static>) -> Result<(),
     // This makes use of the fact that the first iteration always returns the current state.
     // So if the service is already inactive or failed, then this loop will break and we
     // doesn't spin indefinitely.
-    debug!("spinning");
     while let Some(event) = active_state_stream.next().await {
         match &*event.get().await? {
             "inactive" | "failed" => break,
             other => {
-                info!(event = other, "received event");
+                trace!(event = other, "received event");
             }
         }
     }