Skip to content

Validate blobs feature#12574

Merged
terencechain merged 4 commits intodeneb-integrationfrom
validate-blobs-feature
Aug 3, 2023
Merged

Validate blobs feature#12574
terencechain merged 4 commits intodeneb-integrationfrom
validate-blobs-feature

Conversation

@potuz
Copy link
Contributor

@potuz potuz commented Jun 28, 2023

Validate blobs in the beacon chain-package

@potuz potuz requested a review from a team as a code owner June 28, 2023 12:23
@potuz potuz requested review from james-prysm, kasey and rauljordan and removed request for a team June 28, 2023 12:23
@potuz potuz force-pushed the validate-blobs-feature branch from 049445c to 132fbcc Compare June 28, 2023 17:28
Comment on lines +671 to +527
sidecars, err := s.cfg.BeaconDB.BlobSidecarsByRoot(ctx, root)
if err != nil {
return errors.Wrap(err, "could not get blob sidecars")
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

What happens if the sidecars haven't arrived or one of them is missing at this point? What can information can I provide you from sync service or db service to get you to make the right move (wait)

@potuz potuz force-pushed the validate-blobs-feature branch 2 times, most recently from 63404af to ad72215 Compare July 3, 2023 12:28
@potuz potuz force-pushed the validate-blobs-feature branch 2 times, most recently from d2c194e to 8488fba Compare July 14, 2023 11:31
@kasey kasey force-pushed the deneb-integration branch from 2829315 to 20b0fdd Compare July 20, 2023 18:23
@prestonvanloon
Copy link
Member

Can you rebase and/or fix this branch?

@potuz potuz force-pushed the validate-blobs-feature branch 2 times, most recently from 735594e to 5761e38 Compare August 2, 2023 10:39
@potuz potuz force-pushed the validate-blobs-feature branch from 5761e38 to 2b99d6b Compare August 2, 2023 10:41
Copy link
Collaborator

@terencechain terencechain left a comment

Choose a reason for hiding this comment

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

Lgtm

@terencechain terencechain merged commit 03816ea into deneb-integration Aug 3, 2023
@terencechain terencechain deleted the validate-blobs-feature branch August 3, 2023 20:35
james-prysm pushed a commit that referenced this pull request Aug 4, 2023
terencechain pushed a commit that referenced this pull request Aug 4, 2023
terencechain pushed a commit that referenced this pull request Aug 16, 2023
kasey pushed a commit that referenced this pull request Aug 21, 2023
kasey pushed a commit that referenced this pull request Aug 22, 2023
kasey pushed a commit that referenced this pull request Aug 22, 2023
kasey pushed a commit that referenced this pull request Aug 22, 2023
kasey pushed a commit that referenced this pull request Aug 23, 2023
kasey pushed a commit that referenced this pull request Aug 23, 2023
kasey pushed a commit that referenced this pull request Aug 23, 2023
kasey pushed a commit that referenced this pull request Aug 24, 2023
kasey pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 24, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 30, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 30, 2023
prestonvanloon pushed a commit that referenced this pull request Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants