Simplify BMCSettings reconciler and fix bugs (#723)

* Fix bugs and simplify BMCSettings reconciler

* Address review comments

* Fix nitpicks

* Address PR reviews

* Propagate getServers errors instead of converting them to “not granted.”

* Removed failed state cleanup patch

* Fix nitpick issues

Author: afritzler

internal/controller/biossettings_controller_test.go

@@ -733,7 +733,7 @@ var _ = Describe("BIOSSettings Controller", func() {
 			HaveField("Status.State", metalv1alpha1.ServerStateReserved),
 		)
 
-		By("Creating a BMCSetting")
+		By("Creating a BIOSSettings")
 		biosSettings := &metalv1alpha1.BIOSSettings{
 			ObjectMeta: metav1.ObjectMeta{
 				Namespace:    ns.Name,