[main] Update dependencies from dotnet/arcade

[dotnet-maestro]

This pull request updates the following dependencies

From https://github.com/dotnet/arcade

• Subscription: d1dd3c14-f8e9-46f4-7fb7-08d8446de174
• Build: 20221128.1
• Date Produced: November 28, 2022 10:22:58 AM UTC
• Commit: 3817f2b38a05c37c950715f84c95d452bcacfe52
• Branch: refs/heads/main

• Updates:
  • Microsoft.DotNet.Arcade.Sdk: from 8.0.0-beta.22554.2 to 8.0.0-beta.22578.1
  • Microsoft.DotNet.Helix.Sdk: from 8.0.0-beta.22554.2 to 8.0.0-beta.22578.1

[vzarytovskii]

We're getting a bunch of

error NETSDK1192: Targeting .NET 7.0 or higher in Visual Studio 2022 17.3 is not supported.

On most of our windows CI legs.

@MattGal does this ring a bell? Do we need to update images we're running on or something?

[MattGal]

@MattGal does this ring a bell? Do we need to update images we're running on or something?

Not sure, but I definitely touched the default xcopy msbuild version here so I'll take an initial stab at investigation.

[MattGal]

@vzarytovskii I think this is a little convoluted due to the complexities of your build. I did verify you're just using the msbuild that comes with the VS install, so this isn't an xcopy-msbuild problem. I think this can be fixed by temporarily setting your pipeline variable for the image used in CI to windows.vs2022.scout.amd64.open, but this will be global for all runs so I didn't just set it; if you'd like me to set it and retry I can:

The windows.vs2022.scout.amd64.open image is already at 17.4.0 which should make your error message go away, and you can flip back when that becomes the "main" VS 2022 image, which should be around 12/7/2022.

[vzarytovskii]

Yeah, I think it's fine to set it for now for all runs, we would like to start testing compiler on the 7.0.100

[MattGal]

Yeah, I think it's fine to set it for now for all runs, we would like to start testing compiler on the 7.0.100

Flipped it over and hit retry. Due to the way variables work there's a chance you may need to close and reopen the PR to pick up the change.

[MattGal]

Yeah, I think it's fine to set it for now for all runs, we would like to start testing compiler on the 7.0.100

Flipped it over and hit retry. Due to the way variables work there's a chance you may need to close and reopen the PR to pick up the change.

@vzarytovskii please close/reopen at your leisure, I can't and it didn't pick up the new queue as expected.

[vzarytovskii]

Signed builds are broken, I'm going to revert it

[MattGal]

Signed builds are broken, I'm going to revert it

It's the same as the public build hit. We'll have 17.4 soon, just having trouble with rollouts this morning.

[vzarytovskii]

Signed builds are broken, I'm going to revert it

It's the same as the public build hit. We'll have 17.4 soon, just having trouble with rollouts this morning.

Sure, just wanted to unblock us from inserting. Going to re-do the PR once images rollout