Skip to content

Switched base images to Debian 13 (trixie)#345

Merged
lmakarov merged 3 commits intodevelopfrom
feature/debian-13-trixie
Feb 3, 2026
Merged

Switched base images to Debian 13 (trixie)#345
lmakarov merged 3 commits intodevelopfrom
feature/debian-13-trixie

Conversation

@lmakarov
Copy link
Member

@lmakarov lmakarov commented Feb 3, 2026

Summary by CodeRabbit

  • Updates

    • Base OS upgraded to Debian 13 (Trixie); runtime expectations updated (Ruby 3.3, Python 3.13).
  • New Features

    • Images now include hosting tools, cron support, custom startup scripts, VS Code Server, and multi-platform builds.
  • Improvements

    • SQLite included in images.
    • Composer global bin path added to PATH; PHP tool paths updated to Composer v2 locations.
    • Yarn installation moved to Corepack.
    • SSH init made idempotent (mkdir -p).
  • Tests

    • Healthcheck/version assertions tightened with anchored, regex-based matching.

@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

Warning

Rate limit exceeded

@lmakarov has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 25 minutes and 40 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📝 Walkthrough

Walkthrough

Switched php-fpm base images from Debian bookworm → trixie in PHP 8.2–8.5 Dockerfiles; added sqlite3; made SSHD mkdir idempotent; updated Composer global PATH and PHPCS installed_paths; changed profile reloading to source; replaced npm-based Yarn installs with corepack; updated README and tightened tests.

Changes

Cohort / File(s) Summary
PHP Dockerfiles (8.2–8.5)
8.2/Dockerfile, 8.3/Dockerfile, 8.4/Dockerfile, 8.5/Dockerfile
Base image changed from php:...-bookwormphp:...-trixie. Added sqlite3 to apt installs. mkdir /var/run/sshd made idempotent (-p). Removed Debian testing sqlite workaround. Appended $HOME/.config/composer/vendor/bin to PATH (legacy path retained). Replaced . $HOME/.profile with source $HOME/.profile. Updated PHPCS installed_paths to Composer v2 path. Replaced npm global Yarn installs with corepack enable + corepack install -g yarn@.... Minor comment/formatting tweaks.
Docs
README.md
Updated Debian baseline to trixie (Debian 13); bumped system Python and Ruby versions; added/updated feature notes (hosting CLIs, cron, startup script, VS Code Server, multi-platform images).
Tests
tests/test.bats
Tightened healthcheck assertions: switched to grep -E/anchored regexes for version checks (nvm/node/yarn/ruby/python), adjusted msmtp presence/status check, and replaced several bash regex tests with explicit grep assertions for PHPCS outputs.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title accurately reflects the main objective: upgrading base images from Debian 12 (bookworm) to Debian 13 (trixie) across all PHP Dockerfile versions (8.2-8.5) and updating README documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/debian-13-trixie

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
README.md (1)

83-87: ⚠️ Potential issue | 🟡 Minor

Outdated Python and Ruby version documentation.

The README still references Debian 11 versions (Python 3.9.x, Ruby 2.7.x), but the tests now expect Debian 13 versions (Python 3.13.x, Ruby 3.3.x). This documentation is inconsistent with the actual trixie base image.

📝 Suggested fix
 ## Python

-This image comes with a system level installed Python v3.9.x from upstream Debian 11.
+This image comes with a system level installed Python v3.13.x from upstream Debian 13.

 ## Ruby

-This image comes with a system level installed Ruby v2.7.x from upstream Debian 11.
+This image comes with a system level installed Ruby v3.3.x from upstream Debian 13.

Fixed test reporting false positives on mac/zsh (while failing in GitHub Actions).
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In `@README.md`:
- Line 87: Update the phrase in the Ruby section sentence ("This image comes
with a system level installed Ruby v3.3.x from upstream Debian 13.") to use the
hyphenated adjective "system-level" so it reads "This image comes with a
system-level installed Ruby v3.3.x from upstream Debian 13." to correct the
grammar.
- Line 83: Update the sentence fragment "This image comes with a system level
installed Python v3.13.x from upstream Debian 13." in README.md to use the
hyphenated adjective form by replacing "system level" with "system-level" so it
reads "This image comes with a system-level installed Python v3.13.x from
upstream Debian 13." to fix grammar; locate the exact string in the Python
section to apply the change.

@lmakarov lmakarov force-pushed the feature/debian-13-trixie branch from 275e0be to e2fd91c Compare February 3, 2026 12:37
@lmakarov lmakarov merged commit 63e34db into develop Feb 3, 2026
1 check passed
@lmakarov lmakarov deleted the feature/debian-13-trixie branch February 3, 2026 12:38
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.

1 participant