Added debug log of AllocatedIPCount of ippool

[ty-dc]

Thanks for contributing!

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes ##3771

Special notes for your reviewer:

SpiderIPPool's AllocatedIPCount statistics are aligned with SpiderSubnet, and some debug logs are added.

[codecov]

Codecov Report

Attention: Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.

Project coverage is 80.81%. Comparing base (1d5d889) to head (6ef5fc2).
Report is 10 commits behind head on main.

Files with missing lines	Patch %	Lines
pkg/ippoolmanager/ippool_manager.go	71.42%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3926      +/-   ##
==========================================
- Coverage   81.34%   80.81%   -0.54%     
==========================================
  Files          51       51              
  Lines        4514     4519       +5     
==========================================
- Hits         3672     3652      -20     
- Misses        672      700      +28     
+ Partials      170      167       -3     

Flag	Coverage Δ
unittests	80.81% <71.42%> (-0.54%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
pkg/ippoolmanager/ippool_manager.go	88.64% <71.42%> (-0.64%)	⬇️

... and 1 file with indirect coverage changes

[weizhoublue]

需要找出 用量统计的根因，gc并不能解决真正的问题，只是事后补救，这是两个纬度

[ty-dc]

需要找出 用量统计的根因，gc并不能解决真正的问题，只是事后补救，这是两个纬度

原本分配 IP 的流程，是在已有的 allcateIPCount 数量++，如果分配地址时，并发进行一个在分配、一个在释放，而释放的时候 AllocatedIPCount 还没有完成 -- 这个动作，而分配的时候拿到的是未 -- 的 AllocatedIPCount ，并进行了 ++ ，从而导致出现了统计出现不正确的现象（猜测，暂未能证实猜想）

[ty-dc]

用例是在手动更新 AllocatedIPCount 为一个错误的值，期望在新的改进引入后，分配 IP 或则 释放 IP，或则 gc all 回收异常 IP，都能够纠正这个 AllocatedIPCount 值。

IPPool 的状态是健壮的。

[weizhoublue]

(1) 这个日志 真有 能帮助 debug 问题根因么
（2）这个 pr 的 title 并不能 fix

[ty-dc]

(1) 这个日志 真有 能帮助 debug 问题根因么

当前没有任何日志去获悉 ippool 的 AllocatedIPCount 数量在什么时候开始出现异常的。AllocatedIPCount++，AllocatedIPCount-- 只出现在文中两处，补充这样的日志，当 IPPool 分配与释放 IP 打印当前的 AllocatedIPCount，能够知道在哪一次分配和释放出现的记录错误的问题。

[weizhoublue]

(1) 这个日志 真有 能帮助 debug 问题根因么

当前没有任何日志去获悉 ippool 的 AllocatedIPCount 数量在什么时候开始出现异常的。AllocatedIPCount++，AllocatedIPCount-- 只出现在文中两处，补充这样的日志，当 IPPool 分配与释放 IP 打印当前的 AllocatedIPCount，能够知道在哪一次分配和释放出现的记录错误的问题。

你再想想，你这行 日志 真的能 进行 生产监控么
一直在这里 debug 级别 打印 数字

[weizhoublue]

这么检测，是帮助不了 捕获问题的
它的逻辑应该是

if  len( ip 分配数组 ) != AllocatedIPCount {
      log(error, ....... )
}

[ty-dc]

done

[weizhoublue]

同理， subnet 对象的统计 也可能会有问题

[ty-dc]

同理， subnet 对象的统计 也可能会有问题

子网根据实际的来算的

func subnetStatusCount(subnet *spiderpoolv2beta1.SpiderSubnet) (totalCount, allocatedCount int64) {
	s, _ := spiderpoolip.NewCIDR(subnet.Spec.Subnet, subnet.Spec.IPs, subnet.Spec.ExcludeIPs)

	if subnet.Status.ControlledIPPools == nil {
		return 0, 0
	}
	var controlledIPPools spiderpoolv2beta1.PoolIPPreAllocations
	err := json.Unmarshal([]byte(*subnet.Status.ControlledIPPools), &controlledIPPools)
	if nil != err {
		return 0, 0
	}

	// allocated IP Count
	var allocatedIPCount int64
	for _, poolAllocation := range controlledIPPools {
		tmpIPs, _ := spiderpoolip.ParseIPRanges(*subnet.Spec.IPVersion, poolAllocation.IPs)
		allocatedIPCount += int64(len(tmpIPs))
	}
	return int64(s.TotalIPInt()), allocatedIPCount
}

[weizhoublue]

同理， subnet 对象的统计 也可能会有问题

子网根据实际的来算的

func subnetStatusCount(subnet *spiderpoolv2beta1.SpiderSubnet) (totalCount, allocatedCount int64) {
	s, _ := spiderpoolip.NewCIDR(subnet.Spec.Subnet, subnet.Spec.IPs, subnet.Spec.ExcludeIPs)

	if subnet.Status.ControlledIPPools == nil {
		return 0, 0
	}
	var controlledIPPools spiderpoolv2beta1.PoolIPPreAllocations
	err := json.Unmarshal([]byte(*subnet.Status.ControlledIPPools), &controlledIPPools)
	if nil != err {
		return 0, 0
	}

	// allocated IP Count
	var allocatedIPCount int64
	for _, poolAllocation := range controlledIPPools {
		tmpIPs, _ := spiderpoolip.ParseIPRanges(*subnet.Spec.IPVersion, poolAllocation.IPs)
		allocatedIPCount += int64(len(tmpIPs))
	}
	return int64(s.TotalIPInt()), allocatedIPCount
}

为什么不 两个拉齐 一个算法
都使用 切片长度

[ty-dc]

为什么不 两个拉齐 一个算法
都使用 切片长度

开始修改就是对齐子网，使用切片长度来算的，然后被你否了。你说没找到根因怎么知道修改是否有效？

[cyclinder]

建议 PR 加些描述，方便 release-notes 展示

[ty-dc]

建议 PR 加些描述，方便 release-notes 展示

done