Skip to content

Commit 15420da

Browse files
committed
Rename generationChangedPredicate to alwaysTruePredicate
Rename generationChangedPredicate to alwaysTruePredicate
1 parent 7ca54bf commit 15420da

File tree

2 files changed

+8
-10
lines changed

2 files changed

+8
-10
lines changed

internal/mode/static/state/change_processor.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -126,22 +126,22 @@ func NewChangeProcessorImpl(cfg ChangeProcessorConfig) *ChangeProcessorImpl {
126126
{
127127
gvk: extractGVK(&v1.GatewayClass{}),
128128
store: newObjectStoreMapAdapter(clusterStore.GatewayClasses),
129-
predicate: generationChangedPredicate{},
129+
predicate: alwaysTruePredicate{},
130130
},
131131
{
132132
gvk: extractGVK(&v1.Gateway{}),
133133
store: newObjectStoreMapAdapter(clusterStore.Gateways),
134-
predicate: generationChangedPredicate{},
134+
predicate: alwaysTruePredicate{},
135135
},
136136
{
137137
gvk: extractGVK(&v1.HTTPRoute{}),
138138
store: newObjectStoreMapAdapter(clusterStore.HTTPRoutes),
139-
predicate: generationChangedPredicate{},
139+
predicate: alwaysTruePredicate{},
140140
},
141141
{
142142
gvk: extractGVK(&v1beta1.ReferenceGrant{}),
143143
store: newObjectStoreMapAdapter(clusterStore.ReferenceGrants),
144-
predicate: generationChangedPredicate{},
144+
predicate: alwaysTruePredicate{},
145145
},
146146
{
147147
gvk: extractGVK(&apiv1.Namespace{}),

internal/mode/static/state/changed_predicate.go

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,14 +24,12 @@ func (f funcPredicate) delete(object client.Object) bool {
2424
return f.stateChanged(object)
2525
}
2626

27-
// generationChangedPredicate implements stateChangedPredicate based on the generation of the object.
28-
// This predicate will return true on upsert if the object's generation has changed.
29-
// It always returns true on delete.
30-
type generationChangedPredicate struct{}
27+
// FIXME(kevin85421): We should remove this predicate and update changeTrackingUpdater once #1432 is merged.
28+
type alwaysTruePredicate struct{}
3129

32-
func (generationChangedPredicate) delete(_ client.Object) bool { return true }
30+
func (alwaysTruePredicate) delete(_ client.Object) bool { return true }
3331

34-
func (generationChangedPredicate) upsert(_, _ client.Object) bool { return true }
32+
func (alwaysTruePredicate) upsert(_, _ client.Object) bool { return true }
3533

3634
// annotationChangedPredicate implements stateChangedPredicate based on the value of the annotation provided.
3735
// This predicate will return true on upsert if the annotation's value has changed.

0 commit comments

Comments
 (0)