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

bug: cosmoshub does not support faucet type: starship #482

Closed
0xpatrickdev opened this issue Jun 13, 2024 · 3 comments · Fixed by #484
Closed

bug: cosmoshub does not support faucet type: starship #482

0xpatrickdev opened this issue Jun 13, 2024 · 3 comments · Fixed by #484

Comments

@0xpatrickdev
Copy link
Contributor

0xpatrickdev commented Jun 13, 2024

When using the following settings with the 0.2.2 helm chart:

chains:
-  id: cosmoshub-test
    name: cosmoshub
    faucet:
      enabled: true
      type: starship

The faucet container in the cosmoshub-test pod fails to start:

{"level":"error","ts":1718310048.524163,"caller":"app/distributor.go:286","msg":"send token failed","cmd":"gaiad tx bank send cosmos1vhdew4wqu3tp8l2d55aqcc73aqvr0rr9yk4a0l cosmos1prdhkqcdzz2dqp3qwpmecna6e7faghkkqk2p9p 80000000000stake --chain-id=gaialocal --fees=10000000uatom --keyring-backend=test --gas=auto --gas-adjustment=1.5 --yes --node=http://localhost:26657","error":"err: exit status 1, stderr: Error: rpc error: code = Unknown desc = rpc error: code = Unknown desc = failed to execute message; message index: 0: 0stake is smaller than 80000000000stake: insufficient funds


2024-06-13 16:15:36 err: exit status 1, stderr: Error: rpc error: code = Unknown desc = rpc error: code = Unknown desc = failed to execute message; message index: 0: 0stake is smaller than 80000000000stake: insufficient funds [cosmos/cosmos-sdk@v0.45.16/x/bank/keeper/send.go:186] With gas wanted: '0' and gas used: '60680' : unknown request

The @cosmjs/faucet faucet works, but does not support sending tokens to ICA addresses. See this pull request for an upstream fix: cosmos/cosmjs#1593

@Anmol1696
Copy link
Collaborator

let me have a look

@Anmol1696
Copy link
Collaborator

As for cosmjs we can wait for the upstream PR to be merged, then we can create a new version of the faucet based on the new version of cosmjs

@Anmol1696
Copy link
Collaborator

@0xpatrickdev thanks for opening this issue. Fixed the issue in chart version 0.2.4. Please upgrade and let me know if it works.

More over, will upgrade the cosmjs version as well, once your PR is merged.

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

Successfully merging a pull request may close this issue.

2 participants