-
Notifications
You must be signed in to change notification settings - Fork 198
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
Support DataProtection Runtime feature in ACA #3731
Conversation
This change does two things: - Supports using a custom api-version of the ACA CreateOrUpdate request by allowing the yaml deployment template to have an `api-version` property at top level. When set, this value is used as the `api-version` of the request. The request body is the object, rendered as JSON, with the `api-version` key removed. - Uses the above new feature to opt into the `2024-02-02-preview` in Aspire and sets the `configuration.runtime.dotnet.autoConfigureDataProtection` property to `true`, but only when the alpha feature `aspire.autoConfigureDataProtection` is turned on. We are gating turning this feature on behind a feature flag the user must opt-into instead of always sending this request, as we had done in #3711, because currently regions in Azure are rejecting the `2024-02-02-preview` requests (even though the error message implies this should work). This allows partners to test the end to end by running: `azd config set alpha.aspire.autoConfigureDataProtection on` And testing in a supported region, like `centraluseuap` (which can be explicitly selected via `azd env set AZURE_LOCATION centraluseuap`. Fixes #3538
e8338e1
to
4b15134
Compare
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash:
pwsh:
WindowsPowerShell install
MSI install
Standalone Binary
MSIDocumentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for lighting this change up again behind a gate given the regional deployment timing.
This change does two things:
Supports using a custom api-version of the ACA CreateOrUpdate request by allowing the yaml deployment template to have an
api-version
property at top level. When set, this value is used as theapi-version
of the request. The request body is the object, rendered as JSON, with theapi-version
key removed.Uses the above new feature to opt into the
2024-02-02-preview
in Aspire and sets theconfiguration.runtime.dotnet.autoConfigureDataProtection
property totrue
, but only when the alpha featureaspire.autoConfigureDataProtection
is turned on.We are gating turning this feature on behind a feature flag the user must opt-into instead of always sending this request, as we had done in #3711, because currently regions in Azure are rejecting the
2024-02-02-preview
requests (even though the error message implies this should work).This allows partners to test the end to end by running:
azd config set alpha.aspire.autoConfigureDataProtection on
And testing in a supported region, like
centraluseuap
(which can be explicitly selected viaazd env set AZURE_LOCATION centraluseuap
.Fixes #3538