Closed karlkfi closed 2 years ago
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: karlkfi, mortent
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/lgtm
Historically, the objects in ApplyTask were being modified in the Start func by BuildInfo to remove the path annotation, but this can cause a race condition in the task runner if it tried to read the task objects.
Now a DeepCopy is made in BuildInfo instead. This avoids the race condition but broke mutation, which executes after BuildInfo on the original object. So we now extract the object from the into after BuildInfo and use that instead for mutations, filters, and events.