Skip to content

Reduce 6.0 release branch CI runs (6.0 preview 1) - #49168

Merged
hoyosjs merged 1 commit into
dotnet:release/6.0-preview1from
BruceForstall:StopJitStressPipelines60Preview1
Mar 5, 2021
Merged

Reduce 6.0 release branch CI runs (6.0 preview 1)#49168
hoyosjs merged 1 commit into
dotnet:release/6.0-preview1from
BruceForstall:StopJitStressPipelines60Preview1

Conversation

@BruceForstall

@BruceForstallBruceForstall commented Mar 4, 2021

Copy link
Copy Markdown
Contributor

6.0 release branches are running AzDO stress pipelines on a schedule, even though (normally) nothing is changing. Change this to only run if something changes.

This is a port of #49020 to the release branch.

@BruceForstall
Increase jitstress job run frequency
a3e767f
Change dotnet#41856 reduced stress job frequency to once per day, alternating
between main and the release branches. However, the release branches
get few changes yet still run the job. (Also, all the release branches
run the job at the same time.)
Change to running the job daily in main, where most development
happens. Also change the runs to daily in the release branches, but
change to only run if the branch has changed since the last run.
@ghostghost added the area-Infrastructure-coreclr Only use for closed issues label Mar 4, 2021
@ghost

ghost commented Mar 4, 2021

Copy link
Copy Markdown

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

Issue Details

6.0 release branches are running AzDO stress pipelines on a schedule, even though (normally) nothing is changing. Change this to only run if something changes.

This is a port of #49020 to the release branch.

Author:BruceForstall
Assignees:-
Labels:

area-Infrastructure-coreclr

Milestone:-

hoyosjs
hoyosjs approved these changes Mar 4, 2021
View reviewed changes

@hoyosjshoyosjs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Same comment as in the other PR, but thanks!

trylek
trylek approved these changes Mar 4, 2021
View reviewed changes

@trylektrylek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks awesome, thanks for following up!

@BruceForstallBruceForstall changed the title Reduce 6.0 release branch CI runsReduce 6.0 release branch CI runs (6.0 preview 1)Mar 5, 2021
@BruceForstall

BruceForstall commented Mar 5, 2021

Copy link
Copy Markdown
ContributorAuthor

This is a protected branch. Who has permission to merge to preview1 (and preview2) branches?

@BruceForstall

BruceForstall commented Mar 5, 2021

Copy link
Copy Markdown
ContributorAuthor

This is a protected branch. Who has permission to merge to preview1 (and preview2) branches?

@hoyosjs
hoyosjs merged commit 1ef90dd into dotnet:release/6.0-preview1Mar 5, 2021
@hoyosjs

hoyosjs commented Mar 5, 2021

Copy link
Copy Markdown
Member

I've cancelled the CI official build too.

@BruceForstall

BruceForstall commented Mar 6, 2021

Copy link
Copy Markdown
ContributorAuthor

Thanks @hoyosjs !

@BruceForstall
BruceForstall deleted the StopJitStressPipelines60Preview1 branch March 6, 2021 00:19
@BruceForstallBruceForstall mentioned this pull request Mar 8, 2021
Merged
@ghostghost locked as resolved and limited conversation to collaborators Apr 5, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@ViktorHoferViktorHoferViktorHofer approved these changes
@trylektrylektrylek approved these changes
@hoyosjshoyosjshoyosjs approved these changes
@safernsafernAwaiting requested review from safern

Assignees

No one assigned

Labels

area-Infrastructure-coreclrOnly use for closed issues

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@BruceForstall@hoyosjs@ViktorHofer@trylek