Skip to content

Commit

Permalink
Revert "Use LabelsAndGenerationPredicate for ToolchainClusterCache co…
Browse files Browse the repository at this point in the history
…ntroller (#437)"

This reverts commit 463b623.
  • Loading branch information
rajivnathan committed Dec 11, 2024
1 parent 463b623 commit 6d16802
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import (

toolchainv1alpha1 "github.com/codeready-toolchain/api/api/v1alpha1"
"github.com/codeready-toolchain/toolchain-common/pkg/cluster"
commonpredicates "github.com/codeready-toolchain/toolchain-common/pkg/predicate"
"k8s.io/apimachinery/pkg/api/errors"
"k8s.io/apimachinery/pkg/runtime"
ctrl "sigs.k8s.io/controller-runtime"
Expand All @@ -32,7 +31,7 @@ func NewReconciler(mgr manager.Manager, namespace string, timeout time.Duration)
// SetupWithManager sets up the controller with the Manager.
func (r *Reconciler) SetupWithManager(mgr ctrl.Manager) error {
return ctrl.NewControllerManagedBy(mgr).
For(&toolchainv1alpha1.ToolchainCluster{}, builder.WithPredicates(namespacePredicate{namespace: r.namespace}, commonpredicates.LabelsAndGenerationPredicate{})).
For(&toolchainv1alpha1.ToolchainCluster{}, builder.WithPredicates(namespacePredicate{namespace: r.namespace})).

Check warning on line 34 in controllers/toolchainclustercache/toolchaincluster_cache_controller.go

View check run for this annotation

Codecov / codecov/patch

controllers/toolchainclustercache/toolchaincluster_cache_controller.go#L34

Added line #L34 was not covered by tests
Complete(r)
}

Expand Down

0 comments on commit 6d16802

Please sign in to comment.