FuzzForkChoiceResponse in the package github.com/prysmaticlabs/prysm/v5/beacon-chain/execution failed#2
Open
github-actions[bot] wants to merge 342 commits intomainfrom
Conversation
…nLabs#14511) * Switch to compounding when consolidating with source==target * Feedback
… limit (OffchainLabs#14537) * Update default scrape-interval to 2 minutes for Beaconcha.in API rate limit * Update changelog for scrape-interval change --------- Co-authored-by: james-prysm <90280386+james-prysm@users.noreply.github.com>
* remove the cache * linting * changelog * fixing unit tests
* uncommenting lode config tests * fixing minimal config
* removing bodies v2 logic * changelog * fixing unit test
* add endpoint * changelog * correct resp with both attestationSlashings types * fix and comment * fix test * fix version check * review + fixes * fix * James' review * Review items * Radek' review * Radek' review
* recover from panics when writing the event stream * changelog --------- Co-authored-by: Kasey Kirkham <kasey@users.noreply.github.com>
* wip * gaz * rename field * sammy review * updating execution api request and reverting response back * fixing linting * changelog * changelog * adding in serialization of requests * code cleanup * adding some happy path tests and fixing mock * mock still broken * fixing linting * updating name on proto * missed naming * placeholder fix for TestClient_HTTP * removing duplicate change log * adding in test for get payloadv4 as well as some tests * added tests for execution client testing, fixed encode type * adding comment for placeholder test * fixing test and addressing feedback * feedback * flipping the test names, was used in reverse * feedback from kasey * reverting switch back to if statements to fix bug * lint
* Update correlation penalty for EIP-7251 * Potuz and James feedback
* fix endpoint * changelog + gaz * add to endpoints test * Radek' review * James' review * fix duplication a bit * fix changelog
* rollback on SaveState error * add test
* Update the monitor package to Electra * changelog <3
* Use ROBlock in block processing pipeline * Pass blockchain tests * passing forkchoice tests * pass rpc test * change changelog
* new type for tests where errors are only logged * StreamHandler waits for write loop exit * add test case for writer timeout * add changelog * add missing file * logging fix * fix logging test to allow info logs * naming/comments; make response controller private * simplify cancel defers * fix typo in test file name --------- Co-authored-by: Kasey Kirkham <kasey@users.noreply.github.com>
* Use engine api get-blobs for block subscriber Debug changelog add proto marshal and unmarshal Kasey's feedback * Feedback * Preston's feedback * Exist argument should not be hardcoded with kzg count
* Update bazel-lib * Update rules_oci * Compress pkg_tar * Add a note about 401 download for distroless/cc-debian11 * Update CHANGELOG.md * Re-upload distroless/cc-debian11 to prysmaticlabs/distroless/cc-debian11 such that the image is public. Ran the following command: gcrane cp 'gcr.io/distroless/cc-debian11@sha256:b82f113425c5b5c714151aaacd8039bc141821cdcd3c65202d42bdf9c43ae60b' gcr.io/prysmaticlabs/distroless/cc-debian11:latest * Back out "Update rules_oci" This backs out commit 64200fb.
* add version headers * use headstate and add unit test * changelog
* add ListAttestationsV2 endpoint * fix endpoint * changelog * add endpoint to tests * add trailing comma * add version header + lint fix * all reviews * modify v1 and comments * fix linter * Radek' review
* Change it to rollback * Spacing * Reverse order of rollback * Add Rollback Test * Potuz's Test Suggestion * Potuz's Suggestion
* fix --backfill-oldest-slot flag handling * changelog --------- Co-authored-by: Kasey Kirkham <kasey@users.noreply.github.com>
* fixing error handling and returning empty requests with the wrong wallet type * changelog * adding some unit tests * Update validator/rpc/handlers_keymanager.go Co-authored-by: Sammy Rosso <15244892+saolyn@users.noreply.github.com> * Update CHANGELOG.md Co-authored-by: Sammy Rosso <15244892+saolyn@users.noreply.github.com> * Update validator/rpc/handlers_keymanager.go Co-authored-by: Sammy Rosso <15244892+saolyn@users.noreply.github.com> * Update validator/rpc/intercepter.go Co-authored-by: Sammy Rosso <15244892+saolyn@users.noreply.github.com> * Update validator/rpc/intercepter_test.go Co-authored-by: Sammy Rosso <15244892+saolyn@users.noreply.github.com> * Update validator/rpc/intercepter_test.go Co-authored-by: Sammy Rosso <15244892+saolyn@users.noreply.github.com> * lint --------- Co-authored-by: Sammy Rosso <15244892+saolyn@users.noreply.github.com>
* Update pprof image with a recent one * Update CHANGELOG.md
* removing redundant log, and poorly worded log * move log printing message * changelog * reverting a log change
…14592) * Use Read Only Head State * Use Read Only Head State
* fix * fix test * add to comment * changelog
Co-authored-by: Kasey Kirkham <kasey@users.noreply.github.com>
* Remove Fulu block and state. * Add missing tests. * Alias `ProtobufBeaconStateFulu` to `ProtobufBeaconStateElectra`
* fix evaluator in electra * remove function * Fix evaluator --------- Co-authored-by: james-prysm <90280386+james-prysm@users.noreply.github.com>
…ffchainLabs#14928) * Update go.opentelemetry.io/otel to v1.34.0 * Update otel exporter to replace deprecated jaeger exporter * Changelog * Use WithEndpointURL * Clarify potential breaking change
* Fix evaluator * fix deposit activation
Signed-off-by: fuyangpengqi <995764973@qq.com>
* changes needed to support web3signer running on electra for e2e * updating web3signer version and fixing missed configs and test alignment
* Add in Multiclient E2E * Fix Execution Engine * Update testing/endtoend/endtoend_test.go Co-authored-by: Radosław Kapka <rkapka@wp.pl> * Update testing/endtoend/endtoend_test.go Co-authored-by: Radosław Kapka <rkapka@wp.pl> --------- Co-authored-by: Radosław Kapka <rkapka@wp.pl>
* Don't use MaxCover for Electra on-chain aggregates * changelog <3
* Add GetPendingDeposits endpoint * add comment * add changelog * gaz * Radek' review * move JSON object params * gaz * Radek' nits xD * James' review
…chainLabs#14955) * Add request hash to header for builder: executable data to block * go fmt
* add pending partial withdrawals endpoint * changelog * missing new line * fix changelog * removing unneeded header * using generic instead of redundant functions --------- Co-authored-by: james-prysm <james@prysmaticlabs.com>
* `TrackedValidatorsCache`: Implement a 1-hour TTL by uding `go-cache`. * `TrackedValidatorsCache`: Add the `ItemCount` method. * `TrackedValidatorsCache`: Add the `Indices` method. * Add changelog. * `TrackedValidatorsCache`: Add prometheus metrics. * Update beacon-chain/cache/tracked_validators.go Co-authored-by: Preston Van Loon <pvanloon@offchainlabs.com> --------- Co-authored-by: Preston Van Loon <pvanloon@offchainlabs.com>
* `SingleAttestation` support in the monitor service * changelog <3
* adding correct headers when posting for validator registration on builder api * changelog
…ffchainLabs#14962) * improving the error messages for execution request deserialization * changelog
* Test Deposit Requests * Remove extra epochs * Clean up Panic * Fix Slashing Config * Fix Slashing Test --------- Co-authored-by: james-prysm <90280386+james-prysm@users.noreply.github.com>
…inLabs#14953) * created new converstions_execution files moving the from and to consensus functions for execution related items, also created a placeholder for block_execution with the intent of adding execution types there * moving execution types from block.go to block_execution.go * migrating more types and logic * adding all the to consensus functions for payloads * changelog * linting * updating unit tests for conversions * fixing linting * forgot to fix test * updating name based on feedback
…lices is defined as zero (OffchainLabs#14973) * Fix violations of S1009: should omit nil check; len() for nil slices is defined as zero * Changelog fragment
…14972) * Fixed otelhttp client setups. Note: This may not be the best solution as the http client is defined in many places. There should be a canoncial http client with the proper setup. * Changelog fragment * go mod tidy and gazelle
…abs#14974) * Fix violations of non-constant format string in call * Changelog fragment
Fix assertion tests after switching to cmp Changelog fragment
…ysmaticlabs/prysm/v5/beacon-chain/execution.
`go test -run=FuzzForkChoiceResponse/5976311ad0b78cca github.com/prysmaticlabs/prysm/v5/beacon-chain/execution` failed with the following output:
```
--- FAIL: FuzzForkChoiceResponse (0.00s)
--- FAIL: FuzzForkChoiceResponse/5976311ad0b78cca (0.00s)
assertions.go:39: engine_client_fuzz_test.go:43 geth and prysm unmarshaller return inconsistent errors. json: cannot unmarshal hex string without 0x prefix into Go struct field PayloadStatusV1.payloadStatus.witness of type hexutil.Bytes and <nil>, want: true (bool), got: false (bool)
FAIL
FAIL github.com/prysmaticlabs/prysm/v5/beacon-chain/execution 0.027s
FAIL
```
This fuzz data is generated by [actions-go-fuzz](https://github.com/shogo82148/actions-go-fuzz).
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
go test -run=FuzzForkChoiceResponse/5976311ad0b78cca github.com/prysmaticlabs/prysm/v5/beacon-chain/executionfailed with the following output:This pull request is generated by actions-go-fuzz.
See the log.