Skip to content

Conversation

@HaoK
Copy link
Member

@HaoK HaoK commented Jan 10, 2022

Try to get iis tests running on arm64 helix queue

Tests are running now but most no-op as IIS/IISExpress are not installed

@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Jan 10, 2022
@@ -1,5 +1,9 @@
# We only want to run full helix matrix on main
pr: none
pr:
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

revert

<!-- aspnetcore-helix-matrix[-internal] (helix-matrix.yml) and RunHelix.ps1 -->
<!-- aspnetcore-quarantined-tests (quarantined-tests.yml) and RunHelix.ps1 -RunQuarantinedTests -->
<ItemGroup>
<!-- Linux -->
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

revert

@HaoK
Copy link
Member Author

HaoK commented Jan 13, 2022

@dotnet/aspnet-build how does this tweak look for our arm64 testing, I've asked core-eng to turn on IIS for arm64 so we'll be starting to run tests (once we get things working), but basically I split out the windows arm64 queue into its own group for the arm64 native build and split it out from the other queues

publishOnError: true

# Helix ARM64
- ${{ if ne(variables['System.TeamProject'], 'internal') }}:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do this only for public runs e.g. are you dealing w/ another missing internal Helix queue❔ aspnetcore-helix-matrix-internal is disabled in any case.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just copied this from the PR that deleted this job and brought it back, perhaps there are changes that were done since, so we can drop the condition now?

-arch x64
-pack
-all
displayName: Build x64/arm64
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need /p:CrossgenOutput=false here too; might need it in the next build step as well

- script: .\eng\build.cmd -ci -arch arm64 -noBuildRepoTasks -NoRestore -test -all
-projects eng\helix\helix.proj /p:IsHelixWindowsArmCheck=true
/p:IsHelixJob=true /p:RunTemplateTests=true
/p:CrossgenOutput=false /p:ASPNETCORE_TEST_LOG_DIR=artifacts/log
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally, be consistent w/ property settings in all three build steps. This line appears multiple times in the other job for example.

@HaoK HaoK closed this Feb 6, 2023
@wtgodbe wtgodbe deleted the haok/helixarm branch November 15, 2023 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants