-
Notifications
You must be signed in to change notification settings - Fork 298
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
[Deneb] Remove BlindedBlobSidecar
and clean up block production/publishing flow
#7691
[Deneb] Remove BlindedBlobSidecar
and clean up block production/publishing flow
#7691
Conversation
beacon/validator/src/main/java/tech/pegasys/teku/validator/coordinator/BlockFactoryDeneb.java
Fixed
Show fixed
Hide fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just few nits
beacon/validator/src/main/java/tech/pegasys/teku/validator/coordinator/BlockFactoryDeneb.java
Outdated
Show resolved
Hide resolved
assertThat(blobSidecarsUnblinder.blobsBundle) | ||
.isCompletedWithValue(executionPayloadAndBlobsBundle.getBlobsBundle()); | ||
} | ||
|
||
private void prepareBlockProductionWithPayload( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will have the same flow here for blinded and unblinded? Why shouldSetBlobsBundle_whenAcceptingTheBlobSidecarsUnblinderSelector
is no more needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I basically deleted the method, not sure right now what would be the best way to do this, so will add test when working on the block production.
e35d1ad
to
fe54ade
Compare
PR Description
BlindedBlobSidecar
,SignedBlindedBlobSidecar
and all related classes.BlockFactoryDeneb
BlindBlockUtilDeneb
so simplification thereFixed Issue(s)
helps with block production flow for #7654
Documentation
doc-change-required
label to this PR if updates are required.Changelog