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

Outerloop stress legs triggering on PRs they shouldn't #69829

Closed
jkoritzinsky opened this issue May 25, 2022 · 11 comments
Closed

Outerloop stress legs triggering on PRs they shouldn't #69829

jkoritzinsky opened this issue May 25, 2022 · 11 comments

Comments

@jkoritzinsky
Copy link
Member

The GCStress, JITStress, and outerloop legs look to be triggering on PRs they shouldn't.

#69825

cc: @dotnet/runtime-infrastructure if there were any changes we made. @dotnet/dnceng if this is an AzDO issue we need to report.

@ghost ghost added the untriaged New issue has not been triaged by the area owner label May 25, 2022
@ghost
Copy link

ghost commented May 25, 2022

Tagging subscribers to this area: @dotnet/runtime-infrastructure
See info in area-owners.md if you want to be subscribed.

Issue Details

The GCStress, JITStress, and outerloop legs look to be triggering on PRs they shouldn't.

#69825

cc: @dotnet/runtime-infrastructure if there were any changes we made. @dotnet/dnceng if this is an AzDO issue we need to report.

Author: jkoritzinsky
Assignees: -
Labels:

area-Infrastructure

Milestone: -

@elinor-fung
Copy link
Member

5 hours ago (#69808) was fine.
4 hours ago (#69813) was not.

@jkoritzinsky
Copy link
Member Author

cc:@dotnet/dncenghot as this looks like it's an AzDO issue

@garath
Copy link
Member

garath commented May 25, 2022

as this looks like it's an AzDO issue

Acknowledged, checking.

@MattGal
Copy link
Member

MattGal commented May 25, 2022

Also tagging @agocke who touched this pipeline about 5 hours ago (
image
)

@agocke
Copy link
Member

agocke commented May 25, 2022

Oops, I was following security guidance, and I think I changed the PR qualifications. I'll move them back

@ericstj
Copy link
Member

ericstj commented May 25, 2022

It looks like there were some more pipelines impacted: runtime-coreclr jitstress2-jitstressregs, runtime-coreclr jitstress, runtime-coreclr jitstressregs, runtime-coreclr gcstress0x3-gcstress0xc.

I've fixed all of these in the same way @agocke did to outerloop.

@ericstj
Copy link
Member

ericstj commented May 26, 2022

@agocke should we cancel the builds here so that we don't waste resources? Some of these queues are now pretty deep with work that might be unnecessary.

@ericstj
Copy link
Member

ericstj commented May 26, 2022

Just dropping a note here for folks who find this issue. If you see any of the mentioned builds running in your PR between the times of 10:50 AM PDT - 5:00 PM PDT you can ignore them. They are not required to pass.

@agocke
Copy link
Member

agocke commented May 26, 2022

This should be fixed now.

@agocke agocke closed this as completed May 26, 2022
@ghost ghost removed the untriaged New issue has not been triaged by the area owner label May 26, 2022
@agocke
Copy link
Member

agocke commented May 26, 2022

I've canceled a bunch of runs -- feel free to cancel more if things are getting overwhelming

@ghost ghost locked as resolved and limited conversation to collaborators Jun 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants