-
Notifications
You must be signed in to change notification settings - Fork 10
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
Prysm update #575
Prysm update #575
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #575 +/- ##
===========================================
- Coverage 48.54% 48.17% -0.37%
===========================================
Files 117 117
Lines 2091 2115 +24
Branches 225 231 +6
===========================================
+ Hits 1015 1019 +4
- Misses 1047 1067 +20
Partials 29 29
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
try { | ||
attestations = yield retry(2, 0, eth2API.validator.getAttesterDuties, epoch, [validatorState.index]); | ||
attestationsFuture = yield call(eth2API.validator.getAttesterDuties, epoch + 1, [validatorState.index]); | ||
} catch { |
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.
log errors
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.
this try-catch block is in the case of some sort of error, we don't want to break application for failing to fetch some meta data for nonessential work for validating
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.
not sking you to break app, just log error message instead of ignoring it so we have some info when debugging
propositions = yield call(eth2API.validator.getProposerDuties, epoch, [validatorId]); | ||
propositionsFuture = yield call(eth2API.validator.getProposerDuties, epoch + 1, [validatorId]); | ||
propositionsSuperFuture = yield call(eth2API.validator.getProposerDuties, epoch + 2, [validatorId]); | ||
} catch { |
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.
same here
Short description of work done
update prysm docker and client
PR Checklist