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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[馃挕 FEATURE REQUEST]: Service plugin add support stop_wait_secs (graceful termination) #1628

Closed
asanikovich opened this issue Jun 27, 2023 · 3 comments 路 Fixed by roadrunner-server/service#45
Assignees
Labels
C-feature-accepted Category: Feature discussed and accepted
Milestone

Comments

@asanikovich
Copy link

asanikovich commented Jun 27, 2023

Plugin

Service

I have an idea!

Please add to service plugin configuration next parameter:
stop_wait_secs
You have default 5s to wait until process finished after sending SIGINT to process from RR.
It will be good to support custom stop_wait_secs to control termination flow (graceful termination).

Please add to RPC commutation with service plugin next parameters (to support define from PHP):

  • service_name_in_logs - to support define from PHP
  • stop_wait_secs - describe above

Discussion

@asanikovich asanikovich added the C-feature-request Category: feature requested, but need to be discussed label Jun 27, 2023
@rustatian
Copy link
Member

Hey @asanikovich 馃憢馃徎
I will try to implement this for 2023.2.0, but we only have 2 weeks until the release...
Since our options are similar to the systemd unit file options, stop_wait_secs would be TimeoutStopSec -> timeout_stop_sec.

@rustatian rustatian added C-feature-accepted Category: Feature discussed and accepted and removed C-feature-request Category: feature requested, but need to be discussed labels Jun 27, 2023
@rustatian rustatian added this to the v2023.2.0 milestone Jun 27, 2023
@asanikovich
Copy link
Author

@rustatian What's mean global parameter grace_period: 30s (defined at line https://github.com/roadrunner-server/roadrunner/blob/master/.rr.yaml#L2169)?

@rustatian
Copy link
Member

This is the general RR graceful shutdown timeout. When it expires, RR will exit regardless of the fact that some plugin is still in the middle of shutdown.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-feature-accepted Category: Feature discussed and accepted
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants