Skip to content

Revise Yii configuration for RoadRunner setup#385

Merged
samdark merged 6 commits intomasterfrom
roxblnfk-patch-1
Mar 5, 2026
Merged

Revise Yii configuration for RoadRunner setup#385
samdark merged 6 commits intomasterfrom
roxblnfk-patch-1

Conversation

@roxblnfk
Copy link
Member

@roxblnfk roxblnfk commented Mar 3, 2026

Updated the RoadRunner configuration for Yii, including changes to middleware and removed the reload section.

Q A
Is bugfix? ✔️

roxblnfk added 2 commits March 3, 2026 16:14
Updated the RoadRunner configuration for Yii, including changes to middleware and removed the reload section.
Updated the information about HTTP configuration to include TLS, HTTP/2, and HTTP/3.
@vjik vjik requested a review from a team March 3, 2026 12:21
@vjik vjik added the status:code review The pull request needs review. label Mar 3, 2026
Co-authored-by: Sergei Predvoditelev <sergei@predvoditelev.ru>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Yii + RoadRunner tutorial configuration example to reflect a revised RoadRunner HTTP/pool setup and updated run instructions.

Changes:

  • Adds server.env variables and updates the http.pool/middleware configuration example.
  • Removes the reload section from the sample .rr.yaml.
  • Updates the “Starting a server” command and adds an informational link to RoadRunner HTTP docs.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI commented Mar 5, 2026

@roxblnfk I've opened a new pull request, #386, to work on those changes. Once the pull request is ready, I'll request review from you.

* Initial plan

* Update prose to match http.pool config (remove stale 'three workers' reference)

Co-authored-by: roxblnfk <4152481+roxblnfk@users.noreply.github.com>

* Clarify max_worker_memory as a soft limit with post-request restart

Co-authored-by: roxblnfk <4152481+roxblnfk@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: roxblnfk <4152481+roxblnfk@users.noreply.github.com>
@samdark samdark merged commit f6a3955 into master Mar 5, 2026
6 checks passed
@samdark samdark deleted the roxblnfk-patch-1 branch March 5, 2026 14:13
@samdark
Copy link
Member

samdark commented Mar 5, 2026

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code review The pull request needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants