Finalize records before exporting

follow-up https://github.com/docker/actions-toolkit/pull/652#discussion_r2055080629

We should make sure records are finalized before exporting them similar to `history trace`: https://github.com/docker/buildx/blob/9ba3f77219b15cff4cea37262d270083dec334ad/commands/history/trace.go#L57-L70

@tonistiigi I wonder if we should have a `Finalize` attribute in https://github.com/docker/buildx/blob/9ba3f77219b15cff4cea37262d270083dec334ad/commands/history/utils.go#L116-L119

To share logic for both `export` and `trace` in `queryRecords`?

	// build is complete but no trace yet. try to finalize the trace
	time.Sleep(1 * time.Second) // give some extra time for last parts of trace to be written

	c, err := rec.node.Driver.Client(ctx)
	if err != nil {
	return "", nil, err
	}
	_, err = c.ControlClient().UpdateBuildHistory(ctx, &controlapi.UpdateBuildHistoryRequest{
	Ref: rec.Ref,
	Finalize: true,
	})
	if err != nil {
	return "", nil, err
	}

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Finalize records before exporting #3149

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

	type queryOptions struct {
	CompletedOnly bool
	Filters []string
	}

Finalize records before exporting #3149

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions