Fix provisioning logs (#4428)

Add error level log for invalid config so it doesn't get lost in the debug layer.

Author: sjberman

internal/controller/provisioner/provisioner.go

@@ -255,13 +255,22 @@ func (p *NginxProvisioner) provisionNginx(
 			func(ctx context.Context) (bool, error) {
 				res, upsertErr = controllerutil.CreateOrUpdate(ctx, p.k8sClient, obj, objectSpecSetter(obj))
 				if upsertErr != nil {
-					p.cfg.Logger.V(1).Info(
-						"Retrying CreateOrUpdate for nginx resource after error",
-						"namespace", gateway.GetNamespace(),
-						"name", resourceName,
-						"error", upsertErr.Error(),
-					)
-					return false, nil //nolint:nilerr // continue retrying
+					if apierrors.IsInvalid(upsertErr) { // log this error at the error level
+						p.cfg.Logger.Error(
+							upsertErr,
+							"Retrying CreateOrUpdate for nginx resource after error",
+							"namespace", gateway.GetNamespace(),
+							"name", resourceName,
+						)
+					} else {
+						p.cfg.Logger.V(1).Info(
+							"Retrying CreateOrUpdate for nginx resource after error",
+							"namespace", gateway.GetNamespace(),
+							"name", resourceName,
+							"error", upsertErr.Error(),
+						)
+					}
+					return false, nil
 				}
 				return true, nil
 			},