fix: remove requeue after because it halts all other events from bein…

…g reconciled note: the rate limiter will protect us anyways Signed-off-by: Peter Wilcsinszky <[email protected]>
kube-logging · Feb 28, 2024 · 7fe3a82 · 7fe3a82
1 parent 7f1e67b
commit 7fe3a82
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/pkg/resources/fluentd/fluentd.go b/pkg/resources/fluentd/fluentd.go
@@ -17,7 +17,6 @@ package fluentd
 import (
 	"context"
 	"fmt"
-	"time"
 
 	"emperror.dev/errors"
 	"github.com/cisco-open/operator-tools/pkg/reconciler"
@@ -203,7 +202,7 @@ func (r *Reconciler) Reconcile(ctx context.Context) (*reconcile.Result, error) {
 				} else {
 					r.Log.Info("still waiting for the configcheck result...")
 				}
-				return &reconcile.Result{RequeueAfter: time.Minute}, nil
+				return &reconcile.Result{Requeue: true}, nil
 			}
 		}
 	}

diff --git a/pkg/resources/syslogng/syslogng.go b/pkg/resources/syslogng/syslogng.go
@@ -16,7 +16,6 @@ package syslogng
 
 import (
 	"context"
-	"time"
 
 	"emperror.dev/errors"
 	"github.com/cisco-open/operator-tools/pkg/reconciler"
@@ -184,7 +183,7 @@ func (r *Reconciler) Reconcile(ctx context.Context) (*reconcile.Result, error) {
 				} else {
 					r.Log.Info("still waiting for the configcheck result...")
 				}
-				return &reconcile.Result{RequeueAfter: time.Minute}, nil
+				return &reconcile.Result{Requeue: true}, nil
 			}
 		}
 	}