Skip to content

Commit 356ca16

Browse files
Apply suggestions from code review
Co-authored-by: Karl Erickson <[email protected]>
1 parent cfd8eef commit 356ca16

File tree

2 files changed

+16
-16
lines changed

2 files changed

+16
-16
lines changed

articles/container-apps/java-petclinic-ai-overview.md

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
title: Introduction to the Java PetClinic AI Sample in Azure Container Apps
3-
description: Explain the architecture of AI applications deployed to Azure Container Apps.
3+
description: Explains the architecture of AI applications deployed to Azure Container Apps.
44
author: KarlErickson
55
ms.author: sonwan
66
ms.service: azure-container-apps
@@ -37,7 +37,7 @@ This gateway performs the following functions:
3737

3838
The gateway operates within a dedicated virtual network, ensuring secure and isolated communication between the application and external systems.
3939

40-
The following are the key components and services featured in the application:
40+
The following table describes the key components and services featured in the application:
4141

4242
| Service or feature | Description |
4343
|-----------------------------------------------------------------------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
@@ -48,12 +48,12 @@ The following are the key components and services featured in the application:
4848
| [Managed Identities](/entra/identity/managed-identities-azure-resources/overview) | Provides Azure services with automatically managed identities in Azure AD. Eliminates the need for credential management by allowing secure service-to-service authentication without storing credentials in code. |
4949
| [Spring AI](https://spring.io/projects/spring-ai) | Spring framework for AI engineering that applies AI design principles to the Spring ecosystem. Alternatively, [Langchain4j](https://docs.langchain4j.dev/intro) is another AI framework with its own PetClinic sample in [spring-petclinic-langchain4j](https://github.com/Azure-Samples/spring-petclinic-langchain4j). For more information, see [Chat Client API](https://docs.spring.io/spring-ai/reference/api/chatclient.html).|
5050

51-
For more information on infrastructure as code elements of the application, see the [bicep scripts](https://github.com/Azure-Samples/spring-petclinic-ai/blob/main/infra/bicep/main.bicep) in the [Bring your first AI app in Azure Container Apps](https://github.com/Azure-Samples/spring-petclinic-ai/) repository
51+
For more information on the infrastructure as code elements of the application, see the [bicep scripts](https://github.com/Azure-Samples/spring-petclinic-ai/blob/main/infra/bicep/main.bicep) in the [Bring your first AI app in Azure Container Apps](https://github.com/Azure-Samples/spring-petclinic-ai/) repository.
5252
.
5353

5454
## Code implementation
5555

56-
The following information is an introduction to the code for readers to understand the flow of this first AI application:
56+
The following sections provide an introduction to the code to help you understand the flow of this first AI application.
5757

5858
### Making REST calls
5959

@@ -65,7 +65,7 @@ return this.chatClient.prompt().user(u -> u.text(query)).call().content();
6565

6666
### Chat customizations
6767

68-
The [`ChatConfiguration`](https://github.com/Azure-Samples/spring-petclinic-ai/blob/main/src/main/java/org/springframework/samples/petclinic/genai/ChatConfiguration.java) class customizes requests sent to `chatClient`. The following are some key configuration settings of `chatClient`:
68+
The [`ChatConfiguration`](https://github.com/Azure-Samples/spring-petclinic-ai/blob/main/src/main/java/org/springframework/samples/petclinic/genai/ChatConfiguration.java) class customizes requests sent to `chatClient`. The following list describes some key configuration settings of `chatClient`:
6969

7070
- Connections authentication: The client connects to Azure OpenAI. Both API key authentication and managed identity authentication are supported.
7171
- Configuration settings location: For `ChatModel`, deployment `gpt-4o` and temperature `0.7` are set in the configuration file.
@@ -115,4 +115,4 @@ For more information, see [Advisors API](https://docs.spring.io/spring-ai/refere
115115
## Next steps
116116

117117
> [!div class="nextstepaction"]
118-
> [Deploy an AI-enabled instance of Spring PetClinic on Azure Container Apps](java-petclinic-ai-overview.md).
118+
> [Deploy an AI-enabled instance of Spring PetClinic on Azure Container Apps](java-petclinic-ai-overview.md)

articles/container-apps/java-petclinic-ai-tutorial.md

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,13 @@ For information on the architectural details of this application, see [Java PetC
2828

2929
## Prerequisites
3030

31-
- Azure subscription: You can [create an account for free](https://azure.microsoft.com/free/?WT.mc_id=A261C142F) if you don't have an existing account.
32-
- Security context: To perform this quickstart, you need the `Contributor` and `User Access Administrator` roles, or the `Owner` role. For more information, see [Assign Azure roles using the Azure portal](../role-based-access-control/role-assignments-portal.yml?tabs=current).
33-
- GitHub account: Get one for [free](https://github.com/join).
34-
- git: Install the latest version of [git](https://git-scm.com/downloads).
35-
- Microsoft Build of Open JDK: Install [Microsoft Build of Open JDK](/java/openjdk/install). Use version 17 at a minimum.
36-
- Azure Command-Line interface (CLI): Install [Azure CLI](/azure/developer/azure-developer-cli/install-azd) to manage your Azure resources via command line.
37-
- Maven: Install the latest version of [Maven](https://maven.apache.org/download.cgi).
31+
- An Azure subscription. If you don't have an existing account, you can [create one for free](https://azure.microsoft.com/free/?WT.mc_id=A261C142F).
32+
- `Contributor` and `User Access Administrator` roles, or the `Owner` role. For more information, see [Assign Azure roles using the Azure portal](../role-based-access-control/role-assignments-portal.yml?tabs=current).
33+
- [A GitHub account](https://github.com/join).
34+
- The latest version of [git](https://git-scm.com/downloads).
35+
- The [Microsoft Build of Open JDK](/java/openjdk/install), version 17 or higher.
36+
- [Azure CLI](/azure/developer/azure-developer-cli/install-azd).
37+
- [Maven](https://maven.apache.org/download.cgi).
3838

3939
## Setup
4040

@@ -52,7 +52,7 @@ For information on the architectural details of this application, see [Java PetC
5252
cd spring-petclinic-ai
5353
```
5454

55-
1. If you don't already have it, install the Azure CLI.
55+
1. If you don't already have it, install the `containerapp` extension for the Azure CLI.
5656
5757
```azurecli
5858
az extension add --name containerapp --upgrade
@@ -70,13 +70,13 @@ For information on the architectural details of this application, see [Java PetC
7070
7171
## Deploy
7272
73-
1. Automatically deploy the application by with the `azd up` command.
73+
1. Automatically deploy the application by using the `azd up` command.
7474
7575
```azdeveloper
7676
azd up
7777
```
7878
79-
1. When you're prompted you for a name, enter **my-first-ai** for the environment name.
79+
1. When you're prompted, enter **my-first-ai** for the environment name.
8080

8181
After that, enter values for `Azure Subscription` and `Azure location`, substituting your actual values for the `<..>` placeholders in the following prompts:
8282

0 commit comments

Comments
 (0)