Update base image#3768
Merged
patrickelectric merged 2 commits intobluerobotics:masterfrom Feb 12, 2026
Merged
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates the core Docker image to use a newer base image and removes inline apt-based system package installation from the tools installer script, delegating package provisioning to the base image instead. Flow diagram for updated install-system-tools script behaviorflowchart TD
A["Start install-system-tools.sh"] --> B["Run parallel bootstrap scripts in /home/pi/tools"]
B --> C{"Need additional system packages?"}
C -->|No| D["Finish script (all system packages expected from base image)"]
C -->|Yes| E["Manually run apt update and apt install for missing packages"]
E --> F["Manually clean apt cache"]
F --> D
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- Instead of keeping the old apt commands commented out in install-system-tools.sh, consider either removing that block entirely or replacing it with a brief comment that points to where system packages are now managed (e.g., in the base image Dockerfile) to avoid confusion about dead code.
- In the placeholder apt example,
update && apt installis missing theaptbeforeupdate; if you're keeping this as guidance, fix the example so it’s syntactically correct or make it explicitly non-runnable (e.g., mention it in prose rather than as a shell snippet).
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Instead of keeping the old apt commands commented out in install-system-tools.sh, consider either removing that block entirely or replacing it with a brief comment that points to where system packages are now managed (e.g., in the base image Dockerfile) to avoid confusion about dead code.
- In the placeholder apt example, `update && apt install` is missing the `apt` before `update`; if you're keeping this as guidance, fix the example so it’s syntactically correct or make it explicitly non-runnable (e.g., mention it in prose rather than as a shell snippet).Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
3944b1a to
1632c2d
Compare
1632c2d to
a019d41
Compare
patrickelectric
approved these changes
Feb 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This will help reduce the image size, as we are getting some packages updated when we re-run apt in core.
Master:


This PR:
Summary by Sourcery
Update the core image to use the newer blueos-base 0.2.3 and stop installing apt-based system tools in this layer, leaving only commented guidance for future package additions.
Enhancements:
Note
Medium Risk
Base image bump plus removal of installed system packages can change runtime behavior if the new base image doesn’t include all previously-installed dependencies.
Overview
Updates the core Docker build to use
bluerobotics/blueos-base:0.2.4instead of0.2.2.Removes the layer-time
apt installof networking/system utilities fromtools/install-system-tools.sh, leaving only commented guidance so these dependencies are expected to live in the base image.Written by Cursor Bugbot for commit a019d41. This will update automatically on new commits. Configure here.