Skip to content

fix(appset): progressive sync loop when applciation has sync errors #23186

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

Open
wants to merge 17 commits into
base: release-3.0
Choose a base branch
from

Conversation

agaudreault
Copy link
Member

@agaudreault agaudreault commented May 28, 2025

Actual PR on master #23187

  • When an application is stuck in OutOfSync due to sync failures or misconfigured ignoredDifferences, etc. then only sync it once. If it has been synced at least once for the targetRevision and it is still OutOfSync, mmove it to Progressing and wait for a manual/magical intervention. User can/should configure sync retry on their applications.
  • When an application can reconcile, but has invalid conditions, set it to progressing until a manual intervention happen.
  • When an application cannot reconcile, usually it is because of an InvalidSpec/project/namespace configuration, then it is not considered as a valid app and it is left in pending until the errors are resolved and can reconcile.
  • Remove the documented feature that application are move automatically to Healthy becasue the code does not exist for that.
  • Sort ApplicationStatus by application name so it is consistent
  • Sort the conditions by type so it is consistent
  • Do not remove the Progressive Sync conditions when the Application is configured for progressive sync.
  • Add info logs for all progressive sync transitions

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]>
Signed-off-by: Alexandre Gaudreault <[email protected]>
Signed-off-by: Alexandre Gaudreault <[email protected]>
Signed-off-by: Alexandre Gaudreault <[email protected]>
Copy link

codecov bot commented May 28, 2025

Codecov Report

Attention: Patch coverage is 78.73754% with 64 lines in your changes missing coverage. Please review.

Project coverage is 55.96%. Comparing base (5c24b6b) to head (1d23403).

Files with missing lines Patch % Lines
...cationset/controllers/applicationset_controller.go 79.85% 54 Missing and 2 partials ⚠️
.../apis/application/v1alpha1/applicationset_types.go 65.21% 6 Missing and 2 partials ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##           release-3.0   #23186      +/-   ##
===============================================
+ Coverage        55.89%   55.96%   +0.06%     
===============================================
  Files              343      343              
  Lines            57395    57446      +51     
===============================================
+ Hits             32080    32148      +68     
+ Misses           22669    22649      -20     
- Partials          2646     2649       +3     

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

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 marked this pull request as ready for review May 30, 2025 16:47
@agaudreault agaudreault requested review from a team as code owners May 30, 2025 16:47
Signed-off-by: Alexandre Gaudreault <[email protected]>
Signed-off-by: Alexandre Gaudreault <[email protected]>
Signed-off-by: Alexandre Gaudreault <[email protected]>
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.

1 participant