Skip to content

Conversation

@cpuguy83
Copy link
Contributor

@cpuguy83 cpuguy83 commented Dec 7, 2020

This allows builders to opt into determnistic output regardless of
multi-platform output or not.

This resolves the real intention behind 158; deterministic output.

Closes #158

@cpuguy83 cpuguy83 force-pushed the deterministic_output branch from 993267b to c823fcb Compare December 8, 2020 00:55
@tonistiigi
Copy link
Member

if you want to set the frontend opt you should do it directly. multiDriver has a different meaning where build is in multiple nodes and a merge needs to happen from buildx cli.

@tonistiigi tonistiigi changed the title Get multi-platform buildkit frontned opt from args Get multi-platform buildkit frontend opt from args Dec 8, 2020
@tonistiigi
Copy link
Member

nit while you fix the rest: typo in commit title

Copy link
Member

@tonistiigi tonistiigi left a comment

Choose a reason for hiding this comment

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

^

This allows builders to opt into determnistic output regardless of
multi-platform output or not.

Signed-off-by: Brian Goff <[email protected]>
@cpuguy83 cpuguy83 force-pushed the deterministic_output branch from c823fcb to 7f58ad4 Compare December 8, 2020 21:19
@tonistiigi tonistiigi merged commit ea1a71d into docker:master Dec 8, 2020
@cpuguy83 cpuguy83 deleted the deterministic_output branch December 10, 2020 23:27
@tonistiigi
Copy link
Member

@cpuguy83 I think we should add this into the Dockerfile frontend as well. Otherwise, buildx behavior is inconsistent from other tools using BuildKit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants