man-group / dapr-sidekick-dotnet

Dapr Sidekick for .NET - a lightweight lifetime management component for Dapr
Apache License 2.0
175 stars 21 forks source link

client.CheckHealthAsync() return false #50

Closed BartNetJS closed 1 year ago

BartNetJS commented 1 year ago

When execute client.CheckHealthAsync() it return false when running with dapr sidekick When i run it with dapr run ... it returns true.

Also the same with WaitForSidecarAsync, it works with dapr run, but not with the dapr sidekick.

Any idea why? Is it because daprd is not discovered? The same issue with dashboard that is able to discover dapr services started with daprd?

badgeratu commented 1 year ago

Apologies for being late looking at this. I've just tried Dapr Sidekick with the latest Dapr runtime 1.10,2 and it does look to me CheckHealthAsync() is working. To test this I started with the standard Service Invocation Sample and made the following changes:

As for this item:

The same issue with dashboard that is able to discover dapr services started with daprd? That is due to the dashboard looking for Dapr CLI processes (i.e. dapr.exe) which is typically used to launch daprd. As Sidekick does not use the CLI, its managed daprd instances do not show up in the dashboard.

BartNetJS commented 1 year ago

I figured it out when working on the sample for this question https://github.com/man-group/dapr-sidekick-dotnet/issues/51 Thanks!

badgeratu commented 1 year ago

Closing as completed