New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Log Active Validator Count #5923
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5923 +/- ##
==========================================
- Coverage 59.76% 59.43% -0.34%
==========================================
Files 320 318 -2
Lines 27420 26754 -666
==========================================
- Hits 16387 15900 -487
+ Misses 8825 8704 -121
+ Partials 2208 2150 -58 |
@@ -191,7 +191,14 @@ func (s *Service) ProcessDepositLog(ctx context.Context, depositLog gethTypes.Lo | |||
if !s.chainStartData.Chainstarted { | |||
deposits := len(s.chainStartData.ChainstartDeposits) | |||
if deposits%512 == 0 { | |||
log.WithField("deposits", deposits).Info("Processing deposits from Ethereum 1 chain") | |||
valCount, err := helpers.ActiveValidatorCount(s.preGenesisState, 0) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isn't pre genesis state always the same?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no its not, it is updated with every deposit
Co-authored-by: Shay Zluf <thezluf@gmail.com>
What type of PR is this?
Improve Logging
What does this PR do? Why is it needed?
Log Active Validator Count alongside total deposit count.
Which issues(s) does this PR fix?
N.A
Other notes for review
N.A