Skip to content

[release/13.0] Support a docker build arg to specify the source image in PublishWithContainerFiles - #12738

Merged
Jose Perez Rodriguez (joperezr) merged 2 commits into
release/13.0from
backport/pr-12716-to-release/13.0
Nov 6, 2025
Merged

[release/13.0] Support a docker build arg to specify the source image in PublishWithContainerFiles#12738
Jose Perez Rodriguez (joperezr) merged 2 commits into
release/13.0from
backport/pr-12716-to-release/13.0

Conversation

@github-actions

@github-actions github-actions Bot commented Nov 6, 2025

Copy link
Copy Markdown
Contributor

Backport of #12716 to release/13.0

/cc Eric Erhardt (@eerhardt)

Customer Impact

The new python starter template doesn't work with azd because we don't give azd a chance to specify the referenced image tag when we build the dockerfiles when using .PublishWithContainerFiles. Adding a docker build arg allows azd to build the images the same way we do with aspire deploy.

Testing

Existing automated tests. Plus tested the py-starter template still builds a docker image correctly.

Risk

A little bit of risk that we aren't building the dockerfiles correctly now that we generate an argument, but it is low.

Regression?

No

…ContainerFiles

This allows an external system to provide the image name for the source image to copy the files from instead of always hard coding the image tag inline.
@github-actions

github-actions Bot commented Nov 6, 2025

Copy link
Copy Markdown
Contributor Author

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 12738

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 12738"

@eerhardt Eric Erhardt (eerhardt) added the Servicing-consider Issue for next servicing release review label Nov 6, 2025
@joperezr Jose Perez Rodriguez (joperezr) added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Nov 6, 2025
@joperezr
Jose Perez Rodriguez (joperezr) merged commit 396ce67 into release/13.0 Nov 6, 2025
296 checks passed
@joperezr
Jose Perez Rodriguez (joperezr) deleted the backport/pr-12716-to-release/13.0 branch November 6, 2025 18:26
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants