-
Notifications
You must be signed in to change notification settings - Fork 210
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
[Merged by Bors] - cache the right data #4858
[Merged by Bors] - cache the right data #4858
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #4858 +/- ##
=========================================
- Coverage 76.9% 76.8% -0.1%
=========================================
Files 261 261
Lines 29720 29724 +4
=========================================
- Hits 22858 22857 -1
- Misses 5412 5416 +4
- Partials 1450 1451 +1
|
bors merge |
bors cancel |
Canceled. |
bors merge |
## Motivation Closes #4842 ## Changes remove timestamp. it's no longer useful after we made update immutable, and it caused the code to cache the wrong suffix note: this is causing a node to query bootstrap url after restart.
Build failed:
|
bors merge |
## Motivation Closes #4842 ## Changes remove timestamp. it's no longer useful after we made update immutable, and it caused the code to cache the wrong suffix note: this is causing a node to query bootstrap url after restart.
Build failed: |
bors merge |
## Motivation Closes #4842 ## Changes remove timestamp. it's no longer useful after we made update immutable, and it caused the code to cache the wrong suffix note: this is causing a node to query bootstrap url after restart.
Pull request successfully merged into develop. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
## Motivation #4858 was not a backward compatible fix. explorer's node running with latest in develop crashed with ``` {"L":"INFO","T":"2023-08-18T16:28:53.473Z","N":"ff765.bootstrap","M":"loaded bootstrap file","node_id":"ff76543be7a1919e65ac9eac4739aa467b0e0baa202825a77b223fc9c791dc70","module":"bootstrap","persisted":"/root/data/node/bootstrap/2/epoch-2-update-bs-2023-08-10T06-00-08","epoch":"2","beacon":"0xe3e3389e","activeset_size":2367,"name":"bootstrap"} 1 {"L":"FATAL","T":"2023-08-18T16:28:53.473Z","N":"ff765.bootstrap","M":"unexpected suffix for fallback files","node_id":"ff76543be7a1919e65ac9eac4739aa467b0e0baa202825a77b223fc9c791dc70","module":"bootstrap","suffix":"08","name":"bootstrap"} ``` ## changes - rename legacy files with timestamp to the correct one ("epoch-2-update-bs-2023-08-10T06-00-08" -> "epoch-2-update-bs") - delete extra files ## testing - unit test - manually test with mainnet. sync a node with mainnet and restart multiple times so that there are multiple boostrap files for epoch 2. run new code and saw that only 1 file left "epoch-2-update-bs"
Motivation
Closes #4842
Changes
remove timestamp. it's no longer useful after we made update immutable, and it caused the code to cache the wrong suffix
note:
this is causing a node to query bootstrap url after restart.