refactor podUpdater interface

[dbug-dk]

2. fix bug: replace pod by to-replace label, not check new pod status when pod upgrade policy is not replace update

1. Does this PR affect any open issues?(Y/N) and add issue references

fix #182

• N
• Y

2. What is the scope of this PR (e.g. component or file name):

collaset_controller reconcile:

1. syncPods
2. update

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

• Affects user behaviors
• Contains syntax changes
• Contains variable changes
• Contains experimental features
• Performance regression: Consumes more CPU
• Performance regression: Consumes more Memory
• Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

• N
• Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

• Unit test
• Integration test
• Benchmark (add benchmark stats below)
• Manual test (add detailed scripts or steps below)
• Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.

None

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.99%. Comparing base (25ff219) to head (a3f2782).
Report is 2 commits behind head on main.

❗ Current head a3f2782 differs from pull request most recent head ba3463b. Consider uploading reports for the commit ba3463b to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #166      +/-   ##
==========================================
- Coverage   65.66%   63.99%   -1.67%     
==========================================
  Files          74       73       -1     
  Lines        4817     4711     -106     
==========================================
- Hits         3163     3015     -148     
- Misses       1370     1419      +49     
+ Partials      284      277       -7     

Flag	Coverage Δ
unittests	63.99% <ø> (-1.67%)	⬇️

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.

[dbug-dk]

@wu8685 code modified by the cr comments，please review again~

[ColdsteelRail]

lgtm

[wu8685]

Shouldn't we return false here?

[dbug-dk]

Shouldn't we return false here?

yes, I'll change this.

[wu8685]

lgtm