Refactor LB

  Allow place-holder of LB without any VM instance existing
  Fix IP duplicated allocation or release
  Simply VMI controller
  Enhance health check probe
  Refactor lb controller, move lb to none-ready status in various error cases

cmd/webhook/main.go

@@ -115,7 +115,7 @@ func run(ctx context.Context, cfg *rest.Config, options *config.Options) error {
 	webhookServer := server.NewWebhookServer(ctx, cfg, name, options)
 
 	if err := webhookServer.RegisterValidators(ippool.NewIPPoolValidator(poolCache),
-		loadbalancer.NewValidator(vmiCache)); err != nil {
+		loadbalancer.NewValidator()); err != nil {
 		return fmt.Errorf("failed to register ip pool validator: %w", err)
 	}
 

pkg/controller/ippool/controller.go

@@ -81,11 +81,8 @@ func (h *Handler) OnRemove(_ string, ipPool *lbv1.IPPool) (*lbv1.IPPool, error)
 	if ipPool == nil {
 		return nil, nil
 	}
-
-	logrus.Debugf("IP Pool %s has been deleted", ipPool.Name)
-
+	logrus.Infof("IP Pool %s is deleted", ipPool.Name)
 	h.allocatorMap.Delete(ipPool.Name)
-
 	return ipPool, nil
 }