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.

Sign up for GitHub

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

Jump to bottom

integrate ICS with LSM changes #2682

Closed
Tracked by #2649
MSalopek opened this issue Aug 3, 2023 · 0 comments · Fixed by #2643
Closed
Tracked by #2649

integrate ICS with LSM changes #2682

MSalopek opened this issue Aug 3, 2023 · 0 comments · Fixed by #2643
Assignees
@MSalopek
Labels
dependencies Pull requests that update a dependency file state-machine-breaking State machine breaking changes (impacts consensus). type: tech-debt Slows down development in the long run

Comments

@MSalopek
Copy link
Contributor

MSalopek commented Aug 3, 2023 •
edited
Loading

Gaia needs to use a version of ICS v2 that supports the LSM module in order to fully support the feature without any impact on Gaia's livenes.

The changes are state breaking for Gaia and warrant a governance vote to upgrade.

Proposed timeline:

After LSM changes are introduced to cosmos-sdk make sure to bump ICS to the correct version that supports the LSM functionality.

ICS changes will likely land into version v2.1

The issue tracking changes to ICS: cosmos/interchain-security#1183

Instructions

When the ICS PR integrating LSM is merged and ICS v2.1 is cut, refactor gaia to use the latest supported versions of:

  • cosmos-sdk (v0.45.16-lsm-ics)
  • interchain-securirt (v2.1.0)
@MSalopek MSalopek mentioned this issue Aug 3, 2023
Closed
@MSalopek MSalopek self-assigned this Aug 3, 2023
@MSalopek MSalopek added state-machine-breaking State machine breaking changes (impacts consensus). dependencies Pull requests that update a dependency file type: tech-debt Slows down development in the long run labels Aug 3, 2023
@github-project-automation github-project-automation bot moved this to 🩹 Triage in Cosmos Hub Aug 3, 2023
@MSalopek MSalopek moved this from 🩹 Triage to 🏗 In progress in Cosmos Hub Aug 3, 2023
@MSalopek MSalopek mentioned this issue Aug 3, 2023
Merged
8 tasks
@MSalopek MSalopek moved this from 🏗 In progress to 👀 In review in Cosmos Hub Aug 8, 2023
@mpoke mpoke moved this from 👀 In review to 🛑 Blocked in Cosmos Hub Aug 8, 2023
@mpoke mpoke moved this from 🛑 Blocked to 🏗 In progress in Cosmos Hub Aug 16, 2023
@mpoke mpoke moved this from 🏗 In progress to 👀 In review in Cosmos Hub Aug 16, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Cosmos Hub Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MSalopek MSalopek

Labels
dependencies Pull requests that update a dependency file state-machine-breaking State machine breaking changes (impacts consensus). type: tech-debt Slows down development in the long run
Projects
Cosmos Hub
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

deps!: use cosmos-sdk v0.45.16-lsm-ics cosmos/gaia
1 participant
@MSalopek