PR openshift#344 fixup

A chunk was unfortunately left out from PR openshift#344 by mistake.

We also take this opportunity to split the condition in the daemonset
creation error checking to get rid of a long-standing logging annoyance.
No error should be logged if getting the DS returns "not found".

Signed-off-by: Pavel Mores <pmores@redhat.com>

controllers/openshift_controller.go

@@ -177,12 +177,14 @@ func (r *KataConfigOpenShiftReconciler) Reconcile(ctx context.Context, req ctrl.
 			//node. One of the ways to ensure this is to check for the existence of "kata" runtimeclass before
 			//creating the DS
 			//Alternatively we can create the DS post execution of createRuntimeClass()
-			if k8serrors.IsNotFound(err) && r.kataConfig.Status.RuntimeClass == "kata" {
-				r.Log.Info("Creating a new installation monitor daemonset", "ds.Namespace", ds.Namespace, "ds.Name", ds.Name)
-				err = r.Client.Create(context.TODO(), ds)
-				if err != nil {
-					r.Log.Error(err, "error when creating monitor daemonset")
-					res = ctrl.Result{Requeue: true, RequeueAfter: 15 * time.Second}
+			if k8serrors.IsNotFound(err) {
+				if contains(r.kataConfig.Status.RuntimeClass, "kata") {
+					r.Log.Info("Creating a new installation monitor daemonset", "ds.Namespace", ds.Namespace, "ds.Name", ds.Name)
+					err = r.Client.Create(context.TODO(), ds)
+					if err != nil {
+						r.Log.Error(err, "error when creating monitor daemonset")
+						res = ctrl.Result{Requeue: true, RequeueAfter: 15 * time.Second}
+					}
 				}
 			} else {
 				r.Log.Error(err, "could not get monitor daemonset, try again")