Move Server.Status popolation from ServerReconciler to BMCReconciler

[stefanhipfel]

Proposed Changes

BMC controller now loads Server Status Details (Manufacturer, Storages, CPUs) once during creation of a server CRD and when setting the anntotation "metal.ironcore.dev/operation" : "update-server-details".

BMC controller reconcile loop now only gets called when server was deleted or ops annotation was set.

Fixes:
Redfish Processors are now called separately, only when needed.
In General: Less calls to BMC redfish api

Server Controller Test:
Before: 700 API calls
After: 305 API calls

[Nuckal777]

I am still wondering, if the BMC reconciler should be responsible for filling in the state the state of Server objects. 🤔 The main concern is PR wants to address is superfluous calls to the redfish API by introducing some checks on status fields already being present. This can also be achieved in the Server reconciler.

[stefanhipfel]

I am still wondering, if the BMC reconciler should be responsible for filling in the state the state of Server objects. 🤔 The main concern is PR wants to address is superfluous calls to the redfish API by introducing some checks on status fields already being present. This can also be achieved in the Server reconciler.

BMC reconciler already sets state fields of Server objects.

Only updating hardware status fields which are not set, requires to implement a forced update mechanism if hardware was updated..

[stefanhipfel]

superseded by #443