Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

as --no-build is set, set service image to default image name #9179

Merged
merged 1 commit into from Jul 29, 2022

Conversation

ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Feb 16, 2022

What I did
when --no-build is set, enforce service image to have a value, using default image name logic.

Related issue
close #8785

Copy link
Member

@milas milas left a comment

Choose a reason for hiding this comment

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

I think the exported function should be renamed as it's a bit misleading right now, but otherwise LGTM

pkg/api/api.go Show resolved Hide resolved
pkg/api/api.go Outdated Show resolved Hide resolved
Signed-off-by: Nicolas De Loof <nicolas.deloof@gmail.com>
Signed-off-by: Guillaume Lours <guillaume.lours@docker.com>
Copy link
Member

@milas milas left a comment

Choose a reason for hiding this comment

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

🚀

@glours glours enabled auto-merge July 29, 2022 15:47
@glours glours merged commit b49bd7c into docker:v2 Jul 29, 2022
@ndeloof ndeloof deleted the no_build_image branch December 20, 2023 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't use --no-build with services with a build section
3 participants