Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(controller): bad server-side diffs (#18213) #18840

Merged

Conversation

crenshaw-dev
Copy link
Member

@crenshaw-dev crenshaw-dev commented Jun 27, 2024

gitops-engine PR contains the explanation: argoproj/gitops-engine#587

  1. nil gvkParser was causing panics on using server-side diff
  2. nil gvkParser was causing incorrect diffs when using server-side apply

fixes #18548
fixes #18213
fixes #18020
fixes #17968

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
@crenshaw-dev crenshaw-dev requested review from a team as code owners June 27, 2024 16:32
Copy link

bunnyshell bot commented Jun 27, 2024

❌ Preview Environment deleted from Bunnyshell

Available commands (reply to this comment):

  • 🚀 /bns:deploy to deploy the environment

Copy link

bunnyshell bot commented Jun 27, 2024

❌ Preview Environment deleted from Bunnyshell

Available commands (reply to this comment):

  • 🚀 /bns:deploy to deploy the environment

@crenshaw-dev
Copy link
Member Author

/bns:deploy

@crenshaw-dev crenshaw-dev marked this pull request as draft June 27, 2024 17:37
Copy link

codecov bot commented Jun 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.36%. Comparing base (f358e8d) to head (b378c69).
Report is 504 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #18840      +/-   ##
==========================================
+ Coverage   50.29%   50.36%   +0.07%     
==========================================
  Files         315      315              
  Lines       43127    43182      +55     
==========================================
+ Hits        21690    21748      +58     
+ Misses      18958    18950       -8     
- Partials     2479     2484       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
@crenshaw-dev crenshaw-dev marked this pull request as ready for review June 28, 2024 14:58
@ishitasequeira ishitasequeira merged commit 3d3620e into argoproj:master Jun 29, 2024
27 checks passed
@crenshaw-dev crenshaw-dev deleted the fix-bad-server-side-diffs branch June 29, 2024 15:07
@crenshaw-dev
Copy link
Member Author

/cherry-pick release-2.12

Copy link

Cherry-pick failed with Merge error 3d3620e6a0b5fb0933f7e23290437fa3f0ed84c7 into temp-cherry-pick-cc0f64-release-2.12

ggjulio pushed a commit to ggjulio/argo-cd that referenced this pull request Jul 21, 2024
* fix(controller): bad server-side diffs (argoproj#18213)

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* use upstream commit

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

---------

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
jsolana pushed a commit to jsolana/argo-cd that referenced this pull request Jul 24, 2024
* fix(controller): bad server-side diffs (argoproj#18213)

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

* use upstream commit

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>

---------

Signed-off-by: Michael Crenshaw <350466+crenshaw-dev@users.noreply.github.com>
Signed-off-by: Javier Solana <javier.solana@cabify.com>
Signed-off-by: Javier Solana <javier.solana@cabify.com>
@grzesuav
Copy link

@crenshaw-dev any plans to include it in 2.10.xx patch release ?

@crenshaw-dev
Copy link
Member Author

@grzesuav the fix was cherry-picked here: 9c71133

Should be out as of the 2.10.13 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants