Skip to content

bpo-36368: Ignore SIGINT in SharedMemoryManager servers.#12483

Merged
pitrou merged 11 commits intopython:masterfrom
pierreglaser:ignore-SIGINT-in-shared-memory-server
May 10, 2019
Merged

bpo-36368: Ignore SIGINT in SharedMemoryManager servers.#12483
pitrou merged 11 commits intopython:masterfrom
pierreglaser:ignore-SIGINT-in-shared-memory-server

Conversation

@pierreglaser
Copy link
Contributor

@pierreglaser pierreglaser commented Mar 21, 2019 •
edited
Loading

When starting a SharedMemoryManager in an interactive session, any KeyboardInterrupt event will be transmitted to the (sub)process running the shared memory server, which causes the Manager to be unusable thereafter.

This PR catches SIGINT signals in server processes.

https://bugs.python.org/issue36368

@pierreglaser pierreglaser changed the title bpo-36338: Ignore SIGINT in SharedMemoryManager servers. bpo-36368: Ignore SIGINT in SharedMemoryManager servers. Mar 21, 2019
@pierreglaser
Copy link
Contributor Author

pierreglaser commented Mar 22, 2019

FYI: I could not simulate a console CTRL-C event on windows in a test, so I skipped the test on Windows. I did test the fix locally on a Windows 10 VM in an interactive session, and it works.

jdemeyer
jdemeyer reviewed Mar 27, 2019
View reviewed changes
jdemeyer
jdemeyer reviewed Mar 27, 2019
View reviewed changes
jdemeyer
jdemeyer requested changes Mar 27, 2019
View reviewed changes
Copy link
Contributor

@jdemeyer jdemeyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See 2 comments

jdemeyer
jdemeyer reviewed Mar 27, 2019
View reviewed changes
@brettcannon brettcannon added the type-bug An unexpected behavior, bug, or error label Mar 27, 2019
@pitrou
Copy link
Member

pitrou commented May 10, 2019

You may want to rebase/merge this. Also, feel free to ping when you need a core review :-)

@pierreglaser pierreglaser force-pushed the ignore-SIGINT-in-shared-memory-server branch from ec3cfdc to 01a1c0f Compare May 10, 2019 17:44
@pierreglaser
Copy link
Contributor Author

pierreglaser commented May 10, 2019

Rebased.

@pierreglaser pierreglaser mentioned this pull request May 10, 2019
Merged
pitrou
pitrou approved these changes May 10, 2019
View reviewed changes
Copy link
Member

@pitrou pitrou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@pitrou pitrou merged commit d0d64ad into python:master May 10, 2019
@pierreglaser pierreglaser deleted the ignore-SIGINT-in-shared-memory-server branch May 10, 2019 18:43
@matrixise
Copy link
Member

matrixise commented May 15, 2019

@pierreglaser I think this merged PR was not for the right BPO.

https://bugs.python.org/issue36338 is "urlparse of urllib returns wrong hostname"

@pierreglaser
Copy link
Contributor Author

pierreglaser commented May 15, 2019

@matrixise sorry to hear that. The issue number looks correct though. Maybe I mistyped it when I opened the PR and quickly fixed it afterwards. Is there any way I can help fix that?

@matrixise
Copy link
Member

matrixise commented May 15, 2019

just take care for the next time ;-) I just wanted to check my tests with your PR for that BPO and the tests have failed ;-)

I think there is no fix for this issue because your PR has been merged into master.

Thanks again for your PR and your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pitrou pitrou pitrou approved these changes

@jdemeyer jdemeyer jdemeyer approved these changes

Assignees

No one assigned

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@pierreglaser @pitrou @matrixise @jdemeyer @brettcannon @the-knights-who-say-ni @bedevere-bot