Skip to content

Skip tests on internal queues too - #59574

Merged
wtgodbe merged 2 commits into
mainfrom
wtgodbe/SkipTestsOpen
Dec 20, 2024
Merged

Skip tests on internal queues too#59574
wtgodbe merged 2 commits into
mainfrom
wtgodbe/SkipTestsOpen

Conversation

@wtgodbe

@wtgodbe wtgodbe commented Dec 19, 2024

Copy link
Copy Markdown
Member

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.

@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Dec 19, 2024
@BrennanConroy

Copy link
Copy Markdown
Member

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

@wtgodbe

wtgodbe commented Dec 19, 2024

Copy link
Copy Markdown
Member Author

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

wtgodbe commented Dec 19, 2024

Copy link
Copy Markdown
Member Author

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

@wtgodbe

wtgodbe commented Dec 20, 2024

Copy link
Copy Markdown
Member Author

/backport to release/9.0

@wtgodbe

wtgodbe commented Dec 20, 2024

Copy link
Copy Markdown
Member Author

/backport to release/8.0

@github-actions

Copy link
Copy Markdown
Contributor

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

@github-actions

Copy link
Copy Markdown
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
@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
captainsafia pushed a commit that referenced this pull request Feb 11, 2025
* 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