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

CI matrix change: add Windows.Server.Core.20H2 - Second attempt #60054

Merged
merged 3 commits into from
Nov 29, 2021

Conversation

aik-jahoda
Copy link
Contributor

@aik-jahoda aik-jahoda commented Oct 6, 2021

The original CI matrix change was reverted (#60040) because of a docker issue:

Unable to pull image mcr.microsoft.com/dotnet-buildtools/prereqs:windowsservercore-20H2-helix-amd64-20210922171304-edc3733
Exit Code:-4

This is attempt to investigate and re-revert it back.

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

part of #59572

* Add Windows.Server.Core.20H2 to CI matrix

* add windows server 20H2

* Remove 20H2 from PR

* Remove 20H2 from PR run
@ghost
Copy link

ghost commented Oct 6, 2021

Tagging subscribers to this area: @Anipik, @safern, @ViktorHofer
See info in area-owners.md if you want to be subscribed.

Issue Details

The original CI matrix change was reverted (#60040) because of a docker issue:

Exit Code:-4  ```

This is attempt to investigate and re-revert it back.

<table>
  <tr>
    <th align="left">Author:</th>
    <td>aik-jahoda</td>
  </tr>
  <tr>
    <th align="left">Assignees:</th>
    <td>-</td>
  </tr>
  <tr>
    <th align="left">Labels:</th>
    <td>

`area-Infrastructure-libraries`

</td>
  </tr>
  <tr>
    <th align="left">Milestone:</th>
    <td>-</td>
  </tr>
</table>
</details>

@aik-jahoda
Copy link
Contributor Author

@krwq , I try fix the docker issue. We all all green, can we trigger rolling builds on this PR?

@krwq
Copy link
Member

krwq commented Oct 6, 2021

you will need to push your branch to dotnet, i.e.:
git push origin cimatrix20h2:cimatrix20h2 (what's before : is your local branch name, what's after : is remote branch name you want to have or already have) - make sure to not add space before or after : as that might delete the branch
then go to https://dev.azure.com/dnceng/public/_build?definitionId=686&_a=summary and click Run pipeline and choose your branch name and click Run - please share the link for your build in the PR description once it's started

@aik-jahoda
Copy link
Contributor Author

Latest commit is passing in runtime CI: febb494

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.

if it works..

@ericstj
Copy link
Member

ericstj commented Dec 2, 2021

This PR introduced regressions #62205 and is being reverted in #62253

agocke added a commit that referenced this pull request Dec 2, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Jan 1, 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.

4 participants