-
Notifications
You must be signed in to change notification settings - Fork 21
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
chore: add testnet upgrade data #316
Conversation
// Use staking indexer to generate signet blocks until the most recent | ||
// block possible prior to the upgrade launch. | ||
// sid btc-headers 195552 {latestTipBlockHeight} --home ... --output file/to/btc-headers.json | ||
// script how to generate https://github.com/babylonlabs-io/babylon-deployment/blob/9f39a715e54eedf4842632bbbfbdd353fd16951f/deployments/local/upgrades/write-upgrade-btc-headers.sh#L36 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✔️ tested to generate 28k headers and it worked fine 🎉
Close to the date 9 Dec the headers will be generated and inserted
…testnet-upgrade-data
…te the config of btc nety
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general looks good, there are some followups about particular values.
Lets also wait for 👍 from @babylon-devops team that the parameters and data in upgrade is correct.
"min_commission_rate": "0.03", | ||
"delegation_creation_base_gas_fee": 1000, | ||
"allow_list_expiration_height": 26120, | ||
"btc_activation_height": 220637 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as this are the last parameters in the list and the Babylon node does not have concepts of caps, this will be the params required for all new delegations.
We should decide on those values beforehand and make sure the correct ones are in this file. (can be done in separate pr as this require also some product input)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Great work!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
No description provided.