Change variable name in the correct function

Author: t-persson

internal/release/executionspace.go

@@ -25,23 +25,23 @@ import (
 const ExecutionSpaceReleaserName = "execution-space-releaser"
 
 // ExecutionSpaceReleaser returns an ExecutionSpace releaser job specification.
-func ExecutionSpaceReleaser(executionSpace *v1alpha2.ExecutionSpace, environmentrequest *v1alpha1.EnvironmentRequest, image string, noDelete bool) *batchv1.Job {
+func ExecutionSpaceReleaser(executionSpace *v1alpha2.ExecutionSpace, environmentrequest *v1alpha1.EnvironmentRequest, image string, skipDeletingExecutionSpace bool) *batchv1.Job {
 	return ReleaseJob(
 		executionSpace.Name,
 		ExecutionSpaceReleaserName,
 		executionSpace.Namespace,
 		environmentrequest,
-		ExecutionSpaceReleaserContainer(executionSpace, image, noDelete),
+		ExecutionSpaceReleaserContainer(executionSpace, image, skipDeletingExecutionSpace),
 	)
 }
 
 // ExecutionSpaceReleaserContainer returns an ExecutionSpace releaser container specification.
-func ExecutionSpaceReleaserContainer(executionSpace *v1alpha2.ExecutionSpace, image string, noDelete bool) corev1.Container {
+func ExecutionSpaceReleaserContainer(executionSpace *v1alpha2.ExecutionSpace, image string, skipDeletingExecutionSpace bool) corev1.Container {
 	return ReleaseContainer(
 		executionSpace.Name,
 		ExecutionSpaceReleaserName,
 		executionSpace.Namespace,
 		image,
-		noDelete,
+		skipDeletingExecutionSpace,
 	)
 }

internal/release/release.go

@@ -70,13 +70,15 @@ func ReleaseJob(jobName, name, namespace string, environmentrequest *v1alpha1.En
 }
 
 // ReleaseContainer returns a container specification.
-func ReleaseContainer(name, containerName, namespace, image string, skipDeletingExecutionSpace bool) corev1.Container {
+func ReleaseContainer(name, containerName, namespace, image string, noDelete bool) corev1.Container {
 	args := []string{
 		"-release",
 		fmt.Sprintf("-namespace=%s", namespace),
 		fmt.Sprintf("-name=%s", name),
 	}
-	if skipDeletingExecutionSpace {
+	// noDelete tells a provider to not delete the resource it is responsible for.
+	// Resource in this case being 'Iut', 'LogArea' or 'ExecutionSpace'
+	if noDelete {
 		args = append(args, "-nodelete")
 	}
 	return corev1.Container{