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] The DSL example seems incorrect #345

Open
tdeebswihart opened this issue Feb 23, 2024 · 0 comments
Open

[Bug] The DSL example seems incorrect #345

tdeebswihart opened this issue Feb 23, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@tdeebswihart
Copy link
Contributor

tdeebswihart commented Feb 23, 2024

What are you really trying to do?

Run our DSL example and verify that it successfully executes the specified sequence of activities.

Describe the bug

There are a few problems with it

  1. Our docs are wrong and mention workflow outputs that don't exist (we return nothing from our workflows)
  2. It doesn't fully execute the DLS specified by the workflowN.yaml file

workflow1.yaml specifies a sequence of three activities that should be executed but we only execute the first two (see reproduction below). The docs also show this: they contain an example output that only shows the first two activities being executed.

workflow2.yaml exhibits the same problem: it runs two sequences in parallel and the final activity of each sequence is not executed. You'll see neither activity3 nor activity5 execute.

Minimal Reproduction

Run workflow1 of the DSL workflow:

# Run worker
$ npm start &
$ npm run workflow1

Worker logs:

<SNIP>
2024-02-23T22:40:13.586Z [INFO] Worker state changed { state: 'RUNNING' }
2024-02-23T22:42:09.274Z [INFO] Executing activity1 {
  isLocal: false,
  attempt: 1,
  namespace: 'default',
  taskToken: 'CiRhM2MyMzFjZi1lZTk5LTQyNTUtOWE4Mi0zMzU5ZDBkOTMxYTESCW15LWRzbC1pZBokMGNlOTE0MjgtNGQwZS00MDA4LTlmNzItZDEwYWI4NWVmYzAyIAUoATIBMUIJYWN0aXZpdHkxSgkIAhDniYABGAE=',
  workflowId: 'my-dsl-id',
  workflowRunId: '0ce91428-4d0e-4008-9f72-d10ab85efc02',
  workflowType: 'DSLInterpreter',
  activityId: '1',
  activityType: 'activity1',
  taskQueue: 'dsl-interpreter',
  arg1: 'value1'
}
2024-02-23T22:42:09.304Z [INFO] Executing activity2 {
  isLocal: false,
  attempt: 1,
  namespace: 'default',
  taskToken: 'CiRhM2MyMzFjZi1lZTk5LTQyNTUtOWE4Mi0zMzU5ZDBkOTMxYTESCW15LWRzbC1pZBokMGNlOTE0MjgtNGQwZS00MDA4LTlmNzItZDEwYWI4NWVmYzAyIAsoATIBMkIJYWN0aXZpdHkySgkIAhD3iYABGAE=',
  workflowId: 'my-dsl-id',
  workflowRunId: '0ce91428-4d0e-4008-9f72-d10ab85efc02',
  workflowType: 'DSLInterpreter',
  activityId: '2',
  activityType: 'activity2',
  taskQueue: 'dsl-interpreter',
  arg: '[result from activity1: value1]'
}

Workflow output:

undefined

Environment/Versions

  • OS and processor: M1 Mac
  • Temporal Version: Server version 1.22.5, SDK version 1.9.1
  • Are you using Docker or Kubernetes or building Temporal from source? I'm building temporal from source
@tdeebswihart tdeebswihart added the bug Something isn't working label Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant