Skip to content

Commit e59becf

Browse files
committed
refactor: move syncer state transitions log to debug level
1 parent 1e9417f commit e59becf

2 files changed

Lines changed: 15 additions & 2 deletions

File tree

src/event.rs

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ pub trait StateMachineExecutor<
2828
let event = Event::with(endpoints, runtime.identity(), source, request);
2929
let sm_display = sm.to_string();
3030
let sm_name = sm.name();
31+
let sm_log_level = sm.log_level();
3132
if let Some(new_sm) = sm.next(event, runtime)? {
3233
let new_sm_display = new_sm.to_string();
3334
// relegate state transitions staying the same to debug
@@ -38,7 +39,8 @@ pub trait StateMachineExecutor<
3839
new_sm.bright_green_bold()
3940
);
4041
} else {
41-
info!(
42+
log!(
43+
new_sm.log_level(),
4244
"{} state transition {} -> {}",
4345
new_sm.name(),
4446
sm_display.red_bold(),
@@ -47,7 +49,8 @@ pub trait StateMachineExecutor<
4749
}
4850
Ok(Some(new_sm))
4951
} else {
50-
info!(
52+
log!(
53+
sm_log_level,
5154
"{} state machine ended {} -> {}",
5255
sm_name,
5356
sm_display.red_bold(),
@@ -70,7 +73,13 @@ where
7073
where
7174
Self: Sized;
7275

76+
/// Return the display name of the state machine
7377
fn name(&self) -> String;
78+
79+
/// Return the log level to use for state transitions. Info by default
80+
fn log_level(&self) -> log::Level {
81+
log::Level::Info
82+
}
7483
}
7584

7685
/// Event changing state machine state, consisting of a certain P2P or RPC `request` sent from some

src/farcasterd/syncer_state_machine.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,10 @@ impl StateMachine<Runtime, Error> for SyncerStateMachine {
9090
fn name(&self) -> String {
9191
"Syncer".to_string()
9292
}
93+
94+
fn log_level(&self) -> log::Level {
95+
log::Level::Debug
96+
}
9397
}
9498

9599
pub struct SyncerStateMachineExecutor {}

0 commit comments

Comments
 (0)