🐛 Don't log name and namespace twice

[inteon]

Currently, the logs for my controller look like this:

I1206 09:17:57.766036   13278 controller.go:212] "Successfully finished the reconciliation." logger="Reconcile" controller="test" controllerGroup="test.com" controllerKind="Test" Test="namespace1/resource1" namespace="namespace1" name="resource1" reconcileID="6594dad2-87a7-4525-8549-f7967f7447e5"

Notice that Test="namespace1/resource1" just repeats information that was already available in other key-value pairs (controllerKind="Test", namespace="namespace1" and name="resource1").

After this PR, the logs look like this:

I1206 09:17:57.766036   13278 controller.go:212] "Successfully finished the reconciliation." logger="Reconcile" controller="test" controllerGroup="test.com" controllerKind="Test" namespace="namespace1" name="resource1" reconcileID="6594dad2-87a7-4525-8549-f7967f7447e5"

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: inteon
Once this PR has been reviewed and has the lgtm label, please assign vincepri for approval. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files:

• OWNERS

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

[alvaroaleman]

The format of the two is different, this was added to be compatible with the format used by other projects in the ecosystem. Removing this would be a breaking change and I don't see a reason to do that. You can always set up your own logging and make the integrated a no-op.
/hold

[sbueringer]

Agree. I would like to continue to align with the k/k log format