webhook: more checks for netqos

[lucming]

Ⅰ. Describe what this PR does

ralated issue: #1866

The absolute value scenario is not handled, because it requires the actual network bandwidth of the node, which is not yet available here

Ⅱ. 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

[codecov]

Codecov Report

Attention: 49 lines in your changes are missing coverage. Please review.

Comparison is base (a056ccb) 67.26% compared to head (f0d5309) 67.56%.
Report is 5 commits behind head on main.

Files	Patch %	Lines
...bhook/cm/plugins/sloconfig/resource_qos_checker.go	68.58%	33 Missing and 16 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1878      +/-   ##
==========================================
+ Coverage   67.26%   67.56%   +0.30%     
==========================================
  Files         411      413       +2     
  Lines       45848    46206     +358     
==========================================
+ Hits        30838    31221     +383     
+ Misses      12778    12724      -54     
- Partials     2232     2261      +29     

Flag	Coverage Δ
unittests	67.56% <68.58%> (+0.30%)	⬆️

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.

[zwzhang0107]

the node total bandwidth is configured at https://github.com/koordinator-sh/koordinator/blob/main/apis/slo/v1alpha1/nodeslo_types.go#L381
also, the absolute format and percent fomat cannot mix used.

[lucming]

the node total bandwidth is configured at https://github.com/koordinator-sh/koordinator/blob/main/apis/slo/v1alpha1/nodeslo_types.go#L381 also, the absolute format and percent fomat cannot mix used.

but nodeslo also relies on this configmap

[zwzhang0107]

the node total bandwidth is configured at https://github.com/koordinator-sh/koordinator/blob/main/apis/slo/v1alpha1/nodeslo_types.go#L381 also, the absolute format and percent fomat cannot mix used.

but nodeslo also relies on this configmap

the struct is also imported in configmap https://github.com/koordinator-sh/koordinator/blob/main/apis/configuration/slo_controller_config.go#L88

[zwzhang0107]

@eahydra PTAL

[eahydra]

/lgtm
/approve

[koordinator-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: eahydra

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/webhook/OWNERS [eahydra]

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