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

More complete RPC provider instructions #1665

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

martinvol
Copy link
Contributor

No description provided.

@martinvol martinvol requested review from a team as code owners February 13, 2025 23:24
@martinvol martinvol requested review from gastonponti and ezdac and removed request for a team February 13, 2025 23:24
Copy link

netlify bot commented Feb 13, 2025

Deploy Preview for celo-docs failed.

Name Link
🔨 Latest commit e9c6dfd
🔍 Latest deploy log https://app.netlify.com/sites/celo-docs/deploys/67c59803c9c3830008bdd60b

Copy link
Contributor

@ezdac ezdac left a comment

Choose a reason for hiding this comment

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

Looks good to me.
Added two small nits, approve or change at your own discretion.


After Celo mainnet transitions to L1, validators that are elegible, registered and elected must run PRC nodes in order to be elegible for rewards.
After Celo mainnet transitions to Celo L2, validators that are registered and elected can continue to receive rewards by running PRC nodes. These steps outline how validators can publish the URL of their RPC node in order to be eligible for rewards. To reduce the risk of slashing, these steps should be followed before the L2 upgrade.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
After Celo mainnet transitions to Celo L2, validators that are registered and elected can continue to receive rewards by running PRC nodes. These steps outline how validators can publish the URL of their RPC node in order to be eligible for rewards. To reduce the risk of slashing, these steps should be followed before the L2 upgrade.
After Celo mainnet transitions to Celo L2, validators that are registered and elected can continue to receive rewards by running RPC nodes. These steps outline how validators can publish the URL of their RPC node in order to be eligible for rewards. To reduce the risk of slashing, these steps should be followed before the L2 upgrade.


To register as a RPC provider, a public https url needs to be registered on-chain. To do this, we will register the rpc url in a signed metadata file in the Celo Account.
To register as a RPC provider, a public https url needs to be registered on-chain. To do this, we will register the rpc url in a signed metadata file that's used to share additional information about Celo accounts.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
To register as a RPC provider, a public https url needs to be registered on-chain. To do this, we will register the rpc url in a signed metadata file that's used to share additional information about Celo accounts.
To register as a RPC provider, a public HTTPS URL needs to be registered on-chain. To do this, we will register the RPC URL in a signed metadata file that's used to share additional information about Celo accounts.
The metadata file has to be made available publicly and the URL where this file is hosted has to be registered on-chain as well.

@lvpeschke
Copy link
Contributor

@martinvol there's been more changes to https://docs.celo.org/cel2/operators/community-rpc-node since this PR was created. Any change here that should be salvaged, or shall I close?

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