upd: bumped orig wasmd to v0.51 #175
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
* Start implementation * Add implementation + some e2e test * Fix lint * Squashed: sdk upgrade to v0.50 * rebuild protos with newer proto builder (cherry picked from commit fd8f4c1) * update ibc-go (cherry picked from commit fb86679) * bump cosmos-sdk and ibc in the v50 branch (CosmWasm#1616) * tidy * upgade ibc * remove the toolchain command * Bump sdk version * Use correct bech32 prefix * Bump SDK * Enable fraud system test again * Fix genesis param name * Fix import/export simulations * set log level for benchmarks (cherry picked from commit 1cfb930) * Apply review comments * Remove gov beta1 helpers * Bump sdk version to latest in branch * Fix linter * Setup mergify for main * Update mergify for better branch name --------- Co-authored-by: Pino' Surace <pino.surace@live.it> Co-authored-by: Jacob Gadikian <jacobgadikian@gmail.com>
Bumps [actions/checkout](https://github.com/actions/checkout) from 4.0.0 to 4.1.0. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v4.0.0...v4.1.0) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bump sdk to rc1
Bumps [github.com/rs/zerolog](https://github.com/rs/zerolog) from 1.30.0 to 1.31.0. - [Release notes](https://github.com/rs/zerolog/releases) - [Commits](rs/zerolog@v1.30.0...v1.31.0) --- updated-dependencies: - dependency-name: github.com/rs/zerolog dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [github.com/cosmos/ibc-go/modules/capability](https://github.com/cosmos/ibc-go) from 1.0.0-rc5 to 1.0.0-rc6. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/main/CHANGELOG.md) - [Commits](cosmos/ibc-go@modules/capability/v1.0.0-rc5...modules/capability/v1.0.0-rc6) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/modules/capability dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
…github.com/cosmos/ibc-go/modules/capability-1.0.0-rc6 Bump github.com/cosmos/ibc-go/modules/capability from 1.0.0-rc5 to 1.0.0-rc6
…github.com/rs/zerolog-1.31.0 Bump github.com/rs/zerolog from 1.30.0 to 1.31.0
* Add msg update contract label * fix tests
…asm#1641) Bumps [github.com/prometheus/client_golang](https://github.com/prometheus/client_golang) from 1.16.0 to 1.17.0. - [Release notes](https://github.com/prometheus/client_golang/releases) - [Changelog](https://github.com/prometheus/client_golang/blob/v1.17.0/CHANGELOG.md) - [Commits](prometheus/client_golang@v1.16.0...v1.17.0) --- updated-dependencies: - dependency-name: github.com/prometheus/client_golang dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…incompatible (CosmWasm#1647) * Bump github.com/docker/distribution Bumps [github.com/docker/distribution](https://github.com/docker/distribution) from 2.8.2+incompatible to 2.8.3+incompatible. - [Release notes](https://github.com/docker/distribution/releases) - [Commits](distribution/distribution@v2.8.2...v2.8.3) --- updated-dependencies: - dependency-name: github.com/docker/distribution dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * Fix lint --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Pino' Surace <pino.surace@live.it>
…osmWasm#1648) * Bump github.com/cosmos/ibc-go/v8 from 8.0.0-beta.0 to 8.0.0-beta.1 Bumps [github.com/cosmos/ibc-go/v8](https://github.com/cosmos/ibc-go) from 8.0.0-beta.0 to 8.0.0-beta.1. - [Release notes](https://github.com/cosmos/ibc-go/releases) - [Changelog](https://github.com/cosmos/ibc-go/blob/main/CHANGELOG.md) - [Commits](cosmos/ibc-go@v8.0.0-beta.0...v8.0.0-beta.1) --- updated-dependencies: - dependency-name: github.com/cosmos/ibc-go/v8 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> * Fix conflicts * Fix lint * Make lint pass --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Pino' Surace <pino.surace@live.it>
The validation functions on the `tx` type masquerade the root error message for the `Funds` validation. Having the original error helps to save time when debugging the cause for a failed tx. One example is if someone sends multiple funds to a contract execution without sorting the denoms, which is one of the validations in the `Coins.Validate` method. With the error propagation, the developer can quickly determine why the tx failed.
…-funds-err fix: propagate funds validation errors
Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.26.1 to 1.27.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](bufbuild/buf-setup-action@v1.26.1...v1.27.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Retract v0.42.0 release
Bumps [github.com/spf13/viper](https://github.com/spf13/viper) from 1.16.0 to 1.17.0. - [Release notes](https://github.com/spf13/viper/releases) - [Commits](spf13/viper@v1.16.0...v1.17.0) --- updated-dependencies: - dependency-name: github.com/spf13/viper dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Add label validation for non printable chars * Fix printable chars check
* Bump wasmvm to v1.4.1 - bugfix (CosmWasm#1658) * Bump wasmvm to 1.4.1 - bugfix * Update Dockerfile (cherry picked from commit 7270ccd) # Conflicts: # go.mod # go.sum * Resolve conflicts --------- Co-authored-by: Alexander Peters <alpe@users.noreply.github.com>
Bumps [cosmossdk.io/log](https://github.com/cosmos/cosmos-sdk) from 1.3.0 to 1.3.1. - [Release notes](https://github.com/cosmos/cosmos-sdk/releases) - [Changelog](https://github.com/cosmos/cosmos-sdk/blob/main/CHANGELOG.md) - [Commits](cosmos/cosmos-sdk@log/v1.3.0...log/v1.3.1) --- updated-dependencies: - dependency-name: cosmossdk.io/log dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [github.com/rs/zerolog](https://github.com/rs/zerolog) from 1.31.0 to 1.32.0. - [Release notes](https://github.com/rs/zerolog/releases) - [Commits](rs/zerolog@v1.31.0...v1.32.0) --- updated-dependencies: - dependency-name: github.com/rs/zerolog dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.28.1 to 1.29.0. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](bufbuild/buf-setup-action@v1.28.1...v1.29.0) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Upgrade cosmos-sdk to v0.50.4 * Fix lint
Bumps [github.com/golang/protobuf](https://github.com/golang/protobuf) from 1.5.3 to 1.5.4. - [Release notes](https://github.com/golang/protobuf/releases) - [Commits](golang/protobuf@v1.5.3...v1.5.4) --- updated-dependencies: - dependency-name: github.com/golang/protobuf dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This is unused since CosmWasm#1804
Bump wasmvm to 2.0.0
This binrary was added added 5 yeras ago in 2945a05 but most of that work was removed at some point.
Remove contract_tests
* Fix Amino JSON representation * Update amino encoding
Bumps [bufbuild/buf-setup-action](https://github.com/bufbuild/buf-setup-action) from 1.29.0 to 1.30.1. - [Release notes](https://github.com/bufbuild/buf-setup-action/releases) - [Commits](bufbuild/buf-setup-action@v1.29.0...v1.30.1) --- updated-dependencies: - dependency-name: bufbuild/buf-setup-action dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ies() (CosmWasm#1855) * Add BuiltInCapabilities() to x/wasm/keeper and deprecate AllCapabilities() * Replace usage with new wasmkeeper.BuiltInCapabilities()
* Bump cosmos-sdk to v0.50.6 * Fix failing test + fix scripts
* Prepare changelog for v0.51 release * Update changelog with latest changes * Set release date * Update changelog
# Conflicts: # go.mod
swelf19
changed the title
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| maccPerms, | ||
| authcodec.NewBech32Codec(sdk.Bech32MainPrefix), |
Check warning
Code scanning / CodeQL
Directly using the bech32 constants Warning test
Comment on lines
+1029
to
+1036
| for _, m := range app.ModuleManager.Modules { | ||
| if moduleWithName, ok := m.(module.HasName); ok { | ||
| moduleName := moduleWithName.Name() | ||
| if appModule, ok := moduleWithName.(appmodule.AppModule); ok { | ||
| modules[moduleName] = appModule | ||
| } | ||
| } | ||
| } |
Check warning
Code scanning / CodeQL
Iteration over map Warning
Comment on lines
+1062
to
+1064
| for _, key := range app.keys { | ||
| keys = append(keys, key) | ||
| } |
Check warning
Code scanning / CodeQL
Iteration over map Warning
Comment on lines
+169
to
+171
| for eventName, hasEvent := range hasEvents { | ||
| suite.Require().True(hasEvent, "event: %s was not found in events", eventName) | ||
| } |
Check warning
Code scanning / CodeQL
Iteration over map Warning
Comment on lines
+414
to
+418
| go func(i int) { // do parallel | ||
| c.t.Logf("Voting: validator %d\n", i) | ||
| rsp = c.CustomCommand("tx", "gov", "vote", ourProposalID, "yes", "--from", c.GetKeyAddr(fmt.Sprintf("node%d", i))) | ||
| RequireTxSuccess(c.t, rsp) | ||
| }(i) |
Check notice
Code scanning / CodeQL
Spawning a Go routine Note test
Comment on lines
+314
to
+318
| go func() { | ||
| if err := p.Signal(syscall.SIGTERM); err != nil { | ||
| s.Logf("failed to stop node with pid %d: %s\n", p.Pid, err) | ||
| } | ||
| shutdown = true | ||
| default: | ||
| if err := exec.Command(locateExecutable("pgrep"), s.execBinary).Run(); err != nil { //nolint:gosec | ||
| shutdown = true | ||
| }() |
Check notice
Code scanning / CodeQL
Spawning a Go routine Note test
Comment on lines
+573
to
+579
| go func(pid int) { | ||
| _ = cmd.Wait() // blocks until shutdown | ||
| s.pidsLock.Lock() | ||
| delete(s.pids, pid) | ||
| s.pidsLock.Unlock() | ||
| s.Logf("Node stopped: %d\n", pid) | ||
| }(pid) |
Check notice
Code scanning / CodeQL
Spawning a Go routine Note test
Comment on lines
+341
to
+347
| for pid := range pids { | ||
| p, err := os.FindProcess(pid) | ||
| if err != nil { | ||
| continue | ||
| } | ||
| cb(p) | ||
| } |
Check warning
Code scanning / CodeQL
Iteration over map Warning test
pr0n00gler
approved these changes
Jun 10, 2024
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.
no additional changes in neutron'n code, only bumped bas wasmd.
You can see diff between neutron's wasmd and original wasmd wasmd-orig-v0.51.0...neutron-org:wasmd:upd/neutron-upd-0.51