Closed leoluz closed 2 years ago
Merging #460 (50e1255) into master (517c1ff) will increase coverage by
0.01%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #460 +/- ##
==========================================
+ Coverage 55.38% 55.40% +0.01%
==========================================
Files 41 41
Lines 4478 4480 +2
==========================================
+ Hits 2480 2482 +2
Misses 1807 1807
Partials 191 191
Impacted Files | Coverage Δ | |
---|---|---|
pkg/diff/diff.go | 60.88% <100.00%> (+0.12%) |
:arrow_up: |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
Kudos, SonarCloud Quality Gate passed!
0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells
No Coverage information
0.0% Duplication
When applying resources with SSA, sometimes last-applied-configuration are present in the live state but not always. This causes the diff to sometimes show the annotation as removed from the desired state. This is incorrect as desired state should never have the last-applied-configuration annotation provided.
To fix this bug, I am removing the annotation from the live resource as well as from the predicted live so diffs can be displayed correctly.
Signed-off-by: Leonardo Luz Almeida leonardo_almeida@intuit.com