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

Deprecate SDK 0.43 support #2347

Closed
3 of 8 tasks
Tracked by #3005
adizere opened this issue Jun 28, 2022 · 2 comments · Fixed by #3180
Closed
3 of 8 tasks
Tracked by #3005

Deprecate SDK 0.43 support #2347

adizere opened this issue Jun 28, 2022 · 2 comments · Fixed by #3180
Assignees
Labels
I: dependencies Internal: related to dependencies
Milestone

Comments

@adizere
Copy link
Member

adizere commented Jun 28, 2022

Summary

This is currently blocked on some chains still using SDK 0.42. Refs:

Acceptance Criteria

  • Hermes marks any chains that is using SDK 0.43 or older as incompatible, both in guide & code.

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate milestone (priority) applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@adizere adizere added A: blocked Admin: blocked by another (internal/external) issue or PR labels Jun 28, 2022
@adizere adizere added this to the Backlog milestone Jul 26, 2022
@adizere adizere modified the milestones: Backlog, v1.4 Nov 25, 2022
@seanchen1991 seanchen1991 removed the A: blocked Admin: blocked by another (internal/external) issue or PR label Dec 13, 2022
@seanchen1991 seanchen1991 self-assigned this Feb 24, 2023
@seanchen1991 seanchen1991 added the I: dependencies Internal: related to dependencies label Feb 24, 2023
@romac
Copy link
Member

romac commented Mar 17, 2023

We also need to tighten the bounds on the compatibility check:

const SDK_MODULE_VERSION_REQ: &str = ">=0.41, <0.47";
and
const IBC_GO_MODULE_VERSION_REQ: &str = ">=1.1, <=7";

@adizere
Copy link
Member Author

adizere commented Mar 20, 2023

I agree, upgrading the version requirements to match 0.47 and v7 is quite urgent since I saw multiple operators confused that they're seeing health-check warnings since >1 months ago.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I: dependencies Internal: related to dependencies
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

3 participants