Skip to content

Conversation

@HaoK
Copy link
Member

@HaoK HaoK commented May 6, 2020

No description provided.

@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label May 6, 2020
@dougbu
Copy link
Contributor

dougbu commented May 7, 2020

/azp run

@azure-pipelines
Copy link

Pull request contains merge conflicts.

@HaoK
Copy link
Member Author

HaoK commented May 7, 2020

Note: we were running OSX helix queues in both PRs and daily, which is just double running them on rolling builds. Fixing that in this PR too

<ItemGroup Condition="'$(IsWindowsOnlyTest)' != 'true' AND '$(TargetArchitecture)' == 'x64' AND '$(IsHelixDaily)' == 'true'">
<HelixAvailableTargetQueue Include="Windows.81.Amd64.Open" Platform="Windows" />
<HelixAvailableTargetQueue Include="Ubuntu.1804.Amd64.Open" Platform="Linux" />
<HelixAvailableTargetQueue Include="OSX.1014.Amd64.Open" Platform="Linux" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Just curious: Do we have a macOS or OSX platform? Seems less like our Linux platforms every day 😈

Copy link
Member Author

Choose a reason for hiding this comment

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

Platform is really windows or not windows at this point :)

@dougbu
Copy link
Contributor

dougbu commented May 8, 2020

@HaoK are the failures due to flaky tests and not a Win8 specific problem? BTW should close #21421 when this works 😆

@HaoK
Copy link
Member Author

HaoK commented May 8, 2020

That's a good question, I don't even see any test failures in the openapi test logs, just 26 tests passed and 4 skipped :/ https://helix.dot.net/api/2019-06-17/jobs/67e5400f-917e-4779-8e23-ae8f52d8053f/workitems/dotnet-microsoft.openapi.Tests--netcoreapp5.0/files/console.0ff64487.log

@HaoK
Copy link
Member Author

HaoK commented May 8, 2020

Win8 has looked fine, reverting yaml changes

@HaoK HaoK marked this pull request as ready for review May 8, 2020 23:58
@HaoK HaoK requested a review from a team May 9, 2020 03:00
@HaoK HaoK changed the title [Helix] See how win 8 is doing [Helix] Reenable win8 helix queue May 9, 2020
@HaoK HaoK merged commit b062d1b into master May 9, 2020
@HaoK HaoK deleted the helix/win8 branch May 9, 2020 03:34
@HaoK HaoK linked an issue May 9, 2020 that may be closed by this pull request
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.

Restore use of Windows.81.Amd64.Open queue

4 participants