-
Notifications
You must be signed in to change notification settings - Fork 100
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix(security): fix concurrency issues in tree key formats, and CPU us…
…age in genesis tree roots (#7392) * Add tree key format and cached root upgrades * Document the changes in the upgrades * Remove unnecessary clippy::unwrap_in_result * Fix database type * Bump state version * Skip some checks if the database is empty * Fix tests for a short state upgrade * Disable format checks in some tests * Document state performance issues * Clarify upgrade behaviour * Clarify panic messages * Delete incorrect genesis trees write code * Fix metrics handling for genesis * Remove an unused import * Explain why genesis anchors are ok * Update snapshots * Debug a failing test * Fix some tests * Fix missing imports * Move the state check in a test * Fix comment and doc typos Co-authored-by: Marek <[email protected]> Co-authored-by: Arya <[email protected]> * Clarify what a long upgrade is * Rename unused function arguments Co-authored-by: Marek <[email protected]> * Add all_unordered log regex matching methods * Fix timing issues with version upgrades and other logs * Fix argument name in docs Co-authored-by: Marek <[email protected]> * Explain match until first for all regexes behaviour better --------- Co-authored-by: Marek <[email protected]> Co-authored-by: Arya <[email protected]>
- Loading branch information
1 parent
8c01fec
commit 64f7772
Showing
36 changed files
with
997 additions
and
360 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
10 changes: 10 additions & 0 deletions
10
...rvice/finalized_state/disk_format/tests/snapshots/orchard_anchors_raw_data@mainnet_0.snap
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
source: zebra-state/src/service/finalized_state/disk_format/tests/snapshot.rs | ||
expression: cf_data | ||
--- | ||
[ | ||
KV( | ||
k: "ae2935f1dfd8a24aed7c70df7de3a668eb7a49b1319880dde2bbd9031ae5d82f", | ||
v: "", | ||
), | ||
] |
10 changes: 10 additions & 0 deletions
10
...rvice/finalized_state/disk_format/tests/snapshots/orchard_anchors_raw_data@testnet_0.snap
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
source: zebra-state/src/service/finalized_state/disk_format/tests/snapshot.rs | ||
expression: cf_data | ||
--- | ||
[ | ||
KV( | ||
k: "ae2935f1dfd8a24aed7c70df7de3a668eb7a49b1319880dde2bbd9031ae5d82f", | ||
v: "", | ||
), | ||
] |
10 changes: 10 additions & 0 deletions
10
...rvice/finalized_state/disk_format/tests/snapshots/sapling_anchors_raw_data@mainnet_0.snap
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
source: zebra-state/src/service/finalized_state/disk_format/tests/snapshot.rs | ||
expression: cf_data | ||
--- | ||
[ | ||
KV( | ||
k: "fbc2f4300c01f0b7820d00e3347c8da4ee614674376cbc45359daa54f9b5493e", | ||
v: "", | ||
), | ||
] |
10 changes: 10 additions & 0 deletions
10
...rvice/finalized_state/disk_format/tests/snapshots/sapling_anchors_raw_data@testnet_0.snap
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
source: zebra-state/src/service/finalized_state/disk_format/tests/snapshot.rs | ||
expression: cf_data | ||
--- | ||
[ | ||
KV( | ||
k: "fbc2f4300c01f0b7820d00e3347c8da4ee614674376cbc45359daa54f9b5493e", | ||
v: "", | ||
), | ||
] |
10 changes: 10 additions & 0 deletions
10
...ervice/finalized_state/disk_format/tests/snapshots/sprout_anchors_raw_data@mainnet_0.snap
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
source: zebra-state/src/service/finalized_state/disk_format/tests/snapshot.rs | ||
expression: cf_data | ||
--- | ||
[ | ||
KV( | ||
k: "d7c612c817793191a1e68652121876d6b3bde40f4fa52bc314145ce6e5cdd259", | ||
v: "0001d7c612c817793191a1e68652121876d6b3bde40f4fa52bc314145ce6e5cdd259", | ||
), | ||
] |
10 changes: 10 additions & 0 deletions
10
...ervice/finalized_state/disk_format/tests/snapshots/sprout_anchors_raw_data@testnet_0.snap
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
source: zebra-state/src/service/finalized_state/disk_format/tests/snapshot.rs | ||
expression: cf_data | ||
--- | ||
[ | ||
KV( | ||
k: "d7c612c817793191a1e68652121876d6b3bde40f4fa52bc314145ce6e5cdd259", | ||
v: "0001d7c612c817793191a1e68652121876d6b3bde40f4fa52bc314145ce6e5cdd259", | ||
), | ||
] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.