Merge pull request #9 from SchSeba/fix_multiple_issues

Fix multiple issues

cmd/manager/main.go

@@ -27,10 +27,10 @@ import (
 	"github.com/go-logr/logr"
 	"k8s.io/client-go/kubernetes"
 	"sigs.k8s.io/controller-runtime/pkg/client/config"
+	"sigs.k8s.io/controller-runtime/pkg/manager"
 	logf "sigs.k8s.io/controller-runtime/pkg/runtime/log"
 
 	"github.com/K8sNetworkPlumbingWG/kubemacpool/pkg/controller"
-	"github.com/K8sNetworkPlumbingWG/kubemacpool/pkg/manager"
 	poolmanager "github.com/K8sNetworkPlumbingWG/kubemacpool/pkg/pool-manager"
 	"github.com/K8sNetworkPlumbingWG/kubemacpool/pkg/webhook"
 )
@@ -126,10 +126,7 @@ func main() {
 	go waitForSignal()
 
 	log.Info("Setting up manager")
-	mgr, err := manager.New(cfg, manager.Options{MetricsBindAddress: metricsAddr,
-		LeaderElection:          true,
-		LeaderElectionID:        "kubemacpool-mac-election",
-		LeaderElectionNamespace: "kubemacpool-system"})
+	mgr, err := manager.New(cfg, manager.Options{MetricsBindAddress: metricsAddr})
 	ExitIfError(err, "unable to set up manager")
 
 	log.Info("Setting up controller")

pkg/controller/pod/pod_controller.go

@@ -89,5 +89,15 @@ func (r *ReconcilePolicy) Reconcile(request reconcile.Request) (reconcile.Result
 		return reconcile.Result{}, err
 	}
 
+	// allocate only when the pod is ready
+	if instance.Annotations == nil {
+		return reconcile.Result{}, nil
+	}
+
+	err = r.poolManager.AllocatePodMac(instance)
+	if err != nil {
+		log.Error(err, "failed to allocate mac for pod")
+	}
+
 	return reconcile.Result{}, nil
 }