feat: add hidden dumb-llb flag (#225)

without this, you have to manually enable the bool to get a dumped llb.

I don't think it makes sense to enable this as a public flag at any point soon (or ever)
but for development having this available as a CLI flag will make life easier and
making it a hidden flag is more self-documenting to developers working with the project.
This commit is contained in:
Michael Bianco
2025-08-20 11:30:11 -06:00
committed by GitHub
parent 5688136fb8
commit 549afca2bb
+6 -2
View File
@@ -47,6 +47,11 @@ var BuildCommand = &cli.Command{
Name: "cache-key",
Usage: "Unique id to prefix to cache keys",
},
&cli.BoolFlag{
Name: "dump-llb",
Hidden: true,
Value: false,
},
}, commonPlanFlags()...),
Action: func(ctx context.Context, cmd *cli.Command) error {
buildResult, app, env, err := GenerateBuildResultForCommand(cmd)
@@ -71,7 +76,6 @@ var BuildCommand = &cli.Command{
if err != nil {
return cli.Exit(err, 1)
}
fmt.Println(serializedPlan)
}
@@ -89,7 +93,7 @@ var BuildCommand = &cli.Command{
err = buildkit.BuildWithBuildkitClient(app.Source, buildResult.Plan, buildkit.BuildWithBuildkitClientOptions{
ImageName: cmd.String("name"),
DumpLLB: cmd.Bool("llb"),
DumpLLB: cmd.Bool("dump-llb"),
OutputDir: cmd.String("output"),
ProgressMode: cmd.String("progress"),
CacheKey: cmd.String("cache-key"),