From 2ce75c19b75c8f1aa2c18657cc9a75ff5193764f Mon Sep 17 00:00:00 2001 From: Julien Robert Date: Sat, 13 Aug 2022 12:39:33 +0200 Subject: [PATCH 1/6] fix: remove replace directive --- cosmovisor/go.mod | 4 +--- cosmovisor/go.sum | 3 +-- go.work.sum | 28 ++++++++++++++++++++++++++++ 3 files changed, 30 insertions(+), 5 deletions(-) diff --git a/cosmovisor/go.mod b/cosmovisor/go.mod index e2a9da79c262..f2d45a693077 100644 --- a/cosmovisor/go.mod +++ b/cosmovisor/go.mod @@ -29,7 +29,7 @@ require ( github.com/btcsuite/btcd v0.22.1 // indirect github.com/cespare/xxhash v1.1.0 // indirect github.com/cespare/xxhash/v2 v2.1.2 // indirect - github.com/coinbase/rosetta-sdk-go v0.7.10 // indirect + github.com/coinbase/rosetta-sdk-go v0.7.11 // indirect github.com/confio/ics23/go v0.7.0 // indirect github.com/cosmos/btcutil v1.0.4 // indirect github.com/cosmos/cosmos-proto v1.0.0-alpha7 // indirect @@ -126,5 +126,3 @@ require ( gopkg.in/yaml.v3 v3.0.1 // indirect sigs.k8s.io/yaml v1.3.0 // indirect ) - -replace github.com/gogo/protobuf => github.com/regen-network/protobuf v1.3.3-alpha.regen.1 diff --git a/cosmovisor/go.sum b/cosmovisor/go.sum index 84adc74264d8..95f8fd632e32 100644 --- a/cosmovisor/go.sum +++ b/cosmovisor/go.sum @@ -213,8 +213,7 @@ github.com/cncf/xds/go v0.0.0-20211011173535-cb28da3451f1/go.mod h1:eXthEFrGJvWH github.com/cncf/xds/go v0.0.0-20211130200136-a8f946100490/go.mod h1:eXthEFrGJvWHgFFCl3hGmgk+/aYT6PnTQLykKQRLhEs= github.com/cockroachdb/apd/v2 v2.0.2 h1:weh8u7Cneje73dDh+2tEVLUvyBc89iwepWCD8b8034E= github.com/cockroachdb/datadriven v0.0.0-20190809214429-80d97fb3cbaa/go.mod h1:zn76sxSg3SzpJ0PPJaLDCu+Bu0Lg3sKTORVIj19EIF8= -github.com/coinbase/rosetta-sdk-go v0.7.10 h1:m5Prrqg9CD1GFZm0tu8z3LEDF/BY5RKUSkODZRpAkEc= -github.com/coinbase/rosetta-sdk-go v0.7.10/go.mod h1:/glajndJEMrp+D7cO2PJZWKeclK8mDyzSCokCMc6Ftc= +github.com/coinbase/rosetta-sdk-go v0.7.11 h1:T7camDGstlX2ENVE0QHR1AVu3asSXED0vg1xDaF6bYE= github.com/confio/ics23/go v0.7.0 h1:00d2kukk7sPoHWL4zZBZwzxnpA2pec1NPdwbSokJ5w8= github.com/confio/ics23/go v0.7.0/go.mod h1:E45NqnlpxGnpfTWL/xauN7MRwEE28T4Dd4uraToOaKg= github.com/containerd/console v1.0.3/go.mod h1:7LqA/THxQ86k76b8c/EMSiaJ3h1eZkMkXar0TQ1gf3U= diff --git a/go.work.sum b/go.work.sum index 4339e550875c..383b8b05b0ae 100644 --- a/go.work.sum +++ b/go.work.sum @@ -1,8 +1,36 @@ cosmossdk.io/core v0.1.1-0.20220726092710-f848e4300a8a/go.mod h1:NI7jc1kHeG47qN5RCEHuJQ8hpuy+I5l4iQ595nFCZno= cosmossdk.io/math v1.0.0-beta.3.0.20220726092710-f848e4300a8a/go.mod h1:3LYasri3Zna4XpbrTNdKsWmD5fHHkaNAod/mNT9XdE4= +github.com/DATA-DOG/go-sqlmock v1.5.0 h1:Shsta01QNfFxHCfpW6YH2STWB0MudeXXEWMr20OEh60= github.com/DataDog/zstd v1.5.2/go.mod h1:g4AWEaM3yOg3HYfnJ3YIawPnVdXJh9QME85blwSAmyw= +github.com/Zilliqa/gozilliqa-sdk v1.2.1-0.20201201074141-dd0ecada1be6/go.mod h1:eSYp2T6f0apnuW8TzhV3f6Aff2SE8Dwio++U4ha4yEM= +github.com/btcsuite/btcd v0.0.0-20190315201642-aa6e0f35703c/go.mod h1:DrZx5ec/dmnfpw9KyYoQyYo7d0KEvTkk/5M/vbZjAr8= +github.com/btcsuite/btcutil v0.0.0-20190207003914-4c204d697803/go.mod h1:+5NJ2+qvTyV9exUAL/rxXi3DcLg2Ts+ymUAY5y4NvMg= +github.com/bwesterb/go-ristretto v1.2.0/go.mod h1:fUIoIZaG73pV5biE2Blr2xEzDoMj7NFEuV9ekS419A0= +github.com/coinbase/kryptology v1.8.0/go.mod h1:RYXOAPdzOGUe3qlSFkMGn58i3xUA8hmxYHksuq+8ciI= +github.com/consensys/bavard v0.1.8-0.20210915155054-088da2f7f54a/go.mod h1:9ItSMtA/dXMAiL7BG6bqW2m3NdSEObYWoH223nGHukI= +github.com/consensys/gnark-crypto v0.5.3/go.mod h1:hOdPlWQV1gDLp7faZVeg8Y0iEPFaOUnCc4XeCCk96p0= github.com/cosmos/cosmos-sdk/db v1.0.0-beta.1/go.mod h1:JUMM2MxF9wuwzRWZJjb8BjXsn1BmPmdBd3a75pIct4I= +github.com/dgraph-io/ristretto v0.0.3/go.mod h1:KPxhHT9ZxKefz+PCeOGsrHpl1qZ7i70dGTu2u+Ahh6E= +github.com/ethereum/go-ethereum v1.10.19/go.mod h1:IJBNMtzKcNHPtllYihy6BL2IgK1u+32JriaTbdt4v+w= github.com/facebookgo/ensure v0.0.0-20200202191622-63f1cf65ac4c/go.mod h1:Yg+htXGokKKdzcwhuNDwVvN+uBxDGXJ7G/VN1d8fa64= github.com/facebookgo/subset v0.0.0-20200203212716-c811ad88dec4/go.mod h1:5tD+neXqOorC30/tWg0LCSkrqj/AR6gu8yY8/fpw1q0= +github.com/go-playground/assert/v2 v2.0.1 h1:MsBgLAaY856+nPRTKrp3/OZK38U/wa0CcBYNjji3q3A= +github.com/leanovate/gopter v0.2.9/go.mod h1:U2L/78B+KVFIx2VmW6onHJQzXtFb+p5y3y2Sh+Jxxv8= +github.com/lucasjones/reggen v0.0.0-20180717132126-cdb49ff09d77/go.mod h1:5ELEyG+X8f+meRWHuqUOewBOhvHkl7M76pdGEansxW4= +github.com/neilotoole/errgroup v0.1.6/go.mod h1:Q2nLGf+594h0CLBs/Mbg6qOr7GtqDK7C2S41udRnToE= +github.com/segmentio/fasthash v1.0.3/go.mod h1:waKX8l2N8yckOgmSsXJi7x1ZfdKZ4x7KRMzBtS3oedY= github.com/tendermint/tendermint v0.34.20-rc1/go.mod h1:u2xI6q3IeLQQ2NdIpRKLUKBNog0o7EzBpvDCE0/OTmg= +github.com/tidwall/gjson v1.12.1/go.mod h1:/wbyibRr2FHMks5tjHJ5F8dMZh3AcwJEMf5vlfC0lxk= +github.com/tidwall/gjson v1.14.1/go.mod h1:/wbyibRr2FHMks5tjHJ5F8dMZh3AcwJEMf5vlfC0lxk= +github.com/tidwall/match v1.1.1/go.mod h1:eRSPERbgtNPcGhD8UCthc6PmLEQXEWd3PRB5JTxsfmM= +github.com/tidwall/pretty v1.2.0/go.mod h1:ITEVvHYasfjBbM0u2Pg8T2nJnzm8xPwvNhhsoaGGjNU= +github.com/tidwall/sjson v1.2.4/go.mod h1:098SZ494YoMWPmMO6ct4dcFnqxwj9r/gF0Etp19pSNM= +github.com/tyler-smith/go-bip39 v1.0.2/go.mod h1:sJ5fKU0s6JVwZjjcUEX2zFOnvq0ASQ2K9Zr6cf67kNs= +github.com/vmihailenco/msgpack/v5 v5.3.5/go.mod h1:7xyJ9e+0+9SaZT0Wt1RGleJXzli6Q/V5KbhBonMG9jc= +github.com/vmihailenco/tagparser/v2 v2.0.0/go.mod h1:Wri+At7QHww0WTrCBeu4J6bNtoV6mEfg5OIWRZA9qds= +github.com/ybbus/jsonrpc v2.1.2+incompatible/go.mod h1:XJrh1eMSzdIYFbM08flv0wp5G35eRniyeGut1z+LSiE= github.com/yuin/goldmark v1.4.13/go.mod h1:6yULJ656Px+3vBD8DxQVa3kxgyrAnzto9xy5taEt/CY= +golang.org/x/crypto v0.0.0-20190426145343-a29dc8fdc734/go.mod h1:yigFU9vqHzYiE8UmvKecakEJjdnWj3jj499lnFckfCI= +golang.org/x/crypto v0.0.0-20210322153248-0c34fe9e7dc2/go.mod h1:T9bdIzuCu7OtxOm1hfPfRQxPLYneinmdGuTeoZ9dtd4= +golang.org/x/sys v0.0.0-20210420205809-ac73e9fd8988/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs= +rsc.io/tmplfunc v0.0.3/go.mod h1:AG3sTPzElb1Io3Yg4voV9AGZJuleGAwaVRxL9M49PhA= From 6395d6459f2501216f06db0d19845b1961bb069a Mon Sep 17 00:00:00 2001 From: Julien Robert Date: Sat, 13 Aug 2022 13:01:15 +0200 Subject: [PATCH 2/6] chore: prepare cosmovisor 1.2.1 release --- cosmovisor/CHANGELOG.md | 5 ++++ cosmovisor/README.md | 36 +++++++++++++++------------- cosmovisor/RELEASE_NOTES.md | 15 ++++-------- cosmovisor/cmd/cosmovisor/version.go | 2 +- 4 files changed, 30 insertions(+), 28 deletions(-) diff --git a/cosmovisor/CHANGELOG.md b/cosmovisor/CHANGELOG.md index 6f71487a42cb..20be6adeab11 100644 --- a/cosmovisor/CHANGELOG.md +++ b/cosmovisor/CHANGELOG.md @@ -37,6 +37,11 @@ Ref: https://keepachangelog.com/en/1.0.0/ ## [Unreleased] +## v1.2.1 2022-08-15 + +* [\XXXX](https://github.com/cosmos/cosmos-sdk/pull/XXXX) Fix failure when installing cosmovisor via `go install`. +* [\XXXX](https://github.com/cosmos/cosmos-sdk/pull/XXXX) Fix plan path case sensitivity issue. + ## v1.2.0 2022-07-26 ### Features diff --git a/cosmovisor/README.md b/cosmovisor/README.md index 12b5e054e3b3..76614f91607a 100644 --- a/cosmovisor/README.md +++ b/cosmovisor/README.md @@ -3,20 +3,20 @@ `cosmovisor` is a small process manager for Cosmos SDK application binaries that monitors the governance module for incoming chain upgrade proposals. If it sees a proposal that gets approved, `cosmovisor` can automatically download the new binary, stop the current binary, switch from the old binary to the new one, and finally restart the node with the new binary. - - [Design](#design) - - [Contributing](#contributing) - - [Setup](#setup) - - [Installation](#installation) - - [Command Line Arguments And Environment Variables](#command-line-arguments-and-environment-variables) - - [Folder Layout](#folder-layout) - - [Usage](#usage) - - [Initialization](#initialization) - - [Detecting Upgrades](#detecting-upgrades) - - [Auto-Download](#auto-download) - - [Example: SimApp Upgrade](#example-simapp-upgrade) - - [Chain Setup](#chain-setup) - - +* [Design](#design) +* [Contributing](#contributing) +* [Setup](#setup) + * [Installation](#installation) + * [Command Line Arguments And Environment Variables](#command-line-arguments-and-environment-variables) + * [Folder Layout](#folder-layout) +* [Usage](#usage) + * [Initialization](#initialization) + * [Detecting Upgrades](#detecting-upgrades) + * [Auto-Download](#auto-download) +* [Example: SimApp Upgrade](#example-simapp-upgrade) + * [Chain Setup](#chain-setup) + * [Prepare Cosmovisor and Start the Chain](#prepare-cosmovisor-and-start-the-chain) + * [Update App](#update-app) ## Design @@ -35,25 +35,27 @@ Cosmovisor is designed to be used as a wrapper for a `Cosmos SDK` app: Cosmovisor is part of the Cosmos SDK monorepo, but it's a separate module with it's own release schedule. -Release branches have the following format `release/cosmovisor/vA.B.x`, where A and B are a number (e.g. `release/cosmovisor/v0.1.x`). Releases are tagged using the following format: `cosmovisor/vA.B.C`. +Release branches have the following format `release/cosmovisor/vA.B.x`, where A and B are a number (e.g. `release/cosmovisor/v1.2.x`). Releases are tagged using the following format: `cosmovisor/vA.B.C`. ## Setup ### Installation +You can download Cosmovisor from the [Github releases](https://github.com/cosmos/cosmos-sdk/releases/tag/cosmovisor%2Fv1.2.1). + To install the latest version of `cosmovisor`, run the following command: ```sh go install github.com/cosmos/cosmos-sdk/cosmovisor/cmd/cosmovisor@latest ``` -To install a previous version, you can specify the version. IMPORTANT: Chains that use Cosmos-SDK v0.44.3 or earlier (eg v0.44.2) and want to use auto-download feature MUST use Cosmovisor v0.1.0 +To install a previous version, you can specify the version. IMPORTANT: Chains that use Cosmos-SDK v0.44.3 or earlier (eg v0.44.2) and want to use auto-download feature MUST use `cosmovisor v0.1.0` ```sh go install github.com/cosmos/cosmos-sdk/cosmovisor/cmd/cosmovisor@v0.1.0 ``` -You can run `cosmovisor version` to check the Cosmovisor version (works only with Cosmovisor >1.1.0). +Run `cosmovisor version` to check the cosmovisor version. You can also install from source by pulling the cosmos-sdk repository and switching to the correct version and building as follows: diff --git a/cosmovisor/RELEASE_NOTES.md b/cosmovisor/RELEASE_NOTES.md index 495a36049811..c5614e221958 100644 --- a/cosmovisor/RELEASE_NOTES.md +++ b/cosmovisor/RELEASE_NOTES.md @@ -1,15 +1,10 @@ -# Cosmovisor v1.2.0 Release Notes - -### New Features - -With the `cosmovisor init` command, all the necessary folders for using cosmovisor are automatically created. You do not need to manually symlink the chain binary anymore. - -We've added a new configuration option: `DAEMON_RESTART_DELAY` (as env variable). When set, Cosmovisor will wait that delay between the node halt and backup. See the [README](https://github.com/cosmos/cosmos-sdk/blob/main/cosmovisor/README.md#command-line-arguments-and-environment-variables) file for more details. +# Cosmovisor v1.2.1 Release Notes ### Bug Fixes -* Fix Cosmovisor binary usage for pre-upgrade. Cosmovisor was using the wrong binary when running a `pre-upgrade` command. - +* Fix failure when installing cosmovisor via `go install`. +* Fix plan path case sensitivity issue. + ### Changelog -For more details, please see the [CHANGELOG](https://github.com/cosmos/cosmos-sdk/blob/cosmovisor/v1.1.0/cosmovisor/CHANGELOG.md). +For more details, please see the [CHANGELOG](https://github.com/cosmos/cosmos-sdk/blob/cosmovisor/v1.2.1/cosmovisor/CHANGELOG.md). diff --git a/cosmovisor/cmd/cosmovisor/version.go b/cosmovisor/cmd/cosmovisor/version.go index c8e427eb9316..e9806895d677 100644 --- a/cosmovisor/cmd/cosmovisor/version.go +++ b/cosmovisor/cmd/cosmovisor/version.go @@ -17,7 +17,7 @@ func init() { var ( // Version represents Cosmovisor version value. Overwritten during build - Version = "1.2.0" + Version = "1.2.1" // OutputFlag defines the output format flag OutputFlag = "output" ) From 92778f40960595e299f8970b4b8d5e587e99f69e Mon Sep 17 00:00:00 2001 From: Julien Robert Date: Sat, 13 Aug 2022 17:59:36 +0200 Subject: [PATCH 3/6] use version from go module (todo check releaser) --- cosmovisor/CHANGELOG.md | 1 + cosmovisor/Makefile | 4 +--- cosmovisor/cmd/cosmovisor/version.go | 22 ++++++++++++++-------- 3 files changed, 16 insertions(+), 11 deletions(-) diff --git a/cosmovisor/CHANGELOG.md b/cosmovisor/CHANGELOG.md index 20be6adeab11..12c6f45dd0a1 100644 --- a/cosmovisor/CHANGELOG.md +++ b/cosmovisor/CHANGELOG.md @@ -40,6 +40,7 @@ Ref: https://keepachangelog.com/en/1.0.0/ ## v1.2.1 2022-08-15 * [\XXXX](https://github.com/cosmos/cosmos-sdk/pull/XXXX) Fix failure when installing cosmovisor via `go install`. +* [\XXXX](https://github.com/cosmos/cosmos-sdk/pull/XXXX) Automatically set version using module version. * [\XXXX](https://github.com/cosmos/cosmos-sdk/pull/XXXX) Fix plan path case sensitivity issue. ## v1.2.0 2022-07-26 diff --git a/cosmovisor/Makefile b/cosmovisor/Makefile index d904c990cc11..b5a7be601e9c 100644 --- a/cosmovisor/Makefile +++ b/cosmovisor/Makefile @@ -1,11 +1,9 @@ #!/usr/bin/make -f -VERSION := $(shell echo $(shell git describe --always --match "cosmovisor/v*") | sed 's/^cosmovisor[/]//') - all: cosmovisor test cosmovisor: - go build -ldflags="-X 'github.com/cosmos/cosmos-sdk/cosmovisor/cmd/cosmovisor/cmd.Version=$(VERSION)'" -mod=readonly ./cmd/cosmovisor + go build -mod=readonly ./cmd/cosmovisor test: go test -mod=readonly -race ./... diff --git a/cosmovisor/cmd/cosmovisor/version.go b/cosmovisor/cmd/cosmovisor/version.go index e9806895d677..0651da67250e 100644 --- a/cosmovisor/cmd/cosmovisor/version.go +++ b/cosmovisor/cmd/cosmovisor/version.go @@ -3,6 +3,7 @@ package main import ( "encoding/json" "fmt" + "runtime/debug" "strings" "github.com/cosmos/cosmos-sdk/cosmovisor" @@ -15,12 +16,8 @@ func init() { rootCmd.AddCommand(versionCmd) } -var ( - // Version represents Cosmovisor version value. Overwritten during build - Version = "1.2.1" - // OutputFlag defines the output format flag - OutputFlag = "output" -) +// OutputFlag defines the output format flag +var OutputFlag = "output" var versionCmd = &cobra.Command{ Use: "version", @@ -37,8 +34,17 @@ var versionCmd = &cobra.Command{ }, } +func getVersion() string { + version, ok := debug.ReadBuildInfo() + if !ok { + panic("failed to get cosmovisor version") + } + + return strings.TrimSpace(version.Main.Version) +} + func printVersion(logger *zerolog.Logger, args []string) error { - fmt.Println("cosmovisor version: ", Version) + fmt.Printf("cosmovisor version: %s\n", getVersion()) if err := Run(logger, append([]string{"version"}, args...)); err != nil { return fmt.Errorf("failed to run version command: %w", err) @@ -66,7 +72,7 @@ func printVersionJSON(logger *zerolog.Logger, args []string) error { Version string `json:"cosmovisor_version"` AppVersion json.RawMessage `json:"app_version"` }{ - Version: Version, + Version: getVersion(), AppVersion: json.RawMessage(buf.String()), }) if err != nil { From 2aedfb58534dc6b6be86df22fa793bba455c847d Mon Sep 17 00:00:00 2001 From: Julien Robert Date: Sat, 13 Aug 2022 18:04:50 +0200 Subject: [PATCH 4/6] update changelog --- cosmovisor/CHANGELOG.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/cosmovisor/CHANGELOG.md b/cosmovisor/CHANGELOG.md index 12c6f45dd0a1..ddcb744ed72a 100644 --- a/cosmovisor/CHANGELOG.md +++ b/cosmovisor/CHANGELOG.md @@ -39,9 +39,9 @@ Ref: https://keepachangelog.com/en/1.0.0/ ## v1.2.1 2022-08-15 -* [\XXXX](https://github.com/cosmos/cosmos-sdk/pull/XXXX) Fix failure when installing cosmovisor via `go install`. -* [\XXXX](https://github.com/cosmos/cosmos-sdk/pull/XXXX) Automatically set version using module version. -* [\XXXX](https://github.com/cosmos/cosmos-sdk/pull/XXXX) Fix plan path case sensitivity issue. +* [\12918](https://github.com/cosmos/cosmos-sdk/pull/12918) Fix failure when installing cosmovisor via `go install`. +* [\12918](https://github.com/cosmos/cosmos-sdk/pull/12918) Automatically set version using module version. +* [\12918](https://github.com/cosmos/cosmos-sdk/pull/12918) Fix plan path case sensitivity issue. ## v1.2.0 2022-07-26 From e24505d528d16662bbb9d9fd49f6c7b77d08e8b9 Mon Sep 17 00:00:00 2001 From: Julien Robert Date: Sun, 14 Aug 2022 00:27:39 +0200 Subject: [PATCH 5/6] updates --- cosmovisor/args_test.go | 7 ++ cosmovisor/scanner.go | 4 - cosmovisor/scanner_test.go | 79 +++++++++++-------- .../testdata/upgrade-files/f2-good.json | 1 + 4 files changed, 54 insertions(+), 37 deletions(-) create mode 100644 cosmovisor/testdata/upgrade-files/f2-good.json diff --git a/cosmovisor/args_test.go b/cosmovisor/args_test.go index e42749e0c76a..a8bec5bb2e5e 100644 --- a/cosmovisor/args_test.go +++ b/cosmovisor/args_test.go @@ -144,6 +144,13 @@ func (s *argsTestSuite) TestConfigPaths() { expectGenesis: fmt.Sprintf("/longer/prefix/%s/genesis/bin/yourd", rootName), expectUpgrade: "/longer/prefix/cosmovisor/upgrades/some%20spaces/bin/yourd", }, + "handle casing": { + cfg: Config{Home: "/longer/prefix/", Name: "appd"}, + upgradeName: "myUpgrade", + expectRoot: fmt.Sprintf("/longer/prefix/%s", rootName), + expectGenesis: fmt.Sprintf("/longer/prefix/%s/genesis/bin/appd", rootName), + expectUpgrade: "/longer/prefix/cosmovisor/upgrades/myUpgrade/bin/appd", + }, } for _, tc := range cases { diff --git a/cosmovisor/scanner.go b/cosmovisor/scanner.go index 7ed696f48150..14bc316301f9 100644 --- a/cosmovisor/scanner.go +++ b/cosmovisor/scanner.go @@ -158,9 +158,5 @@ func parseUpgradeInfoFile(filename string) (upgradetypes.Plan, error) { return upgradetypes.Plan{}, fmt.Errorf("invalid upgrade-info.json content; name and height must be not empty; got: %v", ui) } - // Normalize name to prevent operator error in upgrade name case sensitivity - // errors. - ui.Name = strings.ToLower(ui.Name) - return ui, err } diff --git a/cosmovisor/scanner_test.go b/cosmovisor/scanner_test.go index f45dfafe85fd..f98d5707ac3b 100644 --- a/cosmovisor/scanner_test.go +++ b/cosmovisor/scanner_test.go @@ -14,39 +14,52 @@ func TestParseUpgradeInfoFile(t *testing.T) { filename string expectUpgrade upgradetypes.Plan expectErr bool - }{{ - filename: "f1-good.json", - expectUpgrade: upgradetypes.Plan{Name: "upgrade1", Info: "some info", Height: 123}, - expectErr: false, - }, { - filename: "f2-bad-type.json", - expectUpgrade: upgradetypes.Plan{}, - expectErr: true, - }, { - filename: "f2-bad-type-2.json", - expectUpgrade: upgradetypes.Plan{}, - expectErr: true, - }, { - filename: "f3-empty.json", - expectUpgrade: upgradetypes.Plan{}, - expectErr: true, - }, { - filename: "f4-empty-obj.json", - expectUpgrade: upgradetypes.Plan{}, - expectErr: true, - }, { - filename: "f5-partial-obj-1.json", - expectUpgrade: upgradetypes.Plan{}, - expectErr: true, - }, { - filename: "f5-partial-obj-2.json", - expectUpgrade: upgradetypes.Plan{}, - expectErr: true, - }, { - filename: "unknown.json", - expectUpgrade: upgradetypes.Plan{}, - expectErr: true, - }} + }{ + { + filename: "f1-good.json", + expectUpgrade: upgradetypes.Plan{Name: "upgrade1", Info: "some info", Height: 123}, + expectErr: false, + }, + { + filename: "f2-good.json", + expectUpgrade: upgradetypes.Plan{Name: "Upgrade2", Info: "some info", Height: 125}, + expectErr: false, + }, + { + filename: "f2-bad-type.json", + expectUpgrade: upgradetypes.Plan{}, + expectErr: true, + }, + { + filename: "f2-bad-type-2.json", + expectUpgrade: upgradetypes.Plan{}, + expectErr: true, + }, + { + filename: "f3-empty.json", + expectUpgrade: upgradetypes.Plan{}, + expectErr: true, + }, + { + filename: "f4-empty-obj.json", + expectUpgrade: upgradetypes.Plan{}, + expectErr: true, + }, + { + filename: "f5-partial-obj-1.json", + expectUpgrade: upgradetypes.Plan{}, + expectErr: true, + }, + { + filename: "f5-partial-obj-2.json", + expectUpgrade: upgradetypes.Plan{}, + expectErr: true, + }, + { + filename: "unknown.json", + expectUpgrade: upgradetypes.Plan{}, + expectErr: true, + }} for i := range cases { tc := cases[i] diff --git a/cosmovisor/testdata/upgrade-files/f2-good.json b/cosmovisor/testdata/upgrade-files/f2-good.json new file mode 100644 index 000000000000..af81a0bbd104 --- /dev/null +++ b/cosmovisor/testdata/upgrade-files/f2-good.json @@ -0,0 +1 @@ +{"name": "Upgrade2", "info": "some info", "height": 125} From 157b856eb884f211901b5665bd3ec1b03a18e661 Mon Sep 17 00:00:00 2001 From: Julien Robert Date: Mon, 15 Aug 2022 00:39:36 +0200 Subject: [PATCH 6/6] updates --- cosmovisor/go.sum | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/cosmovisor/go.sum b/cosmovisor/go.sum index 95f8fd632e32..ad8620120fa7 100644 --- a/cosmovisor/go.sum +++ b/cosmovisor/go.sum @@ -373,6 +373,11 @@ github.com/godbus/dbus/v5 v5.0.6/go.mod h1:xhWf0FNVPg57R7Z0UbKHbJfkEywrmjJnf7w5x github.com/gofrs/flock v0.8.1/go.mod h1:F1TvTiK9OcQqauNUHlbJvyl9Qa1QvF/gOUDKA14jxHU= github.com/gofrs/uuid v4.2.0+incompatible/go.mod h1:b2aQJv3Z4Fp6yNu3cdSllBxTCLRxnplIgP/c0N/04lM= github.com/gogo/gateway v1.1.0 h1:u0SuhL9+Il+UbjM9VIE3ntfRujKbvVpFvNB4HbjeVQ0= +github.com/gogo/protobuf v1.1.1/go.mod h1:r8qH/GZQm5c6nD/R0oafs1akxWv10x8SbQlK7atdtwQ= +github.com/gogo/protobuf v1.2.1/go.mod h1:hp+jE20tsWTFYpLwKvXlhS1hjn+gTNwPg2I6zVXpSg4= +github.com/gogo/protobuf v1.3.0/go.mod h1:SlYgWuQ5SjCEi6WLHjHCa1yvBfUnHcTbrrZtXPKa29o= +github.com/gogo/protobuf v1.3.1/go.mod h1:SlYgWuQ5SjCEi6WLHjHCa1yvBfUnHcTbrrZtXPKa29o= +github.com/gogo/protobuf v1.3.2 h1:Ov1cvc58UF3b5XjBnZv7+opcTcQFZebYjWzi34vdm4Q= github.com/golang-jwt/jwt/v4 v4.0.0/go.mod h1:/xlHOz8bRuivTWchD4jCa+NbatV+wEUSzwAxVc6locg= github.com/golang-sql/civil v0.0.0-20190719163853-cb61b32ac6fe/go.mod h1:8vg3r2VgvsThLBIFL93Qb5yWzgyZWhEmBwUJWevAkK0= github.com/golang-sql/sqlexp v0.0.0-20170517235910-f1bb20e5a188/go.mod h1:vXjM/+wXQnTPR4KqTKDgJukSZ6amVRtWMPEjE6sQoK8= @@ -642,6 +647,8 @@ github.com/julienschmidt/httprouter v1.3.0/go.mod h1:JR6WtHb+2LUe8TCKY3cZOxFyyO8 github.com/julz/importas v0.1.0/go.mod h1:oSFU2R4XK/P7kNBrnL/FEQlDGN1/6WoxXEjSSXO0DV0= github.com/jung-kurt/gofpdf v1.0.3-0.20190309125859-24315acbbda5/go.mod h1:7Id9E/uU8ce6rXgefFLlgrJj/GYY22cpxn+r32jIOes= github.com/k0kubun/colorstring v0.0.0-20150214042306-9440f1994b88/go.mod h1:3w7q1U84EfirKl04SVQ/s7nPm1ZPhiXd34z40TNz36k= +github.com/kisielk/errcheck v1.1.0/go.mod h1:EZBBE59ingxPouuu3KfxchcWSUPOHkagtvWXihfKN4Q= +github.com/kisielk/errcheck v1.2.0/go.mod h1:/BMXB+zMLi60iA8Vv6Ksmxu/1UDYcXs4uQLJ+jE2L00= github.com/kisielk/errcheck v1.5.0/go.mod h1:pFxgyoBC7bSaBwPgfKdkLd5X25qrDl4LWUI2bnpBCr8= github.com/kisielk/gotool v1.0.0/go.mod h1:XhKaO+MFFWcvkIS/tQcRk01m1F5IRFswLeQ+oQHNcck= github.com/kkdai/bstream v0.0.0-20161212061736-f391b8402d23/go.mod h1:J+Gs4SYgM6CZQHDETBtE9HaSEkGmuNXF86RwHhHUvq4= @@ -903,8 +910,6 @@ github.com/rcrowley/go-metrics v0.0.0-20181016184325-3113b8401b8a/go.mod h1:bCqn github.com/rcrowley/go-metrics v0.0.0-20200313005456-10cdbea86bc0 h1:MkV+77GLUNo5oJ0jf870itWm3D0Sjh7+Za9gazKc5LQ= github.com/rcrowley/go-metrics v0.0.0-20200313005456-10cdbea86bc0/go.mod h1:bCqnVzQkZxMG4s8nGwiZ5l3QUCyqpo9Y+/ZMZ9VjZe4= github.com/regen-network/cosmos-proto v0.3.1 h1:rV7iM4SSFAagvy8RiyhiACbWEGotmqzywPxOvwMdxcg= -github.com/regen-network/protobuf v1.3.3-alpha.regen.1 h1:OHEc+q5iIAXpqiqFKeLpu5NwTIkVXUs48vFMwzqpqY4= -github.com/regen-network/protobuf v1.3.3-alpha.regen.1/go.mod h1:2DjTFR1HhMQhiWC5sZ4OhQ3+NtdbZ6oBDKQwq5Ou+FI= github.com/remyoudompheng/go-dbus v0.0.0-20121104212943-b7232d34b1d5/go.mod h1:+u151txRmLpwxBmpYn9z3d1sdJdjRPQpsXuYeY9jNls= github.com/remyoudompheng/go-liblzma v0.0.0-20190506200333-81bf2d431b96/go.mod h1:90HvCY7+oHHUKkbeMCiHt1WuFR2/hPJ9QrljDG+v6ls= github.com/remyoudompheng/go-misc v0.0.0-20190427085024-2d6ac652a50e/go.mod h1:80FQABjoFzZ2M5uEa6FUaJYEmqU2UOKojlFVak1UAwI= @@ -1437,8 +1442,10 @@ golang.org/x/time v0.0.0-20190308202827-9d24e82272b4/go.mod h1:tRJNPiyCQ0inRvYxb golang.org/x/time v0.0.0-20191024005414-555d28b269f0/go.mod h1:tRJNPiyCQ0inRvYxbN9jk5I+vvW/OXSQhTDSoE431IQ= golang.org/x/time v0.0.0-20200416051211-89c76fbcd5d1/go.mod h1:tRJNPiyCQ0inRvYxbN9jk5I+vvW/OXSQhTDSoE431IQ= golang.org/x/time v0.0.0-20210723032227-1f47c861a9ac/go.mod h1:tRJNPiyCQ0inRvYxbN9jk5I+vvW/OXSQhTDSoE431IQ= +golang.org/x/tools v0.0.0-20180221164845-07fd8470d635/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ= golang.org/x/tools v0.0.0-20180525024113-a5b4c53f6e8b/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ= golang.org/x/tools v0.0.0-20180917221912-90fa682c2a6e/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ= +golang.org/x/tools v0.0.0-20181030221726-6c7e314b6563/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ= golang.org/x/tools v0.0.0-20190114222345-bf090417da8b/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ= golang.org/x/tools v0.0.0-20190206041539-40960b6deb8e/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ= golang.org/x/tools v0.0.0-20190226205152-f727befe758c/go.mod h1:9Yl7xja0Znq3iFh3HoIrodX9oNMXvdceNzlUR8zjMvY= @@ -1624,7 +1631,6 @@ google.golang.org/genproto v0.0.0-20200224152610-e50cd9704f63/go.mod h1:55QSHmfG google.golang.org/genproto v0.0.0-20200228133532-8c2c7df3a383/go.mod h1:55QSHmfGQM9UVYDPBsyGGes0y52j32PQ3BqQfXhyH3c= google.golang.org/genproto v0.0.0-20200305110556-506484158171/go.mod h1:55QSHmfGQM9UVYDPBsyGGes0y52j32PQ3BqQfXhyH3c= google.golang.org/genproto v0.0.0-20200312145019-da6875a35672/go.mod h1:55QSHmfGQM9UVYDPBsyGGes0y52j32PQ3BqQfXhyH3c= -google.golang.org/genproto v0.0.0-20200324203455-a04cca1dde73/go.mod h1:55QSHmfGQM9UVYDPBsyGGes0y52j32PQ3BqQfXhyH3c= google.golang.org/genproto v0.0.0-20200331122359-1ee6d9798940/go.mod h1:55QSHmfGQM9UVYDPBsyGGes0y52j32PQ3BqQfXhyH3c= google.golang.org/genproto v0.0.0-20200423170343-7949de9c1215/go.mod h1:55QSHmfGQM9UVYDPBsyGGes0y52j32PQ3BqQfXhyH3c= google.golang.org/genproto v0.0.0-20200430143042-b979b6f78d84/go.mod h1:55QSHmfGQM9UVYDPBsyGGes0y52j32PQ3BqQfXhyH3c=