airship/vino
Code Issues Proposed changes

Merge "Don't reconcile non-spec changes"

Browse Source
This commit is contained in:
Zuul 2021-01-29 19:48:03 +00:00 committed by Gerrit Code Review
commit ceefdc1467
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
pkg/controllers/vino_controller.go
View File

@ -31,8 +31,10 @@ import (
"k8s.io/apimachinery/pkg/types"
kerror "k8s.io/apimachinery/pkg/util/errors"
ctrl "sigs.k8s.io/controller-runtime"
"sigs.k8s.io/controller-runtime/pkg/builder"
"sigs.k8s.io/controller-runtime/pkg/client"
"sigs.k8s.io/controller-runtime/pkg/controller/controllerutil"
"sigs.k8s.io/controller-runtime/pkg/predicate"
"sigs.k8s.io/yaml"
vinov1 "vino/pkg/api/v1"
@ -590,7 +592,9 @@ func (r *VinoReconciler) daemonSet(ctx context.Context, vino *vinov1.Vino) (*app
func (r *VinoReconciler) SetupWithManager(mgr ctrl.Manager) error {
return ctrl.NewControllerManagedBy(mgr).
For(&vinov1.Vino{}).
For(&vinov1.Vino{}, builder.WithPredicates(
predicate.GenerationChangedPredicate{},
)).
Complete(r)
}