🐛 Apply conditions patch should preserve LastTransitionTime

[fabriziopandini]

What this PR does / why we need it:
This PR fixes a nasty bug when applying a condition patch with and add operation with the path happening on a different second than the one where the condition was set on the target object.

The PR also align the variable names used for the objects involved in the three-way merge with the one used by the patch helper (before, after, latest), so there will be less confusion were looking at the code

Which issue(s) this PR fixes
Fixes #3178

[fabriziopandini]

/assign @benmoss

[benmoss]

Great branch name 😄

[benmoss]

Looks like it's not flaky anymore too 😸 . I'll try to make a patch to fix it if I can figure it out.

[benmoss]

This is one solution to fixing this test now: https://gist.github.com/benmoss/14ac414bccdfbce903ee4db43da7f18b

go-cmp will end up using https://golang.org/pkg/time/#Time.Equal rather than Gomega's Equal matcher which ends up using reflect.DeepEqual. It's a whole nightmare then of reflect.DeepEqual compares private fields on time.Time structs, and why UTC times have nil locations but those that are coming back from the API server are in local time.

[JoelSpeed]

+1 for the name changes in patch.go, much clearer this way I think

[fabriziopandini]

/retest

[fabriziopandini]

/retest

[vincepri]

/approve
/milestone v0.3.7

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fabriziopandini, vincepri

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [fabriziopandini,vincepri]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[vincepri]

/lgtm