fix: analysis runs to wait for all metrics to complete

[harikrongali]

Signed-off-by: hari rongali hari_rongali@intuit.com

fixes #1360

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.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

No Coverage information
0.0% Duplication

[codecov]

Codecov Report

Merging #1407 (86d3e88) into master (7a0704a) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1407      +/-   ##
==========================================
- Coverage   81.27%   81.25%   -0.03%     
==========================================
  Files         108      108              
  Lines        9950     9955       +5     
==========================================
+ Hits         8087     8089       +2     
- Misses       1309     1311       +2     
- Partials      554      555       +1     

Impacted Files	Coverage Δ
analysis/analysis.go	85.87% <100.00%> (-0.68%)	⬇️

---

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 cf19b5b...86d3e88. Read the comment docs.

[jessesuen]

Nice fix!