Skip to content

[4844] Fix some Devnet9 Hive tests#5929

Merged
jflo merged 5 commits intohyperledger:mainfrom
Gabriel-Trintinalia:hive-devnet9
Sep 22, 2023
Merged

[4844] Fix some Devnet9 Hive tests#5929
jflo merged 5 commits intohyperledger:mainfrom
Gabriel-Trintinalia:hive-devnet9

Conversation

@Gabriel-Trintinalia
Copy link
Contributor

PR description

Fix null latest valid hash when validation blobs/expected hashes
Fix HISTORICAL_ROOTS_MODULUS used by hive/devnet 9

Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
@github-actions
Copy link

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.
  • I thought about the changelog and included a changelog update if required.
  • If my PR includes database changes (e.g. KeyValueSegmentIdentifier) I have thought about compatibility and performed forwards and backwards compatibility tests

@Gabriel-Trintinalia Gabriel-Trintinalia marked this pull request as ready for review September 22, 2023 07:56
@jflo jflo merged commit 9d7ee2b into hyperledger:main Sep 22, 2023
jflo pushed a commit to jflo/besu that referenced this pull request Nov 10, 2023
* Return latest valid hash for blob validation
* Fix invalid hash
* Fix null hash
* Change HISTORY_BUFFER_LENGTH
---------

Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
Signed-off-by: Justin Florentine <justin+github@florentine.us>
NickSneo pushed a commit to NickSneo/besu that referenced this pull request Nov 12, 2023
* Return latest valid hash for blob validation
* Fix invalid hash
* Fix null hash
* Change HISTORY_BUFFER_LENGTH
---------

Signed-off-by: Gabriel-Trintinalia <gabriel.trintinalia@consensys.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants