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

--idle-timeout should not shut down entire cluster if any single worker is idle #3282

Closed
bensheldon opened this issue Nov 26, 2023 · 1 comment · Fixed by #3283
Closed

--idle-timeout should not shut down entire cluster if any single worker is idle #3282

bensheldon opened this issue Nov 26, 2023 · 1 comment · Fixed by #3283
Labels

Comments

@bensheldon
Copy link

Describe the bug

#3235 was introduced to fix #3228, but now it will shut down the entire cluster if any single worker is idle. This is currently unreleased but staged in v6.4.1.

For example, bundle exec puma -C config/puma.rb -w 10 --idle-timeout 10 seems like if the server gets 1 request every 5 seconds (less than the 10 second idle timeout), it will still shutdown. That doesn't seem right.

Expected behavior

I would expect that the idle-timeout is applied to the entire cluster. So any request to the cluster as a whole would reset the idle timer.

joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 28, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 29, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 29, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 29, 2023
@joshuay03
Copy link
Contributor

Got this up: #3283

joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 30, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 30, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 30, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 30, 2023
joshuay03 added a commit to joshuay03/puma that referenced this issue Nov 30, 2023
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 a pull request may close this issue.

3 participants