Skip to content

Conversation

acroca
Copy link
Member

@acroca acroca commented Sep 23, 2025

Adds support for multi-app workflow invocations, both child workflows and activities, just providing app_id in the call

The multi-app documentation is here: https://docs.dapr.io/developing-applications/building-blocks/workflow/workflow-multi-app/, and at the moment is fully supported in the Go SDK, and partially supported in the Java SDK.

This PR will make the multi-app functionality fully supported in the python SDK.

@acroca acroca requested review from a team as code owners September 23, 2025 07:46
@acroca acroca force-pushed the cross-app branch 2 times, most recently from 9e5e28e to f1e653d Compare September 23, 2025 08:01
Signed-off-by: Albert Callarisa <albert@diagrid.io>
Copy link

codecov bot commented Sep 23, 2025

Codecov Report

❌ Patch coverage is 31.57895% with 13 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (release-1.16@99314a4). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...orkflow/dapr/ext/workflow/dapr_workflow_context.py 23.52% 13 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##             release-1.16     #844   +/-   ##
===============================================
  Coverage                ?   87.17%           
===============================================
  Files                   ?       93           
  Lines                   ?     6159           
  Branches                ?        0           
===============================================
  Hits                    ?     5369           
  Misses                  ?      790           
  Partials                ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Albert Callarisa <albert@diagrid.io>
Copy link
Contributor

@elena-kolevska elena-kolevska left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but can you also pls add an example for non-green paths?

@acroca
Copy link
Member Author

acroca commented Oct 2, 2025

LGTM, but can you also pls add an example for non-green paths?

I've added a couple of examples showing how it handles errors coming from both workflow and activities.

Signed-off-by: Albert Callarisa <albert@diagrid.io>
Signed-off-by: Albert Callarisa <albert@diagrid.io>
@elena-kolevska elena-kolevska merged commit d005a15 into dapr:release-1.16 Oct 7, 2025
19 of 20 checks passed
@acroca acroca deleted the cross-app branch October 7, 2025 11:52
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 this pull request may close these issues.

2 participants