Skip to content

Remove unused DOCKER_REPO variable causing pipeline failures#1495

Merged
mthalman merged 2 commits intomainfrom
copilot/fix-1494
Aug 28, 2025
Merged

Remove unused DOCKER_REPO variable causing pipeline failures#1495
mthalman merged 2 commits intomainfrom
copilot/fix-1494

Conversation

Copy link
Contributor

Copilot AI commented Aug 28, 2025

The recent infrastructure update from PR #1490 introduced a pipeline failure where the stagingRepoPrefix variable was referenced but not defined. The error occurred in the Define ImageBuilder Command Variables step:

stagingRepoPrefix: /mnt/vss/_work/_temp/1a374083-69b0-4f7c-a0ae-80ebc6427d5c.ps1:14
Line |
  14 |  … DOCKER_REPO=msdotnetcontainers.azurecr.io/$(stagingRepoPrefix)dotnet- …
     |                                                ~~~~~~~~~~~~~~~~~
     | The term 'stagingRepoPrefix' is not recognized as a name of a cmdlet,
     | function, script file, or executable program.

The issue was caused by the DOCKER_REPO environment variable in eng/pipelines/variables/common.yml referencing $(stagingRepoPrefix) as a variable, but this value was only defined as a parameter in publishing config templates with no corresponding variable definition.

After analysis, it was determined that the DOCKER_REPO environment variable is not used anywhere else in the codebase and can be safely removed. This change removes the problematic variable definition entirely, resolving the pipeline failure with minimal impact.

Changes made:

  • Removed the unused build.imageBuilderDockerRunExtraOptions variable that set DOCKER_REPO
  • Confirmed no other references to DOCKER_REPO exist in the repository
  • Validated YAML syntax remains correct

Fixes #1494.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copy link
Contributor Author

Copilot AI commented Aug 28, 2025

@lbussell 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Co-authored-by: lbussell <36081148+lbussell@users.noreply.github.com>
Copilot AI changed the title [WIP] Pipeline failures after infra update Remove unused DOCKER_REPO variable causing pipeline failures Aug 28, 2025
Copilot AI requested a review from lbussell August 28, 2025 17:22
@lbussell lbussell marked this pull request as ready for review August 28, 2025 17:45
@lbussell lbussell requested a review from a team as a code owner August 28, 2025 17:45
@lbussell lbussell requested a review from mthalman August 28, 2025 17:45
@mthalman mthalman merged commit 505420f into main Aug 28, 2025
94 checks passed
@mthalman mthalman deleted the copilot/fix-1494 branch August 28, 2025 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pipeline failures after infra update

3 participants