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

Switch to dapr list to obtain running Dapr applications #264

Closed
philliphoff opened this issue Jan 19, 2023 · 0 comments · Fixed by #265
Closed

Switch to dapr list to obtain running Dapr applications #264

philliphoff opened this issue Jan 19, 2023 · 0 comments · Fixed by #265
Labels
enhancement Update to existing feature

Comments

@philliphoff
Copy link
Collaborator

Currently the extension periodically polls all processes to find running Dapr applications (filtering for those. Originally this was necessary because the extension started daprd processes itself (as the only way then to start the sidecar independent of the application being debugged)and dapr list didn't yet exist or, even when it did, would only report processes started via dapr. All of this is no longer necessary; the extension now (by default) uses dapr to start the sidecar and dapr list also is better at listing independent daprd instances.

The extension should now use dapr list to obtain running Dapr applications and dispense with process polling (and it's quirks across platforms).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Update to existing feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant