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.

Sign up for GitHub

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

Jump to bottom

[BUG] interfaces cause scenario service to crash #12475

Closed
lucianojoublanc-da opened this issue Jan 19, 2022 · 1 comment
Closed

[BUG] interfaces cause scenario service to crash #12475

lucianojoublanc-da opened this issue Jan 19, 2022 · 1 comment
Labels
language Language team work
Milestone
Daml Interfaces

Comments

@lucianojoublanc-da
Copy link

sdk-version: 2.0.0-snapshot.20220110.8812.0.3a08380b

Running this test:

https://github.com/lucianojoublanc-da/lib-finance-next/blob/cfc6ccb299745496407ce27c86b18409738555f3/daml/Daml/Test.daml

results in

2022-01-19 09:55:16.50 [ERROR]  [test ScenarioService] 
SCENARIO SERVICE STDERR: com.daml.lf.engine.script.ScriptF$FailedCmd: Command submit failed: null

2022-01-19 09:55:16.50 [ERROR]  [test ScenarioService] 
SCENARIO SERVICE STDERR: Daml stacktrace:

2022-01-19 09:55:16.50 [ERROR]  [test ScenarioService] 
SCENARIO SERVICE STDERR: submit at -homePackageId-:Daml.Test:17File:     daml/Daml/Test.daml
Hidden:   no
Range:    10:1-10:5
Source:   Script
Severity: DsError
Message: 
  "Scenario service backend error: BErrorClient (ClientIOError (GRPCIOBadStatusCode StatusUnknown
  (StatusDetails {unStatusDetails = \"\"})))"

Please request access, it's a private repo at the moment.

The one naughty thing I think may cause this is the implementation of an interface's Eq and Show instances. Otherwise I don't really have anything to go on, given the error message.
@cocreature cocreature added the language Language team work label Jan 19, 2022
@cocreature cocreature added this to the Daml Interfaces milestone Jan 19, 2022
@cocreature
Copy link
Contributor

Your model is unfortunately broken: You are trying to use interface payloads as serializable values in choice arguments which doesn’t work. You hacked around it by manually defining Eq and Show instances which worked because we messed up our checks. I added them in #12483 which “fixes” this issue by making your code not compile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
language Language team work
Projects
None yet
Milestone
Daml Interfaces
Development

No branches or pull requests
2 participants
@cocreature @lucianojoublanc-da