gitops-engine icon indicating copy to clipboard operation
gitops-engine copied to clipboard

fix: Fix argument order in resource filter

Open jannfis opened this issue 3 years ago • 2 comments

The signature for the resourcesFilter function in the sync context is

resourcesFilter        func(key kube.ResourceKey, target *unstructured.Unstructured, live *unstructured.Unstructured) bool

but it's called with live and target exchanged.

This is a prerequisite for fixing https://github.com/argoproj/argo-cd/issues/8683

Signed-off-by: jannfis [email protected]

jannfis avatar Aug 04 '22 11:08 jannfis

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
1.4% 1.4% Duplication

sonarqubecloud[bot] avatar Aug 04 '22 11:08 sonarqubecloud[bot]

Codecov Report

Merging #436 (f86f321) into master (6cde798) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #436   +/-   ##
=======================================
  Coverage   53.96%   53.96%           
=======================================
  Files          41       41           
  Lines        3189     3189           
=======================================
  Hits         1721     1721           
  Misses       1286     1286           
  Partials      182      182           
Impacted Files Coverage Δ
pkg/sync/sync_context.go 70.86% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Aug 04 '22 11:08 codecov[bot]