From a6ca5917b69ff23a1801cabc281fc6bb5b47d67e Mon Sep 17 00:00:00 2001
From: Manuel de la Pena <mandel@microsoft.com>
Date: Fri, 18 Feb 2022 11:02:53 -0500
Subject: [PATCH] [CI] If we disabled the api diff do not event spawn a job.
 (#13875)

---
 .../automation/templates/build/stage.yml      | 67 ++++++++++---------
 1 file changed, 34 insertions(+), 33 deletions(-)

diff --git a/tools/devops/automation/templates/build/stage.yml b/tools/devops/automation/templates/build/stage.yml
index b9e836e43e..36a34f6419 100644
--- a/tools/devops/automation/templates/build/stage.yml
+++ b/tools/devops/automation/templates/build/stage.yml
@@ -116,40 +116,41 @@ jobs:
       skipESRP: ${{ parameters.skipESRP }}
       enableAPIDiff: false # this step should never execute the api diff, that should be left for the parallel job
 
-- job: api_diff
-  dependsOn:
-  - AgentPoolSelector
-  - configure
-  displayName: 'Generate API diff'
-  timeoutInMinutes: 1000
-  variables:
-    AgentPoolComputed: $[ dependencies.AgentPoolSelector.outputs['setAgentPool.AgentPoolComputed'] ]
-    PR_ID: $[ dependencies.configure.outputs['labels.pr-number'] ]
-    # set the branch variable name, this is required by jenkins and we have a lot of scripts that depend on it
-    BRANCH_NAME: $[ replace(variables['Build.SourceBranch'], 'refs/heads/', '') ]
-    XHARNESS_LABELS: $[ dependencies.configure.outputs['labels.xharness-labels'] ]
-  pool:
-    name: $(AgentPoolComputed)
-    demands:
-    - Agent.OS -equals Darwin
-    - macios_image -equals v2.1     # Big Sur image with Xcode 12.4 and 12.5 installed
-  workspace:
-    clean: all
+- ${{ if eq(parameters.enableAPIDiff, true) }}:
+  - job: api_diff
+    dependsOn:
+    - AgentPoolSelector
+    - configure
+    displayName: 'Generate API diff'
+    timeoutInMinutes: 1000
+    variables:
+      AgentPoolComputed: $[ dependencies.AgentPoolSelector.outputs['setAgentPool.AgentPoolComputed'] ]
+      PR_ID: $[ dependencies.configure.outputs['labels.pr-number'] ]
+      # set the branch variable name, this is required by jenkins and we have a lot of scripts that depend on it
+      BRANCH_NAME: $[ replace(variables['Build.SourceBranch'], 'refs/heads/', '') ]
+      XHARNESS_LABELS: $[ dependencies.configure.outputs['labels.xharness-labels'] ]
+    pool:
+      name: $(AgentPoolComputed)
+      demands:
+      - Agent.OS -equals Darwin
+      - macios_image -equals v2.1     # Big Sur image with Xcode 12.4 and 12.5 installed
+    workspace:
+      clean: all
 
-  steps:
-  - template: build.yml
-    parameters:
-      runTests: false
-      runDeviceTests: false
-      uploadBinlogs: false
-      signAndNotarize: false
-      skipPackages: true
-      vsdropsPrefix: ${{ parameters.vsdropsPrefix }}
-      keyringPass: ${{ parameters.keyringPass }}
-      gitHubToken: ${{ parameters.gitHubToken }}
-      xqaCertPass: ${{ parameters.xqaCertPass }}
-      enableDotnet: ${{ parameters.enableDotnet }}
-      enableAPIDiff: ${{ parameters.enableAPIDiff }}
+    steps:
+    - template: build.yml
+      parameters:
+        runTests: false
+        runDeviceTests: false
+        uploadBinlogs: false
+        signAndNotarize: false
+        skipPackages: true
+        vsdropsPrefix: ${{ parameters.vsdropsPrefix }}
+        keyringPass: ${{ parameters.keyringPass }}
+        gitHubToken: ${{ parameters.gitHubToken }}
+        xqaCertPass: ${{ parameters.xqaCertPass }}
+        enableDotnet: ${{ parameters.enableDotnet }}
+        enableAPIDiff: true
 
 - job: upload_azure_blob
   displayName: 'Upload packages to Azure & SBOM'