test: pick up cosmos-sdk with more abci flag tests #9263
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Deploying agoric-sdk with
|
| Latest commit: |
0d9a63d
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4be3c2b9.agoric-sdk.pages.dev |
| Branch Preview URL: | https://9224-abci-flag-tests.agoric-sdk.pages.dev |
gibson042
approved these changes
Apr 19, 2024
JimLarson
force-pushed
the
9224-abci-flag-tests
branch
from
f90a214 to
0d9a63d
Compare
gibson042
pushed a commit
that referenced
this pull request
Apr 19, 2024
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
added a commit
that referenced
this pull request
Apr 19, 2024
## Description Includes commits from the following PRs: * #9030 (partial cherry-pick; refactors go.mod) * #9251 * #9263 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 fff392b 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 548e758 fix: various fixes and features for u15 label 9224-u15-changes reset base-9224-u15-changes merge -C c1ec5f1 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 0d9a63d test: pick up cosmos-sdk with more abci flag tests label 9224-abci-flag-tests reset base-9224-abci-flag-tests merge -C 947973c 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.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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