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 connection reconciliation to handle restarts #882

Merged

Conversation

kevin-bates
Copy link
Member

The changes delivered in #879 did not account for scenarios when ports have changed during restarts - which tends to occur with remote kernels. As a result, we need to reset the connection file when the connection information returned from the provisioner differs from the file's contents, rather than raise a ValueError.

Although this is probably not an issue in typical situations, it is something that should be fixed. I apologize to @blink1073 for having not caught this in my initial changes in #879 since two different releases were cut following those changes.

@blink1073 blink1073 added the bug label Nov 16, 2022
Copy link
Contributor

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

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

Thanks! I'll cut these releases in the morning.

@blink1073 blink1073 merged commit 107ccdd into jupyter:main Nov 16, 2022
@blink1073
Copy link
Contributor

@meeseeksdev please backport to 7.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyter_client that referenced this pull request Nov 16, 2022
blink1073 pushed a commit that referenced this pull request Nov 16, 2022
…le restarts) (#883)

Co-authored-by: Kevin Bates <kbates4@gmail.com>
@kevin-bates kevin-bates deleted the reconcile-connection-w-port-changes branch December 1, 2022 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants