Index named input for NAD, unify across Watches and return nil error in case NAD not found to use RequeueAfter

[stuggi]

• Adds to watch NADs so that the controller reconciles when the configured NAD changes on the GlanceAPI.
• Also unifies the Watches across the named input parameters.
• Currently the reconciler returned both a non-zero result and a non-nil error.
  The result will always be ignored if the error is non-nil and the non-nil error causes reqeueuing with exponential backoff.
  In case of NotFound return nil that the ReqeueAfter is used. For more details, see: https://pkg.go.dev/sigs.k8s.io/controller-runtime/pkg/reconcile#Reconciler

2024-08-05T15:55:35Z    INFO    Warning: Reconciler returned both a non-zero result and a non-nil error. The result will always be ignored if the error is non-nil and the non-nil error causes reqeueuing with exponential backoff. For more details, see: https://pkg.go.dev/sigs.k8s.io/controller-runtime/pkg/reconcile#Reconciler   {"controller": "glance", "controllerGroup": "glance.openstack.org", "controllerKind": "Glance", "Glance": {"name":"glance","namespace":"openstack"}, "namespace": "openstack", "name": "glance", "reconcileID": "d615ee4e-7b07-44d3-a9e8-160540927609"}

[fmount]

Just a minor comment on the usage of the existing const to return RequeueAfter but in general I'm +1 to this change.
@stuggi I can approve and follow up on that part, let me know your thoughts around that.

[fmount]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fmount, stuggi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [fmount,stuggi]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[stuggi]

NAD annotations won't change on NAD details. so it does not make sense right now to watch them. closing in favor of #601