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

chore: Cherry-pick go.mod changes into dev-upgrade-15 #9264

Merged
merged 5 commits into from
Apr 19, 2024

Conversation

gibson042
Copy link
Member

Description

Includes commits from the following PRs:

Constructed using the following git rebase -i HEAD todo list:

# pull request #9030 branch mfig-ibc-revamp
## Replace this commit with just its refactoring of golang/cosmos/go.mod
## and subsequent `go mod tidy`.
edit fff392b9a30af2d60a6dffe3427eb5bb5ecdde7b build(cosmos): add `ibc-go` fork and `go mod tidy`
exec git commit --amend --reset-author -m 'build(cosmos): Refactor go.mod to match master'

# pull request #9251 branch 9224-u15-changes
label base-9224-u15-changes
## Resolve conflict using `go mod tidy`.
pick 548e758962ad348816d2b3d1618688829cc7c62b fix: various fixes and features for u15
label 9224-u15-changes
reset base-9224-u15-changes
merge -C c1ec5f12460b1936090f7d76941dfedf8521b544 9224-u15-changes # fix: various fixes and features for u15 (#9251)

# pull request #9263 branch 9224-abci-flag-tests
label base-9224-abci-flag-tests
pick 0d9a63d95d25723720686dfebb1283adcf0c9465 test: pick up cosmos-sdk with more abci flag tests
label 9224-abci-flag-tests
reset base-9224-abci-flag-tests
merge -C 947973cdb72b040410cd56de221c2b154582dff8 9224-abci-flag-tests # test: pick up cosmos-sdk with more abci flag tests (#9263)

Security Considerations

None in particular.

Scaling Considerations

This brings in the ability to switch between "local" and "committing" query client behavior.

Documentation Considerations

Adopted by reference.

Testing Considerations

No additions.

Upgrade Considerations

To be included in agoric-upgrade-15.

gibson042 and others added 5 commits April 19, 2024 16:08
refs: #9224

## Description

Upgrade our cosmos-sdk fork and cosmos/iavl to pick up bug fixes and
features.

### Security Considerations

See upgrade-15 notes.

### Scaling Considerations

N/A

### Documentation Considerations

N/A - internal changes

### Testing Considerations

See upgrade-15 notes.

### Upgrade Considerations

N/A
closes: #9224

## Description

Pick up new version of cosmos-sdk that includes more tests and
documentation for the abci client type setting.

### Security Considerations

N/A

### Scaling Considerations

If the setting is used to revert to the "local" client, query throughput
drops dramatically.

### Documentation Considerations

Docs added to TOML template, but this is an expert-level switch and no
one should use it without a good understanding of the impact.

### Testing Considerations

N/A

### Upgrade Considerations

N/A
@gibson042 gibson042 added the force:integration Force integration tests to run on PR label Apr 19, 2024
@mhofman mhofman added force:integration Force integration tests to run on PR and removed force:integration Force integration tests to run on PR labels Apr 19, 2024
@gibson042 gibson042 merged commit 5996921 into dev-upgrade-15 Apr 19, 2024
82 of 89 checks passed
@gibson042 gibson042 deleted the gibson-upgrade-15-go-mod branch April 19, 2024 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
force:integration Force integration tests to run on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants