Skip to content

Commit

Permalink
extra session disabling traces
Browse files Browse the repository at this point in the history
  • Loading branch information
Overkillus committed Feb 21, 2025
1 parent 26b5a0b commit dc9fc1b
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions substrate/frame/session/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -754,6 +754,8 @@ impl<T: Config> Pallet<T> {
return false
}

log!(trace, "disabling validator {:?}", i);

DisabledValidators::<T>::mutate(|disabled| {
if let Err(index) = disabled.binary_search_by_key(&i, |(index, _)| *index) {
disabled.insert(index, (i, OffenceSeverity(Perbill::zero())));
Expand Down Expand Up @@ -930,6 +932,7 @@ impl<T: Config> Pallet<T> {
}

pub fn report_offence(validator: T::ValidatorId, severity: OffenceSeverity) {
log!(trace, "reporting offence for {:?} with {:?}", validator, severity);
DisabledValidators::<T>::mutate(|disabled| {
let decision = T::DisablingStrategy::decision(&validator, severity, &disabled);

Expand All @@ -939,11 +942,13 @@ impl<T: Config> Pallet<T> {
// Offender is already disabled, update severity if the new one is higher
Ok(index) => {
let (_, old_severity) = &mut disabled[index];
log!(trace, "Offender was already disabled, updating severity from {:?} to {:?}", old_severity, severity);
if severity > *old_severity {
*old_severity = severity;
}
},
Err(index) => {
log!(trace, "Disabling validator {:?} with severity: {:?}", offender_idx, severity);
// Offender is not disabled, add to `DisabledValidators` and disable it
disabled.insert(index, (offender_idx, severity));
// let the session handlers know that a validator got disabled
Expand All @@ -961,6 +966,7 @@ impl<T: Config> Pallet<T> {
// Remove the validator from `DisabledValidators` and re-enable it.
if let Ok(index) = disabled.binary_search_by_key(&reenable_idx, |(index, _)| *index)
{
log!(trace, "Re-enabling validator {:?} to make space", reenable_idx);
disabled.remove(index);
// Emit event that a validator got re-enabled
let reenabled_stash = Validators::<T>::get()[reenable_idx as usize].clone();
Expand Down

0 comments on commit dc9fc1b

Please sign in to comment.