Skip to content

fix(docker): build the go-judge image on Debian 13 so openjdk-21 installs - #137

Merged
Shreesh-Sree merged 1 commit into
developmentfrom
fix/go-judge-base
Nov 2, 2026
Merged

Shreesh-Sree merged 1 commit into
developmentfrom
fix/go-judge-base

Conversation

@Shreesh-Sree

@Shreesh-Sree Shreesh-Sree commented Nov 2, 2026 •

Copy link
Copy Markdown
Member

The image failed with 'Unable to locate package openjdk-21-jdk-headless' on bookworm. trixie packages openjdk-21, rustc, mono, php and ruby. The deployed go-judge image is already Debian 13.


Summary by cubic

Switches the go-judge Docker base image from debian:bookworm-slim to debian:trixie-slim (Debian 13) so the openjdk-21-jdk-headless install no longer fails with "Unable to locate package". The deployed go-judge image is already Debian 13, so runtime behavior is unchanged.

Written for commit a42b9cb. Summary will update on new commits.

View guided diff Turn on auto-fix

Summary by CodeRabbit

  • Chores
    • Updated the runtime container to use a newer Debian release. This may affect the environment in which the application runs; package installation and other container setup remain unchanged.

@github-actions github-actions Bot added the docker label Nov 2, 2026
@coderabbitai

coderabbitai Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Caution

Review failed

The pull request is closed.

Walkthrough

The go-judge runtime image now uses Debian Trixie instead of Debian Bookworm. The Dockerfile comment notes that Bookworm does not package openjdk-21.

Changes

go-judge runtime image

Layer / File(s) Summary
Runtime base image
docker/go-judge/Dockerfile
The runtime base changes from debian:bookworm-slim to debian:trixie-slim. A comment notes that Bookworm does not package openjdk-21.

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

Change: Bug fix

Possibly related PRs

Suggested labels: docker

Suggested reviewers: gaurrov

Merge Risk: Minimal · up to a42b9

The image base changes to Debian 13. No actionable merge-blocking risk was established.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

View guided diff | Turn on auto-fix | Re-trigger cubic

@github-actions github-actions Bot closed this Nov 2, 2026
@Shreesh-Sree
Shreesh-Sree merged commit 9b92527 into development Nov 2, 2026
12 of 13 checks passed
@Shreesh-Sree
Shreesh-Sree deleted the fix/go-judge-base branch November 2, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant