Skip to content

push also consider build.tags#11201

Merged
ndeloof merged 1 commit intodocker:mainfrom
ndeloof:push_tags
Nov 17, 2023
Merged

push also consider build.tags#11201
ndeloof merged 1 commit intodocker:mainfrom
ndeloof:push_tags

Conversation

@ndeloof
Copy link
Copy Markdown
Contributor

@ndeloof ndeloof commented Nov 17, 2023

What I did

introduce docker compose push --all to also push tags declared by the build section

Related issue
closes #11175

(not mandatory) A picture of a cute animal, if possible in relation to what you did

Signed-off-by: Nicolas De Loof <[email protected]>
@ndeloof ndeloof requested review from a team, StefanScherer, glours, laurazard, milas, nicksieger and ulyssessouza and removed request for a team November 17, 2023 15:19
Copy link
Copy Markdown
Contributor

@glours glours left a comment

Choose a reason for hiding this comment

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

🥳

@ndeloof ndeloof merged commit cb01186 into docker:main Nov 17, 2023
@ndeloof ndeloof deleted the push_tags branch November 17, 2023 15:32
@designermonkey
Copy link
Copy Markdown

I'm coming back to this from the future as I am having the same issue described in the originating issue. This added functionality doesn't appear in compose anymore, and I'm wondering why it was removed?

Docker Desktop: 4.43.2 (199162)
Engine: 28.3.2
Compose: v2.38.2-desktop.1
Credential Helper: v0.9.3
Kubernetes: v1.32.2

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.

add an ability to push all tags specified in compose.yaml

3 participants