-
Notifications
You must be signed in to change notification settings - Fork 99
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
Remove Dapr installation from Radius installation #2952
Comments
If we did take on Dapr as a dependency in Radius it needs to be firewalled from Dapr used by a user for applications (an implementation detail of Radius). So, I don't think we should factor whether Radius uses Dapr into our decision. Dapr should be an optional component from the user's POV. If we were take responsibility for installing/managing Dapr for production use that's a huge deal 😆
Yes 100%. To treat Dapr as an optional component we need to design how users describe it to the environment, and how we implement guardrails for the user. |
Part of: #2952 This change validates the installation status of Dapr inside links RP for Dapr resources. This is step one in removing Dapr from Radius' default installation. After we have a good experience with Dapr missing, we can remove it. This validation is only applied on PUT calls to be more friendly to API consumers. We still allow clients to get and list dapr link resources without Dapr installed. The pattern I followed is similar to the validation we perform for the AKV CSI driver.
Part of: #2952 This change validates the installation status of Dapr inside links RP for Dapr resources. This is step one in removing Dapr from Radius' default installation. After we have a good experience with Dapr missing, we can remove it. This validation is only applied on PUT calls to be more friendly to API consumers. We still allow clients to get and list dapr link resources without Dapr installed. The pattern I followed is similar to the validation we perform for the AKV CSI driver.
As part of investigation into how to lower our Radius installation time #2582
we saw that Radius install time could be significantly reduced by not including Dapr installation.
At this time, we don't use Dapr internally as part of Radius, but we're considering it for a longer term possible change.
I think we'd need to do a little investigation into what it means to remove Dapr and what we'd need to add at that time to keep a great user experience.
Right now, Dapr is commonly used with Radius. We lean into teaching people that Dapr + Radius is a "better together" way of achieving portability. So Dapr installation is one of the common environment dependencies that Radius users are going to want.
When we stop installing Dapr by default, we should probably have the guardrails in place for validating whether environment dependencies are installed: dependency validation work described in #2951
AB#4448
The text was updated successfully, but these errors were encountered: