Skip to content

Commit 4d22cc1

Browse files
committed
[core] Use device label instead of field indexer in LLDP controller
1 parent c8b0208 commit 4d22cc1

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
@@ -388,7 +388,7 @@ func (r *LLDPReconciler) validateUniqueLLDPPerDevice(ctx context.Context, s *lld
388388
var list v1alpha1.LLDPList
389389
if err := r.List(ctx, &list,
390390
client.InNamespace(s.LLDP.Namespace),
391-
client.MatchingFields{".spec.deviceRef.name": s.LLDP.Spec.DeviceRef.Name},
391+
client.MatchingLabels{v1alpha1.DeviceLabel: s.Device.Name},
392392
); err != nil {
393393
return err
394394
}
@@ -422,13 +422,6 @@ func (r *LLDPReconciler) SetupWithManager(ctx context.Context, mgr ctrl.Manager)
422422
return fmt.Errorf("failed to create label selector predicate: %w", err)
423423
}
424424

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

0 commit comments

Comments
 (0)