Add namespace and name field

Author: timebertt

pkg/webhook/admission/webhook.go

@@ -200,6 +200,7 @@ func (wh *Webhook) getLogger(req *Request) logr.Logger {
 func DefaultLogConstructor(base logr.Logger, req *Request) logr.Logger {
 	if req != nil {
 		return base.WithValues("object", klog.KRef(req.Namespace, req.Name),
+			"namespace", req.Namespace, "name", req.Name,
 			"resource", req.Resource, "user", req.UserInfo.Username,
 		)
 	}

pkg/webhook/admission/webhook_test.go

@@ -172,7 +172,7 @@ var _ = Describe("Admission Webhooks", func() {
 		Expect(resp.Allowed).To(BeTrue())
 
 		By("checking that the log message contains the request fields")
-		Eventually(logBuffer).Should(gbytes.Say(`"msg":"Received request","object":{"name":"foo","namespace":"bar"},"resource":{"group":"apps","version":"v1","resource":"deployments"},"user":"tim","requestID":"test123"}`))
+		Eventually(logBuffer).Should(gbytes.Say(`"msg":"Received request","object":{"name":"foo","namespace":"bar"},"namespace":"bar","name":"foo","resource":{"group":"apps","version":"v1","resource":"deployments"},"user":"tim","requestID":"test123"}`))
 	})
 
 	It("should pass a request logger created by LogConstructor via the context", func() {