Pausing the arcadesdk darc subscription and fixing github private repo CI builds

eng/Version.Details.xml

@@ -2,9 +2,14 @@
 <Dependencies>
   <ProductDependencies></ProductDependencies>
   <ToolsetDependencies>
+    <!--
+    Commenting this out to "pause" our subscription to the arcade sdk, because it's blowing away changes
+    needed to support private github repo CI builds. Uncomment once this is fixed in the arcade sdk.
+
     <Dependency Name="Microsoft.DotNet.Arcade.Sdk" Version="1.0.0-beta.18530.4">
       <Uri>https://github.com/dotnet/arcade</Uri>
       <Sha>7ac8b155c0e77a97848f376c54d54305404fc2b9</Sha>
     </Dependency>
+    -->
   </ToolsetDependencies>
 </Dependencies>

eng/common/templates/phases/base.yml

@@ -59,7 +59,7 @@ phases:
 
   - ${{ if eq(parameters.enableMicrobuild, 'true') }}:
     # Internal only resource, and Microbuild signing shouldn't be applied to PRs.
-    - ${{ if and(ne(variables['System.TeamProject'], 'public'), notin(variables['Build.Reason'], 'PullRequest'), ne(variables['Build.Repository.Provider'], 'GitHub')) }}:
+    - ${{ if and(ne(variables['System.TeamProject'], 'public'), notin(variables['Build.Reason'], 'PullRequest'), not(contains(variables['Build.DefinitionName'], 'github'))) }}:
       - task: MicroBuildSigningPlugin@2
         displayName: Install MicroBuild plugin
         inputs:
@@ -77,7 +77,7 @@ phases:
 
   - ${{ if eq(parameters.enableMicrobuild, 'true') }}:
     # Internal only resources
-    - ${{ if and(ne(variables['System.TeamProject'], 'public'), ne(variables['Build.Repository.Provider'], 'GitHub')) }}:
+    - ${{ if and(ne(variables['System.TeamProject'], 'public'), not(contains(variables['Build.DefinitionName'], 'github'))) }}:
       - task: MicroBuildCleanup@1
         displayName: Execute Microbuild cleanup tasks  
         condition: and(always(), in(variables['_SignType'], 'real', 'test'), eq(variables['Agent.Os'], 'Windows_NT'))
@@ -90,7 +90,7 @@ phases:
         helixSource: $(_HelixSource)
         helixType: $(_HelixType)
 
-  - ${{ if and(ne(variables['System.TeamProject'], 'public'), notin(variables['Build.Reason'], 'PullRequest'), ne(variables['Build.Repository.Provider'], 'GitHub')) }}:
+  - ${{ if and(ne(variables['System.TeamProject'], 'public'), notin(variables['Build.Reason'], 'PullRequest'), not(contains(variables['Build.DefinitionName'], 'github'))) }}:
     - task: CopyFiles@2
       displayName: Gather Asset Manifests
       inputs:

eng/common/templates/phases/publish-build-assets.yml

@@ -12,7 +12,7 @@ phases:
     variables:
       config: ${{ parameters.configuration }}
     steps:
-      - ${{ if and(ne(variables['System.TeamProject'], 'public'), notin(variables['Build.Reason'], 'PullRequest'), ne(variables['Build.Repository.Provider'], 'GitHub')) }}:
+      - ${{ if and(ne(variables['System.TeamProject'], 'public'), notin(variables['Build.Reason'], 'PullRequest'), not(contains(variables['Build.DefinitionName'], 'github'))) }}:
         - task: DownloadBuildArtifacts@0
           displayName: Download artifact
           inputs:

eng/common/templates/steps/telemetry-start.yml

@@ -4,7 +4,7 @@ parameters:
   buildConfig: ''
 
 steps:
-- ${{ if and(not(eq(variables['System.TeamProject'], 'public')), ne(variables['Build.Repository.Provider'], 'GitHub')) }}:
+- ${{ if and(not(eq(variables['System.TeamProject'], 'public')), not(contains(variables['Build.DefinitionName'], 'github'))) }}:
   - task: AzureKeyVault@1
     inputs:
       azureSubscription: 'HelixProd_KeyVault'