osmosis
osmosis copied to clipboard
osmosis-labs
SDK v50 Upgrade - Cherry-Picks to Fork
From pre v0.47:
Features:
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/7dbb553fe1716e06c5885a7eac53a1eaa1f4ed05
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/07eae85a8608205a3ff8b4f33b9789eea9395556
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/545021100d920845ce96dad62149863f22b24f0c
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/09e0036436d85fd4b558aa473792e6a9e58256dc
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/05c92b5b93abc4eb77959ddb6d10a00944aa4096
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/b1b5c4a3e3e713337db79790f3eefddacd196714
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/e57bb6e68f1fa322353e13e11efbc9578fae95b1
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/c3c7ab7f52ea5383cfa42c5521b45bc0930b24a9
From post v0.47:
More CI
More CI
Consider adding test backinto integration
For the IAVL wrapper (I dont think this one is needed)
- [ ] https://github.com/osmosis-labs/cosmos-sdk/commit/1434d548a938399772b52a1e9736a4f64fd3b051
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/c4a2b329734f7cfa0b36c10cd05d320e8dfa4140
No longer needed now that Balances is used
- [ ] https://github.com/osmosis-labs/cosmos-sdk/commit/a836ab6f581555f34aa6aa0042d356a9b14a91bd
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/e0f9e4f30a4accb8a350e3bf53e1b02de4fd22a0
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/11b6186e0b308c48b624b376ec20e2ecfd01fa4a
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/8d82953d3433b9c6ff281b1504dcdbf36ae36e43
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/cf09bb59fa0ba3c5d96f3018dbd74131fccef57c
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/ab4bc056639199fce60f3275cf5b24acebc22f2d
Make sure slashing migration changes don't need to be backported (they do, this is where we change key)
Its unclear to me if these wrapper changes are needed, we don't use cometbft-db anymore and use cosmos-db
- [ ] https://github.com/osmosis-labs/cosmos-sdk/commit/dead65a30794f203283e5279cbc0ea1e54e8ea68
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/0a12f9961196369aa0b056e66cd398c8d0838cba
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/de3aaa1afcf4666d61023bbcd11ef86c9cb2cbdf
- [X] https://github.com/osmosis-labs/cosmos-sdk/commit/e8c4bae3796bb27450adbf6ddddf7593a22f4265
Questionable PRs After First Review
Skipping this one because denomOwners looks to be tracked in its own way now
Skipping this one because we don't have the option to get a param as a byte slice anymore
Might need this one when I run e2e in CI
Might need this one when I run e2e in CI
Its unclear to me that this speedup was already implemented in some other way, need Dev to look
Double check that no in-place-testnet changes were made that weren't upstreamed
For the IAVL wrapper (I dont think this one is needed)
No longer needed now that Balances is used
Its unclear to me if these wrapper changes are needed, we don't use cometbft-db anymore and use cosmos-db
Consider adding test backinto integration
