More useful audit error logs

[tallclair]

What type of PR is this?

/kind bug

What this PR does / why we need it:

1. Include the auditID with audit error logs in the kube-apiserver logs, so errors can be correlated with the request. (As an asside, I think we should do this for all apiserver log lines in the request serving path, but that's outside the scope of this PR)
2. Attempt to log the actual request / response object rather than it's reflected type when encoding fails.

---

While debugging a kube-apiserver, I came across a lot of log lines like this:

Auditing failed of  request: encoding failed: request to convert CR to an invalid group/version: v1

Note the double space after the first of. This is logged from

kubernetes/staging/src/k8s.io/apiserver/pkg/audit/request.go

Line 169 in eefcf6a

klog.Warningf("Auditing failed of %v request: %v", reflect.TypeOf(obj).Name(), err)

But reflect.TypeOf(obj).Name() is returning the empty string (from the docs: "Name returns the type's name within its package for a defined type. For other (non-defined) types it returns the empty string.")

Note to reviewers:

The underlying issue generating these log lines was fixed by #110110

Does this PR introduce a user-facing change?

NONE

/sig api-machinery

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tallclair

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:

• staging/src/k8s.io/apiserver/pkg/audit/OWNERS [tallclair]

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

[fedebongio]

/assign @deads2k

[fedebongio]

/triage accepted

[pacoxu]

/sig instrumentation
/lgtm
The log looks better now.