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

[PoC] Update/Reload without downtime #4654

Closed
wants to merge 2 commits into from

Conversation

Watson1978
Copy link
Contributor

@Watson1978 Watson1978 commented Oct 3, 2024

Which issue(s) this PR fixes:
Fixes #

What this PR does / why we need it:
Yet another proof of concept for #4624
This PR depends on treasure-data/serverengine#149

Docs Changes:

Release Note:

daipom and others added 2 commits October 3, 2024 14:16
To make it easier to re-use plugin shutdown process.

Signed-off-by: Daijiro Fukuda <[email protected]>
Signed-off-by: Shizuo Fujita <[email protected]>
start_new_supervisor

# TODO: 少なくとも worker を1つでも起動できるように戻さないと自身が死ななかった
scale_up
Copy link
Contributor Author

Choose a reason for hiding this comment

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

タイミングによっては、scale_up によってワーカーが起動してしまうかもしれない。
scale_up しなくても「古いプロセス(自分自身)を止める」ことができるようにしないと危ないだろう。

@daipom
Copy link
Contributor

daipom commented Oct 11, 2024

Thanks for making another PoC for #4624!
Some of this concept is reflected in #4624.
We will proceed with this feature development in #4624.

@daipom daipom closed this Oct 11, 2024
@daipom daipom mentioned this pull request Oct 11, 2024
3 tasks
@Watson1978 Watson1978 deleted the downtime branch October 28, 2024 06:47
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.

2 participants