Clarify blob sidecars broadcast section#4772
Clarify blob sidecars broadcast section#4772bharath-123 wants to merge 1 commit intoethereum:masterfrom
Conversation
specs/gloas/validator.md
Outdated
| @@ -165,9 +165,10 @@ construct the `payload_attestations` field in `BeaconBlockBody`: | |||
| #### Blob sidecars | |||
|
|
|||
| The blob sidecars are no longer broadcast by the validator, and thus their | |||
There was a problem hiding this comment.
why does this talk about blob sidecars instead of data column sidecars? seems outdated as well
There was a problem hiding this comment.
you are right here! I can update it too
bharath-123
changed the title
specs/gloas/validator.md
Outdated
| their construction is not necessary unless they self-build. This deprecates the | ||
| corresponding sections from the Honest Validator specifications in the Fulu | ||
| fork, moving them, albeit with some modifications, to the | ||
| [Honest Builder](./builder.md) specifications. |
There was a problem hiding this comment.
If a validator needs to do this, can "the corresponding sections" be deprecated?
There was a problem hiding this comment.
I would think so. Validators still need to construct and broadcast the data column sidecars if they are self-building
bharath-123
force-pushed
the
clarify-blob-sidecars-broadcast
branch
from
b1e7feb to
428ac96
Compare
bharath-123
force-pushed
the
clarify-blob-sidecars-broadcast
branch
from
428ac96 to
43d5dbe
Compare
| the Honest Validator specifications in the Fulu fork, moving them, albeit with | ||
| some modifications, to the [Honest Builder](./builder.md) specifications. | ||
| Unless a validator self-builds, the data column sidecars are no longer broadcast | ||
| by the validator, and thus their construction is not necessary. |
There was a problem hiding this comment.
thus their construction is not necessary
So this is still inaccurate. If validators can/will self-build, their construction is necessary.
This sentence needs a complete re-write in my opinion.
There was a problem hiding this comment.
yup it does require more clarification
There was a problem hiding this comment.
what about something like:
Data column sidecar construction and broadcast will be handled by builders.
Validators will still have to construct and broadcast the data column sidecars if
they choose to self-build.
|
If the validator choose to self-build, then they are also a builder, therefore they have to act in accordance with the honest builder spec. I see no points to specify this in validator spec. so I think the current spec is clear enough, except the blob/data column sidecars confusion. |
|
I'm going to open a new PR which simply deletes this section. |
4 participants
In Glamsterdam, validators who self-build (i.e are both the validator and the builder for the slot) still have to broadcast the
BlobSidecarsby themselves. The spec today mentions that validators do not need to broadcast blob sidecars which feels a bit misleading as validators would still have to broadcast the blob sidecars if they self-build.Also post-Fulu, validators will broadcast
DataColumnSidecarsinstead ofBlobSidecars.