Skip to content

chore(deps): bump the docker-base-images group with 3 updates - #1220

Closed
dependabot[bot] wants to merge 3 commits into
developfrom
dependabot/docker/docker-base-images-7c97bd0883
Closed

chore(deps): bump the docker-base-images group with 3 updates#1220
dependabot[bot] wants to merge 3 commits into
developfrom
dependabot/docker/docker-base-images-7c97bd0883

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Aug 3, 2026

Copy link
Copy Markdown
Contributor

Bumps the docker-base-images group with 3 updates: python, node and ollama/ollama.

Updates python from b877e50 to cea0e60

Updates node from ffc7838 to deae974

Updates ollama/ollama from 509fdf5 to 4dea9fb

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore <dependency name> major version will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)
  • @dependabot ignore <dependency name> minor version will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)
  • @dependabot ignore <dependency name> will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)
  • @dependabot unignore <dependency name> will remove all of the ignore conditions of the specified dependency
  • @dependabot unignore <dependency name> <ignore condition> will remove the ignore condition of the specified dependency and ignore conditions

Summary by CodeRabbit

  • Chores
    • Updated container base-image versions for the Python backend, Node frontend builder, and Ollama runtime.
    • Improved build reproducibility and security by pinning refreshed image digests.

Bumps the docker-base-images group with 3 updates: python, node and ollama/ollama.


Updates `python` from `b877e50` to `cea0e60`

Updates `node` from `ffc7838` to `deae974`

Updates `ollama/ollama` from `509fdf5` to `4dea9fb`

---
updated-dependencies:
- dependency-name: python
  dependency-version: 3.14-slim
  dependency-type: direct:production
  dependency-group: docker-base-images
- dependency-name: node
  dependency-version: 26-slim
  dependency-type: direct:production
  dependency-group: docker-base-images
- dependency-name: ollama/ollama
  dependency-version: 4dea9fb511947e24a84237bb636b0203abcb2ff0d3fbc7b4ff865deb91362131
  dependency-type: direct:production
  dependency-group: docker-base-images
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file docker Pull requests that update docker code labels Aug 3, 2026
@dependabot
dependabot Bot requested a review from seonghobae as a code owner August 3, 2026 04:04
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file docker Pull requests that update docker code labels Aug 3, 2026
@coderabbitai

coderabbitai Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The Docker build files now use updated SHA256 digests for the Python backend runtime, Node.js frontend builder, and Ollama base image.

Changes

Container image pins

Layer / File(s) Summary
Update pinned base image digests
Dockerfile, Dockerfile.ollama
The Dockerfiles now reference updated digests for the python:3.14-slim, node:26-slim, and Ollama base images.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: seonghobae

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the grouped Docker base-image dependency updates covered by the pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dependabot/docker/docker-base-images-7c97bd0883

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
Dockerfile.ollama (1)

1-1: 🩺 Stability & Availability | 🔵 Trivial

Verify Dockerfile.ollama in pull-request validation.

The supplied workflow snippet lists validation for Dockerfile and frontend/Dockerfile, but it does not show a Dockerfile.ollama entry. docker-compose.yml builds this file directly. Confirm that a required check builds this image, validates both model pulls, and runs the resulting image health check. If no check exists, add one before merging this digest update.

As per coding guidelines, include the exact focused verification command in the PR body.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@Dockerfile.ollama` at line 1, Verify that pull-request validation includes a
required check for Dockerfile.ollama: build the image, validate both model
pulls, and run the resulting image health check. If absent, add the check and
document the exact focused verification command in the PR body.

Source: Coding guidelines

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@Dockerfile.ollama`:
- Line 1: Verify that pull-request validation includes a required check for
Dockerfile.ollama: build the image, validate both model pulls, and run the
resulting image health check. If absent, add the check and document the exact
focused verification command in the PR body.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: e2cdbc0a-0c22-49c0-a30d-4bdff2c0375f

📥 Commits

Reviewing files that changed from the base of the PR and between 298c9f4 and 01f2652.

📒 Files selected for processing (2)
  • Dockerfile
  • Dockerfile.ollama

@seonghobae seonghobae left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This grouped Dependabot update supersedes #1221 and keeps Python, Node, and Ollama base-image pins synchronized. No review thread is open. Required repository checks and current-head OpenCode/Strix review must complete before merging; #1203 should then be refreshed against these current digests.

@seonghobae
seonghobae marked this pull request as draft August 3, 2026 05:08
@seonghobae
seonghobae marked this pull request as ready for review August 3, 2026 05:08
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

PR governance metadata gate is not ready for afe54b96821bd3889b43989fa989a667f4847c02:

  • Review decision is CHANGES_REQUESTED; address requested changes before merge.
  • Required check backend (Python 3.14) is FAILURE on the current head.

@seonghobae seonghobae left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Grouped base-image digest branch refreshed onto live develop. Current head 98e95acd75d14b5009c3baa38ce9a58c2f8db7b9 is zero commits behind and changes only Dockerfile and Dockerfile.ollama. Revalidating current-head build/security checks and independent review.

@opencode-agent opencode-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

OpenCode could not approve from deterministic current-head evidence because GitHub Checks have failed.

Findings

1. HIGH Current-head GitHub Checks - Fix failed required checks before approval

  • Problem: Failed same-head checks remain for afe54b96821bd3889b43989fa989a667f4847c02.
  • Root cause: The model-unavailable evidence fallback is allowed only when peer GitHub Checks are complete and clean.
  • Fix: Read and fix the failed check logs below, then rerun the current-head checks.
  • Regression test: Keep the model-unavailable fallback gated on an empty failed-check rollup.

Failed checks:

Changed-File Evidence Map

flowchart LR
  PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
  Evidence --> S1["Changed file (2 files)"]
  S1 --> I1["repository behavior"]
  I1 --> R1["Review risk: Changed file (2 files)"]
  R1 --> V1["required checks"]
Loading

@opencode-agent

opencode-agent Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

OpenCode Review Overview

  • Head SHA: afe54b96821bd3889b43989fa989a667f4847c02
  • Workflow run: 30854809803
  • Workflow attempt: 1
  • Gate result: REQUEST_CHANGES (approval step)

Pull request overview

OpenCode could not approve from deterministic current-head evidence because GitHub Checks have failed.

Findings

1. HIGH Current-head GitHub Checks - Fix failed required checks before approval

  • Problem: Failed same-head checks remain for afe54b96821bd3889b43989fa989a667f4847c02.
  • Root cause: The model-unavailable evidence fallback is allowed only when peer GitHub Checks are complete and clean.
  • Fix: Read and fix the failed check logs below, then rerun the current-head checks.
  • Regression test: Keep the model-unavailable fallback gated on an empty failed-check rollup.

Failed checks:

Changed-File Evidence Map

flowchart LR
  PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
  Evidence --> S1["Changed file (2 files)"]
  S1 --> I1["repository behavior"]
  I1 --> R1["Review risk: Changed file (2 files)"]
  R1 --> V1["required checks"]
Loading

Copy link
Copy Markdown
Contributor

This digest-only PR is now held as a draft because #1203 is the required superset: it synchronizes all container entry points, derives OCI provenance from the actual first FROM, validates multi-architecture manifests, and adds regression governance. The newer digests from this PR have been explicitly handed to #1203 for integration. Once #1203 publishes an exact head containing these three pins and passes its validation gates, this PR should close as superseded rather than merge separately.

Copy link
Copy Markdown
Contributor

@dependabot rebase

@dependabot @github

dependabot Bot commented on behalf of github Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry!

If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request @dependabot recreate.

Copy link
Copy Markdown
Contributor

Closing as superseded by #1203. This Dependabot branch only changes three raw FROM digests, while #1203 carries the reviewed current Python/Node/Ollama pins together with synchronized Docker entry points, OCI base.name/base.digest derivation, amd64/arm64 manifest validation, permanent governance tests, provenance doctoring, and clean exact-head repository checks. Merging both would reintroduce conflicting Node/Ollama digests and bypass the provenance contract. No changes from this PR are being treated as independent completion evidence.

@seonghobae seonghobae closed this Aug 5, 2026
@dependabot @github

dependabot Bot commented on behalf of github Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

This pull request was built based on a group rule. Closing it will not ignore any of these versions in future pull requests.

To ignore these dependencies, configure ignore rules in dependabot.yml

@dependabot
dependabot Bot deleted the dependabot/docker/docker-base-images-7c97bd0883 branch August 5, 2026 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file docker Pull requests that update docker code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant