Skip to content

Fix default values and move checkpoint sync URLs to network variants #63

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

Merged
Marketen merged 3 commits into from
Apr 8, 2026
Merged

Fix default values and move checkpoint sync URLs to network variants #63

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
059d035
Initial plan
Copilot Apr 1, 2026
142e8dd
Replicate Prysm changes: fix defaults and checkpoint sync URLs
Copilot Apr 1, 2026
6e294b3
Apply same port and checkpoint sync URL changes to gnosis variant
Copilot Apr 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 2 additions & 3 deletions docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,7 @@ services:
- seccomp:unconfined
environment:
LOG_LEVEL: info
CHECKPOINT_SYNC_URL: ""
FEE_RECIPIENT_ADDRESS: ""
FEE_RECIPIENT_ADDRESS: "0x0000000000000000000000000000000000000000"
EXTRA_OPTS: ""
validator:
build:
Expand All @@ -34,7 +33,7 @@ services:
GRAFFITI: validating_from_DAppNode
DOPPELGANGER_PROTECTION: "true"
EXTRA_OPTS: ""
FEE_RECIPIENT_ADDRESS: ""
FEE_RECIPIENT_ADDRESS: "0x0000000000000000000000000000000000000000"
BACKUP_BEACON_NODES: ""
volumes:
beacon-data: {}
Expand Down
4 changes: 3 additions & 1 deletion package_variants/gnosis/docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,11 @@ services:
P2P_PORT: 19012
QUIC_PORT: 19013
ports:
- 19012:19012/tcp
- 19012:19012
- 19012:19012/udp
Comment on lines +10 to 11
Copy link
Copy Markdown
Collaborator

@nflaig nflaig Apr 1, 2026

Choose a reason for hiding this comment

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

we don't need the extra udp case, I think the default is both udp and tcp if you don't specify it, curious why this is changed at all thouhg

Copy link
Copy Markdown
Contributor

@Marketen Marketen Apr 1, 2026

Choose a reason for hiding this comment

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

I think its worth it to leave it like this for now. Dappmanager expects these defaults

- 19012:19012
- 19012:19012/udp

and restarts the package overwriting previous port config to this one.

Ideal fix is changing dappmanager behaviour and make it accept whatever comes with the package, but this works for now and AFAIK is harmless to the package

- 19013:19013/udp
environment:
CHECKPOINT_SYNC_URL: "https://checkpoint-sync-gnosis.dappnode.net"
validator:
build:
args:
Expand Down
4 changes: 3 additions & 1 deletion package_variants/hoodi/docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,11 @@ services:
P2P_PORT: 9712
QUIC_PORT: 9713
ports:
- 9712:9712/tcp
- 9712:9712
- 9712:9712/udp
- 9713:9713/udp
environment:
CHECKPOINT_SYNC_URL: "https://checkpoint-sync-hoodi.dappnode.net"
validator:
build:
args:
Expand Down
4 changes: 3 additions & 1 deletion package_variants/mainnet/docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,11 @@ services:
P2P_PORT: 9112
QUIC_PORT: 9113
ports:
- 9112:9112/tcp
- 9112:9112
- 9112:9112/udp
- 9113:9113/udp
environment:
CHECKPOINT_SYNC_URL: "https://checkpoint-sync.dappnode.net"
validator:
build:
args:
Expand Down