leoluz commented 2 years ago

This PR introduces one additional configuration to the SyncContext allowing the manager to be specified and used in server-side apply syncs.

Relates to https://github.com/argoproj/argo-cd/issues/2267

Signed-off-by: Leonardo Luz Almeida leonardo_almeida@intuit.com

codecov[bot] commented 2 years ago

Codecov Report

Merging #418 (57c4f66) into master (67ddccd) will decrease coverage by 0.38%. The diff coverage is 41.11%.

@@            Coverage Diff             @@
##           master     #418      +/-   ##
==========================================
- Coverage   54.42%   54.03%   -0.39%     
==========================================
  Files          41       41              
  Lines        3107     3194      +87     
==========================================
+ Hits         1691     1726      +35     
- Misses       1249     1286      +37     
- Partials      167      182      +15

Impacted Files	Coverage Δ
pkg/diff/diff_options.go	`55.55% <0.00%> (-27.78%)`	:arrow_down:
pkg/utils/kube/resource_ops.go	`0.00% <0.00%> (ø)`
pkg/sync/sync_context.go	`70.86% <33.33%> (-0.24%)`	:arrow_down:
pkg/diff/diff.go	`60.68% <47.36%> (-2.93%)`	:arrow_down:

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 67ddccd...57c4f66. Read the comment docs.

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
3 Code Smells

No Coverage information
0.1% Duplication

argoproj / gitops-engine

feat: Add server-side apply manager config #418

Codecov Report