Skip to content

Commit 7d80e70

Browse files
authored
Fix IdentityModel pipeline (#55527)
* Fix spacing on pipeline * Force reinstall * Change step order * Try local source * Fix package ref * Move binlog * Fix open queue usage * How bout now * Don't run template tests * Really skip template tests * NoWarn
1 parent d28d0c5 commit 7d80e70

File tree

11 files changed

+24
-14
lines changed

11 files changed

+24
-14
lines changed

.azure/pipelines/identitymodel-helix-matrix.yml

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -49,46 +49,48 @@ extends:
4949
timeoutInMinutes: 300
5050
steps:
5151
- task: NuGetAuthenticate@1
52+
inputs:
53+
forceReinstallCredentialProvider: true
5254
- task: NuGetCommand@2
5355
displayName: Install Microsoft.IdentityModel.Logging
5456
inputs:
5557
command: 'custom'
5658
arguments: 'install Microsoft.IdentityModel.Logging
5759
-Source https://pkgs.dev.azure.com/dnceng/internal/_packaging/identitymodel-nightlies/nuget/v3/index.json
58-
-DependencyVersion Highest -PreRelease'
60+
-DependencyVersion Highest -OutputDirectory $(Build.StagingDirectory) -PreRelease'
5961
- task: NuGetCommand@2
60-
inputs:
6162
displayName: Install Microsoft.IdentityModel.Protocols.OpenIdConnect
63+
inputs:
6264
command: 'custom'
6365
arguments: 'install Microsoft.IdentityModel.Protocols.OpenIdConnect
6466
-Source https://pkgs.dev.azure.com/dnceng/internal/_packaging/identitymodel-nightlies/nuget/v3/index.json
65-
-DependencyVersion Highest -PreRelease'
67+
-DependencyVersion Highest -OutputDirectory $(Build.StagingDirectory) -PreRelease'
6668
- task: NuGetCommand@2
67-
inputs:
6869
displayName: Install Microsoft.IdentityModel.Protocols.WsFederation
70+
inputs:
6971
command: 'custom'
7072
arguments: 'install Microsoft.IdentityModel.Protocols.WsFederation
7173
-Source https://pkgs.dev.azure.com/dnceng/internal/_packaging/identitymodel-nightlies/nuget/v3/index.json
72-
-DependencyVersion Highest -PreRelease'
74+
-DependencyVersion Highest -OutputDirectory $(Build.StagingDirectory) -PreRelease'
7375
- task: NuGetCommand@2
74-
inputs:
7576
displayName: System.IdentityModel.Tokens.Jwt
77+
inputs:
7678
command: 'custom'
7779
arguments: 'install System.IdentityModel.Tokens.Jwt
7880
-Source https://pkgs.dev.azure.com/dnceng/internal/_packaging/identitymodel-nightlies/nuget/v3/index.json
79-
-DependencyVersion Highest -PreRelease'
81+
-DependencyVersion Highest -OutputDirectory $(Build.StagingDirectory) -PreRelease'
8082
- task: PowerShell@2
8183
displayName: Add IdentityModel feel to NuGet.config
8284
inputs:
8385
filePath: $(Build.SourcesDirectory)/eng/scripts/SetupIdentitySources.ps1
84-
arguments: -ConfigFile $(Build.SourcesDirectory)/NuGet.config
86+
arguments: -ConfigFile $(Build.SourcesDirectory)/NuGet.config -IdentityModelPackageSource $(Build.StagingDirectory)
8587
# Build the shared framework
8688
- script: ./eng/build.cmd -ci -prepareMachine -nativeToolsOnMachine -nobl -all -pack -arch x64
8789
/p:CrossgenOutput=false /p:IsIdentityModelTestJob=true /p:ASPNETCORE_TEST_LOG_DIR=artifacts/log
8890
displayName: Build shared fx
8991
# -noBuildRepoTasks -noBuildNative -noBuild to avoid repeating work done in the previous step.
9092
- script: .\eng\build.cmd -ci -prepareMachine -nobl -all -noBuildRepoTasks -noBuildNative -noBuild -test
91-
-projects eng\helix\helix.proj /p:IsHelixJob=true
93+
-projects eng\helix\helix.proj /p:IsHelixJob=true /p:RunTemplateTests=false
9294
/p:CrossgenOutput=false /p:IsIdentityModelTestJob=true /p:ASPNETCORE_TEST_LOG_DIR=artifacts/log
9395
displayName: Run build.cmd helix target
9496
env:

Directory.Build.props

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,8 @@
121121
<!-- Ensure API docs are available. -->
122122
<NoWarn>$(NoWarn.Replace('1591', ''))</NoWarn>
123123
<NoWarn Condition=" '$(BuildingInsideVisualStudio)' == 'true' ">$(NoWarn);0105</NoWarn>
124+
<!-- Allow prerelease dependencies in stable packages when we're running the IdentityModel nightly tests -->
125+
<NoWarn Condition=" '$(IsIdentityModelTestJob)' == 'true' ">$(NoWarn);NU5104</NoWarn>
124126

125127
<!-- For local builds, don't make missing XML docs a fatal build error, but still surface so we have visibility into undocumented APIs. -->
126128
<WarningsNotAsErrors Condition=" '$(ContinuousIntegrationBuild)' != 'true' ">$(WarningsNotAsErrors);CS1591</WarningsNotAsErrors>

eng/Versions.props

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
<PreReleaseVersionIteration>5</PreReleaseVersionIteration>
1313
<ValidateBaseline>true</ValidateBaseline>
1414
<IdentityModelVersion Condition="'$(IsIdentityModelTestJob)' != 'true'">7.4.1</IdentityModelVersion>
15-
<IdentityModelVersion Condition="'$(IsIdentityModelTestJob)' == 'true'">7.*</IdentityModelVersion>
15+
<IdentityModelVersion Condition="'$(IsIdentityModelTestJob)' == 'true'">*-*</IdentityModelVersion>
1616
<!--
1717
When StabilizePackageVersion is set to 'true', this branch will produce stable outputs for 'Shipping' packages
1818
-->

eng/helix/helix.proj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
<!-- Microsoft.DotNet.Helix.Sdk.MultiQueue.targets splits $(HelixTargetQueues) into @(HelixTargetQueue) items. -->
2222
<ItemGroup Condition=" '$(HelixTargetQueues)' == '' ">
2323
<HelixTargetQueue Condition="'$(_UseHelixOpenQueues)' == 'true'" Include="@(HelixAvailableTargetQueue)" />
24-
<HelixTargetQueue Condition="'$(_UseHelixOpenQueues)' != 'true'" Include="@(HelixAvailableTargetQueue->'%(Identity)'->Replace('.Open',''))" />
24+
<HelixTargetQueue Condition="'$(_UseHelixOpenQueues)' != 'true'" Include="@(HelixAvailableTargetQueue->'%(Identity)'->Replace('.Open','')->Replace('.open',''))" />
2525
</ItemGroup>
2626

2727
<PropertyGroup>

eng/scripts/SetupIdentitySources.ps1

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
[CmdletBinding()]
22
param (
3-
[Parameter(Mandatory = $true)][string]$ConfigFile
3+
[Parameter(Mandatory = $true)][string]$ConfigFile,
4+
[Parameter(Mandatory = $true)][string]$IdentityModelPackageSource
45
)
56

67
$ErrorActionPreference = "Stop"
@@ -40,6 +41,6 @@ if ($sources -eq $null) {
4041
$doc.DocumentElement.AppendChild($sources) | Out-Null
4142
}
4243

43-
AddPackageSource -Sources $sources -SourceName "identitymodel-nightlies" -SourceEndPoint "https://pkgs.dev.azure.com/dnceng/internal/_packaging/identitymodel-nightlies/nuget/v3/index.json"
44+
AddPackageSource -Sources $sources -SourceName "identitymodel-nightlies" -SourceEndPoint $IdentityModelPackageSource
4445

4546
$doc.Save($filename)

eng/targets/Helix.targets

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@
141141
<ItemGroup Condition="'$(_UseHelixOpenQueues)' != 'true'">
142142
<_Temp Include="@(HelixAvailableTargetQueue)" />
143143
<HelixAvailableTargetQueue Remove="@(HelixAvailableTargetQueue)" />
144-
<HelixAvailableTargetQueue Include="@(_Temp->'%(Identity)'->Replace('.Open',''))" />
144+
<HelixAvailableTargetQueue Include="@(_Temp->'%(Identity)'->Replace('.Open','')->Replace('.open',''))" />
145145
</ItemGroup>
146146

147147
<ItemGroup>

src/ProjectTemplates/test/Templates.Blazor.Tests/Templates.Blazor.Tests.csproj

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
<RunTemplateTests Condition="'$(RunTemplateTests)' == ''">true</RunTemplateTests>
1010
<RunBlazorPlaywrightTemplateTests Condition="'$(RunBlazorPlaywrightTemplateTests)' == ''">$(RunTemplateTests)</RunBlazorPlaywrightTemplateTests>
1111
<SkipTests Condition="'$(RunBlazorPlaywrightTemplateTests)' != 'true'">true</SkipTests>
12+
<BuildHelixPayload Condition="'$(RunBlazorPlaywrightTemplateTests)' != 'true'">false</BuildHelixPayload>
1213

1314
<BaseOutputPath />
1415
<OutputPath />

src/ProjectTemplates/test/Templates.Blazor.WebAssembly.Auth.Tests/Templates.Blazor.WebAssembly.Auth.Tests.csproj

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
<RunTemplateTests Condition="'$(RunTemplateTests)' == ''">true</RunTemplateTests>
1111
<SkipTests Condition="'$(RunTemplateTests)' != 'true'">true</SkipTests>
12+
<BuildHelixPayload Condition="'$(RunTemplateTests)' != 'true'">false</BuildHelixPayload>
1213

1314
<BaseOutputPath />
1415
<OutputPath />

src/ProjectTemplates/test/Templates.Blazor.WebAssembly.Tests/Templates.Blazor.WebAssembly.Tests.csproj

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
<RunTemplateTests Condition="'$(RunTemplateTests)' == ''">true</RunTemplateTests>
1111
<SkipTests Condition="'$(RunTemplateTests)' != 'true'">true</SkipTests>
12+
<BuildHelixPayload Condition="'$(RunTemplateTests)' != 'true'">false</BuildHelixPayload>
1213

1314
<BaseOutputPath />
1415
<OutputPath />

src/ProjectTemplates/test/Templates.Mvc.Tests/Templates.Mvc.Tests.csproj

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99

1010
<RunTemplateTests Condition="'$(RunTemplateTests)' == ''">true</RunTemplateTests>
1111
<SkipTests Condition="'$(RunTemplateTests)' != 'true'">true</SkipTests>
12+
<BuildHelixPayload Condition="'$(RunTemplateTests)' != 'true'">false</BuildHelixPayload>
1213

1314
<BaseOutputPath />
1415
<OutputPath />

0 commit comments

Comments
 (0)