Skip to content

Commit

Permalink
Added fixes for build environment and npm log level
Browse files Browse the repository at this point in the history
  • Loading branch information
DCMattyG committed Sep 3, 2024
1 parent fb7fe29 commit 2a82c87
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/azure-ipam-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ jobs:
init:
name: Initialize Azure IPAM Build
runs-on: ubuntu-latest
environment: production
outputs:
ipamVersion: ${{ steps.extractVersion.outputs.ipamVersion }}
steps:
Expand All @@ -34,6 +35,7 @@ jobs:
build:
name: Build Azure IPAM Containers
runs-on: ubuntu-latest
environment: production
needs: [ init ]
steps:
- name: Azure Login
Expand Down Expand Up @@ -61,6 +63,7 @@ jobs:
build-legacy:
name: Build Legacy Azure IPAM Containers
runs-on: ubuntu-latest
environment: production
needs: [ init ]
steps:
- name: Azure Login
Expand Down
10 changes: 5 additions & 5 deletions tools/build.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -154,17 +154,17 @@ try {

Write-Host "INFO: Running NPM Install..." -ForegroundColor Green

# Build Azure IPAM UI
# Install Azure IPAM UI Dependencies
$npmInstallErr = $(
$npmInstall = npm ci --no-progress --no-update-notifier --no-fund
$npmInstall = npm ci --no-progress --no-update-notifier --no-fund --loglevel error
) 2>&1

# Switch back to original dir
Pop-Location

# Exit if NPM Build fails
# Exit if NPM Install fails
if($npmInstallErr) {
Write-Host "ERROR: NPM Build failed!" -ForegroundColor red
Write-Host "ERROR: NPM Install failed!" -ForegroundColor red
throw $npmInstallErr
}

Expand All @@ -183,7 +183,7 @@ try {

# Exit if NPM Build fails
if($npmBuildErr) {
Write-Host "ERROR: NPM Install failed!" -ForegroundColor red
Write-Host "ERROR: NPM Build failed!" -ForegroundColor red
throw $npmBuildErr
}

Expand Down

0 comments on commit 2a82c87

Please sign in to comment.