Skip to content

Commit

Permalink
Fix typo for CorrelatedInvocationsID
Browse files Browse the repository at this point in the history
This cl make correction to the identifier's CorrelatedInvocationsID field name to make it match the proto. It was CorrelatedInvocationID before.

We need to have CorrelatedInvocationsID in the ToProto method to deduplicate builds that have the same invocation id.
  • Loading branch information
ywmei-brt1 committed Jul 25, 2024
1 parent 380596e commit baabf15
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions go/pkg/rexec/rexec.go
Original file line number Diff line number Diff line change
Expand Up @@ -57,11 +57,11 @@ func (c *Client) NewContext(ctx context.Context, cmd *command.Command, opt *comm
return nil, err
}
grpcCtx, err := contextmd.WithMetadata(ctx, &contextmd.Metadata{
ToolName: cmd.Identifiers.ToolName,
ToolVersion: cmd.Identifiers.ToolVersion,
ActionID: cmd.Identifiers.CommandID,
InvocationID: cmd.Identifiers.InvocationID,
CorrelatedInvocationID: cmd.Identifiers.CorrelatedInvocationsID,
ToolName: cmd.Identifiers.ToolName,
ToolVersion: cmd.Identifiers.ToolVersion,
ActionID: cmd.Identifiers.CommandID,
InvocationID: cmd.Identifiers.InvocationID,
CorrelatedInvocationsID: cmd.Identifiers.CorrelatedInvocationsID,
})
if err != nil {
return nil, err
Expand Down

0 comments on commit baabf15

Please sign in to comment.