Skip to content

fix: ArgoCD 2.11 - Loop of PATCH calls to Application objects#19569

Merged
todaywasawesome merged 1 commit intoargoproj:release-2.9from
alexmt:patch-loop-fix-release-2.9-cherry-pick
Aug 21, 2024
Merged

fix: ArgoCD 2.11 - Loop of PATCH calls to Application objects#19569
todaywasawesome merged 1 commit intoargoproj:release-2.9from
alexmt:patch-loop-fix-release-2.9-cherry-pick

Conversation

@alexmt
Copy link
Collaborator

@alexmt alexmt commented Aug 16, 2024

PR cherrypicks #19340 into release-2.9

@alexmt alexmt requested a review from a team as a code owner August 16, 2024 18:58
@gdsoumya
Copy link
Member

@alexmt same here, CI codegen/lint failed.

…oj#19340)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
@alexmt alexmt force-pushed the patch-loop-fix-release-2.9-cherry-pick branch from d457b43 to 50cc61b Compare August 18, 2024 19:54
@codecov
Copy link

codecov bot commented Aug 18, 2024

Codecov Report

Attention: Patch coverage is 43.75000% with 9 lines in your changes missing coverage. Please review.

Project coverage is 50.13%. Comparing base (212d320) to head (50cc61b).
Report is 6 commits behind head on release-2.9.

Files with missing lines Patch % Lines
controller/appcontroller.go 43.75% 6 Missing and 3 partials ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##           release-2.9   #19569      +/-   ##
===============================================
- Coverage        50.14%   50.13%   -0.02%     
===============================================
  Files              268      268              
  Lines            46370    46409      +39     
===============================================
+ Hits             23252    23266      +14     
- Misses           20864    20884      +20     
- Partials          2254     2259       +5     

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

Copy link
Contributor

@todaywasawesome todaywasawesome left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks for backporting to 2.9

@todaywasawesome todaywasawesome merged commit 90c83dc into argoproj:release-2.9 Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants