Shawn

After debugging with @jtremback it seems this issue can be solved by removing the custom tendermint config values [here](https://github.com/cosmos/interchain-security/blob/main/integration-tests/actions.go#L110), and instead using the default values for ```timeout_commit``` and ```peer_gossip_sleep_duration```. Is...

@eswak this might have to do with shorter block times. I'm facing this issue on a 200ms block time sidechain

Set this to draft, but once the merge conflicts are fixed I'm happy to review

There's an sdk pattern that binary names have a `d` appended, was there reasoning around avoiding that pattern? Ex: `providerd` or `provider-d`

If we're taking the time to change/review the binary names I'd prefer we do it the proper way with the d appended, thanks @faddat

@MSalopek I assigned you cause I bet you can knock this out in minutes :) thanks

Regarding lessons for the future: Standardizing test conventions, intention, organization, etc. should be the first thing we prioritize for a new repo! Test standardization should not be an afterthought that...

My two cents on this issue -> someone with a deep understanding of the protocol would need to audit what we currently call integration tests. If the auditor cannot decipher...

Here's my thoughts: The idea of boosting the downtime signal does not seem like the way to go imo. Relying on a subset of headers to make extrapolated decisions about...

Re `Submit the last N LastCommitInfo in IBC packet metadata`, maybe the LastCommitInfos could be included in the ack for VSC packets. But this solution is not the most elegant..