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

chores: upgrade from v0.42.9 to v0.43.0 #109

Merged
merged 577 commits into from
Aug 20, 2021

Conversation

daeMOn63
Copy link
Contributor

All commits are from patches from cosmos/cosmos-sdk, expect:

  • d07b5bb: fixed tests and some merge errors (mostly due to our changes to the inflation / powerReduction)
  • 0b23060: port airdrop modules to v0.43

@codecov-commenter
Copy link

codecov-commenter commented Aug 19, 2021

Codecov Report

Merging #109 (2347b8d) into upgrade_v0.42.9 (e1be32d) will decrease coverage by 2.21%.
The diff coverage is 70.37%.

Impacted file tree graph

@@                 Coverage Diff                 @@
##           upgrade_v0.42.9     #109      +/-   ##
===================================================
- Coverage            63.02%   60.80%   -2.22%     
===================================================
  Files                  635      604      -31     
  Lines                41679    37847    -3832     
===================================================
- Hits                 26267    23012    -3255     
+ Misses               12991    12852     -139     
+ Partials              2421     1983     -438     
Impacted Files Coverage Δ
baseapp/grpcrouter.go 80.00% <ø> (-0.36%) ⬇️
client/config/cmd.go 38.88% <0.00%> (-3.97%) ⬇️
client/flags/flags.go 21.73% <0.00%> (+0.98%) ⬆️
client/keys/codec.go 100.00% <ø> (ø)
client/keys/list.go 72.22% <ø> (-10.39%) ⬇️
client/keys/migrate.go 56.33% <0.00%> (-2.33%) ⬇️
client/keys/parse.go 76.27% <0.00%> (+0.92%) ⬆️
client/keys/root.go 100.00% <ø> (ø)
client/query.go 16.98% <0.00%> (-1.77%) ⬇️
client/rpc/validators.go 0.00% <0.00%> (-2.00%) ⬇️
... and 775 more

@daeMOn63
Copy link
Contributor Author

failed Protobuf CI task is expected, because IBC have been moved to it's own repo: https://github.com/cosmos/ibc-go

Base automatically changed from upgrade_v0.42.9 to master August 20, 2021 12:36
@daeMOn63 daeMOn63 changed the base branch from master to integration/caelum August 20, 2021 12:50
fdymylja and others added 25 commits August 20, 2021 14:59
* use docker for proto-swagger-gen

fixes #7933

* fix deprecated commands and flags

see https://docs.buf.build/faq/ for detail

* run proto-gen and proto-swagger-gen

* add changelog entry
* refactor cli tests

* add norace

* fix typos

* update docs

* go imports

Co-authored-by: Amaury <[email protected]>
…#9061)

Currently, AddressFromBalancesStore uses the input key without any
validation, so an empty key or an invalid key length cause it panics.

This commit fixes the problem, by returning an error in case of invalid
key was passed.

Found by fuzzing added in #9060.

Fixed #9062
… vesting accounts (#8865)

Delegations and undelegations calculations performed during
accounting operations for vesting accounts were correct but
were not written back to the account store.

closes: #8601
closes: #8812

Co-authored-by: Alessio Treglia <[email protected]>
Co-authored-by: Aleksandr Bezobchuk <[email protected]>
Co-authored-by: Jonathan Gimeno <[email protected]>
Co-authored-by: Frojdi Dymylja <[email protected]>
Co-authored-by: Adam Bozanich <[email protected]>
Co-authored-by: Amaury <[email protected]>
Co-authored-by: SaReN <[email protected]>
* WIP

* Add consume gas

* update authz.go

* add build flag

* Update x/staking/types/authz.go

* Update x/staking/types/authz.go

* add tests docs

* review changes

* fix operations

* Update x/authz/types/msgs.go

Co-authored-by: Amaury <[email protected]>

* review changes

* update gas cost

* review changes

Co-authored-by: SaReN <[email protected]>
Co-authored-by: Amaury <[email protected]>
* build(deps): bump github.com/otiai10/copy from 1.5.0 to 1.5.1

Bumps [github.com/otiai10/copy](https://github.com/otiai10/copy) from 1.5.0 to 1.5.1.
- [Release notes](https://github.com/otiai10/copy/releases)
- [Commits](otiai10/copy@v1.5.0...v1.5.1)

Signed-off-by: dependabot[bot] <[email protected]>

* fix lint

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Marko Baricevic <[email protected]>
* WIP: add filtered message

* updated `Accept` interface method

* fix tests

* add cli tests for filtered fee allowance

* fix tests

* review changes

* rename `filteredFeeAllowance` message

* review changes

* review changes

* review changes

* review changes

* review changes

* update errors

* remove validation

* fix conflicts

* add `ctx` to `Accept` method

* fix test

* add gas consumption

* review changes

* review changes

* revert

* review changes

* improve error handling

* fix test

* Merge branch 'master' of github.com:cosmos/cosmos-sdk into atheesh/feegrant-filtered-msgs

* review changes

* update gas

* update type

* review changes

Co-authored-by: Alessio Treglia <[email protected]>
Co-authored-by: SaReN <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* Update docs

* Tweak

Co-authored-by: Marko <[email protected]>
…#9096)

Bumps [styfle/cancel-workflow-action](https://github.com/styfle/cancel-workflow-action) from 0.8.0 to 0.9.0.
- [Release notes](https://github.com/styfle/cancel-workflow-action/releases)
- [Commits](styfle/cancel-workflow-action@0.8.0...89f242e)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Marko <[email protected]>
* Adding disclosure process to security.md

* Apply suggestions from code review

Co-authored-by: Barrie Byron <[email protected]>

Co-authored-by: Alessio Treglia <[email protected]>
Co-authored-by: Barrie Byron <[email protected]>
Co-authored-by: Marko <[email protected]>
Co-authored-by: Amaury <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* 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 <[email protected]>

* fix linter issues

* minor fixes

Co-authored-by: Alessio Treglia <[email protected]>
Co-authored-by: Amaury <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
* update docs/code to utilize map access return values

* remove mock module in fromVM to simulate truly non-existent module

* update test docs to refelect change

Co-authored-by: technicallyty <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Bumps [actions/cache](https://github.com/actions/cache) from v2.1.4 to v2.1.5.
- [Release notes](https://github.com/actions/cache/releases)
- [Commits](actions/cache@v2.1.4...1a9e213)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Marko <[email protected]>
* upgrade draft docs

* upgrade draft docs2

* change to generic id

* Update docs/building-modules/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/building-modules/README.md

Co-authored-by: Amaury <[email protected]>

* Update docs/building-modules/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/building-modules/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/building-modules/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/building-modules/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/building-modules/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/building-modules/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/README.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* added lines for version map and consensus versions

* fix headers, add synopsis tag

* docs for new modules

* fix

* remove line

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/building-modules/upgrade.md

Co-authored-by: Amaury <[email protected]>

* SetUpgradeHandler example snippet

* -general clean up of docs
-include PR #9007 information

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Update docs/core/upgrade.md

Co-authored-by: Amaury <[email protected]>

* Apply suggestions from code review

Co-authored-by: Barrie Byron <[email protected]>

* fix self-reference

* clearer definitions for overwriting genesis functions

* add sync section

* forgot to save this

* update description of initgenesis modules

* specify upgrade method

* Update docs/core/upgrade.md

Co-authored-by: Barrie Byron <[email protected]>

Co-authored-by: technicallyty <[email protected]>
Co-authored-by: Amaury <[email protected]>
Co-authored-by: Barrie Byron <[email protected]>
Co-authored-by: Alessio Treglia <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
mergify bot and others added 24 commits August 20, 2021 14:59
## Description

While looking into #9704 I found that we don't need to wrap grpc web handler into HTTP HandlerFun.
<!--
The default pull request template is for types feat, fix, or refactor.
For other templates, add one of the following parameters to the url:
- template=docs.md
- template=other.md
-->

## Description

Closes: #XXXX

<!-- Add a description of the changes that this PR introduces and the files that
are the most critical to review. -->
* refactor: Improve txRaw decoder code (#9769)

<!--
The default pull request template is for types feat, fix, or refactor.
For other templates, add one of the following parameters to the url:
- template=docs.md
- template=other.md
-->

## Description

Apply post-merge reviews to a previous PR

ref: cosmos/cosmos-sdk#9754 (review)

<!-- Add a description of the changes that this PR introduces and the files that
are the most critical to review. -->
* feat: Query txs by signature and by address+seq (#9750)

<!--
The default pull request template is for types feat, fix, or refactor.
For other templates, add one of the following parameters to the url:
- template=docs.md
- template=other.md
-->

## Description

Closes: #9741

<!-- Add a description of the changes that this PR introduces and the files that
are the most critical to review. -->
* added low-s normalization to ecdsa secp256r1 signing

* go fmt fixes

* removed else block as golint required

* implement raw signature encoding for secp256r1

* move the creation of signature to after the check for sig string length

* fake commit to re-run checks? (move the creation of signature to after the check for sig string length)

* added a signature test for high s signature that requires sig validation to fail after the valid signature was mutated by extracting and scalar negating its s value

* reordered code to prevent mutated message from being used in sig verify

* added test for successful high_s signature with the ecdsa portion of the publicKey

* Remove comment for self-explanatory code.

Co-authored-by: Robert Zaremba <[email protected]>

* Missing quote

Co-authored-by: Robert Zaremba <[email protected]>

* Apply minor suggestions from code review

Co-authored-by: Robert Zaremba <[email protected]>

* normalize comments for godoc

* refactored p256Order functions as private vars

* Div -> Rsh optimizing time for division

* resolve two code coverage issues; fix some small review issues

* test using private signatureRaw function instead of copying code. Added tests to improve code coverage

Co-authored-by: Aaron Craelius <[email protected]>
Co-authored-by: Robert Zaremba <[email protected]>
Co-authored-by: Aleksandr Bezobchuk <[email protected]>
(cherry picked from commit aa37ae9)

Co-authored-by: John Kemp <[email protected]>
* docs: fix cosmovisor quickstart (#9730)

## Description

<!-- Add a description of the changes that this PR introduces and the files that
are the most critical to review. -->

Automatically create a copy of [Cosmovisor Quickstart](https://docs.cosmos.network/v0.43/run-node/cosmovisor.html) from [Cosmovisor README](https://github.com/cosmos/cosmos-sdk/blob/master/cosmovisor/README.md) and minor fixes to Cosmovisor README.
## Description

Fix typos in baseapp.md and tx-lifecycle.md
* x/capability: Cap Initialization Fix (#9392)

(cherry picked from commit 3776793ce6492f1df0d7134655e82dc12cdf2052)

# Conflicts:
#	x/ibc/applications/transfer/simulation/params.go

* remove ibc file

Co-authored-by: Aditya <[email protected]>
Co-authored-by: Amaury M <[email protected]>
… #9796) (#9798)

* chore: added changelog for 9793 (secp256r1 tx malleability) (#9796)

## Description

Added missing changelog and updated a `p256Order` comment to remove misleading part.
* fix: hardcoded ledger algo on `keys add` (#9766)

<!--
The default pull request template is for types feat, fix, or refactor.
For other templates, add one of the following parameters to the url:
- template=docs.md
- template=other.md
-->

## Description

Closes: #9734

cc: @jleni

<!-- Add a description of the changes that this PR introduces and the files that
are the most critical to review. -->
* fix: feegrant grant cli bug (#9720)

<!--
The default pull request template is for types feat, fix, or refactor.
For other templates, add one of the following parameters to the url:
- template=docs.md
- template=other.md
-->

## Description
feegrant grant cli granter now accepts key name as well as address in general and accepts only address in `--generate-only` mode

Closes: #9659

<!-- Add a description of the changes that this PR introduces and the files that
are the most critical to review. -->
* feat: Improve withdraw-all-rewards UX (#9781)

## Description
Related to #9489, this PR improves the UX when using the `withdraw-all-rewards` command by forcing the broadcast mode to `block` if the chunk size is greater than 0. This will ensure that the transactions do not fail even if the user uses invalid broadcast modes for this command (`sync` and `async`).
…86) (#9844)

* fix: fixed missing coin denomination in msg server logs (#9786)

<!--
The default pull request template is for types feat, fix, or refactor.
For other templates, add one of the following parameters to the url:
- template=docs.md
- template=other.md
-->

## Description

Closes: [#9785](cosmos/cosmos-sdk#9785)

This PR modifies _x/staking/keeper/msg_server.go_ file and fixes coin denomination issues in the emitted events.
… (#9841)

* fix!: Capability Issue on Restart, Backport to v0.43 (#9836)

<!--
The default pull request template is for types feat, fix, or refactor.
For other templates, add one of the following parameters to the url:
- template=docs.md
- template=other.md
-->

## Description

Closes: #9800

The following is a breaking fix to #9800. In the non-breaking fix, the initialization logic was moved out of `InitializeAndSeal` but the API was preserved. I've now removed `InitializeAndSeal` and replaced it with just the `Seal` function, which may be called much more cleanly in `app.go`
…ackport #9845) (#9846)

* style(capability)!: update go doc comments and remove BeginBlocker (#9845)

## Description

+ Backported go doc comment updates from cosmos/cosmos-sdk#9835
+ Removed BeginBlocker function which only wraps the `InitMemStore` -> https://github.com/cosmos/cosmos-sdk/pull/9835/files#r681987005  -- this is not a breaking change because RC3 was not yet released.
+ Updated changelog
* chore: Changelog and Release Notes for 0.43.0-rc3

* Update release notes

* Better release notes

* Update CHANGELOG.md
* chore: v0.43.0 Changelog and Release Notes

* Update RELEASE_NOTES.md

* Update CHANGELOG.md

Co-authored-by: Aaron Craelius <[email protected]>

* Update RELEASE_NOTES.md

Co-authored-by: Tomas Tauber <[email protected]>

* Update CHANGELOG.md

Co-authored-by: Aaron Craelius <[email protected]>
Co-authored-by: Tomas Tauber <[email protected]>
adjust rosetta test chain setup to match our minSelfDelegation
requirements (1**18 instead of 1)
@daeMOn63 daeMOn63 force-pushed the upgrade_v0.42.9-v0.43.0 branch from 2347b8d to 66496ee Compare August 20, 2021 13:00
@daeMOn63 daeMOn63 force-pushed the integration/caelum branch from 68251bb to bb935b2 Compare August 20, 2021 13:00
@daeMOn63 daeMOn63 merged commit aa6e50b into integration/caelum Aug 20, 2021
@daeMOn63 daeMOn63 deleted the upgrade_v0.42.9-v0.43.0 branch August 20, 2021 13:01
@daeMOn63 daeMOn63 restored the upgrade_v0.42.9-v0.43.0 branch August 20, 2021 13:03
@daeMOn63 daeMOn63 deleted the upgrade_v0.42.9-v0.43.0 branch November 24, 2021 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.