Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

[OCTRL-1076] Unusual number of goroutines stuck in controllable task Kill in executor#783

Merged
knopers8 merged 1 commit intomasterAliceO2Group/Control:masterfrom
fix_controllable_killAliceO2Group/Control:fix_controllable_killCopy head branch name to clipboard
Dec 11, 2025
Merged

[OCTRL-1076] Unusual number of goroutines stuck in controllable task Kill in executor#783
knopers8 merged 1 commit intomasterAliceO2Group/Control:masterfrom
fix_controllable_killAliceO2Group/Control:fix_controllable_killCopy head branch name to clipboard

Conversation

@justonedev1
Copy link
Collaborator

@justonedev1 justonedev1 commented Dec 10, 2025

part of OCTRL-1076, if OCC transition of a task is not fast enough it timeouts and leaks the goroutine which waits until commitDone channel is read. I changed the channel to buffered one as we don't care to wait until the commit is read.

@justonedev1 justonedev1 changed the title [core] part of OCTRL-1076, fixed leaked controllable task Kill goroutine [OCTRL-1076] Unusual number of goroutines stuck in controllable task Kill in executor Dec 10, 2025
@knopers8 knopers8 merged commit 2317e45 into master Dec 11, 2025
4 checks passed
@knopers8 knopers8 deleted the fix_controllable_kill branch December 11, 2025 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.