Skip to content

Commit

Permalink
Merge pull request #242 from openshift-cherrypick-robot/cherry-pick-2…
Browse files Browse the repository at this point in the history
…39-to-18.0-fr1

[18.0-fr1] Fix the nil pointing issue
  • Loading branch information
openshift-merge-bot[bot] authored Nov 12, 2024
2 parents 9b081b3 + 6c90ba1 commit 026395b
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions pkg/ansibletest/volumes.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ func GetVolumes(
volumes = append(volumes, extraVol)
}

if len(instance.Spec.Workflow) > 0 {
if len(instance.Spec.Workflow) > 0 && *instance.Spec.Workflow[externalWorkflowCounter].ExtraConfigmapsMounts != nil {
for _, vol := range *instance.Spec.Workflow[externalWorkflowCounter].ExtraConfigmapsMounts {
extraWorkflowVol := corev1.Volume{
Name: vol.Name,
Expand Down Expand Up @@ -204,7 +204,7 @@ func GetVolumeMounts(mountCerts bool, instance *testv1beta1.AnsibleTest, externa
volumeMounts = append(volumeMounts, extraConfigmapsMounts)
}

if len(instance.Spec.Workflow) > 0 {
if len(instance.Spec.Workflow) > 0 && *instance.Spec.Workflow[externalWorkflowCounter].ExtraConfigmapsMounts != nil {
for _, vol := range *instance.Spec.Workflow[externalWorkflowCounter].ExtraConfigmapsMounts {

extraConfigmapsMounts := corev1.VolumeMount{
Expand All @@ -217,5 +217,6 @@ func GetVolumeMounts(mountCerts bool, instance *testv1beta1.AnsibleTest, externa
volumeMounts = append(volumeMounts, extraConfigmapsMounts)
}
}

return volumeMounts
}

0 comments on commit 026395b

Please sign in to comment.