Skip to content

Conversation

bbcorp
Copy link
Contributor

@bbcorp bbcorp commented Aug 18, 2025

Closes #5259

📑 Description

Alert enrichment: I've noticed when you acknowledge an incident, it acknowledges the alert too by adding the enrichment "status": "acknowledged" but all other previous enrichments are gone. Same goes when you resolve and incident.

Now in batch_enrich function, we merge enrichments before committing them to the db.

✅ Checks

  • My pull request adheres to the code style of this project
  • All the tests have passed

@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Aug 18, 2025
Copy link

vercel bot commented Aug 18, 2025

@bbcorp is attempting to deploy a commit to the KeepHQ Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added the Bug Something isn't working label Aug 18, 2025
Copy link

vercel bot commented Aug 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
keep Ignored Ignored Preview Aug 19, 2025 6:49am

talboren
talboren previously approved these changes Aug 19, 2025
Copy link
Member

@talboren talboren left a comment

Choose a reason for hiding this comment

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

lgtm though it will impact performance in large bulk updates

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 19, 2025
@bbcorp
Copy link
Contributor Author

bbcorp commented Aug 19, 2025

Thats what I thought too, do you see a way to make it in bulk SQL query anyway?
I was thinking we could merge at code level but then I wonder how we could do the SQL in bulk?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[🐛 Bug]: [Alert enrichment] when changing status of incident, other enrichments of alerts are gone
2 participants