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

ag-cosmos-helper status should write to stdout, not stderr #2888

Closed
warner opened this issue Apr 15, 2021 · 3 comments
Closed

ag-cosmos-helper status should write to stdout, not stderr #2888

warner opened this issue Apr 15, 2021 · 3 comments
Assignees
Labels
bug Something isn't working cosmic-swingset package: cosmic-swingset

Comments

@warner
Copy link
Member

warner commented Apr 15, 2021

Describe the bug

I used ag-cosmos-helper status for the first time today. I was surprised that the machine-parseable JSON output is written to stderr, rather than stdout. It's a drag to require a redirect to parse the output.

Instead of:

% ag-cosmos-helper status 2>&1 | jq .SyncInfo

I'd prefer to run:

% ag-cosmos-helper status | jq .SyncInfo

If we can fix this, we could also update the guides at https://github.com/Agoric/agoric-sdk/wiki/Validator-Guide-for-Devnet#syncing-your-node and https://github.com/Agoric/agoric-sdk/wiki/Validator-Guide-for-Incentivized-Testnet#syncing-your-node to remove the 2>&1 bit.

@warner warner added bug Something isn't working cosmic-swingset package: cosmic-swingset labels Apr 15, 2021
@michaelfig
Copy link
Member

We're working on it in cosmos/cosmos-sdk#8628 which should be backported to cosmos-sdk v0.42.x.

I don't know if there's value added by keeping the current issue open. Tracking, I guess.

@michaelfig
Copy link
Member

Solved in cosmos-sdk v0.43.0-alpha1. We will be adopting the 0.43.0 release sooner or later.

@michaelfig
Copy link
Member

Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cosmic-swingset package: cosmic-swingset
Projects
None yet
Development

No branches or pull requests

2 participants