Skip to content

Conversation

developer-guy and others added 10 commits January 26, 2023 13:32
Signed-off-by: Batuhan Apaydın <[email protected]>
(cherry picked from commit edb16f8)
Signed-off-by: CrazyMax <[email protected]>
(cherry picked from commit 523a16a)
Signed-off-by: David Karlsson <[email protected]>
(cherry picked from commit ba8fa6c)
So it doesn't show up in the History API.

Signed-off-by: Tonis Tiigi <[email protected]>
(cherry picked from commit be55b41)
This change has been introduced in e7b5ee7
but we should not check context builders when removing inactive
ones.

Signed-off-by: CrazyMax <[email protected]>
(cherry picked from commit 6cd0c11)
Signed-off-by: Justin Chadwell <[email protected]>
(cherry picked from commit 7145e02)
Signed-off-by: Justin Chadwell <[email protected]>
(cherry picked from commit 347417e)
Signed-off-by: Justin Chadwell <[email protected]>
(cherry picked from commit 1180d91)
Using the syncronization primitive, we can avoid needing to create a
separate wait group.

This allows us to sidestep the issue where the wait group could be
completed, but the build invocation functions had not terminated - if
one of the functions was to terminate with an error, then it was
possible to encounter a race condition, where the result handling code
would begin executing, despite an error.

The refactor to use a separate error group which more elegantly handles
the concept of function returns and errors, ensures that we can't
encounter this issue.

Signed-off-by: Justin Chadwell <[email protected]>
(cherry picked from commit 8b7aa1a)
Signed-off-by: CrazyMax <[email protected]>
(cherry picked from commit c1058c1)
@crazy-max
Copy link
Member Author

@jedevc @tonistiigi When BuildKit 0.11.2 is released, will open a PR to vendor on Buildx and cherry-pick here.

Signed-off-by: Tonis Tiigi <[email protected]>
(cherry picked from commit f6da7ee)
@tonistiigi
Copy link
Member

added #1559

@tonistiigi tonistiigi marked this pull request as ready for review January 26, 2023 18:35
@tonistiigi tonistiigi merged commit 481796f into docker:v0.10 Jan 26, 2023
@crazy-max crazy-max deleted the 0.10.1_cherry_picks branch February 8, 2023 13:27
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.

5 participants