Read object before compare to avoid wrong cached data (#1216)

* Read object before compare to avoid wrong cached data

#1208 Reading the object before checking if historical to avoid reading cached annotation.

* Keep error check close to err assigment

* Change log location to comply with code style requirements

pkg/webhook/webhookreporter.go

@@ -72,12 +72,7 @@ func (r *WebhookReconciler) SetupWithManager(mgr ctrl.Manager) error {
 func (r *WebhookReconciler) reconcileReport(reportType client.Object) reconcile.Func {
 	return func(ctx context.Context, request reconcile.Request) (reconcile.Result, error) {
 		log := r.Logger.WithValues("report", request.NamespacedName)
-		if ignoreHistoricalReport(reportType) {
-			log.V(1).Info("Ignoring historical report")
-			return ctrl.Result{}, nil
-		}
 		verb := Update
-
 		err := r.Client.Get(ctx, request.NamespacedName, reportType)
 		if err != nil {
 			if !errors.IsNotFound(err) {
@@ -89,6 +84,11 @@ func (r *WebhookReconciler) reconcileReport(reportType client.Object) reconcile.
 			}
 			verb = Delete
 		}
+
+		if ignoreHistoricalReport(reportType) {
+			log.V(1).Info("Ignoring historical report")
+			return ctrl.Result{}, nil
+		}
 
 		if r.WebhookSendDeletedReports {
 			msg := WebhookMsg{OperatorObject: reportType, Verb: verb}