fabric8-analytics-server icon indicating copy to clipboard operation
fabric8-analytics-server copied to clipboard

[On-HOLD] feat: Ignore vulnerability feature for VA

Open AkshayBhansali18 opened this issue 4 years ago • 5 comments
trafficstars

Ignore vulnerability feature for VA with test cases

AkshayBhansali18 avatar Oct 29 '21 06:10 AkshayBhansali18

Codecov Report

Merging #817 (d32d5c4) into master (a8c33bc) will decrease coverage by 0.13%. The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #817      +/-   ##
==========================================
- Coverage   79.33%   79.19%   -0.14%     
==========================================
  Files          21       21              
  Lines        1679     1692      +13     
==========================================
+ Hits         1332     1340       +8     
- Misses        347      352       +5     
Impacted Files Coverage Δ
bayesian/utility/v2/component_analyses.py 90.00% <61.53%> (-1.83%) :arrow_down:
bayesian/api/api_v2.py 74.47% <100.00%> (+0.13%) :arrow_up:

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 a8c33bc...d32d5c4. Read the comment docs.

codecov-commenter avatar Oct 29 '21 06:10 codecov-commenter

@jparsai Do we really need this for VA? VA is very specific for clair.

rafiu007 avatar Oct 29 '21 06:10 rafiu007

@rafiu007 Dont we want to ignore in Clair?

jparsai avatar Oct 29 '21 06:10 jparsai

@rafiu007 Dont we want to ignore in Clair?

I cant think of a use case where we would like to ignore. By default clair serves all packages

rafiu007 avatar Oct 29 '21 06:10 rafiu007

Okay, for now we can keep this PR open, later if Clair requires ignore feature we can take this PR in.

jparsai avatar Oct 29 '21 06:10 jparsai