Skip to content

bake: check for empty build network with compose - #2790

Merged
tonistiigi merged 1 commit into
docker:masterfrom
crazy-max:fix-network-attr-yaml
Nov 15, 2024
Merged

bake: check for empty build network with compose#2790
tonistiigi merged 1 commit into
docker:masterfrom
crazy-max:fix-network-attr-yaml

Conversation

@crazy-max

Copy link
Copy Markdown
Member

@crazy-maxcrazy-max changed the title test: bake print compose filebake: check for empty build network with composeNov 14, 2024
@crazy-maxcrazy-max added this to the v0.18.1 milestone Nov 14, 2024
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@crazy-max
crazy-max marked this pull request as ready for review November 14, 2024 20:53

@tonistiigitonistiigi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What case is it fixing? Is the issue that compose parser is reusing some of the pointers it returns?

@crazy-max

Copy link
Copy Markdown
MemberAuthor

What case is it fixing? Is the issue that compose parser is reusing some of the pointers it returns?

This is similar to what we got with hcl in #2689 but this time for compose

@tonistiigi
tonistiigi merged commit 3b943bd into docker:masterNov 15, 2024
@crazy-max
crazy-max deleted the fix-network-attr-yaml branch November 15, 2024 08:27
Sign up for freeto 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.

2 participants

@crazy-max@tonistiigi