Skip to content

Commit

Permalink
Address comments
Browse files Browse the repository at this point in the history
  • Loading branch information
p-offtermatt committed Apr 29, 2024
1 parent 062a574 commit 3ae1561
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion tests/e2e/steps_partial_set_security.go
Original file line number Diff line number Diff line change
Expand Up @@ -765,7 +765,7 @@ func stepsTopNChain() []Step {
},
ChainID("provi"): ChainState{
HasToValidate: &map[ValidatorID][]ChainID{
ValidatorID("alice"): {}, // alice has opted out and the epoch is over, so definitely doesnt have to validate anymore
ValidatorID("alice"): {}, // alice has opted out and the epoch is over, so definitely does not have to validate anymore
ValidatorID("bob"): {"consu"},
ValidatorID("carol"): {"consu"},
},
Expand Down
6 changes: 3 additions & 3 deletions x/ccv/provider/keeper/keeper.go
Original file line number Diff line number Diff line change
Expand Up @@ -1259,10 +1259,10 @@ func (k Keeper) HasToValidate(

minPowerToOptIn := k.ComputeMinPowerToOptIn(ctx, chainID, k.stakingKeeper.GetLastValidators(ctx), topN)

if power > minPowerToOptIn {
return true, nil
if power < minPowerToOptIn {
return false, nil
}
return false, nil
return true, nil
}

// SetConsumerCommissionRate sets a per-consumer chain commission rate
Expand Down

0 comments on commit 3ae1561

Please sign in to comment.