Skip to content

feat(gallery): add foundry-sql-containerapp scenario#107

Draft
petender wants to merge 1 commit into
MicrosoftLearning:mainfrom
petender:contribute/foundry-sql-containerapp
Draft

feat(gallery): add foundry-sql-containerapp scenario#107
petender wants to merge 1 commit into
MicrosoftLearning:mainfrom
petender:contribute/foundry-sql-containerapp

Conversation

@petender

Copy link
Copy Markdown
Collaborator

New Template Registration

Field Value
Scenario foundry-sql-containerapp
Repo petender/tdd-azd-foundry-sql-containerapp
Tags ai-102,msft,aifoundry,aca,acr,azuresql,azurestorage,loganalytics,dotnet
Sample App Yes
Demo Guide Yes
Contributor @petender

Tag Validation

All tags validated against src/data/tags.tsx TagType union: ✅ PASSED

Deployment

gh repo clone petender/tdd-azd-foundry-sql-containerapp cd tdd-azd-foundry-sql-containerapp azd init azd up

@rob-foulkrod

Copy link
Copy Markdown
Collaborator

Validation is progressing, but this template is still not reaching a successful azd up.

Two SQL issues appear to be resolved in the latest local validation:

  1. Removed the unused SQL admin login/password path. The prior generated password was not guaranteed to satisfy Azure SQL complexity and caused:
PasswordNotComplex: Password validation failed. The password does not meet policy requirements because it is not complex enough.
  1. Added an explicit Basic-tier database size of 2 GB:
maxSizeBytes: 2147483648

This addressed:

InvalidMaxSizeTierCombination: The tier 'Basic' does not support the database max size '34359738368'.

After those fixes, validation got further. The following resources provisioned successfully:

  • Log Analytics workspace
  • Application Insights
  • Container Registry
  • Storage account
  • AI Foundry
  • AI model deployment
  • Foundry project
  • Azure SQL Server
  • Container Apps Environment

The current blocker is now the Container App revision:

ContainerAppOperationError: Failed to provision revision for container app 'ca-fsc-p1072250'. Error details: Operation expired.

So the SQL path looks improved, but the template still needs investigation around the Container App revision/image/startup/configuration before it can be approved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants