feat(qrm-plugins): synchronous GetAdvice API between qrm-plugins and sys-advisor

[gary-lgy]

What type of PR is this?

Replace ListAndWatch api from sysadvisor with synchronous GetAdvice rpc.

What this PR does / why we need it:

Which issue(s) this PR fixes:

Special notes for your reviewer:

[codecov]

Codecov Report

Attention: Patch coverage is 67.41935% with 202 lines in your changes missing coverage. Please review.

Project coverage is 60.54%. Comparing base (161358c) to head (8274d33).
Report is 7 commits behind head on main.

Files with missing lines	Patch %	Lines
...nt/sysadvisor/plugin/qosaware/server/cpu_server.go	62.20%	47 Missing and 18 partials ⚠️
...ins/memory/dynamicpolicy/policy_advisor_handler.go	58.69%	33 Missing and 5 partials ⚠️
...sysadvisor/plugin/qosaware/server/memory_server.go	42.18%	30 Missing and 7 partials ⚠️
...lugins/cpu/dynamicpolicy/policy_advisor_handler.go	75.43%	23 Missing and 5 partials ⚠️
...g/agent/qrm-plugins/memory/dynamicpolicy/policy.go	64.10%	12 Missing and 2 partials ⚠️
pkg/agent/qrm-plugins/commonstate/pool.go	76.66%	4 Missing and 3 partials ⚠️
pkg/agent/qrm-plugins/cpu/dynamicpolicy/policy.go	93.87%	2 Missing and 1 partial ⚠️
cmd/katalyst-agent/app/options/qrm/cpu_plugin.go	50.00%	2 Missing ⚠️
...md/katalyst-agent/app/options/qrm/memory_plugin.go	80.00%	2 Missing ⚠️
...g/agent/qrm-plugins/advisorsvc/advisor_svc_stub.go	0.00%	2 Missing ⚠️
... and 2 more

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #747      +/-   ##
==========================================
+ Coverage   56.78%   60.54%   +3.75%     
==========================================
  Files         601      598       -3     
  Lines       69829    62933    -6896     
==========================================
- Hits        39655    38100    -1555     
+ Misses      25927    20985    -4942     
+ Partials     4247     3848     -399     

Flag	Coverage Δ
unittest	60.54% <67.41%> (+3.75%)	⬆️

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.

[luomingmeng]

it can also use generic match such as "*.pb.go"

[luomingmeng]

whether it will dead lock for it acquiring metaCache lock multiple times?

[gary-lgy]

no, line 141 acquires metaCache.Mutex, while line 153 acquires metaCache.podMutex.