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.

Sign up for GitHub

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

Jump to bottom

build(prysm): update default build method to go (forces bazel for minimal) #215

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

build(prysm): update default build method to go (forces bazel for minimal) #215

wants to merge 3 commits into from

Conversation

mattevans
Copy link
Member

No description provided.

@mattevans mattevans self-assigned this Dec 6, 2024
@mattevans mattevans marked this pull request as ready for review December 6, 2024 09:36
@barnabasbusa
Copy link
Collaborator

Lets hold this off actually till we got the minimal build thing working #217

@mattevans
Merge branch 'master' into chore/default-prysm-build-go
5b1c993 
* master:
  autobuild grandine mekong branch
  feat: expose full commit hash to build scripts (#216)
@mattevans
refactor(prysm): force minimal builds to use bazel
c4c1d92
@mattevans
Copy link
Member Author

mattevans commented Dec 12, 2024 •
edited
Loading

@barnabasbusa

lets do mainnet in go, and minimal in bazel

c4c1d92

@mattevans mattevans changed the title build(prysm): update default build method to go build(prysm): update default build method to go (forces bazel for minimal) Dec 12, 2024
@barnabasbusa
Copy link
Collaborator

we good to merge this?

@mattevans
Copy link
Member Author

Yeah, seems ok? Ran another test this morning to be sure.

https://github.com/ethpandaops/eth-client-docker-image-builder/actions/runs/12342543936

beacon (go)

Example kurtosis cfg.

participants:
- el_type: geth
  cl_type: lighthouse
- cl_type: prysm
  cl_image: ethpandaops/prysm-beacon-chain:check-defaults-linux-arm64-11aa51e # beacon, go build.

Booted fine, started syncing.

time="2024-12-15 22:10:18" level=info msg="Synced new block" block=0xf81f287b... epoch=0 finalizedEpoch=0 finalizedRoot=0x00000000... prefix=blockchain slot=12
time="2024-12-15 22:10:18" level=info msg="Finished applying state transition" attestations=1 kzgCommitmentCount=1 payloadHash=0x9b597fe6696f prefix=blockchain slot=12 syncBitsCount=512 txCount=114
time="2024-12-15 22:10:18" level=info msg="Forkchoice updated with payload attributes for proposal" blockRoot=0xf81f287bb9c0 headSlot=12 payloadID=0x03c00af98d4f prefix=blockchain
beacon-minimal (bazel)

Example kurtosis cfg.

participants:
- el_type: geth
  cl_type: lighthouse
- cl_type: prysm
  cl_image: ethpandaops/prysm-beacon-chain:check-defaults-minimal-linux-arm64-11aa51e # beacon-minimal, bazel build.
network_params:
  preset: minimal

Booted fine, started syncing.

time="2024-12-15 22:11:40" level=warning msg="Using minimal config" prefix=node
time="2024-12-15 22:16:06" level=info msg="Synced new block" block=0x79c48f9b... epoch=5 finalizedEpoch=3 finalizedRoot=0x7f77ada7... prefix=blockchain slot=40
time="2024-12-15 22:16:06" level=info msg="Finished applying state transition" attestations=4 payloadHash=0xb9b16c4713b5 prefix=blockchain slot=40 syncBitsCount=32 txCount=100
time="2024-12-15 22:16:06" level=info msg="Forkchoice updated with payload attributes for proposal" blockRoot=0x79c48f9b296e headSlot=40 payloadID=0x0312bcabb174 prefix=blockchain

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@barnabasbusa barnabasbusa barnabasbusa approved these changes

Assignees

@mattevans mattevans

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mattevans @barnabasbusa