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

Cosmovisor doesn't report its version properly #15360

Closed
freak12techno opened this issue Mar 10, 2023 · 9 comments
Closed

Cosmovisor doesn't report its version properly #15360

freak12techno opened this issue Mar 10, 2023 · 9 comments
Assignees
Labels
C:Cosmovisor Issues and PR related to Cosmovisor

Comments

@freak12techno
Copy link
Contributor

Summary of Bug

When running cosmovisor version it displays its version as (devel) instead of a proper version (like 1.4.0, the one I've tested it on).

validator@cosmos-validator ~/cosmos-sdk #tools/cosmovisor/v1.4.0 ?1 ❯ cosmovisor version
cosmovisor version: (devel)
11:16PM INF running app args=["version"] module=cosmovisor path=/home/validator/.gaia/cosmovisor/upgrades/v8-rho/bin/gaiad
v8.0.1

Version

validator@cosmos-validator ~/cosmos-sdk #tools/cosmovisor/v1.4.0 ?1 ❯ git status
HEAD detached at tools/cosmovisor/v1.4.0

Steps to Reproduce

  1. Run cosmovisor version
@github-actions github-actions bot added the needs-triage Issue that needs to be triaged label Mar 10, 2023
@freak12techno
Copy link
Contributor Author

Additionally I am facing the following when running it as --output json, not sure if that's related or it should be another issue.

validator@cosmos-validator ~/cosmos-sdk #tools/cosmovisor/v1.4.0 ?1 ❯ cosmovisor version --output json
{"name":"gaia","server_name":"gaiad","version":"v8.0.1","commit":"890ab3aa2e5788537b0d2ebc9bafdc968340e0e5","build_tags":"netgo ledger,","go":"go version go1.18.5 linux/amd64","build_deps":["cosmossdk.io/api@v0.2.6","cosmossdk.io/core@v0.5.1","cosmossdk.io/depinject@v1.0.0-alpha.3","filippo.io/edwards25519@v1.0.0-rc.1","github.com/99designs/keyring@v1.2.1 =\u003e github.com/cosmos/keyring@v1.2.0","github.com/ChainSafe/go-schnorrkel@v0.0.0-20200405005733-88cbf1b4c40d","github.com/Workiva/go-datastructures@v1.0.53","github.com/armon/go-metrics@v0.4.0","github.com/beorn7/perks@v1.0.1","github.com/bgentry/speakeasy@v0.1.1-0.20220910012023-760eaf8b6816","github.com/btcsuite/btcd@v0.23.0 =\u003e github.com/btcsuite/btcd@v0.22.2","github.com/btcsuite/btcd/btcec/v2@v2.3.2","github.com/cenkalti/backoff/v4@v4.1.3","github.com/cespare/xxhash/v2@v2.1.2","github.com/coinbase/rosetta-sdk-go@v0.7.9","github.com/confio/ics23/go@v0.9.0 =\u003e github.com/confio/ics23/go@v0.9.0","github.com/cosmos/btcutil@v1.0.4","github.com/cosmos/cosmos-db@v0.0.0-20221226095112-f3c38ecb5e32","github.com/cosmos/cosmos-proto@v1.0.0-beta.1","github.com/cosmos/cosmos-sdk@v0.45.14 =\u003e github.com/cosmos/cosmos-sdk@v0.45.14","github.com/cosmos/go-bip39@v1.0.0","github.com/cosmos/iavl@v0.19.5","github.com/cosmos/ibc-go/v3@v3.4.0 =\u003e github.com/cosmos/ibc-go/v3@v3.4.0","github.com/cosmos/ledger-cosmos-go@v0.12.2","github.com/creachadair/taskgroup@v0.3.2","github.com/davecgh/go-spew@v1.1.1","github.com/decred/dcrd/dcrec/secp256k1/v4@v4.0.1","github.com/desertbit/timer@v0.0.0-20180107155436-c41aec40b27f","github.com/dvsekhvalnov/jose2go@v1.5.0","github.com/felixge/httpsnoop@v1.0.1","github.com/fsnotify/fsnotify@v1.6.0","github.com/go-kit/kit@v0.12.0","github.com/go-kit/log@v0.2.1","github.com/go-logfmt/logfmt@v0.5.1","github.com/godbus/dbus@v0.0.0-20190726142602-4481cbc300e2","github.com/gogo/gateway@v1.1.0","github.com/gogo/protobuf@v1.3.3 =\u003e github.com/regen-network/protobuf@v1.3.3-alpha.regen.1","github.com/golang/protobuf@v1.5.2","github.com/golang/snappy@v0.0.4","github.com/google/btree@v1.1.2","github.com/google/orderedcode@v0.0.1","github.com/gorilla/handlers@v1.5.1","github.com/gorilla/mux@v1.8.0","github.com/gorilla/websocket@v1.5.0","github.com/gravity-devs/liquidity@v1.5.3","github.com/grpc-ecosystem/go-grpc-middleware@v1.3.0","github.com/grpc-ecosystem/grpc-gateway@v1.16.0","github.com/grpc-ecosystem/grpc-gateway/v2@v2.0.1","github.com/gsterjov/go-libsecret@v0.0.0-20161001094733-a6f4afe4910c","github.com/gtank/merlin@v0.1.1","github.com/gtank/ristretto255@v0.1.2","github.com/hashicorp/go-immutable-radix@v1.3.1","github.com/hashicorp/golang-lru@v0.5.5-0.20210104140557-80c98217689d","github.com/hashicorp/hcl@v1.0.0","github.com/hdevalence/ed25519consensus@v0.0.0-20220222234857-c00d1f31bab3","github.com/improbable-eng/grpc-web@v0.15.0","github.com/klauspost/compress@v1.15.11","github.com/lib/pq@v1.10.6","github.com/libp2p/go-buffer-pool@v0.1.0","github.com/magiconair/properties@v1.8.6","github.com/mattn/go-colorable@v0.1.13","github.com/mattn/go-isatty@v0.0.16","github.com/matttproud/golang_protobuf_extensions@v1.0.2-0.20181231171920-c182affec369","github.com/mimoo/StrobeGo@v0.0.0-20210601165009-122bf33a46e0","github.com/minio/highwayhash@v1.0.2","github.com/mitchellh/mapstructure@v1.5.0","github.com/mtibben/percent@v0.2.1","github.com/pelletier/go-toml/v2@v2.0.5","github.com/pkg/errors@v0.9.1","github.com/pmezard/go-difflib@v1.0.0","github.com/prometheus/client_golang@v1.14.0","github.com/prometheus/client_model@v0.3.0","github.com/prometheus/common@v0.37.0","github.com/prometheus/procfs@v0.8.0","github.com/rakyll/statik@v0.1.7","github.com/rcrowley/go-metrics@v0.0.0-20201227073835-cf1acfcdf475","github.com/regen-network/cosmos-proto@v0.3.1","github.com/rs/cors@v1.8.2","github.com/rs/zerolog@v1.27.0","github.com/spf13/afero@v1.9.2","github.com/spf13/cast@v1.5.0","github.com/spf13/cobra@v1.6.1","github.com/spf13/jwalterweatherman@v1.1.0","github.com/spf13/pflag@v1.0.5","github.com/spf13/viper@v1.14.0","github.com/strangelove-ventures/packet-forward-middleware/v3@v3.1.1","github.com/stretchr/testify@v1.8.1","github.com/subosito/gotenv@v1.4.1","github.com/syndtr/goleveldb@v1.0.1-0.20210819022825-2ae1ddf74ef7","github.com/tendermint/go-amino@v0.16.0","github.com/tendermint/tendermint@v0.34.26 =\u003e github.com/informalsystems/tendermint@v0.34.26","github.com/tendermint/tm-db@v0.6.7","github.com/tidwall/btree@v1.5.0","github.com/zondax/hid@v0.9.1","github.com/zondax/ledger-go@v0.14.1","golang.org/x/crypto@v0.5.0","golang.org/x/exp@v0.0.0-20221205204356-47842c84f3db","golang.org/x/net@v0.5.0","golang.org/x/sys@v0.4.0","golang.org/x/term@v0.4.0","golang.org/x/text@v0.6.0","google.golang.org/genproto@v0.0.0-20230125152338-dcaf20b6aeaa","google.golang.org/grpc@v1.52.3 =\u003e google.golang.org/grpc@v1.33.2","google.golang.org/protobuf@v1.28.2-0.20220831092852-f930b1dc76e8","gopkg.in/ini.v1@v1.67.0","gopkg.in/yaml.v2@v2.4.0","gopkg.in/yaml.v3@v3.0.1","nhooyr.io/websocket@v1.8.6"],"cosmos_sdk_version":"v0.45.14"}
Error: can't print version output, expected valid json from APP, got:  - json: error calling MarshalJSON for type json.RawMessage: unexpected end of JSON input
11:12PM ERR error="can't print version output, expected valid json from APP, got:  - json: error calling MarshalJSON for type json.RawMessage: unexpected end of JSON input" module=cosmovisor

@julienrbrt
Copy link
Member

julienrbrt commented Mar 11, 2023

How have you installed cosmovisor?
Can you try to install it using go install?

$ go install cosmossdk.io/tools/cosmovisor/cmd/cosmovisor@v1.4.0
$ DAEMON_NAME=simd DAEMON_HOME=~/.simapp DAEMON_DATA_BACK_DIR=~/.simapp/bak cosmovisor version
cosmovisor version: v1.4.0
...

@julienrbrt julienrbrt added S:needs more info This bug can't be addressed until more information is provided by the reporter. C:Cosmovisor Issues and PR related to Cosmovisor and removed needs-triage Issue that needs to be triaged labels Mar 11, 2023
@julienrbrt
Copy link
Member

Additionally I am facing the following when running it as --output json, not sure if that's related or it should be another issue.

For the second point, this is related to gaia not returning a valid JSON and not to Cosmovisor

@freak12techno
Copy link
Contributor Author

@julienrbrt

How have you installed cosmovisor?

I compiled it from source, like described here: https://docs.cosmos.network/main/tooling/cosmovisor#installation.

Can you try to install it using go install?

Just tested it, got the same result as you did. I assume that this is not working only when compiling Cosmovisor from source then. It was totally working before though, I've seen some changes where the version injected into app from ldflags was removed and ReadBuildInfo was used instead, my assumption is that it was broken at that time.

For the second point, this is related to gaia not returning a valid JSON and not to Cosmovisor

Seems not, I also have it on other nodes, for example this is bitsong (also compiled from source):

$ cosmovisor version --output json
{"name":"go-bitsong","server_name":"bitsongd","version":"0.14.0","commit":"e5847ac0b4f0319811d62140f4e438d33163ad0d","build_tags":"netgo,ledger","go":"go version go1.19.3 linux/amd64","build_deps":["filippo.io/edwards25519@v1.0.0-beta.2","github.com/99designs/keyring@v1.2.1","github.com/ChainSafe/go-schnorrkel@v0.0.0-20200405005733-88cbf1b4c40d","github.com/CosmWasm/wasmd@v0.29.2","github.com/CosmWasm/wasmvm@v1.1.1","github.com/Workiva/go-datastructures@v1.0.53","github.com/armon/go-metrics@v0.4.0","github.com/beorn7/perks@v1.0.1","github.com/bgentry/speakeasy@v0.1.0","github.com/btcsuite/btcd@v0.22.1","github.com/cespare/xxhash/v2@v2.1.2","github.com/coinbase/rosetta-sdk-go@v0.7.0","github.com/confio/ics23/go@v0.7.0 =\u003e github.com/cosmos/cosmos-sdk/ics23/go@v0.8.0","github.com/cosmos/btcutil@v1.0.4","github.com/cosmos/cosmos-proto@v1.0.0-alpha7","github.com/cosmos/cosmos-sdk@v0.45.11","github.com/cosmos/go-bip39@v1.0.0","github.com/cosmos/gogoproto@v1.4.2","github.com/cosmos/iavl@v0.19.4","github.com/cosmos/ibc-go/v3@v3.3.1","github.com/cosmos/ledger-cosmos-go@v0.11.1","github.com/cosmos/ledger-go@v0.9.2","github.com/creachadair/taskgroup@v0.3.2","github.com/davecgh/go-spew@v1.1.1","github.com/desertbit/timer@v0.0.0-20180107155436-c41aec40b27f","github.com/dvsekhvalnov/jose2go@v1.5.0","github.com/felixge/httpsnoop@v1.0.1","github.com/fsnotify/fsnotify@v1.5.4","github.com/go-kit/kit@v0.12.0","github.com/go-kit/log@v0.2.1","github.com/go-logfmt/logfmt@v0.5.1","github.com/godbus/dbus@v0.0.0-20190726142602-4481cbc300e2","github.com/gogo/gateway@v1.1.0","github.com/gogo/protobuf@v1.3.3 =\u003e github.com/regen-network/protobuf@v1.3.3-alpha.regen.1","github.com/golang/protobuf@v1.5.2","github.com/golang/snappy@v0.0.4","github.com/google/btree@v1.1.2","github.com/google/gofuzz@v1.2.0","github.com/google/orderedcode@v0.0.1","github.com/gorilla/handlers@v1.5.1","github.com/gorilla/mux@v1.8.0","github.com/gorilla/websocket@v1.5.0","github.com/grpc-ecosystem/go-grpc-middleware@v1.3.0","github.com/grpc-ecosystem/grpc-gateway@v1.16.0","github.com/gsterjov/go-libsecret@v0.0.0-20161001094733-a6f4afe4910c","github.com/gtank/merlin@v0.1.1","github.com/gtank/ristretto255@v0.1.2","github.com/hashicorp/go-immutable-radix@v1.3.1","github.com/hashicorp/golang-lru@v0.5.4","github.com/hashicorp/hcl@v1.0.0","github.com/hdevalence/ed25519consensus@v0.0.0-20210204194344-59a8610d2b87","github.com/improbable-eng/grpc-web@v0.14.1","github.com/klauspost/compress@v1.15.11","github.com/lib/pq@v1.10.6","github.com/libp2p/go-buffer-pool@v0.1.0","github.com/magiconair/properties@v1.8.6","github.com/mattn/go-colorable@v0.1.13","github.com/mattn/go-isatty@v0.0.16","github.com/matttproud/golang_protobuf_extensions@v1.0.2-0.20181231171920-c182affec369","github.com/mimoo/StrobeGo@v0.0.0-20210601165009-122bf33a46e0","github.com/minio/highwayhash@v1.0.2","github.com/mitchellh/mapstructure@v1.5.0","github.com/mtibben/percent@v0.2.1","github.com/pelletier/go-toml/v2@v2.0.5","github.com/pkg/errors@v0.9.1","github.com/pmezard/go-difflib@v1.0.0","github.com/prometheus/client_golang@v1.13.0","github.com/prometheus/client_model@v0.2.0","github.com/prometheus/common@v0.37.0","github.com/prometheus/procfs@v0.8.0","github.com/rakyll/statik@v0.1.7","github.com/rcrowley/go-metrics@v0.0.0-20201227073835-cf1acfcdf475","github.com/regen-network/cosmos-proto@v0.3.1","github.com/rs/cors@v1.8.2","github.com/rs/zerolog@v1.27.0","github.com/spf13/afero@v1.8.2","github.com/spf13/cast@v1.5.0","github.com/spf13/cobra@v1.6.0","github.com/spf13/jwalterweatherman@v1.1.0","github.com/spf13/pflag@v1.0.5","github.com/spf13/viper@v1.13.0","github.com/strangelove-ventures/packet-forward-middleware/v2@v2.1.1","github.com/stretchr/testify@v1.8.0","github.com/subosito/gotenv@v1.4.1","github.com/syndtr/goleveldb@v1.0.1-0.20210819022825-2ae1ddf74ef7","github.com/tendermint/btcd@v0.1.1","github.com/tendermint/crypto@v0.0.0-20191022145703-50d29ede1e15","github.com/tendermint/go-amino@v0.16.0","github.com/tendermint/tendermint@v0.34.24","github.com/tendermint/tm-db@v0.6.7","github.com/zondax/hid@v0.9.0","golang.org/x/crypto@v0.1.0","golang.org/x/exp@v0.0.0-20220722155223-a9213eeb770e","golang.org/x/net@v0.1.0","golang.org/x/sys@v0.1.0","golang.org/x/term@v0.1.0","golang.org/x/text@v0.4.0","google.golang.org/genproto@v0.0.0-20221014213838-99cd37c6964a","google.golang.org/grpc@v1.50.1 =\u003e google.golang.org/grpc@v1.33.2","google.golang.org/protobuf@v1.28.2-0.20220831092852-f930b1dc76e8","gopkg.in/ini.v1@v1.67.0","gopkg.in/yaml.v2@v2.4.0","gopkg.in/yaml.v3@v3.0.1","nhooyr.io/websocket@v1.8.6"],"cosmos_sdk_version":"v0.45.11"}
Error: can't print version output, expected valid json from APP, got:  - json: error calling MarshalJSON for type json.RawMessage: unexpected end of JSON input
11:41PM ERR error="can't print version output, expected valid json from APP, got:  - json: error calling MarshalJSON for type json.RawMessage: unexpected end of JSON input" module=cosmovisor

I am not sure whether these bitsongd/gaiad daemons are writing to stderr or stdout though.

@julienrbrt
Copy link
Member

Oh yes, we wanted to make it work with go install, in a more "automatic" fashion so using ReadBuildInfo was perfect.
So it is probably why it shows devel when building from source.

On your second point I am not sure. Does it work for you too with simapp? I know Gaia had some inconsistencies with where some commands were returning (seen a issue recently on their repo about that).

@freak12techno
Copy link
Contributor Author

@julienrbrt

Oh yes, we wanted to make it work with go install, in a more "automatic" fashion so using ReadBuildInfo was perfect.
So it is probably why it shows devel when building from source.

I guess it makes sense to either deprecate building from binary or make it work properly for binaries built from source, what do you think?

On your second point I am not sure. Does it work for you too with simapp? I know Gaia had some inconsistencies with where some commands were returning (seen a issue recently on their repo about that).

Actually just tested and it seem to work file with simd:

➜  cosmos-sdk git:(main) DAEMON_NAME=simd DAEMON_HOME=~/.simapp DAEMON_DATA_BACK_DIR=~/.simapp/bak cosmovisor version --output json
{"cosmovisor_version":"v1.4.0","app_version":{"name":"sim","server_name":"simd","version":"0.46.0-beta2-1616-g00154fec3","commit":"00154fec385c6e1b3242b2c5bd785bc1b419f153","build_tags":"netgo,ledger","go":"go version go1.19.6 darwin/arm64","build_deps":["cloud.google.com/go@v0.110.0","cloud.google.com/go/compute/metadata@v0.2.3","cloud.google.com/go/iam@v0.11.0","cloud.google.com/go/storage@v1.29.0","cosmossdk.io/api@v0.3.1","cosmossdk.io/client/v2@v2.0.0-20230309163709-87da587416ba","cosmossdk.io/collections@v0.0.0-20230309163709-87da587416ba","cosmossdk.io/core@v0.6.1-0.20230309163709-87da587416ba","cosmossdk.io/depinject@v1.0.0-alpha.3","cosmossdk.io/errors@v1.0.0-beta.7","cosmossdk.io/log@v0.0.0-20230306220716-5e55f56d39d5","cosmossdk.io/math@v1.0.0-beta.6.0.20230216172121-959ce49135e4","cosmossdk.io/store@v0.0.0-20230309163709-87da587416ba","cosmossdk.io/tools/confix@v0.0.0-20230120150717-4f6f6c00021f =\u003e ../tools/confix@(devel)","cosmossdk.io/tools/rosetta@v0.2.0 =\u003e ../tools/rosetta@(devel)","cosmossdk.io/x/evidence@v0.1.0 =\u003e ../x/evidence@(devel)","cosmossdk.io/x/feegrant@v0.0.0-20230117113717-50e7c4a4ceff =\u003e ../x/feegrant@(devel)","cosmossdk.io/x/nft@v0.0.0-20230113085233-fae3332d62fc =\u003e ../x/nft@(devel)","cosmossdk.io/x/tx@v0.2.3-0.20230309163709-87da587416ba","cosmossdk.io/x/upgrade@v0.0.0-20230127052425-54c8e1568335 =\u003e ../x/upgrade@(devel)","filippo.io/edwards25519@v1.0.0","github.com/99designs/go-keychain@v0.0.0-20191008050251-8e49817e8af4","github.com/99designs/keyring@v1.2.1 =\u003e github.com/cosmos/keyring@v1.2.0","github.com/ChainSafe/go-schnorrkel@v0.0.0-20200405005733-88cbf1b4c40d","github.com/armon/go-metrics@v0.4.1","github.com/aws/aws-sdk-go@v1.44.203","github.com/beorn7/perks@v1.0.1","github.com/bgentry/go-netrc@v0.0.0-20140422174119-9fd32a8b3d3d","github.com/bgentry/speakeasy@v0.1.1-0.20220910012023-760eaf8b6816","github.com/btcsuite/btcd/btcec/v2@v2.3.2","github.com/cenkalti/backoff/v4@v4.1.3","github.com/cespare/xxhash/v2@v2.2.0","github.com/chzyer/readline@v1.5.1","github.com/cockroachdb/apd/v2@v2.0.2","github.com/cockroachdb/errors@v1.9.1","github.com/cockroachdb/logtags@v0.0.0-20230118201751-21c54148d20b","github.com/cockroachdb/redact@v1.1.3","github.com/coinbase/rosetta-sdk-go/types@v1.0.0","github.com/cometbft/cometbft@v0.37.0","github.com/cometbft/cometbft-db@v0.7.0","github.com/confio/ics23/go@v0.9.0","github.com/cosmos/btcutil@v1.0.5","github.com/cosmos/cosmos-db@v1.0.0-rc.1","github.com/cosmos/cosmos-proto@v1.0.0-beta.3","github.com/cosmos/cosmos-sdk@v0.48.0 =\u003e ../.@(devel)","github.com/cosmos/go-bip39@v1.0.0","github.com/cosmos/gogogateway@v1.2.0","github.com/cosmos/gogoproto@v1.4.6","github.com/cosmos/iavl@v0.21.0-alpha.1","github.com/cosmos/ledger-cosmos-go@v0.13.0","github.com/cosmos/rosetta-sdk-go@v0.10.0","github.com/creachadair/atomicfile@v0.2.8","github.com/creachadair/taskgroup@v0.4.2","github.com/creachadair/tomledit@v0.0.24","github.com/davecgh/go-spew@v1.1.1","github.com/decred/dcrd/dcrec/secp256k1/v4@v4.1.0","github.com/desertbit/timer@v0.0.0-20180107155436-c41aec40b27f","github.com/dvsekhvalnov/jose2go@v1.5.0","github.com/felixge/httpsnoop@v1.0.2","github.com/fsnotify/fsnotify@v1.6.0","github.com/getsentry/sentry-go@v0.18.0","github.com/go-kit/kit@v0.12.0","github.com/go-kit/log@v0.2.1","github.com/go-logfmt/logfmt@v0.6.0","github.com/gogo/googleapis@v1.4.1","github.com/gogo/protobuf@v1.3.2","github.com/golang/groupcache@v0.0.0-20210331224755-41bb18bfe9da","github.com/golang/mock@v1.6.0","github.com/golang/protobuf@v1.5.3","github.com/golang/snappy@v0.0.4","github.com/google/btree@v1.1.2","github.com/google/go-cmp@v0.5.9","github.com/google/orderedcode@v0.0.1","github.com/google/uuid@v1.3.0","github.com/googleapis/enterprise-certificate-proxy@v0.2.3","github.com/googleapis/gax-go/v2@v2.7.0","github.com/gorilla/handlers@v1.5.1","github.com/gorilla/mux@v1.8.0","github.com/gorilla/websocket@v1.5.0","github.com/grpc-ecosystem/go-grpc-middleware@v1.3.0","github.com/grpc-ecosystem/grpc-gateway@v1.16.0","github.com/gtank/merlin@v0.1.1","github.com/gtank/ristretto255@v0.1.2","github.com/hashicorp/go-cleanhttp@v0.5.2","github.com/hashicorp/go-getter@v1.7.0","github.com/hashicorp/go-immutable-radix@v1.3.1","github.com/hashicorp/go-safetemp@v1.0.0","github.com/hashicorp/go-version@v1.6.0","github.com/hashicorp/golang-lru@v0.5.5-0.20210104140557-80c98217689d","github.com/hashicorp/hcl@v1.0.0","github.com/hdevalence/ed25519consensus@v0.1.0","github.com/huandu/skiplist@v1.2.0","github.com/improbable-eng/grpc-web@v0.15.0","github.com/jmespath/go-jmespath@v0.4.0","github.com/klauspost/compress@v1.16.0","github.com/kr/pretty@v0.3.1","github.com/kr/text@v0.2.0","github.com/lib/pq@v1.10.7","github.com/libp2p/go-buffer-pool@v0.1.0","github.com/magiconair/properties@v1.8.7","github.com/manifoldco/promptui@v0.9.0","github.com/mattn/go-colorable@v0.1.13","github.com/mattn/go-isatty@v0.0.17","github.com/matttproud/golang_protobuf_extensions@v1.0.4","github.com/mimoo/StrobeGo@v0.0.0-20210601165009-122bf33a46e0","github.com/minio/highwayhash@v1.0.2","github.com/mitchellh/go-homedir@v1.1.0","github.com/mitchellh/go-testing-interface@v1.14.1","github.com/mitchellh/mapstructure@v1.5.0","github.com/mtibben/percent@v0.2.1","github.com/pelletier/go-toml/v2@v2.0.7","github.com/pkg/errors@v0.9.1","github.com/pmezard/go-difflib@v1.0.0","github.com/prometheus/client_golang@v1.14.0","github.com/prometheus/client_model@v0.3.0","github.com/prometheus/common@v0.42.0","github.com/prometheus/procfs@v0.9.0","github.com/rcrowley/go-metrics@v0.0.0-20201227073835-cf1acfcdf475","github.com/rogpeppe/go-internal@v1.9.0","github.com/rs/cors@v1.8.3","github.com/rs/zerolog@v1.29.0","github.com/spf13/afero@v1.9.3","github.com/spf13/cast@v1.5.0","github.com/spf13/cobra@v1.6.1","github.com/spf13/jwalterweatherman@v1.1.0","github.com/spf13/pflag@v1.0.5","github.com/spf13/viper@v1.15.0","github.com/stretchr/testify@v1.8.2","github.com/subosito/gotenv@v1.4.2","github.com/syndtr/goleveldb@v1.0.1-0.20220721030215-126854af5e6d =\u003e github.com/syndtr/goleveldb@v1.0.1-0.20210819022825-2ae1ddf74ef7","github.com/tendermint/go-amino@v0.16.0","github.com/tidwall/btree@v1.6.0","github.com/ulikunitz/xz@v0.5.11","github.com/zondax/hid@v0.9.1","github.com/zondax/ledger-go@v0.14.1","go.opencensus.io@v0.24.0","golang.org/x/crypto@v0.7.0","golang.org/x/exp@v0.0.0-20230224173230-c95f2b4c22f2","golang.org/x/net@v0.8.0","golang.org/x/oauth2@v0.5.0","golang.org/x/sync@v0.1.0","golang.org/x/sys@v0.6.0","golang.org/x/term@v0.6.0","golang.org/x/text@v0.8.0","golang.org/x/xerrors@v0.0.0-20220907171357-04be3eba64a2","google.golang.org/api@v0.110.0","google.golang.org/genproto@v0.0.0-20230216225411-c8e22ba71e44","google.golang.org/grpc@v1.53.0","google.golang.org/protobuf@v1.29.0","gopkg.in/ini.v1@v1.67.0","gopkg.in/yaml.v2@v2.4.0","gopkg.in/yaml.v3@v3.0.1","gotest.tools/v3@v3.4.0","nhooyr.io/websocket@v1.8.6","pgregory.net/rapid@v0.5.5","sigs.k8s.io/yaml@v1.3.0"],"cosmos_sdk_version":"v0.48.0"}}

On the other hand I just noticed I have some chains that have it working (emoney, simd, jackal) and some have it not working (gaia, bitsong, decentr, sentinel). Wonder if it has something to do with cosmos-sdk version, not sure about it.

@julienrbrt julienrbrt self-assigned this Mar 12, 2023
@julienrbrt
Copy link
Member

I think this is fine, given that it works as intended with go install, which is the way we advise to install our tooling (everything under tools/). The fact it shows devel seems to be an issue when using go build (golang/go#29228).

I can add a disclaimer in the docs.

@freak12techno
Copy link
Contributor Author

I can add a disclaimer in the docs.

I'm all in, my concern here is that one of the officially recommended ways of installing it is by building from source, so it's nice to have it noted somewhere that the versioning won't work in that case. Adding a disclaimer would work I guess.

On the other issue I've mentioned here in comments, should I open another issue on this repository or it isn't something to be fixed and should be rather asked to be fixed by the chain maintainers for the chains it doesn't work properly with?

@julienrbrt
Copy link
Member

Right, I've added a disclaimer for that, so it is more clear go install is the way to install our tooling.

On the other issue I've mentioned here in comments, should I open another issue on this repository or it isn't something to be fixed and should be rather asked to be fixed by the chain maintainers for the chains it doesn't work properly with?

Given that it works with (latest) simapp v0.45, (latest) simapp v0.46, (latest) simapp v0.47 and (latest) simapp from main,
this is indeed something for the chain maintainers to verify that they have the following in their persistentPreRunE of their rootCmd:

// set the default command outputs
cmd.SetOut(cmd.OutOrStdout())
cmd.SetErr(cmd.ErrOrStderr())

@julienrbrt julienrbrt added T:Sprint and removed S:needs more info This bug can't be addressed until more information is provided by the reporter. T:Sprint labels Mar 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:Cosmovisor Issues and PR related to Cosmovisor
Projects
None yet
Development

No branches or pull requests

2 participants