Tracking issue surfaced during PR #166 production deploy.
Context
During PR #166's deploy window we attempted to flip the production deploy tool from azd deploy to aspire deploy. The attempt blocked on two real issues:
-
AppHost does not register AddAzureContainerAppEnvironment. src/SentenceStudio.AppHost/AppHost.cs lines 5–9 contain an explicit note that the registration was removed because it changes azd's manifest template format incompatibly with existing azd env vars. Switching to aspire deploy requires uncommenting builder.AddAzureContainerAppEnvironment("aca-env").WithAzdResourceNaming();.
-
aspire deploy prompts for all 7 parameters interactively: openaikey, elevenlabskey, jwtkey, githubpat, dbUser, dbPassword, syncfusionkey. azd stores these in its env state; aspire deploy doesn't read azd state. A non-interactive path (env vars, config file, or keyvault refs) needs to be wired before aspire-deploy is viable.
Work required
Why this is worth doing
aspire deploy is incremental (no reprovision), faster than azd deploy for code-only changes
- Keeps deploy tooling aligned with Aspire's evolving story rather than diverging
Why we didn't bundle it into PR #166
Separating concerns — PR #166 is the server-side App Insights slice and was already dress-rehearsed for that scope. A deploy-tool migration needs its own rehearsal.
Context: PR #166
Tracking issue surfaced during PR #166 production deploy.
Context
During PR #166's deploy window we attempted to flip the production deploy tool from
azd deploytoaspire deploy. The attempt blocked on two real issues:AppHost does not register
AddAzureContainerAppEnvironment.src/SentenceStudio.AppHost/AppHost.cslines 5–9 contain an explicit note that the registration was removed because it changes azd's manifest template format incompatibly with existing azd env vars. Switching toaspire deployrequires uncommentingbuilder.AddAzureContainerAppEnvironment("aca-env").WithAzdResourceNaming();.aspire deployprompts for all 7 parameters interactively:openaikey,elevenlabskey,jwtkey,githubpat,dbUser,dbPassword,syncfusionkey.azdstores these in its env state;aspire deploydoesn't read azd state. A non-interactive path (env vars, config file, or keyvault refs) needs to be wired before aspire-deploy is viable.Work required
AddAzureContainerAppEnvironment("aca-env").WithAzdResourceNaming()inAppHost.csdocs/deploy-runbook.mdto recommendaspire deployand keepazd deploydocumented as fallbackWhy this is worth doing
aspire deployis incremental (no reprovision), faster thanazd deployfor code-only changesWhy we didn't bundle it into PR #166
Separating concerns — PR #166 is the server-side App Insights slice and was already dress-rehearsed for that scope. A deploy-tool migration needs its own rehearsal.
Context: PR #166