-
Notifications
You must be signed in to change notification settings - Fork 334
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
[Workflow] Fix issue with ignored external event payload #1119
Conversation
Signed-off-by: Chris Gillum <cgillum@microsoft.com>
Signed-off-by: Chris Gillum <cgillum@microsoft.com>
Codecov Report
@@ Coverage Diff @@
## master #1119 +/- ##
=======================================
Coverage 67.22% 67.22%
=======================================
Files 170 170
Lines 5687 5687
Branches 605 605
=======================================
Hits 3823 3823
Misses 1723 1723
Partials 141 141
Flags with carried forward coverage won't be shown. Click here to find out more. |
@cgillum - You have test failures that I think are related to legacy steps in the itests workflow. Can you remove these steps and repush to see if that fixes things? https://github.com/dapr/dotnet-sdk/blob/master/.github/workflows/itests.yml#L108-L116 |
Signed-off-by: Chris Gillum <cgillum@microsoft.com>
* [Workflow] Fix issue with ignored external event payload Signed-off-by: Chris Gillum <cgillum@microsoft.com> * Pushing missing commits Signed-off-by: Chris Gillum <cgillum@microsoft.com> * Remove unnecessary steps from itests.yml Signed-off-by: Chris Gillum <cgillum@microsoft.com> --------- Signed-off-by: Chris Gillum <cgillum@microsoft.com> Signed-off-by: Bradley Cotier <bcotier@microsoft.com>
* [Workflow] Fix issue with ignored external event payload Signed-off-by: Chris Gillum <cgillum@microsoft.com> * Pushing missing commits Signed-off-by: Chris Gillum <cgillum@microsoft.com> * Remove unnecessary steps from itests.yml Signed-off-by: Chris Gillum <cgillum@microsoft.com> --------- Signed-off-by: Chris Gillum <cgillum@microsoft.com> Signed-off-by: Bradley Cotier <bcotier@microsoft.com>
* [Workflow] Fix issue with ignored external event payload Signed-off-by: Chris Gillum <cgillum@microsoft.com> * Pushing missing commits Signed-off-by: Chris Gillum <cgillum@microsoft.com> * Remove unnecessary steps from itests.yml Signed-off-by: Chris Gillum <cgillum@microsoft.com> --------- Signed-off-by: Chris Gillum <cgillum@microsoft.com> Signed-off-by: Bradley Cotier <bcotier@microsoft.com>
* [Workflow] Fix issue with ignored external event payload Signed-off-by: Chris Gillum <cgillum@microsoft.com> * Pushing missing commits Signed-off-by: Chris Gillum <cgillum@microsoft.com> * Remove unnecessary steps from itests.yml Signed-off-by: Chris Gillum <cgillum@microsoft.com> --------- Signed-off-by: Chris Gillum <cgillum@microsoft.com> Signed-off-by: Artur Souza <asouza.pro@gmail.com>
Description
In dapr/dapr#6614, a user observed that the
RaiseEventAsync
method was always resulting in a null payload in the workflow code. This PR fixes the issue, which is isolated to the Dapr Workflow SDK.I also made a couple other small changes as part of this PR:
getInputsAndOutputs
totrue
for the workflow status query APIs (I decided to do this to improve usability after some frustration while setting up the local repro for the raise event issue)I also updated the Workflow SDK nuget version to
0.3.0
.Issue reference
Closes dapr/dapr#6614
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: