Skip to content

[main] Update Image Builder tag reference#2038

Merged
lbussell merged 3 commits intodotnet:mainfrom
dotnet-docker-bot:docker-tools-main-image-builder-tag
Mar 26, 2026
Merged

[main] Update Image Builder tag reference#2038
lbussell merged 3 commits intodotnet:mainfrom
dotnet-docker-bot:docker-tools-main-image-builder-tag

Conversation

@dotnet-docker-bot
Copy link
Copy Markdown
Contributor

Updates the common pipeline variables to reference an updated tag of Image Builder.

@dotnet-docker-bot dotnet-docker-bot requested a review from a team as a code owner March 19, 2026 18:54
@dotnet-docker-bot dotnet-docker-bot force-pushed the docker-tools-main-image-builder-tag branch 4 times, most recently from 8952c09 to 8235e73 Compare March 24, 2026 19:05
@lbussell lbussell force-pushed the docker-tools-main-image-builder-tag branch from f6192d4 to ec108fa Compare March 25, 2026 16:30
@lbussell lbussell requested a review from mthalman March 25, 2026 16:31
@dotnet-docker-bot dotnet-docker-bot force-pushed the docker-tools-main-image-builder-tag branch from ec108fa to c705260 Compare March 26, 2026 01:27
Add createManifestList step to post-build.yml after mergeImageInfo, with
reference-service-connections.yml for build registry auth.

Remove the publishManifest step from publish.yml - manifest lists are
now
copied via copyAcrImages instead of being recreated.

Fixes: dotnet#2002

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Update changelog with Manifest changes

Update pipelines to use imagebuilder executable name

Update CHANGELOG
@lbussell
Copy link
Copy Markdown
Member

/azp run

@lbussell lbussell closed this Mar 26, 2026
@lbussell lbussell reopened this Mar 26, 2026
@lbussell lbussell merged commit a3fbe4a into dotnet:main Mar 26, 2026
1 check passed
lbussell added a commit that referenced this pull request Mar 30, 2026
The `Post Publish Notification` step in the Publish stage fails with a
`NullReferenceException` on PR builds
([example](https://dev.azure.com/dnceng-public/public/_build/results?buildId=1358504)).

**Root cause:** The `Create Manifest Lists` step in `post-build.yml` is
skipped for PR builds (`ne(variables['Build.Reason'], 'PullRequest')`)
because it requires registry access. Without that step, manifest digests
in `image-info.json` are null. The `postPublishNotification` command
then crashes when it tries to read those digests.

This was introduced by #2030 (moved manifest list creation to
Post_Build) and activated by #2038 (updated the ImageBuilder tag to
include that change).

**Fix:** Add `ne(variables['Build.Reason'], 'PullRequest')` to the `Post
Publish Notification` condition, consistent with how `Create Manifest
Lists` handles PR builds.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

3 participants