-
Notifications
You must be signed in to change notification settings - Fork 19
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
feat: stargate upgrade #85
Merged
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
daeMOn63
force-pushed
the
feature/stargate
branch
2 times, most recently
from
April 22, 2021 14:53
40dcdfb
to
34830ac
Compare
* crypto/hd: add 'm/' prefix to hd path * update fundraiser path * fix some tests * tests * fix test case * changelog * fix ledger tests
* ibc: commit packet and hash * minor changes * fix build * update err msgs * don't use Any * update commitment hash to used fixed length preimage The commitment hash will now use 8 bytes timeout timestamp + 8 bytes timeout height version number + 8 bytes timeout height version height + sha 256 hash of data * remove error from CommitPacket return values * update godoc * rm proto-tools-stamp Co-authored-by: Colin Axner <colinaxner@berkeley.edu> Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: SaReN <sahithnarahari@gmail.com>
…… (#7839) * ADR-031: elaborate consequences and encapsulation of module client-server implementation * Update docs/architecture/adr-031-msg-service.md Co-authored-by: Aaron Craelius <aaron@regen.network> Co-authored-by: Aaron Craelius <aaron@regen.network>
Testable examples were accidentally converted into tests.
* init commit * ibc: remove redundant casting * ci: attempt fix liveness job
…(#7989) * types/Coin: compile and reuse Regexps to reduce massive RAM+CPU burn Fixes a resource (CPU+RAM) burn from recompiling Regexps which would consume 1.2+MiB of RAM and 1.1ms per ParseCoin invocation, which means that 1,000 invocations would consume 1.2GiB of RAM and >1second. With the change, ParseCoin now takes 3.2us down from 1.1ms, with dramatic results: name old time/op new time/op delta ParseCoin-8 1.04ms ± 4% 0.00ms ± 1% -99.69% (p=0.000 n=9+10) name old alloc/op new alloc/op delta ParseCoin-8 1.20MB ± 0% 0.00MB ± 0% -99.94% (p=0.000 n=10+10) name old allocs/op new allocs/op delta ParseCoin-8 17.8k ± 0% 0.0k ± 0% -99.89% (p=0.000 n=10+10) Fixes #7986. * replace the CoinDenomRegex variable with SetCoinDenomRegex() (#7998) Co-authored-by: Federico Kunze <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Alessio Treglia <alessio@tendermint.com>
* use TrustingPeriod instead of UnbondingPeriod when verifying the age of the trusted consensus state * update comment
* Fix showing height in rest endpoints * Fix query height * Manually inject req.Height * Small tweaks * Use withHeight Co-authored-by: Aleksandr Bezobchuk <alexanderbez@users.noreply.github.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* upgrade progress * fix build minus cli * write state at block before upgrade height * refix build * Apply suggestions from code review Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com> * fix upgrade and start with tests * fix tendermint tests * add tests and remove unnecessary relayer options on upgradedClient * fix all tests except weird msg panic * add invalid final client test and codec stuff * fix everything but msg issue * remove problematic test for now * safer self validation * upgrade fixes * proto-linting * remove unnecessary last height * add timestamp to committed upgrade consensus state * address final nits * address nit Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com> Co-authored-by: Federico Kunze <31522760+fedekunze@users.noreply.github.com>
Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com>
* Add REST endpoints migration guide * add bank, distrib, evidence * Add slashing, mint * Add staking * Add query * Add todo * Typos * Add link to interact node * grammar * Better wording * Update docs/migrations/rest.md Co-authored-by: Robert Zaremba <robert@zaremba.ch> * Update docs/migrations/rest.md * Add footnote Co-authored-by: Robert Zaremba <robert@zaremba.ch>
* Update "Core Concepts" docs * Update tags * Update more subsctions * Revert basics * ADd link to run-node * Typo * Add sign modes * Finish transaction.md * Update docs/core/baseapp.md Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com> * Update docs/core/baseapp.md Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com> * Update docs/core/baseapp.md Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com> * Update docs/core/baseapp.md Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com> * Update docs/core/baseapp.md Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com> * Update docs/core/baseapp.md Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com> * Wording * Update docs/core/baseapp.md Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com> * Revert basics changes * revert space change * typo * Update docs/core/transactions.md Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com> * Update docs/core/node.md Co-authored-by: Robert Zaremba <robert@zaremba.ch> * Update docs/core/store.md Co-authored-by: Robert Zaremba <robert@zaremba.ch> Co-authored-by: Marie Gauthier <marie.gauthier63@gmail.com> Co-authored-by: Robert Zaremba <robert@zaremba.ch> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* add generate identifier functions for connection and channel * update proto types and begin open init changes remove unnecessary field from open init msgs. Update connectionopeninit handshake, begin updating channel handshake * finish connection handshake chanes and some channel handshake changes Finish updating connection handshake changes, update genesis for connection and channel, some channel handshake changes, fix build for tests but still failing * finish channel handshake changes * fix more tests * fix connection handshake tests * fix more tests * fix build * changes from self review * address @fedekunze review suggestions * update spec * fix build * fix tests * Update x/ibc/testing/chain.go Co-authored-by: Aditya <adityasripal@gmail.com> * Update x/ibc/core/03-connection/types/msgs.go Co-authored-by: Aditya <adityasripal@gmail.com> * address @AdityaSripal comments * reflect spec changes * address @AdityaSripal review suggestions * nit * add connection/channel identifier parsing and validation as per @AdityaSripal suggestion, cc @fedekunze * Fix auth rest test Change dummy channel id to a valid channel id given the updated stricter channel identifier checks cc @amaurymartiny Co-authored-by: Aditya <adityasripal@gmail.com>
* rename Version to Revision in ibc Height * lint Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Aleksandr Bezobchuk <alexanderbez@users.noreply.github.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* Add v038 upgrade types * Add migrate x/upgrade * remove nolint * Add ParameterChangeProposal * Revert change * Don't register crypto twice Co-authored-by: Anil Kumar Kammari <anil@vitwit.com> Co-authored-by: Federico Kunze <31522760+fedekunze@users.noreply.github.com>
Remove terminated containers once their execution has finished. This saves the users a bit of disk space. Closes: #8023
* Add tests for query txs * Add test for IBC * Refactor checkSignModeError * Fix lint * Add successful IBC test * Remove logs * break * Remove known errors * Update error message * Better comments * Revert variable renaming * Fix lint * Add mention of gRPC as TODO
Co-authored-by: Marko <marbar3778@yahoo.com> (cherry picked from commit 0836361) Co-authored-by: Hanjun Kim <hallazzang@gmail.com>
Co-authored-by: Barrie Byron <barrie.byron@tendermint.com> Co-authored-by: Alessio Treglia <alessio@tendermint.com> (cherry picked from commit 1a4418b) Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com>
Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>
Closes: #8923 (cherry picked from commit 7b09f95) Co-authored-by: Alessio Treglia <alessio@tendermint.com>
* Staking spec updates (#8843) * Overview of keepers in object capability model (OCM) * Updates to the spec, making clarifications * Create a sequence diagram of a (fresh) delegation * Misc notes, not yet decided where to put them * Description of the shares abstraction in validators * Model all keeper dependencies and move the UML file to docs * Move and rename delegation sequence diagram * Move shares description * Remove TODO * Diagram touch-ups * Add how consensus power is calculated * remove temp file * Diagram improvements * Describe slashing in more detail * Describe redelegation * Describe unbonding * Delegation updates * Delegation updates * Make a diagram describing overall transaction flow * Add delegation flows for the events of tokens being bonded/unbonding/etc. * Grammar fix * Diagram updates: distinguish alts, remove numbering. * Use groups instead of "func:" participants * Remove unused keepers from dependency diagram * Add title to unbonding diagram * Move keeper dependencies * small doc updates * remove numbers on sequence diagram * !!!WIP EndBlock * Explain "Last"-prefix in storage * Remove `panic` step (they are supposed to never happen) * EndBlock sequence diagram (with TODOs) * Add TODO * More visible TODOs * Remove numbering * Complete EndBlock * Remove numbering * Remove TODOs and update title * add title back * remove endblock seq-diagram * spec updates * Make power index update conditional on not being jailed * update title Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> (cherry picked from commit 0b24970) Co-authored-by: Rikard Hjort <8545447+hjorthjort@users.noreply.github.com> Co-authored-by: Amaury M <1293565+amaurym@users.noreply.github.com>
…(#9041) Co-authored-by: Alessio Treglia <alessio@tendermint.com>
Co-authored-by: Alessio Treglia <alessio@tendermint.com> Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com>
* bump tendermint core Depends on 0.34.9. * fix FTBFS
* update changelog for v0.42.4 * Update RELEASE_NOTES.md * Update CHANGELOG.md Co-authored-by: Barrie Byron <barrie.byron@tendermint.com> * Update CHANGELOG.md Co-authored-by: Barrie Byron <barrie.byron@tendermint.com> * Update RELEASE_NOTES.md Co-authored-by: Barrie Byron <barrie.byron@tendermint.com> * Update CHANGELOG.md Co-authored-by: Barrie Byron <barrie.byron@tendermint.com> Co-authored-by: Alessio Treglia <alessio@tendermint.com> Co-authored-by: Barrie Byron <barrie.byron@tendermint.com>
* feat: custom minting with fixed rate inflation * fix: set default minter inflation to 3% * fix: tests from mint default settings changes * fix: mint simulation tests
the power reduction have to be applied by the application itself
ejfitzgerald
previously approved these changes
May 5, 2021
* chores: bump tendermint to v0.34.10 * fix show multisig query (#9108) closes: #9056 * update legacy rest swagger (#9049) (#9087) Co-authored-by: SaReN <sahithnarahari@gmail.com> Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> * crypto/types: check for overflow and unreasonably large element count (#9173) From: #9163 Fixes: #9162 Thanks: @odeke-em for the patch. Co-authored-by: Emmanuel T Odeke <emmanuel@orijtech.com> * backport: make NewDecFromStr returns error for too large decimal (#9175) From: #9157 Closes: #9160 Co-authored-by: Cuong Manh Le <cuong.manhle.vn@gmail.com> * backport 0.42.x: Fix IBC doc links (#9210) closes: #9180 * setup: reuses proto container (backport #9192) (#9203) Co-authored-by: marbar3778 <marbar3778@yahoo.com> Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> (cherry picked from commit 71b7fb8) Co-authored-by: Robert Zaremba <robert@zaremba.ch> * Add more details in error message for invalid vote option (#9185) (#9217) Co-authored-by: Marko <marbar3778@yahoo.com> Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> (cherry picked from commit 15edf3c) Co-authored-by: Devashish Dixit <devashish@crypto.com> * x/gov/keeper: fix flaky TestPaginatedVotesQuery (backport #9223) (#9225) * x/gov/keeper: fix flaky TestPaginatedVotesQuery (#9223) When testing with -race, sometimes the random source generate the same string for consecutive calls, causing duplicated voter address. So the number of votes in DB is not 20. To fix this, we ensure unique addresses are generated, by using a map for tracking which one was produced, and skip the duplicated address and generated new one. Testing with: go test -race -v -count=1000 -run=TestPaginatedVotesQuery now passes. Updates #9010 (cherry picked from commit 6ad84c5) # Conflicts: # x/gov/keeper/querier_test.go * fi merge conflict Co-authored-by: Cuong Manh Le <cuong@orijtech.com> Co-authored-by: Amaury M <1293565+amaurym@users.noreply.github.com> * Keyring migrate command doc (#8979) (#9254) Co-authored-by: SaReN <sahithnarahari@gmail.com> Co-authored-by: Barrie Byron <barrie.byron@tendermint.com> Co-authored-by: Alessio Treglia <alessio@tendermint.com> Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> (cherry picked from commit d5f5fe6) Co-authored-by: Andrei Ivasko <cyberbono3@gmail.com> * store/internal: validate keys before calling ProofsFromMap (backport #9235) (#9247) Fixes #9233 (cherry picked from commit 711976e) Co-authored-by: Cuong Manh Le <cuong@orijtech.com> Co-authored-by: Amaury M <1293565+amaurym@users.noreply.github.com> * Update chain migration docs to use v0.42 (#9090) (#9317) * Update docs * Tweak Co-authored-by: Marko <marbar3778@yahoo.com> (cherry picked from commit 47c399f) Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> * Update IBC overview document to latest ICS-24 link (backport #9044) (#9316) * Update overview.md (#9044) Co-authored-by: SaReN <sahithnarahari@gmail.com> (cherry picked from commit cad3987) # Conflicts: # docs/ibc/overview.md * Fix conflits Co-authored-by: Calvin Lau <38898718+calvinaco@users.noreply.github.com> Co-authored-by: Amaury M <1293565+amaurym@users.noreply.github.com> * fix arm (#9345) * Add client config subcommand to CLI (backport #8953) (#9255) * Add client config subcommand to CLI (#8953) * add client config * addressed reviewers comments * refactored,ready for review * fixed linter issues and addressed reviewers comments * Bump golangci-lint * fix linter warnings * fix some tests Co-authored-by: Alessio Treglia <alessio@tendermint.com> Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> * Fix build Co-authored-by: Andrei Ivasko <cyberbono3@gmail.com> Co-authored-by: Alessio Treglia <alessio@tendermint.com> * fix SA5011 static check in ibc/../chain.go (#9333) * Add env variable to cmd flags (backport #9040) (#9318) * Add env variable to cmd flags (#9040) * first draft * add tests in config_test package * refactored, all tests pass, r4r * ready for review * add envPrefix * Update simapp/simd/cmd/root.go Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> * fix linter issues * minor fixes Co-authored-by: Alessio Treglia <alessio@tendermint.com> Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> (cherry picked from commit a465ae1) # Conflicts: # client/context.go # simapp/simd/cmd/root.go * Remove line Co-authored-by: Andrei Ivasko <cyberbono3@gmail.com> Co-authored-by: Amaury M <1293565+amaurym@users.noreply.github.com> Co-authored-by: Alessio Treglia <alessio@tendermint.com> * Update Changelog and RL for v0.42.5 (#9350) * Update Changelog and RL for v0.42.5 * Reword * Tweak * fix client config don't take effect (backport #9211) (#9360) * fix client config don't take effect (#9211) * fix client keyring config * fix output flag of keys commads Co-authored-by: Robert Zaremba <robert@zaremba.ch> Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> (cherry picked from commit b4d1a5e) # Conflicts: # client/keys/add.go * Fix conflicts Co-authored-by: yihuang <huang@crypto.com> Co-authored-by: Amaury M <1293565+amaurym@users.noreply.github.com> * fix: add missing nil check in store.GetStore (#9354) (#9359) * fix: add missing nil check in store.GetStore * check nil in rootmulti as well (cherry picked from commit b065e20) Co-authored-by: Robert Zaremba <robert@zaremba.ch> * fix: invalid dockerfile volume for simd Co-authored-by: atheeshp <59333759+atheeshp@users.noreply.github.com> Co-authored-by: SaReN <sahithnarahari@gmail.com> Co-authored-by: Amaury <1293565+amaurym@users.noreply.github.com> Co-authored-by: Alessio Treglia <alessio@tendermint.com> Co-authored-by: Emmanuel T Odeke <emmanuel@orijtech.com> Co-authored-by: Cuong Manh Le <cuong.manhle.vn@gmail.com> Co-authored-by: colin axnér <25233464+colin-axner@users.noreply.github.com> Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> Co-authored-by: marbar3778 <marbar3778@yahoo.com> Co-authored-by: Robert Zaremba <robert@zaremba.ch> Co-authored-by: Devashish Dixit <devashish@crypto.com> Co-authored-by: Cuong Manh Le <cuong@orijtech.com> Co-authored-by: Barrie Byron <barrie.byron@tendermint.com> Co-authored-by: Andrei Ivasko <cyberbono3@gmail.com> Co-authored-by: Calvin Lau <38898718+calvinaco@users.noreply.github.com> Co-authored-by: yihuang <huang@crypto.com>
ejfitzgerald
previously approved these changes
Jun 8, 2021
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.
pristine version of cosmos-sdk v0.42.4 merged on top of master