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

Production guideline is missing information about using memory over storage when running Dapr in HA mode with actors #4329

Open
KrylixZA opened this issue Aug 28, 2024 · 3 comments
Labels
content/incorrect-information Content in the docs is incorrect

Comments

@KrylixZA
Copy link
Contributor

Describe the issue

The documentation which provides Dapr users with production guidelines for installing Dapr fails to mention any information about the fact that there can be issues when running Dapr in HA mode with actors. Specifically, there needs to be information about setting the following configuration for the placement service when running in HA mode:

forceInMemory: true

URL of the docs

https://docs.dapr.io/operations/hosting/kubernetes/kubernetes-production#high-availability-mode

Expected content

As mentioned above, there should be a paragraph under the HA mode header that states that there are known issues when sharing a PVC for storage of the actor addresses as after a long time with a large number of pod/node restarts, the state can become an issue for the placement service. Thus we need two sections of documentation:

  1. For new installations, users should set forceInMemory: true before installing.
  2. For those who have already installed Dapr with HA mode, they will need to first delete the stateful set and then run a Dapr upgrade with the forceInMemory: true set to true.

Screenshots

N/A

Additional context

Discord thread: https://discord.com/channels/778680217417809931/1277561831988265021

@KrylixZA KrylixZA added the content/incorrect-information Content in the docs is incorrect label Aug 28, 2024
@elena-kolevska
Copy link
Contributor

elena-kolevska commented Aug 28, 2024

This was indeed a recommendation a while ago and it wasn't documented, but this issue was addressed in 1.13 and 1.14. I did a few benchmarks and adjustments of the default values in this pr: dapr/dapr#7837

@KrylixZA
Copy link
Contributor Author

Great! If the community is happy this is no longer a recommendation, then please feel free to close this issue based on your experience and judgement.

@elena-kolevska
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content/incorrect-information Content in the docs is incorrect
Projects
None yet
Development

No branches or pull requests

2 participants