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

deploy fails without error message #3375

Open
jimscarver opened this issue Apr 8, 2021 · 9 comments
Open

deploy fails without error message #3375

jimscarver opened this issue Apr 8, 2021 · 9 comments
Labels

Comments

@jimscarver
Copy link
Collaborator

What did you expect to see?

output to deployId - counts, {"red": 2, ...

What did you see instead?

Deploy error when executing Rholang code.

Steps to reproduce the bug

create an issue, adding a voter, tallying a vote having at least one voter.
See https://docs.google.com/document/d/1V5k9tDjztEVDYiXJG1-Hpb0rNtVcV22GrsdXb0pDUx8/edit#

What version of RNode are you using?

0.10.0/1/2

What is your environment?

https://github.com/rchain-community/liquid-democracy/blob/reorganize-actions/src/actions/tallyVotes.rho
shows correct execution in the log but fails returning a value at deployId but the test when defining tally works fine. https://github.com/rchain-community/liquid-democracy/blob/reorganize-actions/Issue.rho
http://rhobot.net:40403/api/block/d4407236ee4408c4584404c22e0e6bd4d7c6f20dd90a910b5d6d30096703d7f8
see rchain-community/rchain-api#86

@dckc
Copy link
Collaborator

dckc commented Apr 10, 2021

@jimscarver did you say a couple other people reproduced this bug? Who were they?

@jimscarver
Copy link
Collaborator Author

@w2vy and @Bill-Keuntje confirmed the bug

@jimscarver
Copy link
Collaborator Author

Any vote causes the choice and tally to fail independently but in this case adding immediate tally/choice does not cause the error. https://gist.github.com/jimscarver/8b5a7733592855a117ded183d8d55e0b

https://github.com/rchain-community/liquid-democracy/blob/reorganize-actions/Issue.rho also runs without an error.

@Bill-Kunj
Copy link

The message appeared because of the code
log!((name ++ "delegates to" ++ delegate))
Where 'name' is a URI. But, the '++' operator is not defined for URI. The interpreter silently (no log message) indicates an error which gets reported to the API, which in turn produces "Deploy error when executing Rholang code".

@dckc
Copy link
Collaborator

dckc commented Apr 13, 2021

Thanks for the update, @Bill-Keuntje

@jimscarver
Copy link
Collaborator Author

using version 0.10.2 (4220ea8) there is no error message if you misspell a function name. The code does nothing and if you do not listen on deployId you get no indication at all that there was an error.
image

A whole section of code was silently not being executed because I misspelled contain. The deploy and propose succeeds without any indication of an error.

@w2vy
Copy link

w2vy commented May 1, 2021 via email

@tgrospic
Copy link
Collaborator

Closing because it seems this is not an issue with RNode. Please reopen if you think otherwise.

@tgrospic tgrospic added question and removed bug labels Jun 28, 2021
@dckc
Copy link
Collaborator

dckc commented Jun 28, 2021

Collapsing all method call errors to a single "Deploy error" diagnostic sure seems like an issue in rnode to me. It makes the dev UX pretty horrible.

log!((name ++ "delegates to" ++ delegate))
Where 'name' is a URI. But, the '++' operator is not defined for URI. The interpreter silently (no log message) indicates an error which gets reported to the API, which in turn produces "Deploy error when executing Rholang code".

@dckc dckc reopened this Jun 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants