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

sapphire-localnet: -amount broken #585

Open
matevz opened this issue Jul 1, 2024 · 3 comments
Open

sapphire-localnet: -amount broken #585

matevz opened this issue Jul 1, 2024 · 3 comments
Labels
bug Something isn't working docker Docker related issues

Comments

@matevz
Copy link
Member

matevz commented Jul 1, 2024

SUMMARY

Reported by @csillag:

oa@matevz-oa:~/playground$ docker run -it -p8545:8545 -p8546:8546 -e OASIS_NODE_LOG_LEVEL=debug -e LOG__LEVEL=debug ghcr.io/oasisprotocol/sapphire-localnet -amount 10 -test-mnemonic
sapphire-localnet 2024-06-17-gitb807232 (oasis-core: 24.0-gitfb49717, sapphire-paratime: 0.7.3-testnet, oasis-web3-gateway: 5.1.0-gitb807232)

 * Starting oasis-net-runner with sapphire...
 * Waiting for Postgres to start.....
 * Waiting for Oasis node to start...
 * Starting oasis-web3-gateway...
 * Bootstrapping network (this might take a minute).
 * Waiting for nodes to be ready....
 * Setting epoch to 1...
 * Waiting for key manager to generate ephemeral secret....
 * Setting epoch to 2...
 * Waiting for key manager......
 * Populating accounts...

panic: can't deposit: client: transaction check failed: runtime error: module: consensus code: 5 message: consensus: amount not representable

goroutine 1 [running]:
main.main()
        /go/oasis-web3-gateway/docker/common/oasis-deposit/main.go:273 +0x1005

I think we need to bump the oasis-sdk Go bindings in the deposit helper.

@matevz matevz added bug Something isn't working docker Docker related issues labels Jul 1, 2024
@kostko
Copy link
Member

kostko commented Jul 1, 2024

Is the amount passed as an argument in base units and 10 is less than 1 consensus layer base unit so you cannot deposit such a small amount? Try increasing the amount ;)

@CedarMist
Copy link
Member

So the intuition is people expect to pass whole units of ROSE to amount but it's actually base units...

@matevz
Copy link
Member Author

matevz commented Sep 25, 2024

Should be implicitly fixed by #617.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working docker Docker related issues
Projects
None yet
Development

No branches or pull requests

3 participants