scheduler: improve NodeNUMAResource handling node cpu bind policy

[eahydra]

Ⅰ. Describe what this PR does

If the node enables label node.koordinator.sh/cpu-bind-policy and configures the corresponding value to FullPCPUsOnly or SpreadByPCPUs, then the scheduler should ensure that the allocation result corresponding to ResourceSpec.RequiredCPUBindPolicy is consistent.

And the node policy may conflict with the ResourceSpec.RequiredCPUBindPolicy defined in the Pod. For example: the node defined the policy with FullPCPUsOnly, and the Pod required SpreadByPCPUs, this creates a conflict that should be filtered.

If the Pod does not declare the ResourceSpec with CPUBindPolicy(either required or preferred), and the node declares the CPUBindPolicy, the scheduler should allocate CPUs by node's CPUBindPolicy, even though the Pod's QoSClass is LS. Although the previous implementation could work, but the implementation was not complete enough.
As for whether koordlet or kubelet will set the cgroup according to the scheduler allocation result on the LS Pod, I think this is a strategic issue.

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it

Ⅳ. Special notes for reviews

V. Checklist

• I have written necessary docs and comments
• I have added necessary unit tests and integration tests
• All checks passed in make test

[eahydra]

/hold

[codecov]

Codecov Report

Attention: Patch coverage is 75.52448% with 35 lines in your changes are missing coverage. Please review.

Project coverage is 67.53%. Comparing base (93f2bc2) to head (d465bf3).
Report is 17 commits behind head on main.

Files	Patch %	Lines
pkg/scheduler/plugins/nodenumaresource/plugin.go	74.07%	13 Missing and 8 partials ⚠️
pkg/scheduler/plugins/nodenumaresource/scoring.go	60.00%	4 Missing and 2 partials ⚠️
...cheduler/plugins/nodenumaresource/topology_hint.go	53.84%	4 Missing and 2 partials ⚠️
...duler/plugins/nodenumaresource/resource_manager.go	50.00%	0 Missing and 1 partial ⚠️
pkg/scheduler/plugins/nodenumaresource/service.go	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1892   +/-   ##
=======================================
  Coverage   67.52%   67.53%           
=======================================
  Files         413      413           
  Lines       46072    46115   +43     
=======================================
+ Hits        31111    31143   +32     
- Misses      12705    12710    +5     
- Partials     2256     2262    +6     

Flag	Coverage Δ
unittests	67.53% <75.52%> (+<0.01%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[eahydra]

/hold cancel

PTAL

[hormes]

/lgtm
/approve

[koordinator-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hormes

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• pkg/scheduler/OWNERS [hormes]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment