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

Update FLIP 204 #218

Merged
merged 1 commit into from
Nov 7, 2023
Merged

Update FLIP 204 #218

merged 1 commit into from
Nov 7, 2023

Conversation

jordanschalm
Copy link
Member

  • Add more detailed deployment plan, based on timing of Mainnet24 spork.
  • Add link to GitHub epic.

Add more detailed deployment plan, based on timing of Mainnet24 spork. Add link to GitHub epic.
Copy link
Member

@joshuahannan joshuahannan left a comment

Choose a reason for hiding this comment

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

Since it looks like we are wanting to do the epoch smart contract performance upgrades sooner rather than later (maybe within the next 5 weeks, meaning canary upgrade would likely have to be in 3 weeks), do you think you'll have time to get this change into that upgrade, or should we just plan on doing it during a future upgrade?

@jordanschalm
Copy link
Member Author

do you think you'll have time to get this change into that upgrade, or should we just plan on doing it during a future upgrade?

I should have time to implement the change. However, it might be tight to complete the full deployment plan in that time. (I'm not sure how long it takes to get all node operators to upgrade 😬 and this would be a breaking change, including for Observer nodes etc.) I'll aim to have the code ready and will ask around about how long it takes to get upgrades installed by all operators.

@jordanschalm jordanschalm merged commit d7d691b into main Nov 7, 2023
1 check passed
@jordanschalm jordanschalm deleted the jordan/update-flip-204 branch November 7, 2023 18:19
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