Skip to content

Re-register all triggers when a capability LOOPP restarts #1763

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
bolekk merged 2 commits into from
Jan 15, 2026
Merged

Re-register all triggers when a capability LOOPP restarts #1763

bolekk merged 2 commits into from
Jan 15, 2026

Conversation

@bolekk
Copy link
Contributor

@bolekk bolekk commented Jan 9, 2026 •
edited
Loading

Updating an underlying trigger capability (e.g. after a crash or job spec update) requires re-playing all trigger registrations.

@github-actions
Copy link

github-actions bot commented Jan 9, 2026 •
edited
Loading

✅ API Diff Results - No breaking changes

📄 View full apidiff report

@bolekk bolekk force-pushed the base_cap_trigger_resubs branch from 96866ed to 00d5782 Compare January 9, 2026 02:14
@bolekk bolekk changed the title Re-subscribe all triggers when a capability LOOPP restarts [WIP] Re-subscribe all triggers when a capability LOOPP restarts Jan 9, 2026
@bolekk bolekk force-pushed the base_cap_trigger_resubs branch from 00d5782 to edb1eae Compare January 15, 2026 03:06
@bolekk bolekk force-pushed the base_cap_trigger_resubs branch from edb1eae to e3ed6cd Compare January 15, 2026 03:40
@bolekk bolekk force-pushed the base_cap_trigger_resubs branch from e3ed6cd to 199c514 Compare January 15, 2026 03:49
@bolekk bolekk force-pushed the base_cap_trigger_resubs branch from 199c514 to 99841b5 Compare January 15, 2026 04:07
@bolekk bolekk changed the title [WIP] Re-subscribe all triggers when a capability LOOPP restarts Re-subscribe all triggers when a capability LOOPP restarts Jan 15, 2026
@bolekk bolekk changed the title Re-subscribe all triggers when a capability LOOPP restarts Re-register all triggers when a capability LOOPP restarts Jan 15, 2026
@bolekk bolekk force-pushed the base_cap_trigger_resubs branch from 99841b5 to 227075b Compare January 15, 2026 04:13
@bolekk bolekk force-pushed the base_cap_trigger_resubs branch from 227075b to 8de2df6 Compare January 15, 2026 04:18
@bolekk bolekk marked this pull request as ready for review January 15, 2026 04:19
@bolekk bolekk requested a review from a team as a code owner January 15, 2026 04:19
@bolekk bolekk enabled auto-merge January 15, 2026 18:52
@bolekk bolekk added this pull request to the merge queue Jan 15, 2026
Merged via the queue into main with commit 7777312 Jan 15, 2026
35 of 36 checks passed
@bolekk bolekk deleted the base_cap_trigger_resubs branch January 15, 2026 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jmank88 jmank88 jmank88 approved these changes

@MStreet3 MStreet3 MStreet3 approved these changes

@Unheilbar Unheilbar Unheilbar approved these changes

@krehermann krehermann Awaiting requested review from krehermann

@fernandezlautaro fernandezlautaro Awaiting requested review from fernandezlautaro

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@bolekk @jmank88 @MStreet3 @Unheilbar