Skip to content

Commit 2643dcc

Browse files
Merge pull request #233 from microsoft/dev
fix: Prefix/Suffix fixes & package upgrades - dev to main
2 parents 8b76892 + 45b3630 commit 2643dcc

File tree

12 files changed

+10335
-5012
lines changed

12 files changed

+10335
-5012
lines changed

.github/workflows/azure-dev.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ jobs:
2020

2121
# Step 2: Validate the Azure template using microsoft/template-validation-action
2222
- name: Validate Azure Template
23-
uses: microsoft/template-validation-action@v0.3.5
23+
uses: microsoft/template-validation-action@v0.4.3
2424
id: validation
2525
env:
2626
AZURE_CLIENT_ID: ${{ secrets.AZURE_CLIENT_ID }}

.github/workflows/build-docker-image.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,10 @@ jobs:
1818

1919
steps:
2020
- name: Checkout repository
21-
uses: actions/checkout@v2
21+
uses: actions/checkout@v4
2222

2323
- name: Set up Docker Buildx
24-
uses: docker/setup-buildx-action@v1
24+
uses: docker/setup-buildx-action@v3
2525

2626
- name: Get current date
2727
id: date

.github/workflows/deploy.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424

2525
steps:
2626
- name: Checkout Code
27-
uses: actions/checkout@v3
27+
uses: actions/checkout@v4
2828

2929
- name: Setup Azure CLI
3030
run: |

.github/workflows/pylint.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ jobs:
1515

1616
# Step 2: Set up Python environment
1717
- name: Set up Python ${{ matrix.python-version }}
18-
uses: actions/setup-python@v3
18+
uses: actions/setup-python@v5
1919
with:
2020
python-version: ${{ matrix.python-version }}
2121

.github/workflows/test.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,10 @@ jobs:
2323

2424
steps:
2525
- name: Checkout code
26-
uses: actions/checkout@v3
26+
uses: actions/checkout@v4
2727

2828
- name: Set up Python
29-
uses: actions/setup-python@v4
29+
uses: actions/setup-python@v5
3030
with:
3131
python-version: "3.11"
3232

@@ -66,7 +66,7 @@ jobs:
6666
#
6767
# steps:
6868
# - name: Checkout code
69-
# uses: actions/checkout@v3
69+
# uses: actions/checkout@v4
7070
#
7171
# - name: Set up Node.js
7272
# uses: actions/setup-node@v3

infra/main.bicep

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,8 +86,7 @@ param existingLogAnalyticsWorkspaceId string = ''
8686
param existingFoundryProjectResourceId string = ''
8787

8888
// ========== Variables ========== //
89-
var solutionPrefix = 'cps-${padLeft(take(toLower(uniqueString(subscription().id, environmentName, resourceGroup().location)), 12), 12, '0')}'
90-
89+
var solutionPrefix = 'cps-${padLeft(take(toLower(uniqueString(subscription().id, environmentName, resourceGroup().location, resourceGroup().name)), 12), 12, '0')}'
9190
// ============== //
9291
// Resources //
9392
// ============== //

infra/scripts/docker-build.sh

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,12 @@ get_azd_env_value_or_default() {
1414
local default="$2"
1515
local required="${3:-false}"
1616

17-
value=$(azd env get-value "$key" 2>/dev/null || echo "")
17+
local value
18+
value=$(azd env get-value "$key" 2>/dev/null)
19+
local exit_code=$?
1820

19-
if [ -z "$value" ]; then
20-
if [ "$required" = true ]; then
21+
if [ $exit_code -ne 0 ] || [ -z "$value" ]; then
22+
if [ "$required" = "true" ]; then
2123
echo "❌ Required environment key '$key' not found." >&2
2224
exit 1
2325
else
@@ -27,6 +29,7 @@ get_azd_env_value_or_default() {
2729

2830
echo "$value"
2931
}
32+
3033
# Required env variables
3134
AZURE_SUBSCRIPTION_ID=$(get_azd_env_value_or_default "AZURE_SUBSCRIPTION_ID" "" true)
3235
ENV_NAME=$(get_azd_env_value_or_default "AZURE_ENV_NAME" "" true)
@@ -37,8 +40,8 @@ AZURE_ENV_IMAGETAG=$(get_azd_env_value_or_default "AZURE_ENV_IMAGETAG" "latest"
3740
CONTAINER_WEB_APP_NAME=$(get_azd_env_value_or_default "CONTAINER_WEB_APP_NAME" "" true)
3841
CONTAINER_API_APP_NAME=$(get_azd_env_value_or_default "CONTAINER_API_APP_NAME" "" true)
3942
CONTAINER_APP_NAME=$(get_azd_env_value_or_default "CONTAINER_APP_NAME" "" true)
40-
$ACR_NAME = $(get_azd_env_value_or_default "CONTAINER_REGISTRY_NAME" "" true)
41-
$ACR_ENDPOINT = $(get_azd_env_value_or_default "CONTAINER_REGISTRY_LOGIN_SERVER" "" true)
43+
ACR_NAME=$(get_azd_env_value_or_default "CONTAINER_REGISTRY_NAME" "" true)
44+
ACR_ENDPOINT=$(get_azd_env_value_or_default "CONTAINER_REGISTRY_LOGIN_SERVER" "" true)
4245

4346
echo "Using the following parameters:"
4447
echo "AZURE_SUBSCRIPTION_ID = $AZURE_SUBSCRIPTION_ID"

src/ContentProcessorAPI/requirements.txt

Lines changed: 215 additions & 80 deletions
Large diffs are not rendered by default.

src/ContentProcessorAPI/uv.lock

Lines changed: 425 additions & 408 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)