[rainersigwald]

Fix the official build by removing this group that should no longer be needed after dotnet/arcade#17245. Similar to dotnet/arcade#17259.

Exp job was an easy test and got past the failure point: https://dev.azure.com/devdiv/DevDiv/_build/results?buildId=14979069

[Copilot]

Pull request overview

This PR removes references to the AzureDevOps-Artifact-Feeds-Pats variable group from MSBuild’s Azure Pipelines YAML definitions to unblock official builds after upstream Arcade changes removed/obsoleted that secret group.

Changes:

• Removed the AzureDevOps-Artifact-Feeds-Pats variable group from the exp perf pipeline variables.
• Removed the AzureDevOps-Artifact-Feeds-Pats variable group from the official .vsts-dotnet.yml variables.
• Removed the conditional AzureDevOps-Artifact-Feeds-Pats variable group import from the CI pipeline.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
azure-pipelines/.vsts-dotnet-exp-perf.yml	Drops the stale secret variable group from the exp perf pipeline variables list.
.vsts-dotnet.yml	Drops the stale secret variable group from the official pipeline variables list.
.vsts-dotnet-ci.yml	Removes the DevDiv-only variable-group import for the stale secret group.

---

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

[github-actions]

✅ 24/24 dimensions clean — no findings.

This removes an obsolete variable group AzureDevOps-Artifact-Feeds-Pats from the ExpPerf pipeline. The full file was checked — no remaining pipeline steps reference variables from this group, so the removal is safe.

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• patchdiff.githubusercontent.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "patchdiff.githubusercontent.com"

See Network Configuration for more information.

Generated by Expert Code Review (on open) for #14710 · sonnet46 · 29.2 AIC · ⌖ 7.46 AIC · ⊞ 5.3K

[ViktorHofer]

/backport to vs16.11

[github-actions]

Started backporting to vs16.11 (link to workflow run)

[github-actions]

@ViktorHofer backporting to vs16.11 failed, the patch most likely resulted in conflicts. Please backport manually!

git am output

$ git cherry-pick b6d968a085d1010fd04b7a65d3adb1684d297153

Auto-merging .vsts-dotnet-ci.yml
CONFLICT (content): Merge conflict in .vsts-dotnet-ci.yml
Auto-merging .vsts-dotnet.yml
CONFLICT (content): Merge conflict in .vsts-dotnet.yml
Auto-merging azure-pipelines/.vsts-dotnet-exp-perf.yml
error: could not apply b6d968a08... Drop stale secret group (#14710)
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".
hint: Disable this message with "git config set advice.mergeConflict false"


$ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch

Applying: Remove obsolete artifact feeds variable group
Using index info to reconstruct a base tree...
M	.vsts-dotnet-ci.yml
M	.vsts-dotnet.yml
Falling back to patching base and 3-way merge...
Auto-merging .vsts-dotnet-ci.yml
CONFLICT (content): Merge conflict in .vsts-dotnet-ci.yml
Auto-merging .vsts-dotnet.yml
CONFLICT (content): Merge conflict in .vsts-dotnet.yml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Remove obsolete artifact feeds variable group
Error: The process '/usr/bin/git' failed with exit code 128

Link to workflow output

[ViktorHofer]

/backport to vs17.8

[github-actions]

Started backporting to vs17.8 (link to workflow run)

[github-actions]

@ViktorHofer backporting to vs17.8 failed, the patch most likely resulted in conflicts. Please backport manually!

git am output

$ git cherry-pick b6d968a085d1010fd04b7a65d3adb1684d297153

Auto-merging .vsts-dotnet-ci.yml
CONFLICT (content): Merge conflict in .vsts-dotnet-ci.yml
Auto-merging .vsts-dotnet.yml
CONFLICT (content): Merge conflict in .vsts-dotnet.yml
Auto-merging azure-pipelines/.vsts-dotnet-exp-perf.yml
error: could not apply b6d968a08... Drop stale secret group (#14710)
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".
hint: Disable this message with "git config set advice.mergeConflict false"


$ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch

Applying: Remove obsolete artifact feeds variable group
Using index info to reconstruct a base tree...
M	.vsts-dotnet-ci.yml
M	.vsts-dotnet.yml
Falling back to patching base and 3-way merge...
Auto-merging .vsts-dotnet-ci.yml
CONFLICT (content): Merge conflict in .vsts-dotnet-ci.yml
Auto-merging .vsts-dotnet.yml
CONFLICT (content): Merge conflict in .vsts-dotnet.yml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Remove obsolete artifact feeds variable group
Error: The process '/usr/bin/git' failed with exit code 128

Link to workflow output