Skip to content

Commit f9de3df

Browse files
committed
documentation polished
1 parent 91afc98 commit f9de3df

3 files changed

Lines changed: 3 additions & 11 deletions

File tree

docs/aca/README-ACA.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -174,7 +174,7 @@ azd auth login
174174
azd env set AZURE_OPENAI_CHATGPT_MODEL "gpt-4" # Change the chatgpt model to gpt-4
175175
azd up
176176
```
177-
> NOTE: This sample has been tested with gpt-35-turbo/0613 model version.Using other models or different versions might result in unexpected behavior.
177+
> NOTE: This sample has been tested with gpt-4o-mini.Using other models or different versions might result in unexpected behavior.
178178
### Examples of an azd deployment reusing an existing Azure OpenAI and Azure AI Search resources
179179
180180
```shell

docs/aks/README-AKS.md

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -168,7 +168,7 @@ azd env set AZURE_OPENAI_CHATGPT_MODEL "gpt-4" # Change the chatgpt model to gpt
168168
azd up
169169
```
170170

171-
> NOTE: This sample has been tested with gpt-35-turbo/0613 model version.Using other models or different versions might result in unexpected behavior.
171+
> NOTE: This sample has been tested with gpt-4o-mini.Using other models or different versions might result in unexpected behavior.
172172
173173
### Examples of an azd deployment reusing an existing Azure OpenAI and Azure AI Search resources
174174

@@ -387,11 +387,3 @@ Here are the most common failure scenarios and solutions:
387387
7. After running `./app/start.ps1` or `./app/start.sh` you get `"Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.10.1:compile (default-compile) on project myproject: Fatal error compiling: invalid target release: 17"`. It means you are not using JDK 17 but a previous version. Be sure to set the `JAVA_HOME` env variable to your Java 17 installation directory and update your `PATH` env variable to have the Java 17 bin folder as the first occurrence amongst the listed directories. More info [here](https://learn.microsoft.com/en-us/java/openjdk/install)
388388
8. After running `./app/start.sh` on Ubuntu 16.04 or later, the first time you try to create a virtual environment with Python 3.6, Python 3.7, Python 3.8 or Python 3.9, you'll get the following error `"The virtual environment was not created successfully because ensurepip is not available"`. Just follow the hint provided in the error message and run `apt-get install python3-venv` to install the missing packages. More info [here](https://www.softwarepragmatism.com/cannot-create-a-python-virtual-environment-on-ubuntu-ensurepip-is-not-available)
389389
9. While running `azd up` in VS Code Dev Containers you got this error `".. Maven: failed finding mvnw in repository path: exec: /azure-search-openai-demo-java/app/backend/mvnw: permission denied "`. Run `chmod +x ./azure-search-openai-demo-javaapp/backend/mvnw` to fix it and rerun `azd up`.
390-
10. Github App CI pipeline might fail in some scenarios where the provisioned App Service instance doesn't have "Basic Auth Publishing Credentials" enabled in your subscription. To fix it, you can go to your App Service instance in Azure Portal, click on "Settings/Configuration(Panel)->General Settins (Tab)" and flag to ON the "Basic Auth Publishing Credentials" checkbox group. Or you can run the following azd cli commands:
391-
392-
```
393-
az resource update --resource-group <resource-group> --name ftp --namespace Microsoft.Web --resource-type basicPublishingCredentialsPolicies --parent sites/<site-name> --set properties.allow=true
394-
az resource update --resource-group <resource-group> --name scm --namespace Microsoft.Web --resource-type basicPublishingCredentialsPolicies --parent sites/<site-name> --set properties.allow=true
395-
```
396-
397-
For more details see this [issue](https://github.com/Azure-Samples/azure-search-openai-demo-java/issues/7).

docs/aks/login_and_acl.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ Two Microsoft Entra applications must be registered in order to make the optiona
3636

3737
### Prerequisites : Self signed certificate for Nginx Ingress Controller
3838

39-
When enabling user authentication, the [managed Nginx Ingress Controller](https://learn.microsoft.com/en-us/azure/aks/app-routing) requires a self-signed certificate to be created and used for TLS termination. This is necessary as we use [MSAL React library](https://learn.microsoft.com/en-us/samples/azure-samples/ms-identity-ciam-javascript-tutorial/ms-identity-ciam-javascript-tutorial-1-sign-in-react/) to authenticate users with Entra ID, and it only works for HTTPS connections
39+
When enabling user authentication, the [managed Nginx Ingress Controller](https://learn.microsoft.com/en-us/azure/aks/app-routing) requires a self-signed certificate to be created and used for TLS termination. This is necessary as we use [MSAL React library](https://learn.microsoft.com/en-us/samples/azure-samples/ms-identity-ciam-javascript-tutorial/ms-identity-ciam-javascript-tutorial-1-sign-in-react/) to authenticate users with Entra ID, and it only works on HTTPS connections
4040

4141
Follow these steps to create a self-signed certificate that will be automatically configured in the Nginx Ingress Controller by azd [post-deploy hooks](../../deploy/aks/scripts/set_ingress_tls.ps1)
4242

0 commit comments

Comments
 (0)