Skip to content
This repository has been archived by the owner on Nov 20, 2023. It is now read-only.

Removes Dapr placement server management #1191

Merged
merged 5 commits into from
Oct 5, 2021

Conversation

philliphoff
Copy link
Contributor

@philliphoff philliphoff commented Oct 1, 2021

Gets Tye out of the business of creating Dapr placement services, as most applications can use the one provided by the standard Dapr initialization (e.g. via dapr init) and that's what dapr assumes when running sidecars unless otherwise directed. (If no placement service was running, just spinning one up as Tye was previously doing isn't sufficient due to dependencies on a properly configured state store.) Maintains the ability to have Dapr sidecars use a different port via placement-port. The Dapr extension will check whether Dapr has been initialized (e.g. via dapr --version) and fails if not, as a further hint to the user that something is amiss.

NOTE: The placement service is required only for applications using Dapr actors. Applications using other Dapr features need not have it running.

Resolves #1175
Resolves #1177

@philliphoff philliphoff merged commit 565b7d3 into main Oct 5, 2021
@philliphoff philliphoff deleted the philliphoff-no-implicit-placement branch October 5, 2021 21:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tye tries to docker pull dapr/dapr:latest Tye seems to start an adjacent Dapr placement service
2 participants