🌱 Remove usage of FailureReason and FailureMessage (hcloud)

[guettli]

HCloud PR

Deprecated: This field is deprecated and is
going to be removed when support for v1beta1
will be dropped. Please see
https://github.com/kubernetes-sigs/cluster-api/blob/main/docs/proposals/20240916-improve-status-in-CAPI-resources.md
for more details.

Related Baremetal PR 1716

Changes:

• Extracted function SetErrorAndRemediateMachine(), so that it can be used without a MachineScope (for example in a test).
• Do not reconcile hbmm, when RemediateMachineAnnotation is set (except the hbmm gets deleted).
• In test: set PlacementGroupName: &defaultPlacementGroupName. Otherwise, machine would not get provisioned (in test).
• added test: It("should delete machine if SetErrorAndRemediate() was called" ...
• tests: removed machinePatchHelper, because it was not used.
• Reconcile of hcloud remediation: Do not try to reboot, when NoRemediateMachineAnnotationCondition is set. Use setOwnerRemediatedConditionToFailed() and stop reconciling.
• Reconcile of hcloud remediation: Better err msg to differentiate between ProviderId is not set and server-not-found.
• Avoid using named returns, when not needed: return res, nil --> return reconcile.Result{}, nil
• Better err msg when a PlacementGroup does not exist.
• create server: Align message in condition to message in error.
• avoid using capierrors (will be removed in future versions of capi).
• test: ensure RemediateMachineAnnotation is set.
• tests: align capi-machine name to hbmm name.
• Add helper noErrorOccured() to check that machineScope contains no error.