-
Notifications
You must be signed in to change notification settings - Fork 198
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
Refined import db process #2338
Conversation
… to have the time to commit received data - added ThresholdMinConnectedPeers as a config in order to be overridden by the import db process - made the import-db process automatically generate a new BLS key in order to avoid accidental usage of a validator BLS key that migh trigger the node shuffle-out
cmd/node/main.go
Outdated
importCheckpointRoundsModulus := uint(config.EpochStartConfig.RoundsPerEpoch) | ||
log.Info("import DB directory is set, altering config values!", | ||
log.Warn("the node is import mode! Will auto-set some config values", |
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.
is in
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.
done
factory/cryptoSigningParams.go
Outdated
@@ -78,6 +91,21 @@ func (cspf *cryptoSigningParamsLoader) Get() (*CryptoParams, error) { | |||
return cryptoParams, nil | |||
} | |||
|
|||
func (cspf *cryptoSigningParamsLoader) generateCryptoParams(cryptoParams *CryptoParams) (*CryptoParams, error) { | |||
log.Warn("the node is import mode! Will generate a fresh new BLS key") |
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.
is in
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.
done
fca91fb
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.
System tests passed.
Improved the import-db process: