Skip to content

Fix stale worker-count prose after http.pool config refactor#386

Merged
roxblnfk merged 3 commits intoroxblnfk-patch-1from
copilot/sub-pr-385
Mar 5, 2026
Merged

Fix stale worker-count prose after http.pool config refactor#386
roxblnfk merged 3 commits intoroxblnfk-patch-1from
copilot/sub-pr-385

Conversation

Copy link
Contributor

Copilot AI commented Mar 5, 2026

  • Update narrative text after YAML config to remove stale "three workers" reference
  • Clarify max_worker_memory behaviour: it is a soft limit — the worker restarts after finishing the current request, not immediately

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…reference)

Co-authored-by: roxblnfk <4152481+roxblnfk@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on Yii configuration for RoadRunner setup Fix stale worker-count prose after http.pool config refactor Mar 5, 2026
files are static ones except `.php` and `.htaccess`. Also, we're sending an additional header.
We're specifying that the entry script is `worker.php`, the server listens on port 8080, `public` directory
files are served statically except `.php` and `.htaccess`, and workers are limited to 192 MB of memory before
being recycled. Also, we're sending an additional header.
Copy link
Member

@roxblnfk roxblnfk Mar 5, 2026

Choose a reason for hiding this comment

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

@copilot Read the RoadRunner documentation and fix this statement. Workers do not terminate when the limit is exceeded. It's a soft limit. If a worker exceeds it, the worker will restart after processing the request.

Copy link
Member

Choose a reason for hiding this comment

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

@copilot Read the RoadRunner documentation and fix this statement. Workers do not terminate when the limit is exceeded. It's a soft limit. If a worker exceeds it, the worker will restart after processing the request.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 4d06eaf — the sentence now describes max_worker_memory as a soft limit and explains that the worker restarts after finishing its current request when the limit is exceeded.

Co-authored-by: roxblnfk <4152481+roxblnfk@users.noreply.github.com>
@roxblnfk roxblnfk marked this pull request as ready for review March 5, 2026 13:42
@roxblnfk roxblnfk merged commit 9b82e75 into roxblnfk-patch-1 Mar 5, 2026
@roxblnfk roxblnfk deleted the copilot/sub-pr-385 branch March 5, 2026 13:42
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