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

Upgrade to SDK v0.47 #2413

Closed
9 of 13 tasks
mpoke opened this issue Apr 17, 2023 · 2 comments
Closed
9 of 13 tasks

Upgrade to SDK v0.47 #2413

mpoke opened this issue Apr 17, 2023 · 2 comments
Assignees
Labels
admin: key-result A key result (in the context of OKRs) status: waiting-triage This issue/PR has not yet been triaged by the team.

Comments

@mpoke
Copy link
Contributor

mpoke commented Apr 17, 2023

Problem

Currently, Gaia uses SDK v0.45 which leads to the following issues:

  • The changes needed for Interchain Security are in a custom SDK release (e.g., https://github.com/cosmos/cosmos-sdk/releases/tag/v0.45.15-ics). These changes are part of v0.47. Thus, upgrading would enable us to drop the -ics suffix.
  • IBC 4 will be EoL in August, 2023. IBC 5 or newer requires SDK v0.46 or newer.
  • The Liquid Staking module (LSM) will not be backported to SDK v0.45. This means that if the community decides to add the LSM to the Hub, we need SDK v0.47.
  • SDK v0.45 is EoL. Although fixes to security bugs are still backported to the custom SDK release for ICS, bug fixes are not.

Closing criteria

Gaia release with SDK v0.47.

Task list

Must have

  1. state-machine-breaking type: tech-debt
    mpoke
  2. dependencies
    sainoe
  3. sdk-v0.47 type: tech-debt
    sainoe
  4. sainoe
  5. type: tech-debt
  6. S: ImprovingThings

Tasks

  1. S: Productivity scope: testing
    sainoe
  2. S: Productivity scope: testing
    MSalopek bermuell
  3. type: tech-debt
    ruthishvitwit
@mpoke mpoke added sdk-v0.47 admin: key-result A key result (in the context of OKRs) labels Apr 17, 2023
@mpoke mpoke mentioned this issue Apr 17, 2023
@mpoke mpoke added the status: waiting-triage This issue/PR has not yet been triaged by the team. label Apr 17, 2023
@faddat
Copy link
Contributor

faddat commented Apr 18, 2023

This issue doesn't reflect the findings that the -ics suffix is still going to be needed, discussed with the informal team yesterday. This will affect both consumer and producer chains using ics.

This was referenced May 8, 2023
@mpoke mpoke pinned this issue May 9, 2023
@mpoke mpoke assigned sainoe and MSalopek and unassigned glnro Jun 8, 2023
@mpoke
Copy link
Contributor Author

mpoke commented Jun 28, 2023

Closing in favor of #2540

@mpoke mpoke closed this as completed Jun 28, 2023
@mpoke mpoke unpinned this issue Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
admin: key-result A key result (in the context of OKRs) status: waiting-triage This issue/PR has not yet been triaged by the team.
Projects
Status: ✅ Done
Development

No branches or pull requests

5 participants