Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the build infrastructure to ensure .NET 8.0 is installed when using the Darc (Maestro) tooling for publishing build assets. The changes include adding runtime source feed configuration and updating build agent images to newer versions.
- Adds explicit .NET 8.0 installation before Darc operations to ensure compatibility
- Configures internal runtime source feeds with authentication for Darc and SDK tasks
- Updates validation job images from VS2022 to VS2026 preview
- Reformats YAML indentation for consistency (parameters and steps sections)
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| eng/common/core-templates/post-build/post-build.yml | Reformats parameters indentation, updates validation job images to VS2026 preview, adds .NET 8.0 installation and internal runtime configuration for Darc publishing stage |
| eng/common/core-templates/job/publish-build-assets.yml | Adds internal source/runtime configuration templates, installs .NET 8.0 before Darc operations, configures runtime source feeds for SDK tasks |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
@premun these changes will get overwritten with the next VMR re-bootstrap and won't flow anywhere. If you want to make the eng/common changes so that they flow to all repos, either make them in arcade or in src/arcade here.
|
@ViktorHofer I wanted to temporarily override this to fix publishing. I would not care about the overwrites. But we rolled back Maestro instead in the end. |
|
Okay, we will have another go at unpinning so this needs to be in (before we rebootstrap which will bring in these files anyway). |
This reverts commit 5d44331.
Copies of latest
eng/commonfiles from Arcade to allow us to unpin darc.