Use RWX PVs only for parallel execution (#96)

This change is needed as we are going to switch from local storage
to LVMS based storage that does not support RWX. Because of that
the test-operator will now be by default requesting RWO PVs only.
RWX will be requested when parallel is set to true in
the test-operator related CR.

Co-authored-by: root <[email protected]>

Author: lpiwowar

controllers/common.go

@@ -119,6 +119,7 @@ func (r *Reconciler) EnsureLogsPVCExists(
 	helper *helper.Helper,
 	labels map[string]string,
 	StorageClassName string,
+	parallel bool,
 ) (ctrl.Result, error) {
 	instanceNamespace := instance.GetNamespace()
 	pvcName := r.GetPVCLogsName(instance)
@@ -129,17 +130,19 @@ func (r *Reconciler) EnsureLogsPVCExists(
 		return ctrl.Result{}, nil
 	}
 
+	pvcAccessMode := []corev1.PersistentVolumeAccessMode{corev1.ReadWriteOnce}
+	if parallel {
+		pvcAccessMode = append(pvcAccessMode, corev1.ReadWriteMany)
+	}
+
 	testOperatorPvcDef := &corev1.PersistentVolumeClaim{
 		ObjectMeta: metav1.ObjectMeta{
 			Name:      pvcName,
 			Namespace: instanceNamespace,
 			Labels:    labels,
 		},
 		Spec: corev1.PersistentVolumeClaimSpec{
-			AccessModes: []corev1.PersistentVolumeAccessMode{
-				corev1.ReadWriteMany,
-				corev1.ReadWriteOnce,
-			},
+			AccessModes: pvcAccessMode,
 			Resources: corev1.ResourceRequirements{
 				Requests: corev1.ResourceList{
 					corev1.ResourceStorage: k8sresource.MustParse("1Gi"),

controllers/tempest_controller.go

@@ -208,6 +208,7 @@ func (r *TempestReconciler) Reconcile(ctx context.Context, req ctrl.Request) (re
 		helper,
 		serviceLabels,
 		instance.Spec.StorageClass,
+		instance.Spec.Parallel,
 	)
 
 	if err != nil {

controllers/tobiko_controller.go

@@ -152,6 +152,7 @@ func (r *TobikoReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctr
 		helper,
 		serviceLabels,
 		instance.Spec.StorageClass,
+		instance.Spec.Parallel,
 	)
 	if err != nil {
 		return ctrlResult, err