fix: Improve and refactor validation for AnalysisTemplates

[khhirani]

Refactor validation for AnalysisTemplates due to the Rollout spec changes made in #1015

Closes #872

Signed-off-by: khhirani [email protected]

[codecov]

Codecov Report

Merging #1117 (28e6d27) into master (32bfe85) will decrease coverage by 0.05%.
The diff coverage is 86.79%.

@@            Coverage Diff             @@
##           master    #1117      +/-   ##
==========================================
- Coverage   81.05%   80.99%   -0.06%     
==========================================
  Files         105      105              
  Lines        9462     9473      +11     
==========================================
+ Hits         7669     7673       +4     
- Misses       1276     1282       +6     
- Partials      517      518       +1     

Impacted Files	Coverage Δ
rollout/controller.go	75.98% <84.61%> (+0.40%)	⬆️
.../apis/rollouts/validation/validation_references.go	76.05% <88.46%> (+3.25%)	⬆️
utils/analysis/helpers.go	70.64% <100.00%> (ø)
rollout/analysis.go	80.60% <0.00%> (-3.05%)	⬇️
rollout/canary.go	77.93% <0.00%> (-0.94%)	⬇️

---

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 32bfe85...28e6d27. Read the comment docs.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
3 Code Smells

No Coverage information
24.8% Duplication