Skip to content
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

PART 2 - GET/Attestation Pool API - Retrieve attestations and metadata #8437

Merged
merged 3 commits into from
Jul 22, 2024

Conversation

mehdi-aouadi
Copy link
Contributor

@mehdi-aouadi mehdi-aouadi commented Jul 10, 2024

PR Description

Rebased on top of #8431
Retrieve attestations and metadata for the GET attestations from aggregation pool API

Fixed Issue(s)

#8029

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

@mehdi-aouadi mehdi-aouadi self-assigned this Jul 10, 2024
@mehdi-aouadi mehdi-aouadi force-pushed the 8029-2-node-data-provider branch 2 times, most recently from 818a283 to c4ba502 Compare July 18, 2024 08:50
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

  • Added spec parameter to NodeDataProvider constructor in DataProvider.java
  • Implemented getAttestationsAndMetaData tests in NodeDataProviderTest.java
  • Modified getAttestations in AggregatingAttestationPool.java to handle committee bits
  • Updated attestation filtering based on schema definitions for given slot
  • Enhanced error handling for missing slot and empty attestations

3 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings

@mehdi-aouadi mehdi-aouadi force-pushed the 8029-2-node-data-provider branch from c4ba502 to 496f3c9 Compare July 19, 2024 11:42
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

(updates since last review)

  • Refactored waitForNonDefaultExecutionPayload in acceptance-tests/src/testFixtures/java/tech/pegasys/teku/test/acceptance/dsl/TekuBeaconNode.java
  • Updated SyncCommitteeSubnetSubscription usage in beacon/validator/src/main/java/tech/pegasys/teku/validator/coordinator/ValidatorApiHandler.java
  • Added getAttestationsAndMetaData method in data/provider/src/main/java/tech/pegasys/teku/api/NodeDataProvider.java
  • Introduced PostSyncCommitteeData class in ethereum/json-types/src/main/java/tech/pegasys/teku/ethereum/json/types/validator/PostSyncCommitteeData.java
  • Added support for Electra milestone in data/provider/src/main/java/tech/pegasys/teku/api/ValidatorDataProvider.java

37 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings

@mehdi-aouadi mehdi-aouadi requested a review from tbenr July 22, 2024 09:13
Copy link
Contributor

@tbenr tbenr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mehdi-aouadi mehdi-aouadi enabled auto-merge (squash) July 22, 2024 11:28
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

(updates since last review)

This pull request introduces the GET/Attestation Pool API to retrieve attestations and metadata, enhancing the attestation retrieval process.

  • acceptance-tests/src/acceptance-test/java/tech/pegasys/teku/test/acceptance/MergedGenesisAcceptanceTest.java: Updated genesisExecutionPayloadHeaderSource to use a lambda function for correct payload creation.
  • data/provider/src/main/java/tech/pegasys/teku/api/DataProvider.java: Added recentChainData parameter to NodeDataProvider instantiation for accurate data retrieval.
  • data/provider/src/main/java/tech/pegasys/teku/api/NodeDataProvider.java: Introduced RecentChainData dependency and getSlot method for improved attestation retrieval.
  • validator/remote/src/main/java/tech/pegasys/teku/validator/remote/typedef/OkHttpValidatorMinimalTypeDefClient.java: Added functionality to retrieve Genesis data.
  • validator/remote/src/integration-test/java/tech/pegasys/teku/validator/remote/typedef/handlers/GetGenesisRequestTest.java: Added integration tests for GetGenesisRequest to ensure correct handling of HTTP responses.

14 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings

@mehdi-aouadi mehdi-aouadi merged commit e5994f3 into Consensys:master Jul 22, 2024
16 checks passed
@mehdi-aouadi mehdi-aouadi deleted the 8029-2-node-data-provider branch July 22, 2024 12:10
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.

2 participants