Skip to content

Commit

Permalink
chore(sequencer): remove misplaced logs
Browse files Browse the repository at this point in the history
Sequencer checks if the storage path exists and
then reports on actions that it thinks what the
call `cnidarium::Storage::load` will do.

However, on the one hand cnidarium itself already
reports that information. On the other hand,
`Sequencer::run_until_stopped` is the wrong place
to emit these events because it does not have
authority over which actions cnidarium will take.
Should the construct ever change, then the events
will be immediately out of whack, which will lead
to confusion.
  • Loading branch information
SuperFluffy committed Jan 8, 2025
1 parent ffbd008 commit a5c2e4d
Showing 1 changed file with 2 additions and 22 deletions.
24 changes: 2 additions & 22 deletions crates/astria-sequencer/src/sequencer.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ use astria_core::generated::astria::sequencerblock::v1::sequencer_service_server
use astria_eyre::{
anyhow_to_eyre,
eyre::{
eyre,
OptionExt as _,
Result,
WrapErr as _,
eyre,
},
};
use penumbra_tower_trace::{
Expand All @@ -17,8 +17,8 @@ use tendermint::v0_38::abci::ConsensusRequest;
use tokio::{
select,
signal::unix::{
signal,
SignalKind,
signal,
},
sync::{
oneshot,
Expand Down Expand Up @@ -54,26 +54,6 @@ impl Sequencer {
register_histogram_global("cnidarium_nonverifiable_get_raw_duration_seconds");
let span = info_span!("Sequencer::run_until_stopped");

if config
.db_filepath
.try_exists()
.context("failed checking for existence of db storage file")?
{
span.in_scope(|| {
info!(
path = %config.db_filepath.display(),
"opening storage db"
);
});
} else {
span.in_scope(|| {
info!(
path = %config.db_filepath.display(),
"creating storage db"
);
});
}

let mut signals = spawn_signal_handler();

let substore_prefixes = vec![penumbra_ibc::IBC_SUBSTORE_PREFIX];
Expand Down

0 comments on commit a5c2e4d

Please sign in to comment.