Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

Revisit wasmd setup doc #417

Open
alpe opened this issue Aug 25, 2022 · 4 comments
Open

Revisit wasmd setup doc #417

alpe opened this issue Aug 25, 2022 · 4 comments

Comments

@alpe
Copy link
Contributor

alpe commented Aug 25, 2022

People report issues with the env setup from the docs on the cosmwasm discord or github.
The whole "Setting up ..." should be revisited to make it work on all machines.

The "client config" module was added back to the SDK and wasmd long time ago. This may provide a more resilient setup config and can avoid the unsafe source <internet. Please see the wasmd config command

See https://stackoverflow.com/questions/72051270/cosmwasm-environment-setup-error-first-path-segment-in-url-cannot-contain-colo/73487969#73487969

Tag: Error: couldn't get client from nodeURI: parse

@findolor
Copy link
Contributor

@alpe wasmd config does not have other flags defined. We still need to get other transaction related (gas-prices, gas, gas-adjustment, ...) flags for sending out TXs.

We can configure chain-id and node but cannot change these flags. Our best option is to pull it from a repo.

@findolor
Copy link
Contributor

This is also made for zsh, that's why some people using bash are having problems with NODE and TXFLAGS. They still need to export them after sourcing.

@orkunkl Maybe It might be worth to have two different env files for zsh and bash?

@MightOfOaks
Copy link
Contributor

@alpe - Would it be an option to add news keys to the client config in Cosmos SDK for the remaining flags?

@alpe
Copy link
Contributor Author

alpe commented Sep 2, 2022

Would it be an option to add news keys to the client config in Cosmos SDK for the remaining flags?

The client config comes from the SDK. Best would be to make a PR to their repo to add the values you need.
When introduced proper, I think the gas params to calculate the fees don't need not be hidden: --gas-prices 0.25umlg --gas auto --gas-adjustment 1.3. This is what people would need to setup anyway when working with other chains.
But this is just my opinion.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants