-
Notifications
You must be signed in to change notification settings - Fork 78
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
cmd: write deposit datas to disk #2395
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #2395 +/- ##
==========================================
- Coverage 53.42% 53.35% -0.07%
==========================================
Files 196 196
Lines 26254 26282 +28
==========================================
- Hits 14025 14024 -1
- Misses 10490 10520 +30
+ Partials 1739 1738 -1
☔ View full report in Codecov by Sentry. |
cmd/addvalidators.go
Outdated
return err | ||
} | ||
|
||
filename := fmt.Sprintf("deposit-data-%s.pb", currTime) // Ex: "cluster-manifest-backup-20060102150405.pb" |
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.
we need to log that this is the new deposit datas that was created
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.
905bd83
to
89939de
Compare
Write deposit data file for new validators created in
add-validators-solo
command to each node directory. The new deposit files are named asdeposit-data-20230704173152.json
.Also deletes logic for backing up cluster manifests.
category: feature
ticket: #2379