Skip to content

Fix Linux CI registry authentication before network isolation - #617

Merged
Wiktor Kopec (wiktork) merged 2 commits into
mainfrom
wiktork/fix-linux-ci-acr-auth
Oct 23, 2026
Merged

Wiktor Kopec (wiktork) merged 2 commits into
mainfrom
wiktork/fix-linux-ci-acr-auth

Conversation

@wiktork

Copy link
Copy Markdown
Member
Summary

Fix the Linux CI failures in run 2026101601 by authenticating to the build-image registry before 1ES network isolation starts. The existing Docker Login user step ran after isolation blocked registry access, preventing all four Linux builds from starting.

Change Log
  • Configure templateContext.authenticatedContainerRegistries for 1ES Linux binary jobs so the supported template hook authenticates before network isolation.
  • Declare the Linux steps template's OneESPT parameter and retain the explicit Azure CLI login only for non-1ES pipelines, avoiding duplicate authentication.
  • Document the authentication ordering requirement. Network isolation and existing build behavior remain unchanged.
Test Methodology
  • Passed: ClrInstrumentationEngine-CI-Yaml run 2026102201 completed with result Succeeded on branch wiktork/fix-linux-ci-acr-auth, commit e92e644f217223682caec7f9f981c6880db334ad. This exercises Linux and Windows builds, tests, and packaging.
  • Passed: git diff --check origin/main...HEAD.
  • Passed: the following command parsed all 51 pipeline YAML files:
    python -c "import pathlib, yaml; files = list(pathlib.Path('build/yaml').rglob('*.yaml')); [yaml.safe_load(p.read_text(encoding='utf-8-sig')) for p in files]; print(f'Parsed {len(files)} pipeline YAML files successfully')"
  • No local compilation was run for this pipeline-only change; compilation and tests were performed by the successful CI run.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: c9884e9b-9dae-467a-899f-e09dd9f6bd1c
@wiktork
Wiktor Kopec (wiktork) requested a review from a team as a code owner October 23, 2026 16:46
@wiktork
Wiktor Kopec (wiktork) merged commit 92b54b6 into main Oct 23, 2026
8 checks passed
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.

2 participants