Skip to content

Commit 790fdb4

Browse files
committed
[core] Use device label instead of field indexer in LLDP controller
1 parent 28c177e commit 790fdb4

1 file changed

Lines changed: 1 addition & 8 deletions

File tree

‎internal/controller/core/lldp_controller.go‎

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -389,7 +389,7 @@ func (r *LLDPReconciler) validateUniqueLLDPPerDevice(ctx context.Context, s *lld
389389
var list v1alpha1.LLDPList
390390
if err := r.List(ctx, &list,
391391
client.InNamespace(s.LLDP.Namespace),
392-
client.MatchingFields{".spec.deviceRef.name": s.LLDP.Spec.DeviceRef.Name},
392+
client.MatchingLabels{v1alpha1.DeviceLabel: s.Device.Name},
393393
); err != nil {
394394
return err
395395
}
@@ -423,13 +423,6 @@ func (r *LLDPReconciler) SetupWithManager(ctx context.Context, mgr ctrl.Manager)
423423
return fmt.Errorf("failed to create label selector predicate: %w", err)
424424
}
425425

426-
if err := mgr.GetFieldIndexer().IndexField(ctx, &v1alpha1.LLDP{}, ".spec.deviceRef.name", func(obj client.Object) []string {
427-
lldp := obj.(*v1alpha1.LLDP)
428-
return []string{lldp.Spec.DeviceRef.Name}
429-
}); err != nil {
430-
return err
431-
}
432-
433426
c := ctrl.NewControllerManagedBy(mgr).
434427
For(&v1alpha1.LLDP{}).
435428
Named("lldp").

0 commit comments

Comments
 (0)