MINI_EPIC: Wormchain should upgrade from Tendermint to (not necessarily latest)CometBFT

[jonathanpberger]

Tasks

Beta Give feedback

1. SPIKE: review TILT test {suite, tool} . #52
   1 of 2
   +0
   
2. Network-accessible TILT servers should be available #61
   6 of 8
   +0
   
3. write tilt tests

[pharr117]

There are currently 2 options for migration that are being explored:

1. Do a manual migration and resolution of changes by replacing the current branches' core library from Tendermint to CometBFT - work captured here https://github.com/strangelove-ventures/wh-cosmos-sdk/tree/pharr117/tendermint-to-cometbft
2. Bump the patch version of the Wormhole Cosmos SDK from v0.45.9 to v0.45.16, where v0.45.15 was the version that made the migration - work captured here https://github.com/strangelove-ventures/wh-cosmos-sdk/tree/strangelove-wormhole-chain-v0.45.16

Option 1 provides the following benefits:

1. No need to check subsequent versions for breakages as this just changes the cometbft dependency version
2. Easy one line change lacking a full audit

Option 2 provides the following benefits:

1. Bumps the v0.45 line to latest v0.45 with bug fixes and security updates
2. A more canonical way to migrate to CometBFT as the migration was handled by the core Cosmos SDK team

We are leaning toward (2), but will need to ensure bumping to latest v0.45 does not introduce changes that the Wormhole project wants to avoid, like consensus breaking changes requiring a coordinated upgrade. We will also need to ensure that the change does not impact the Wormhole chain repo too significantly.

[pharr117]

ADR here #54. This ADR goes with option (2) above. This will require extensive analysis for consensus breaking changes/requirements for coordinated upgrade. One of the goals of this migration is to achieve it without a coordinated upgrade, as that would require the team to push off the change to a later date.

[jonathanpberger]

Done and in a branch; @pharr117 will cut a PR this week and then we'll close the ticket.

[jonathanpberger]

@pharr117 sez "for these early PRs, we'll just run TILT (and possibly ICT if it's ready"

[jonathanpberger]

Blocking this story til TILTS VPS situation is sorted (#61 )