fix: High reconciliation activity and CPU load for invalid rollout

[perenesenko]

Signed-off-by: Andrii Perenesenko [email protected]

Checklist:

• Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
• The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
• I've signed my commits with DCO
• I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
• My builds are green. Try syncing with master if they are not.
• My organization is added to USERS.md.

[perenesenko]

Fix for #2023

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
3.4% Duplication

[codecov]

Codecov Report

Merging #2091 (e0eba31) into master (3f5960e) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2091   +/-   ##
=======================================
  Coverage   82.43%   82.43%           
=======================================
  Files         120      120           
  Lines       17440    17440           
=======================================
  Hits        14376    14376           
  Misses       2352     2352           
  Partials      712      712           

Impacted Files	Coverage Δ
pkg/apis/rollouts/validation/validation.go	94.94% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3f5960e...e0eba31. Read the comment docs.