Skip to content

Update and add new Raspberry Pi OS version#137

Open
mtouzot wants to merge 6 commits intopguyot:mainfrom
mtouzot:update/raspios
Open

Update and add new Raspberry Pi OS version#137
mtouzot wants to merge 6 commits intopguyot:mainfrom
mtouzot:update/raspios

Conversation

@mtouzot
Copy link
Copy Markdown
Contributor

@mtouzot mtouzot commented May 1, 2026

Bonjour,

Merci pour la précédente MR validée et les correctifs du workflow.

Cette fois-ci, je me suis permit d'ajouter quelques versions de Raspberry Pi OS lite.

N'hésitez pas à me dire si le travail vous convient.

Merci et bonne journée

Summary by CodeRabbit

  • New Features

    • Broader support for additional Raspberry Pi OS image versions and dated releases across 32-bit and 64-bit stable and oldstable variants; CI now includes these images for build coverage.
  • Documentation

    • README updated to list the newly recognized Raspberry Pi OS image tags.
  • Other

    • Image retrieval logic expanded to map and recognize the additional image tags.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 1, 2026

Warning

Rate limit exceeded

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

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ 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.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 90837354-018b-4a8b-b33b-ef4c41641b9c

📥 Commits

Reviewing files that changed from the base of the PR and between 57c43e0 and d06ee33.

📒 Files selected for processing (1)
  • README.md
📝 Walkthrough

Walkthrough

Adds many dated Raspberry Pi OS "lite" and "oldstable_lite" image tags (armhf and arm64) across CI workflow matrix, README documentation, and the download script; download URLs and case mappings in the script are extended and reorganized.

Changes

Cohort / File(s) Summary
CI Workflow Configuration
​.github/workflows/test-base_images.yml
Expands the build_base_images job matrix with multiple additional raspios_lite, raspios_oldstable_lite, and *_arm64 dated tags (no existing entries removed; steps unchanged).
Documentation
README.md
Augments the base_image input documentation to list the newly added dated Raspberry Pi OS lite and oldstable-lite tags for both armhf and arm64 variants.
Download Script
download_image.sh
Reworks the case $1 in mappings: reorders/remaps raspios_lite/raspios_lite_arm64:latest, adds many dated armhf and arm64 tag cases with corresponding .img.xz download URLs, and inserts section comments for grouped sources (Raspbian/RaspiOS/Debian/Custom). (Net changes: +68/-6 lines.)

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Poem

🐇 I hopped through tags both new and old,
Armhf and arm64, bold and cold.
Matrix widened, downloads queued,
Case lines mapped and neatly viewed.
A crunchy carrot for every Pi to hold! 🥕

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Update and add new Raspberry Pi OS version' is directly related to the main changes in the pull request, which add new Raspberry Pi OS image tags across workflow configuration, documentation, and download script.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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
Review rate limit: 0/1 reviews remaining, refill in 50 minutes and 2 seconds.

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

Copy link
Copy Markdown

@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: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Around line 117-118: Update the two oldstable arm64 entries so they include
the explicit architecture suffix to match the other entries; replace
`raspios_oldstable_lite_arm64:2025-05-07` with
`raspios_oldstable_lite_arm64(arm64):2025-05-07` and
`raspios_oldstable_lite_arm64:2026-04-14` with
`raspios_oldstable_lite_arm64(arm64):2026-04-14` in the README so all `*_arm64`
values consistently display `(arm64)`.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f87cdc41-d620-49ae-8677-8d729351d2d0

📥 Commits

Reviewing files that changed from the base of the PR and between 9b88933 and 087686e.

📒 Files selected for processing (3)
  • .github/workflows/test-base_images.yml
  • README.md
  • download_image.sh

Comment thread README.md Outdated
Comment on lines +117 to +118
- `raspios_oldstable_lite_arm64:2025-05-07`
- `raspios_oldstable_lite_arm64:2026-04-14`
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Add explicit architecture label on oldstable arm64 entries.

Line 117 and Line 118 omit (arm64), while the other *_arm64 values in this section include it. Keeping the same suffix here improves scanability and avoids ambiguity.

Suggested doc tweak
--   `raspios_oldstable_lite_arm64:2025-05-07`
--   `raspios_oldstable_lite_arm64:2026-04-14`
+-   `raspios_oldstable_lite_arm64:2025-05-07` (arm64)
+-   `raspios_oldstable_lite_arm64:2026-04-14` (arm64)
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- `raspios_oldstable_lite_arm64:2025-05-07`
- `raspios_oldstable_lite_arm64:2026-04-14`
- `raspios_oldstable_lite_arm64:2025-05-07` (arm64)
- `raspios_oldstable_lite_arm64:2026-04-14` (arm64)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` around lines 117 - 118, Update the two oldstable arm64 entries so
they include the explicit architecture suffix to match the other entries;
replace `raspios_oldstable_lite_arm64:2025-05-07` with
`raspios_oldstable_lite_arm64(arm64):2025-05-07` and
`raspios_oldstable_lite_arm64:2026-04-14` with
`raspios_oldstable_lite_arm64(arm64):2026-04-14` in the README so all `*_arm64`
values consistently display `(arm64)`.

@mtouzot mtouzot force-pushed the update/raspios branch 2 times, most recently from ce32fea to 23b8dc9 Compare May 1, 2026 12:27
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