Skip to content

ref(aci): Minor cleanup to delayed workflow processing#115758

Merged
saponifi3d merged 2 commits into
masterfrom
jcallender/aci/reduce-tracking
May 18, 2026
Merged

ref(aci): Minor cleanup to delayed workflow processing#115758
saponifi3d merged 2 commits into
masterfrom
jcallender/aci/reduce-tracking

Conversation

@saponifi3d
Copy link
Copy Markdown
Contributor

Description

  • Change an info log to debug to reduce the logs by ~5M per day
  • Only invoke fire_actions_for_groups if there are actions to trigger; approx 50% of the calls to this method there aren't groups_to_dcgs or group_to_group_event.

…educe tracking / debug logs -- turns out this is about 50% of the calls to fire_actions_for_groups.
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 18, 2026
@saponifi3d saponifi3d marked this pull request as ready for review May 18, 2026 21:45
@saponifi3d saponifi3d requested a review from a team as a code owner May 18, 2026 21:45
@saponifi3d saponifi3d requested a review from kcons May 18, 2026 21:45
)

fire_actions_for_groups(project.organization, groups_to_dcgs, group_to_groupevent)
if groups_to_dcgs and group_to_groupevent:
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this should be a noop since it's already handled internally, but this will remove all the .info logs and the timing information when there isn't anything to fire.

@saponifi3d saponifi3d merged commit c4247ed into master May 18, 2026
64 checks passed
@saponifi3d saponifi3d deleted the jcallender/aci/reduce-tracking branch May 18, 2026 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants