Add error context to log messages for ansible proxy (operator-framework#2640)

Add some of the logging messages do not have any error context included.
Motivation for the change: Related operator-framework#2623

Author: kramvan1

pkg/ansible/proxy/cache_response.go

@@ -81,7 +81,7 @@ func (c *cacheResponseHandler) ServeHTTP(w http.ResponseWriter, req *http.Reques
 		k, err := getGVKFromRequestInfo(r, c.restMapper)
 		if err != nil {
 			// break here in case resource doesn't exist in cache
-			log.Info("Cache miss, can not find in rest mapper")
+			log.Error(err, "Cache miss, can not find in rest mapper")
 			break
 		}
 
@@ -94,7 +94,7 @@ func (c *cacheResponseHandler) ServeHTTP(w http.ResponseWriter, req *http.Reques
 		isVR, err := c.apiResources.IsVirtualResource(k)
 		if err != nil {
 			// break here in case we can not understand if virtual resource or not
-			log.Info("Unable to determine if virtual resource", "gvk", k)
+			log.Error(err, "Unable to determine if virtual resource", "gvk", k)
 			break
 		}
 

pkg/ansible/proxy/inject_owner.go

@@ -74,15 +74,15 @@ func (i *injectOwnerReferenceHandler) ServeHTTP(w http.ResponseWriter, req *http
 		k, err := getGVKFromRequestInfo(r, i.restMapper)
 		if err != nil {
 			// break here in case resource doesn't exist in cache
-			log.Info("Cache miss, can not find in rest mapper")
+			log.Error(err, "Cache miss, can not find in rest mapper")
 			break
 		}
 
 		// Determine if the resource is virtual. If it is then we should not attempt to use cache
 		isVR, err := i.apiResources.IsVirtualResource(k)
 		if err != nil {
 			// break here in case we can not understand if virtual resource or not
-			log.Info("Unable to determine if virtual resource", "gvk", k)
+			log.Error(err, "Unable to determine if virtual resource", "gvk", k)
 			break
 		}