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

[releases/1.2] Fix the discovery service to not terminate on parent process closure #391

Merged

Conversation

samchris007
Copy link
Contributor

What does this Pull Request accomplish?

Cherry-pick change to fix the issue with the discovery service termination when terminating/closing the measurement service #383

Why should this Pull Request be merged?

Fixes Bug 2508628: Python measurements - Discovery service is not completely detached from VSCode console/terminal and terminates on terminal closure

@samchris007 samchris007 changed the title Users/sam/fixdiscovery termination parentprocessclosure [releases/1.2] Fix the discovery service to not terminate on parent process closure Sep 21, 2023
@github-actions
Copy link

github-actions bot commented Sep 21, 2023

Test Results

     12 files  ±0       12 suites  ±0   1m 58s ⏱️ -1s
   194 tests ±0     165 ✔️ ±0    29 💤 ±0  0 ±0 
2 316 runs  ±0  1 968 ✔️ ±0  348 💤 ±0  0 ±0 

Results for commit 38be10c. ± Comparison against base commit a9d542d.

♻️ This comment has been updated with latest results.

@dixonjoel dixonjoel merged commit 8656f5a into releases/1.2 Sep 21, 2023
17 checks passed
@dixonjoel dixonjoel deleted the users/sam/fixdiscovery-termination-parentprocessclosure branch September 21, 2023 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants