Skip to content

fix(workflow_engine): Remove spammy log#99127

Merged
saponifi3d merged 1 commit into
masterfrom
jcallender/aci/reduce-log-noise
Sep 9, 2025
Merged

fix(workflow_engine): Remove spammy log#99127
saponifi3d merged 1 commit into
masterfrom
jcallender/aci/reduce-log-noise

Conversation

@saponifi3d

Copy link
Copy Markdown
Contributor

Description

Remove the workflow_engine.process_data_sources detectors log which logs each time we successfully get a detector + data source. Removing this log will remove >30,000 logs per minute.

…ide much value, just logging that we're successfully fetching the data source
@saponifi3d saponifi3d requested a review from a team as a code owner September 9, 2025 17:24
@saponifi3d saponifi3d requested a review from ceorourke September 9, 2025 17:24
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 9, 2025
@codecov

codecov Bot commented Sep 9, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #99127   +/-   ##
=======================================
  Coverage   81.26%   81.26%           
=======================================
  Files        8544     8544           
  Lines      377564   377546   -18     
  Branches    24003    24003           
=======================================
+ Hits       306814   306829   +15     
+ Misses      70400    70367   -33     
  Partials      350      350           

@saponifi3d saponifi3d merged commit 5eb68fa into master Sep 9, 2025
64 checks passed
@saponifi3d saponifi3d deleted the jcallender/aci/reduce-log-noise branch September 9, 2025 18:10
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants