Skip to content

Commit

Permalink
update TODOs
Browse files Browse the repository at this point in the history
  • Loading branch information
ckittel authored Oct 29, 2024
1 parent f55ffc0 commit 0886df7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -192,6 +192,8 @@ The following steps are required to deploy the infrastructure from the command l

1. Apply workaround for Azure AI Studio not deploying its managed network.

TODO (P1 Jon): Open issue with PG and track to resolution.

Azure AI Studio tends to delay deploying its managed network, which causes problems when trying to access Azure AI Studio's portal experience in the next step. Your final IaC implementation must account for this.
:clock8: *This might take about 15 minutes.*
Expand Down Expand Up @@ -236,7 +238,7 @@ To test this scenario, you'll be deploying a pre-built prompt flow. The prompt f

If this happens, simply choose a new folder name and click the **Clone** button again. You'll need to remember the new folder name to adjust the instructions later.
TODO (P1 Jon): The UI produces an error when you do this for the first time. But if you do again (with a different directory name), it usally works just fine (sometimes it takes a few additional tries). What is going on?
TODO (P1 Jon): Open issue with PG and track to resolution.
1. Connect the `extract_query_from_question` prompt flow step to your Azure OpenAI model deployment.
Expand Down
2 changes: 1 addition & 1 deletion infra-as-code/bicep/webapp.bicep
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,7 @@ resource appServicePlan 'Microsoft.Web/serverfarms@2023-12-01' = {
capacity: 3
}
properties: {
zoneRedundant: false // TODO (P2 - Set back at end): My subscription doesn't have enough quota to set this to 'true', but before we ship this must go back to true.
zoneRedundant: false // TODO (P1 - CK): My subscription doesn't have enough quota to set this to 'true', but before we ship this must go back to true.
reserved: true
}
}
Expand Down

0 comments on commit 0886df7

Please sign in to comment.