From 66df01257a1abce26e4b3577527a8fbe651358d6 Mon Sep 17 00:00:00 2001 From: RealAnna <89971034+RealAnna@users.noreply.github.com> Date: Tue, 4 Apr 2023 07:47:04 +0200 Subject: [PATCH] fix: removed failure branch (#1175) Signed-off-by: realanna --- scheduler/pkg/klcpermit/permit.go | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/scheduler/pkg/klcpermit/permit.go b/scheduler/pkg/klcpermit/permit.go index f06ca8dd79..4fd5a40f8a 100644 --- a/scheduler/pkg/klcpermit/permit.go +++ b/scheduler/pkg/klcpermit/permit.go @@ -36,14 +36,11 @@ func (pl *Permit) Permit(ctx context.Context, state *framework.CycleState, p *v1 // check the permit immediately, to fail early in case the pod cannot be queued switch pl.workloadManager.Permit(ctx, p) { - case Failure: - klog.Infof("[Keptn Permit Plugin] failed pre-deployment checks on %s", p.GetObjectMeta().GetName()) - return framework.NewStatus(framework.Error), 0 * time.Second case Success: klog.Infof("[Keptn Permit Plugin] passed pre-deployment checks on %s", p.GetObjectMeta().GetName()) return framework.NewStatus(framework.Success), 0 * time.Second default: - klog.Infof("[Keptn Permit Plugin] waiting for pre-deployment checks on %s", p.GetObjectMeta().GetName()) + klog.Infof("[Keptn Permit Plugin] waiting for pre-deployment checks on %s to succeed", p.GetObjectMeta().GetName()) go func() { // create a new context since we are in a new goroutine ctx2, cancel := context.WithCancel(context.Background()) @@ -60,9 +57,6 @@ func (pl *Permit) monitorPod(ctx context.Context, p *v1.Pod) { for { switch pl.workloadManager.Permit(ctx, p) { - case Failure: - waitingPodHandler.Reject(PluginName, "Pre Deployment Check failed") - return case Success: waitingPodHandler.Allow(PluginName) return