Skip to content
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.

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

fix: add contexts to clean up fsnotify goroutines #2427

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

rbtr
Copy link
Contributor

@rbtr rbtr commented Dec 6, 2023

Reason for Change:

Issue Fixed:

Requirements:

Notes:

@rbtr rbtr requested a review from a team as a code owner December 6, 2023 18:38
@rbtr rbtr self-assigned this Dec 6, 2023
@rbtr rbtr added cns Related to CNS. fix Fixes something. release/1.4 Change affects v1.4 release train release/latest Change affects latest release train needs-backport Change needs to be backported to previous release trains labels Dec 6, 2023
@rbtr rbtr force-pushed the fix/async-goroutine-leak branch 2 times, most recently from 611ff3a to 2d8b331 Compare December 6, 2023 19:06
@rbtr rbtr requested a review from a team as a code owner December 6, 2023 21:10
@rbtr rbtr force-pushed the fix/async-goroutine-leak branch 2 times, most recently from 9bf8433 to 8341849 Compare December 6, 2023 21:46
@rbtr rbtr force-pushed the fix/async-goroutine-leak branch from 8341849 to 7bddc7f Compare December 6, 2023 22:27
@rbtr rbtr force-pushed the fix/async-goroutine-leak branch from 7bddc7f to d7f90c1 Compare December 6, 2023 23:22
ramiro-gamarra
ramiro-gamarra previously approved these changes Dec 6, 2023
@rbtr rbtr enabled auto-merge (squash) December 6, 2023 23:25
camrynl
camrynl previously approved these changes Dec 6, 2023
@rbtr rbtr requested a review from neaggarwMS December 7, 2023 16:08
@rbtr rbtr force-pushed the fix/async-goroutine-leak branch from d7f90c1 to e4b246d Compare December 7, 2023 16:50
@rbtr rbtr requested a review from tyler-lloyd December 7, 2023 17:40
@rbtr rbtr dismissed stale reviews from ramiro-gamarra and camrynl via e603778 December 7, 2023 19:08
@rbtr rbtr force-pushed the fix/async-goroutine-leak branch from e603778 to cf3e738 Compare December 8, 2023 15:56
@rbtr rbtr requested a review from huntergregory December 12, 2023 15:13
@rbtr rbtr force-pushed the fix/async-goroutine-leak branch from cf3e738 to 3b4393d Compare December 12, 2023 15:13
@rbtr rbtr merged commit 7dd8342 into master Dec 12, 2023
@rbtr rbtr deleted the fix/async-goroutine-leak branch December 12, 2023 21:55
@rbtr rbtr mentioned this pull request Dec 13, 2023
4 tasks
paulyufan2 pushed a commit that referenced this pull request Dec 18, 2023
@rbtr rbtr removed release/1.4 Change affects v1.4 release train needs-backport Change needs to be backported to previous release trains labels Dec 19, 2023
matmerr pushed a commit that referenced this pull request Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cns Related to CNS. fix Fixes something. release/latest Change affects latest release train
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants