Skip to content

Commit

Permalink
Release 1.2.0 fix
Browse files Browse the repository at this point in the history
  • Loading branch information
dantelmomsft committed Mar 21, 2024
1 parent 4742a0e commit 4986d52
Show file tree
Hide file tree
Showing 13 changed files with 58 additions and 28 deletions.
38 changes: 33 additions & 5 deletions .github/workflows/nightly-jobs.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,19 +7,47 @@ on:
- cron: '0 0 * * *' # Run at midnight every day
workflow_dispatch:

permissions:
id-token: write
contents: read
security-events: write

jobs:
infra:
validate-bicep:
name: "Infra Biceps Validation"
runs-on: ubuntu-latest
steps:
- name: Checkout
uses: actions/checkout@v4

- name: Build Bicep for linting
- name: Filter Changes
uses: dorny/paths-filter@v2
id: changes
with:
filters: |
app-service:
- 'deploy/app-service/**'
aks:
- 'deploy/aks/**'
aca:
- 'deploy/aca/**'
- name: Build App Service Bicep for linting
if: steps.changes.outputs.app-service == 'true'
uses: azure/CLI@v1
with:
inlineScript: az config set bicep.use_binary_from_path=false && az bicep build -f deploy/app-service/infra/main.bicep --stdout

- name: Build AKS Bicep for linting
if: steps.changes.outputs.aks == 'true'
uses: azure/CLI@v1
with:
inlineScript: az config set bicep.use_binary_from_path=false && az bicep build -f deploy/aks/infra/main.bicep --stdout

- name: Build ACA Bicep for linting
if: steps.changes.outputs.aca == 'true'
uses: azure/CLI@v1
with:
inlineScript: az config set bicep.use_binary_from_path=false && az bicep build -f infra/main.bicep --stdout
inlineScript: az config set bicep.use_binary_from_path=false && az bicep build -f deploy/aca/infra/main.bicep --stdout

- name: Run Microsoft Security DevOps Analysis
uses: microsoft/security-devops-action@v1
id: msdo
Expand Down
2 changes: 1 addition & 1 deletion app/backend/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
</parent>
<groupId>com.microsoft.openai.samples.rag</groupId>
<artifactId>search-demo-java</artifactId>
<version>1.2.0-SNAPSHOT</version>
<version>1.2.0</version>
<name>openai-search-demo-java</name>
<description>This sample demonstrates a few approaches for creating ChatGPT-like experiences over your own data using the Retrieval Augmented Generation pattern</description>
<properties>
Expand Down
7 changes: 2 additions & 5 deletions app/frontend/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion app/frontend/package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "frontend",
"private": true,
"version": "0.0.0",
"version": "1.2.0",
"type": "module",
"engines": {
"node": ">=14.0.0"
Expand Down
2 changes: 1 addition & 1 deletion app/indexer/cli/dependency-reduced-pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
<parent>
<artifactId>indexer-parent</artifactId>
<groupId>com.microsoft.openai.samples</groupId>
<version>1.2.0-SNAPSHOT</version>
<version>1.2.0</version>
</parent>
<modelVersion>4.0.0</modelVersion>
<artifactId>indexer-cli</artifactId>
Expand Down
4 changes: 2 additions & 2 deletions app/indexer/cli/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
<parent>
<groupId>com.microsoft.openai.samples</groupId>
<artifactId>indexer-parent</artifactId>
<version>1.2.0-SNAPSHOT</version>
<version>1.2.0</version>
<relativePath>../pom.xml</relativePath>
</parent>

Expand All @@ -22,7 +22,7 @@
<dependency>
<groupId>com.microsoft.openai.samples</groupId>
<artifactId>indexer-core</artifactId>
<version>1.2.0-SNAPSHOT</version>
<version>1.2.0</version>
</dependency>
</dependencies>

Expand Down
2 changes: 1 addition & 1 deletion app/indexer/core/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
<parent>
<groupId>com.microsoft.openai.samples</groupId>
<artifactId>indexer-parent</artifactId>
<version>1.2.0-SNAPSHOT</version>
<version>1.2.0</version>
<relativePath>../pom.xml</relativePath>
</parent>

Expand Down
17 changes: 10 additions & 7 deletions app/indexer/functions/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,13 @@
<parent>
<groupId>com.microsoft.openai.samples</groupId>
<artifactId>indexer-parent</artifactId>
<version>1.2.0-SNAPSHOT</version>
<version>1.2.0</version>
<relativePath>../pom.xml</relativePath>
</parent>


<artifactId>indexer-functions</artifactId>
<version>1.0-SNAPSHOT</version>
<version>1.0</version>


<properties>
Expand All @@ -31,7 +31,7 @@
<dependency>
<groupId>com.microsoft.openai.samples</groupId>
<artifactId>indexer-core</artifactId>
<version>1.0-SNAPSHOT</version>
<version>1.2.0</version>
</dependency>
</dependencies>

Expand All @@ -47,16 +47,18 @@
<encoding>${project.build.sourceEncoding}</encoding>
</configuration>
</plugin>
<!--
Functions deployment generation is disabled as we are still figuring out how to make it azd compliant.
The problem is that we haven't found a working example of passing the app name generated by azd provision as env variables to maven during azd package phase.
That's because probably azd try to package the app before provision phase
<plugin>
<groupId>com.microsoft.azure</groupId>
<artifactId>azure-functions-maven-plugin</artifactId>
<version>${azure.functions.maven.plugin.version}</version>
<configuration>
<!-- function app name -->
<!--suppress UnresolvedMavenProperty -->
<appName>${env.INDEXER_FUNCTIONAPP_NAME}</appName>
<!-- function app resource group -->
<!--suppress UnresolvedMavenProperty -->
<resourceGroup>${env.AZURE_RESOURCE_GROUP}</resourceGroup>
<appSettings>
<property>
Expand All @@ -74,6 +76,7 @@
</execution>
</executions>
</plugin>
</-->
<!--Remove obj folder generated by .NET SDK in maven clean-->
<plugin>
<artifactId>maven-clean-plugin</artifactId>
Expand Down
4 changes: 2 additions & 2 deletions app/indexer/microservice/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@

<groupId>com.microsoft.openai.samples</groupId>
<artifactId>indexer-microservice</artifactId>
<version>1.2.0-SNAPSHOT</version>
<version>1.2.0</version>


<properties>
Expand Down Expand Up @@ -41,7 +41,7 @@
<dependency>
<groupId>com.microsoft.openai.samples</groupId>
<artifactId>indexer-core</artifactId>
<version>1.2.0-SNAPSHOT</version>
<version>1.2.0</version>
</dependency>
<dependency>
<groupId>org.springframework.boot</groupId>
Expand Down
3 changes: 2 additions & 1 deletion app/indexer/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

<groupId>com.microsoft.openai.samples</groupId>
<artifactId>indexer-parent</artifactId>
<version>1.2.0-SNAPSHOT</version>
<version>1.2.0</version>
<packaging>pom</packaging>

<properties>
Expand All @@ -25,6 +25,7 @@
<module>core</module>
<module>cli</module>
<module>microservice</module>
<module>functions</module>
</modules>

<dependencyManagement>
Expand Down
2 changes: 1 addition & 1 deletion deploy/aca/azure.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

name: azure-search-openai-demo-java-aca
metadata:
template: [email protected]-alpha
template: [email protected]
services:
api:
project: ../../app/backend
Expand Down
2 changes: 1 addition & 1 deletion deploy/aks/azure.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

name: azure-search-openai-demo-java-aks
metadata:
template: [email protected]-alpha
template: [email protected]
services:
api:
project: ../../app/backend
Expand Down
1 change: 1 addition & 0 deletions docs/app-service/README-App-Service.md
Original file line number Diff line number Diff line change
Expand Up @@ -443,3 +443,4 @@ Here are the most common failure scenarios and solutions:
az resource update --resource-group <resource-group> --name scm --namespace Microsoft.Web --resource-type basicPublishingCredentialsPolicies --parent sites/<site-name> --set properties.allow=true
```
For more details see this [issue](https://github.com/Azure-Samples/azure-search-openai-demo-java/issues/7).
10. When running azd up the first time it successfully complete but the zip file extraction fails on App Service during deployment. You can fix this by running azd deploy after azd up completes. This will only deploy the app code and not provision the resources again.

0 comments on commit 4986d52

Please sign in to comment.