pkg,test: Avoid setting metadata.GracePeriodSeconds to zero seconds

Update any references where we set the metadata.GracePeriodSeconds field
to zero seconds. If we need a Pod to be deleted as quick as possible,
use a single second.

Signed-off-by: timflannagan <[email protected]>

Author: timflannagan

pkg/api/wrappers/deployment_install_client.go

@@ -3,6 +3,7 @@ package wrappers
 
 import (
 	"context"
+
 	"github.com/pkg/errors"
 	appsv1 "k8s.io/api/apps/v1"
 	corev1 "k8s.io/api/core/v1"
@@ -102,7 +103,8 @@ func (c *InstallStrategyDeploymentClientForNamespace) CreateDeployment(deploymen
 
 func (c *InstallStrategyDeploymentClientForNamespace) DeleteDeployment(name string) error {
 	foregroundDelete := metav1.DeletePropagationForeground // cascading delete
-	immediate := int64(0)
+	// Note(tflannag): See https://bugzilla.redhat.com/show_bug.cgi?id=1939294.
+	immediate := int64(1)
 	immediateForegroundDelete := &metav1.DeleteOptions{GracePeriodSeconds: &immediate, PropagationPolicy: &foregroundDelete}
 	return c.opClient.DeleteDeployment(c.Namespace, name, immediateForegroundDelete)
 }

pkg/controller/operators/suite_test.go

@@ -49,7 +49,7 @@ var (
 	ctx       context.Context
 
 	scheme            = runtime.NewScheme()
-	gracePeriod int64 = 0
+	gracePeriod int64 = 1
 	propagation       = metav1.DeletePropagationForeground
 	deleteOpts        = &client.DeleteOptions{
 		GracePeriodSeconds: &gracePeriod,

test/rh-operators/operator_util.go

@@ -7,7 +7,7 @@ import (
 	"testing"
 	"time"
 
-	"github.com/operator-framework/api/pkg/operators/v1"
+	v1 "github.com/operator-framework/api/pkg/operators/v1"
 	"github.com/operator-framework/api/pkg/operators/v1alpha1"
 	"github.com/operator-framework/operator-lifecycle-manager/pkg/api/client/clientset/versioned"
 	"github.com/operator-framework/operator-lifecycle-manager/pkg/lib/operatorclient"
@@ -26,7 +26,7 @@ var (
 	pollDuration        = 20 * time.Minute
 	terminationDuration = 5 * time.Minute
 
-	immediate             = int64(0)
+	immediate             = int64(1)
 	immediateDeleteOption = &metav1.DeleteOptions{GracePeriodSeconds: &immediate}
 )