Skip to content

Conversation

@dotnet-docker-bot
Copy link
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 May 16, 2025 05:18
@dotnet-docker-bot dotnet-docker-bot force-pushed the docker-tools-main-image-builder-tag branch from d2c6393 to 2bf0383 Compare May 28, 2025 01:24
@dotnet-docker-bot dotnet-docker-bot force-pushed the docker-tools-main-image-builder-tag branch from 2bf0383 to 7b245f7 Compare May 28, 2025 17:56
@lbussell lbussell requested a review from mthalman May 28, 2025 19:22
@lbussell lbussell force-pushed the docker-tools-main-image-builder-tag branch from 24cd703 to 7b1d3fe Compare May 28, 2025 19:29
Comment on lines +62 to 66
'-e SYSTEM_ACCESSTOKEN=$(System.AccessToken)'
'-e SYSTEM_OIDCREQUESTURI=$(System.OidcRequestUri)'
)
$dockerRunCmd = $dockerRunBaseCmd + $dockerRunArgs
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think some or all of this step (Define ImageBuilder Command Variables) could now be embedded directly into run-imagebuilder.yml.

@lbussell
Copy link
Member

@mthalman I've added the pipeline changes for #1712.

@lbussell
Copy link
Member

/cc @dagood @gdams more breaking changes coming down the line. We have changed the way auth works in ImageBuilder. You will need a few more values for each service connection - Service Connection ID, Tenant ID, and Client ID. You can find all of these in the Service Connection's "edit" menu. Format TBD - you will either need to specify these via pipeline template parameter or more pipeline variables.

@lbussell lbussell merged commit 052f71e into dotnet:main May 29, 2025
11 checks passed
lbussell added a commit to lbussell/docker-tools that referenced this pull request Jun 9, 2025
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