Skip to content

Commit

Permalink
Address React Native pipeline component detection timeout (microsoft#…
Browse files Browse the repository at this point in the history
…20871)

mac-react-native-ci-pipeline.yml:
- We don't need to run component detection for PR builds so just disable it there.

npm-packaging-pipeline.yml:
- Manually added component detection task was being added twice - removed one.
- Increased timeout of stage where component detection is run since the existing timeout was close for some builds.
  • Loading branch information
edgchen1 authored May 30, 2024
1 parent 2200a0b commit a508130
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ resources:
ref: 5eda9aded5462201e6310105728d33016e637ea7

variables:
ComponentDetection.Timeout: 660
skipComponentGovernanceDetection: true
${{ if eq(parameters.NpmPublish, 'nightly (@dev)') }}:
NpmPackagingMode: 'dev'
${{ if eq(parameters.NpmPublish, 'release candidate (@rc)') }}:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,6 @@ stages:

variables:
runCodesignValidationInjection: false
ComponentDetection.Timeout: 660
TODAY: $[format('{0:dd}{0:MM}{0:yyyy}', pipeline.startTime)]
ORT_CACHE_DIR: '$(Pipeline.Workspace)/ccache_ort'

Expand Down Expand Up @@ -120,7 +119,7 @@ stages:
vmImage: 'macOS-12'
variables:
runCodesignValidationInjection: false
timeoutInMinutes: 60
timeoutInMinutes: 90
steps:
- task: mspremier.PostBuildCleanup.PostBuildCleanup-task.PostBuildCleanup@3
displayName: Clean Agent Directories
Expand Down Expand Up @@ -399,10 +398,6 @@ stages:
targetFolder: $(Build.ArtifactStagingDirectory)
displayName: Create Artifacts onnxruntime-react-native

- template: component-governance-component-detection-steps.yml
parameters :
condition : 'succeeded'

- task: PublishPipelineArtifact@1
inputs:
artifact: e2e_test_logs_$(Build.BuildId)_$(Build.BuildNumber)_$(System.JobAttempt)
Expand Down

0 comments on commit a508130

Please sign in to comment.