Skip to content

fix: flush session after incident creation in topology processor#5660

Closed
themavik wants to merge 3 commits intokeephq:mainfrom
themavik:fix-topology-incident-fk-violation
Closed

fix: flush session after incident creation in topology processor#5660
themavik wants to merge 3 commits intokeephq:mainfrom
themavik:fix-topology-incident-fk-violation

Conversation

@themavik
Copy link
Contributor

@themavik themavik commented Feb 20, 2026

Fixes #5463

Adds session.flush() after creating incidents in the topology processor to ensure the incident ID is available in the database before linking alerts via LastAlertToIncident, preventing FK constraint violations.

@vercel
Copy link

vercel bot commented Feb 20, 2026

@themavik 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 size:XS This PR changes 0-9 lines, ignoring generated files. label Feb 20, 2026
@CLAassistant
Copy link

CLAassistant commented Feb 20, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ shahargl
❌ themavik
You have signed the CLA already but the status is still pending? Let us recheck it.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 20, 2026

Target branch is not in the allowed branches list.

@dosubot dosubot bot added the Bug Something isn't working label Feb 20, 2026
@shahargl
Copy link
Member

shahargl commented Mar 2, 2026

@themavik is this relevant?

@themavik themavik closed this Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🐛 Bug]: Topology Correlation not working

3 participants