dapr / dotnet-sdk

Dapr SDK for .NET
Apache License 2.0
1.12k stars 340 forks source link

Workflow status displays as `Unknown` after being paused using #1215

Closed MregXN closed 2 months ago

MregXN commented 11 months ago

Expected Behavior

workflow status should be Suspended

Actual Behavior

displays as Unknown

Steps to Reproduce the Problem

use SuspendWorkflowAsync in DaprWorkflowClient to pause a workflow instance


...
await workflowClient.SuspendWorkflowAsync(instanceId);
var getResponse = await workflowClient.GetWorkflowStateAsync(instanceId);
Console.WriteLine(Enum.GetName(typeof(WorkflowRuntimeStatus), getResponse.RuntimeStatus));
...
WhitWaldo commented 2 months ago

Has any testing been done with the other language SDKs to confirm that this is an issue with the .NET SDK itself and not an issue with the sidecar implementation?

mikeee commented 2 months ago

Has any testing been done with the other language SDKs to confirm that this is an issue with the .NET SDK itself and not an issue with the sidecar implementation?

The durabletask-go implementation and downstream dapr go-sdk is working as expected [displays the code - SUSPENDED] (although missing tests)

cgillum commented 2 months ago

Looks like there's a switch case missing for Suspended in the code here. Should be an easy fix.