From 09980780d8870082977d399dc172d86ef6a45818 Mon Sep 17 00:00:00 2001
From: Billy O'Neal <bion@microsoft.com>
Date: Tue, 28 Sep 2021 13:51:42 -0700
Subject: [ci] Move formatting and doc checks into x86-windows (#20402)

* [ci] Move formatting and doc checks into x86-windows to avoid burning a whole job for that 1 run, don't run file lists generation in PRs.

* Deduplicate x86-windows and restore PR file lists, as requested/suggested by @dg0yt

* More quotes

Co-authored-by: nicole mazzuca <83086508+strega-nil-ms@users.noreply.github.com>

Co-authored-by: nicole mazzuca <83086508+strega-nil-ms@users.noreply.github.com>
---
 scripts/azure-pipelines/azure-pipelines.yml        | 139 ++++++++-------------
 .../azure-pipelines/windows/azure-pipelines.yml    |  31 ++++-
 2 files changed, 79 insertions(+), 91 deletions(-)

(limited to 'scripts')

diff --git a/scripts/azure-pipelines/azure-pipelines.yml b/scripts/azure-pipelines/azure-pipelines.yml
index e5102c32c..f57de8835 100644
--- a/scripts/azure-pipelines/azure-pipelines.yml
+++ b/scripts/azure-pipelines/azure-pipelines.yml
@@ -6,92 +6,53 @@ variables:
   linux-pool: 'PrLin-2021-09-08'
   osx-pool: 'PrOsx-2021-07-27'
 
-stages:
-- stage: FormatChecks
-  displayName: 'Formatting and Documentation Checks'
-  pool: $(windows-pool)
-  jobs:
-  - job:
-    workspace:
-      clean: resources
-    variables:
-    - name: VCPKG_DOWNLOADS
-      value: D:\downloads
-    - name: DiffFile
-      value: $(Build.ArtifactStagingDirectory)\format.diff
-    steps:
-    - task: Powershell@2
-      displayName: 'Generate Documentation'
-      inputs:
-        filePath: 'docs/regenerate.ps1'
-        arguments: '-VcpkgRoot . -WarningAction Stop'
-        pwsh: true
-    - script: .\bootstrap-vcpkg.bat
-      displayName: 'Bootstrap vcpkg'
-    - script: '.\vcpkg.exe format-manifest --all'
-      displayName: 'Format Manifests'
-    - task: Powershell@2
-      displayName: 'Create Diff'
-      inputs:
-        filePath: scripts/azure-pipelines/Create-PRDiff.ps1
-        arguments: '-DiffFile $(DiffFile)'
-        pwsh: true
-    - task: PublishBuildArtifacts@1
-      condition: failed()
-      displayName: 'Publish Format and Documentation Diff'
-      inputs:
-        PathtoPublish: '$(DiffFile)'
-        ArtifactName: 'format.diff'
-- stage: RunPrTests
-  displayName: 'Run PR Tests:'
-  dependsOn: FormatChecks
-  jobs:
-  - template: windows/azure-pipelines.yml
-    parameters:
-      triplet: x86-windows
-      jobName: x86_windows
-      poolName: $(windows-pool)
-
-  - template: windows/azure-pipelines.yml
-    parameters:
-      triplet: x64-windows
-      jobName: x64_windows
-      poolName: $(windows-pool)
-
-  - template: windows/azure-pipelines.yml
-    parameters:
-      triplet: x64-windows-static
-      jobName: x64_windows_static
-      poolName: $(windows-pool)
-
-  - template: windows/azure-pipelines.yml
-    parameters:
-      triplet: x64-windows-static-md
-      jobName: x64_windows_static_md
-      poolName: $(windows-pool)
-
-  - template: windows/azure-pipelines.yml
-    parameters:
-      triplet: x64-uwp
-      jobName: x64_uwp
-      poolName: $(windows-pool)
-
-  - template: windows/azure-pipelines.yml
-    parameters:
-      triplet: arm64-windows
-      jobName: arm64_windows
-      poolName: $(windows-pool)
-
-  - template: windows/azure-pipelines.yml
-    parameters:
-      triplet: arm-uwp
-      jobName: arm_uwp
-      poolName: $(windows-pool)
-
-  - template: osx/azure-pipelines.yml
-    parameters:
-      poolName: $(osx-pool)
-
-  - template: linux/azure-pipelines.yml
-    parameters:
-      poolName: $(linux-pool)
+jobs:
+- template: windows/azure-pipelines.yml
+  parameters:
+    triplet: x86-windows
+    jobName: x86_windows
+    poolName: $(windows-pool)
+
+- template: windows/azure-pipelines.yml
+  parameters:
+    triplet: x64-windows
+    jobName: x64_windows
+    poolName: $(windows-pool)
+
+- template: windows/azure-pipelines.yml
+  parameters:
+    triplet: x64-windows-static
+    jobName: x64_windows_static
+    poolName: $(windows-pool)
+
+- template: windows/azure-pipelines.yml
+  parameters:
+    triplet: x64-windows-static-md
+    jobName: x64_windows_static_md
+    poolName: $(windows-pool)
+
+- template: windows/azure-pipelines.yml
+  parameters:
+    triplet: x64-uwp
+    jobName: x64_uwp
+    poolName: $(windows-pool)
+
+- template: windows/azure-pipelines.yml
+  parameters:
+    triplet: arm64-windows
+    jobName: arm64_windows
+    poolName: $(windows-pool)
+
+- template: windows/azure-pipelines.yml
+  parameters:
+    triplet: arm-uwp
+    jobName: arm_uwp
+    poolName: $(windows-pool)
+
+- template: osx/azure-pipelines.yml
+  parameters:
+    poolName: $(osx-pool)
+
+- template: linux/azure-pipelines.yml
+  parameters:
+    poolName: $(linux-pool)
diff --git a/scripts/azure-pipelines/windows/azure-pipelines.yml b/scripts/azure-pipelines/windows/azure-pipelines.yml
index b88a5f120..d341a7a9d 100644
--- a/scripts/azure-pipelines/windows/azure-pipelines.yml
+++ b/scripts/azure-pipelines/windows/azure-pipelines.yml
@@ -14,6 +14,10 @@ jobs:
     value: D:\
   - name: VCPKG_DOWNLOADS
     value: D:\downloads
+  - name: DiffFile
+    value: $(Build.ArtifactStagingDirectory)\format.diff
+  - name: ExtraChecksTriplet
+    value: x86-windows
   - group: vcpkg-asset-caching-credentials
   - name: X_VCPKG_ASSET_SOURCES
     value: "x-azurl,$(root-url),$(sas),readwrite"
@@ -28,12 +32,35 @@ jobs:
     inputs:
       filePath: 'scripts/azure-pipelines/windows/disk-space.ps1'
       pwsh: true
-  # Note: E: is the Azure machines' temporary disk.
   - script: .\bootstrap-vcpkg.bat
     displayName: 'Bootstrap vcpkg'
   - script: |
       if exist ${{ variables.VCPKG_DOWNLOADS }} rmdir /S /Q ${{ variables.VCPKG_DOWNLOADS }} 2>&1
     displayName: 'Clean downloads'
+  # Note that we run docs checks before PR checks because they are likely to invalidate a whole run anyway
+  - task: Powershell@2
+    displayName: 'Generate Documentation'
+    condition: eq('${{ parameters.triplet }}', '${{ variables.ExtraChecksTriplet }}')
+    inputs:
+      filePath: 'docs/regenerate.ps1'
+      arguments: '-VcpkgRoot . -WarningAction Stop'
+      pwsh: true
+  - script: '.\vcpkg.exe format-manifest --all'
+    displayName: 'Format Manifests'
+    condition: eq('${{ parameters.triplet }}', '${{ variables.ExtraChecksTriplet }}')
+  - task: Powershell@2
+    displayName: 'Create Diff'
+    condition: eq('${{ parameters.triplet }}', '${{ variables.ExtraChecksTriplet }}')
+    inputs:
+      filePath: scripts/azure-pipelines/Create-PRDiff.ps1
+      arguments: "-DiffFile '$(DiffFile)'"
+      pwsh: true
+  - task: PublishBuildArtifacts@1
+    displayName: 'Publish Format and Documentation Diff'
+    condition: and(eq('${{ parameters.triplet }}', '${{ variables.ExtraChecksTriplet }}'), failed())
+    inputs:
+      PathtoPublish: '$(DiffFile)'
+      ArtifactName: 'format.diff'
   - task: PowerShell@2
     displayName: '*** Test Modified Ports and Prepare Test Logs ***'
     inputs:
@@ -43,7 +70,7 @@ jobs:
       pwsh: true
   - task: PowerShell@2
     displayName: 'Validate version files'
-    condition: eq('${{ parameters.triplet }}', 'x86-windows')
+    condition: eq('${{ parameters.triplet }}', '${{ variables.ExtraChecksTriplet }}')
     inputs:
       targetType: inline
       script: |
-- 
cgit v1.2.3