Added controller for running ansible playbooks

This addition should be flexible enough to run ansible playbooks for testing popuses. it is dependent on this patch openstack-k8s-operators/tcib#182
openstack-k8s-operators · Jul 2, 2024 · 2692b58 · 2692b58
1 parent 3e04b6c
commit 2692b58
Show file tree

Hide file tree

Showing 4 changed files with 32 additions and 30 deletions.
diff --git a/config/rbac/ansibleTests_editor_role.yaml b/config/rbac/ansibleTests_editor_role.yaml
@@ -9,12 +9,12 @@ metadata:
     app.kubernetes.io/created-by: test-operator
     app.kubernetes.io/part-of: test-operator
     app.kubernetes.io/managed-by: kustomize
-  name: ansibleTests-editor-role
+  name: ansibletests-editor-role
 rules:
 - apiGroups:
   - test.openstack.org
   resources:
-  - ansibleTests
+  - ansibletests
   verbs:
   - create
   - delete
@@ -26,6 +26,6 @@ rules:
 - apiGroups:
   - test.openstack.org
   resources:
-  - ansibleTests/status
+  - ansibletests/status
   verbs:
   - get
diff --git a/config/rbac/ansibleTests_viewer_role.yaml b/config/rbac/ansibleTests_viewer_role.yaml
@@ -9,19 +9,19 @@ metadata:
     app.kubernetes.io/created-by: test-operator
     app.kubernetes.io/part-of: test-operator
     app.kubernetes.io/managed-by: kustomize
-  name: ansibleTests-viewer-role
+  name: ansibletests-viewer-role
 rules:
 - apiGroups:
   - test.openstack.org
   resources:
-  - ansibleTests
+  - ansibletests
   verbs:
   - get
   - list
   - watch
 - apiGroups:
   - test.openstack.org
   resources:
-  - ansibleTests/status
+  - ansibletests/status
   verbs:
   - get
diff --git a/config/rbac/role.yaml b/config/rbac/role.yaml
@@ -121,7 +121,7 @@ rules:
 - apiGroups:
   - test.openstack.org
   resources:
-  - ansibleTests
+  - ansibletests
   verbs:
   - create
   - delete
@@ -133,13 +133,13 @@ rules:
 - apiGroups:
   - test.openstack.org
   resources:
-  - ansibleTests/finalizers
+  - ansibletests/finalizers
   verbs:
   - update
 - apiGroups:
   - test.openstack.org
   resources:
-  - ansibleTests/status
+  - ansibletests/status
   verbs:
   - get
   - patch

diff --git a/pkg/ansibleTests/volumes.go b/pkg/ansibleTests/volumes.go
@@ -122,22 +122,23 @@ func GetVolumes(
 		volumes = append(volumes, extraVol)
 	}
 
-	for _, vol := range instance.Spec.Workflow[externalWorkflowCounter].ExtraMounts {
-		extraWorkflowVol := corev1.Volume{
-			Name: vol.Name,
-			VolumeSource: corev1.VolumeSource{
-				ConfigMap: &corev1.ConfigMapVolumeSource{
-					DefaultMode: &publicInfoMode,
-					LocalObjectReference: corev1.LocalObjectReference{
-						Name: vol.Name,
+	if len(instance.Spec.Workflow) > 0 {
+		for _, vol := range instance.Spec.Workflow[externalWorkflowCounter].ExtraMounts {
+			extraWorkflowVol := corev1.Volume{
+				Name: vol.Name,
+				VolumeSource: corev1.VolumeSource{
+					ConfigMap: &corev1.ConfigMapVolumeSource{
+						DefaultMode: &publicInfoMode,
+						LocalObjectReference: corev1.LocalObjectReference{
+							Name: vol.Name,
+						},
 					},
 				},
-			},
-		}
+			}
 
-		volumes = append(volumes, extraWorkflowVol)
+			volumes = append(volumes, extraWorkflowVol)
+		}
 	}
-
 	return volumes
 }
 
@@ -229,17 +230,18 @@ func GetVolumeMounts(mountCerts bool, instance *testv1beta1.AnsibleTests, extern
 		volumeMounts = append(volumeMounts, extraMounts)
 	}
 
-	for _, vol := range instance.Spec.Workflow[externalWorkflowCounter].ExtraMounts {
+	if len(instance.Spec.Workflow) > 0 {
+		for _, vol := range instance.Spec.Workflow[externalWorkflowCounter].ExtraMounts {
 
-		extraMounts := corev1.VolumeMount{
-			Name:      vol.Name,
-			MountPath: vol.MountPath,
-			SubPath:   vol.SubPath,
-			ReadOnly:  true,
-		}
+			extraMounts := corev1.VolumeMount{
+				Name:      vol.Name,
+				MountPath: vol.MountPath,
+				SubPath:   vol.SubPath,
+				ReadOnly:  true,
+			}
 
-		volumeMounts = append(volumeMounts, extraMounts)
+			volumeMounts = append(volumeMounts, extraMounts)
+		}
 	}
-
 	return volumeMounts
 }