Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skip tests on internal queues too #59574

Merged
merged 2 commits into from
Dec 20, 2024
Merged

Skip tests on internal queues too #59574

merged 2 commits into from
Dec 20, 2024

Conversation

wtgodbe
Copy link
Member

@wtgodbe wtgodbe commented Dec 19, 2024

We run tests on internal queues sometimes (internal PRs, IdentityModel pipeline) - anywhere we skip a test on a specific public queue, we should skip it on the internal queue too.

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Dec 19, 2024
@BrennanConroy
Copy link
Member

Seems like it'd be easier and more future proof if we instead changed the SkipOnHelixAttribute check to ignore .Open when comparing.

@wtgodbe
Copy link
Member Author

wtgodbe commented Dec 19, 2024

Seems like it'd be easier and more future proof if we instead changed the SkipOnHelixAttribute check to ignore .Open when comparing.

Yeah I just got there too, testing that w/ https://dev.azure.com/dnceng/internal/_git/dotnet-aspnetcore/pullrequest/46296

@wtgodbe wtgodbe force-pushed the wtgodbe/SkipTestsOpen branch from 6e1e619 to b50f4bb Compare December 19, 2024 23:51
@wtgodbe
Copy link
Member Author

wtgodbe commented Dec 19, 2024

Alright, this should be a more general-purpose solution now

@wtgodbe
Copy link
Member Author

wtgodbe commented Dec 20, 2024

/backport to release/9.0

@wtgodbe
Copy link
Member Author

wtgodbe commented Dec 20, 2024

/backport to release/8.0

Copy link
Contributor

Started backporting to release/9.0: https://github.com/dotnet/aspnetcore/actions/runs/12426551923

Copy link
Contributor

Started backporting to release/8.0: https://github.com/dotnet/aspnetcore/actions/runs/12426552541

@wtgodbe wtgodbe merged commit 794124f into main Dec 20, 2024
27 checks passed
@wtgodbe wtgodbe deleted the wtgodbe/SkipTestsOpen branch December 20, 2024 16:30
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0-preview1 milestone Dec 20, 2024
ellahathaway pushed a commit to ellahathaway/aspnetcore that referenced this pull request Dec 20, 2024
* Skip tests on internal too

* StringComparison
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.

2 participants