You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When enabling Progressive Sync, the applicationset-controller spams logs even if the AppSet only contains 2 elements. The problem is exponentially worse if there are multiple AppSets and/or more than a few elements in the set. I assume this is because there's no delay in the for-loop.
This is problematic and costly for anyone exporting logs to Grafana/Datadog/etc. But also the spam effectively makes the appSet controller logs useless from a triage/debugging standpoint.
To Reproduce
Enable Progressive Sync on the appset controller, example helm values:
AppSet logs should still be usable, spammy logs should be pushed to higher log levels.
Or perhaps progressive sync's step list should be evaluated on a delay, configurable or otherwise.
Version
Chart Version: 5.54.0
App Version: v2.9.6
Logs
this section is spammed until sync is totally complete
Checklist:
argocd version
.Describe the bug
When enabling Progressive Sync, the applicationset-controller spams logs even if the AppSet only contains 2 elements. The problem is exponentially worse if there are multiple AppSets and/or more than a few elements in the set. I assume this is because there's no delay in the for-loop.
This is problematic and costly for anyone exporting logs to Grafana/Datadog/etc. But also the spam effectively makes the appSet controller logs useless from a triage/debugging standpoint.
To Reproduce
Enable Progressive Sync on the appset controller, example helm values:
Create some arbitrary ApplicationSet that uses
RollingSync
strategy:Expected behavior
AppSet logs should still be usable, spammy logs should be pushed to higher log levels.
Or perhaps progressive sync's step list should be evaluated on a delay, configurable or otherwise.
Version
Chart Version: 5.54.0
App Version: v2.9.6
Logs
this section is spammed until sync is totally complete
The text was updated successfully, but these errors were encountered: