Skip to content

Commit

Permalink
staking: don't double log rm index
Browse files Browse the repository at this point in the history
  • Loading branch information
erwanor committed Feb 26, 2024
1 parent 4b8258e commit b7ec91a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
4 changes: 0 additions & 4 deletions crates/core/component/stake/src/component/epoch_handler.rs
Original file line number Diff line number Diff line change
Expand Up @@ -356,10 +356,6 @@ pub trait EpochHandler: StateWriteExt + ConsensusIndexRead {
// be in the CS index. We should replace the union set approach with a merged
// stream that tags items with their source. See #3874.
if !self.belongs_in_index(&validator.identity_key).await {
tracing::debug!(
validator_id = %validator.identity_key,
"removing validator from consensus set"
);
self.remove_consensus_set_index(&validator.identity_key);
}

Expand Down
4 changes: 2 additions & 2 deletions crates/core/component/stake/src/component/stake.rs
Original file line number Diff line number Diff line change
Expand Up @@ -503,11 +503,11 @@ pub trait ConsensusIndexRead: StateRead {

match state {
validator::State::Active | validator::State::Inactive => {
tracing::debug!(?state, "validator belongs in consensus set");
tracing::debug!(?state, "validator belongs in the consensus set");
true
}
_ => {
tracing::debug!(?state, "validator does not belong in consensus set");
tracing::debug!(?state, "validator does not belong in the consensus set");
false
}
}
Expand Down

0 comments on commit b7ec91a

Please sign in to comment.