Skip to content

Conversation

@agaudreault
Copy link
Member

When an app has error and is expired, it will constantly try to reconcile with level 2 because the lastTransitionTime is always updated.

Related to #23018

@agaudreault agaudreault requested a review from a team as a code owner May 19, 2025 17:00
agaudreault and others added 7 commits May 19, 2025 13:05
Signed-off-by: Alexandre Gaudreault <[email protected]>
Signed-off-by: Alexandre Gaudreault <[email protected]>
Signed-off-by: Alexandre Gaudreault <[email protected]>
Signed-off-by: Alexandre Gaudreault <[email protected]>
@agaudreault agaudreault enabled auto-merge (squash) May 20, 2025 14:36
@agaudreault agaudreault merged commit 5bc6f47 into argoproj:release-2.14 May 20, 2025
12 of 13 checks passed
@codecov
Copy link

codecov bot commented May 20, 2025

Codecov Report

❌ Patch coverage is 91.66667% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 55.31%. Comparing base (3a9ab77) to head (e8dbb1a).
⚠️ Report is 22 commits behind head on release-2.14.

Files with missing lines Patch % Lines
controller/state.go 50.00% 1 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           release-2.14   #23047      +/-   ##
================================================
- Coverage         55.32%   55.31%   -0.02%     
================================================
  Files               339      339              
  Lines             57355    57353       -2     
================================================
- Hits              31733    31726       -7     
- Misses            22927    22937      +10     
+ Partials           2695     2690       -5     

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

@agaudreault
Copy link
Member Author

/cherry-pick release-3.0

@gcp-cherry-pick-bot
Copy link

Cherry-pick failed with Merge error 5bc6f4722bf056e3d02ce3479800e50d2637accb into temp-cherry-pick-4f2fda-release-3.0

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.

2 participants