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

Backport all CI matrix changes from 7.0 to reelase/6.0 #62538

Merged
merged 13 commits into from
Dec 15, 2021

Conversation

aik-jahoda
Copy link
Contributor

@aik-jahoda aik-jahoda commented Dec 8, 2021

Most of the changes are tracked there: #57947 (comment)

There is no product change so impact of the change is only in CI not in product itself.

am11 and others added 12 commits December 8, 2021 16:34
* Drop OSX 10.13 from CI matrix

* Drop OSX 10.14 from PR  CI matrix
* Remove ubuntu 16.04

* Remove another occurence
* CI matrix change: debian+centos+sles

* Adjust the CI matrix based on latest comments
* Rev Alpine WithNode from 3.9 to 3.13
Also,
* update helix queues from 3.12 to 3.13
* add 3.14 helix queues
* update performance job to install cargo from main branch

* Add Alpine 3.13 in rolling CI

* Add missing closing paren
* Add Windows.Server.Core.20H2 to CI matrix

* add windows server 20H2

* Remove 20H2 from PR

* Remove 20H2 from PR run
* Revert "CI matrix update: add Ubuntu 21.04 (#59578)"

This reverts commit 127a498.

* Revert "CI matrix: add openSUSE 15.2 (#59574)"

This reverts commit f9025ae.

* Revert "CI matrix change: add Windows.Server.Core.20H2 (#59572)"

This reverts commit 5ab435e.
@ghost
Copy link

ghost commented Dec 8, 2021

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

Issue Details

Most of the changes are tracked there: #57947 (comment)

Author: aik-jahoda
Assignees: -
Labels:

area-Infrastructure-mono

Milestone: -

@aik-jahoda
Copy link
Contributor Author

Copy link
Member

@danmoseley danmoseley left a comment

Choose a reason for hiding this comment

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

I can't review detail of YAML changes, but I approve the change.

@danmoseley danmoseley requested a review from safern December 9, 2021 16:49
@danmoseley
Copy link
Member

No tactics approval needed.

@@ -0,0 +1,59 @@
trigger: none
Copy link
Member

Choose a reason for hiding this comment

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

This template doesn't make sense to port it as it doesn't even have triggers for release/* branches.

Copy link
Member

@safern safern left a comment

Choose a reason for hiding this comment

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

Other than one comment, looks good.

@aik-jahoda
Copy link
Contributor Author

@safern

Other than one comment, looks good.

I will revert whole 4199aa1 from the backport.

@aik-jahoda
Copy link
Contributor Author

aik-jahoda commented Dec 14, 2021

Passing tests: https://dev.azure.com/dnceng/public/_build/results?buildId=1510275&view=results

There is one unrelated failure.

cc @safern

@aik-jahoda aik-jahoda merged commit 309879d into release/6.0 Dec 15, 2021
@aik-jahoda aik-jahoda deleted the jajahoda/cimatrix6.0 branch December 15, 2021 17:26
@ghost ghost locked as resolved and limited conversation to collaborators Jan 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants