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.

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

nimbus.kiln: enable --*-light-client-data #89

Merged
merged 1 commit into from
Mar 28, 2022
Merged

nimbus.kiln: enable --*-light-client-data #89

merged 1 commit into from
Mar 28, 2022

Conversation

etan-status
Copy link
Contributor

No description provided.

@etan-status etan-status requested a review from jakubgs March 18, 2022 23:50
@etan-status
Copy link
Contributor Author

  • Rebase to master.

@etan-status etan-status marked this pull request as ready for review March 25, 2022 15:20
@etan-status etan-status marked this pull request as draft March 25, 2022 15:22
@etan-status etan-status marked this pull request as ready for review March 25, 2022 15:23
@etan-status etan-status marked this pull request as draft March 25, 2022 16:09
@etan-status
Copy link
Contributor Author

  • Updated variable names, and marked as draft until the kiln branch is up to date and actually has support for the new feature.

@etan-status
Copy link
Contributor Author

  • Rename branch to kiln-dev-auth.

@etan-status etan-status marked this pull request as ready for review March 28, 2022 06:39
@etan-status
Copy link
Contributor Author

  • Move light client variables below eth1 sync config and separate with comment

@@ -38,12 +38,15 @@ geth_websocket_port: '{{ 9546 + idx|int + 1 }}'
beacon_node_service_name: 'beacon-node-{{ beacon_node_network }}-devauth-{{ "%02d"|format(idx|int+1) }}'
beacon_node_network: 'kiln'
beacon_node_proposer_boosting_debug: true
# FIXME: Pinned because bf127920 breaks the testnet config.
beacon_node_repo_branch: 'a35c5f83a8f7880165e19784f73463663db9fdd8'
beacon_node_repo_branch: 'kiln-dev-auth'
Copy link
Contributor Author

@etan-status etan-status Mar 28, 2022

Choose a reason for hiding this comment

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

@jakubgs
Copy link
Member

jakubgs commented Mar 28, 2022

When I try to deploy this I get:

{"lvl":"FAT","ts":"2022-03-28 07:05:59.468+00:00","msg":"Could not load head state, database corrupt?","head":"3f54beeb:1599","tail":"d30e447c:0","genesis":"d30e447c:0"}

If I try to start from a fresh DB I get:

{"lvl":"FAT","ts":"2022-03-28 07:32:51.131+00:00","msg":"No database and no genesis snapshot found: supply a genesis.ssz with the network configuration, or compile the beacon node with the -d:has_genesis_detection option in order to support monitoring for genesis events","topics":"beacnde"}

So the current state of kiln-dev-auth is not usable. The second issue appears to be fixed in status-im/nimbus-eth2#3549.

@jakubgs
Copy link
Member

jakubgs commented Mar 28, 2022

Deployment worked after merging status-im/nimbus-eth2#3549:

 > a nimbus.kiln -a "systemctl cat 'beacon-node-*' | grep light-client"      
metal-01.he-eu-hel1.nimbus.kiln | CHANGED | rc=0 >>
    --serve-light-client-data=true \
    --import-light-client-data=only-new \
    --serve-light-client-data=true \
    --import-light-client-data=only-new \
    --serve-light-client-data=true \
    --import-light-client-data=only-new \
    --serve-light-client-data=true \
    --import-light-client-data=only-new \

@jakubgs jakubgs merged commit 91cc24b into status-im:master Mar 28, 2022
@etan-status etan-status deleted the lc-kiln branch March 29, 2022 09:55
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.

2 participants