Skip to content

Conversation

mdelapenya
Copy link
Member

What does this PR do?

Bump Ryuk to v0.14.0, whose image is considerably smaller.

Why is it important?

Reduce pull time and footprint of the ryuk image

Related issues

@mdelapenya mdelapenya requested a review from a team as a code owner September 26, 2025 14:29
@mdelapenya mdelapenya added the dependencies Dependencies or external services label Sep 26, 2025
@mdelapenya mdelapenya self-assigned this Sep 26, 2025
Copy link

netlify bot commented Sep 26, 2025

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit 1634baa
🔍 Latest deploy log https://app.netlify.com/projects/testcontainers-go/deploys/68de40e4066fc600082154fd
😎 Deploy Preview https://deploy-preview-3313--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

coderabbitai bot commented Sep 26, 2025

Summary by CodeRabbit

  • Chores
    • Updated the default Ryuk image from 0.13.0 to 0.14.0.
    • Future runs will automatically pull and use the newer image; existing custom image overrides remain unaffected.
    • No functional behavior changes beyond the version bump.
    • Users may see a one-time image download on first use after upgrading.

Walkthrough

Updates the default Ryuk image version in configuration from 0.13.0 to 0.14.0 by changing the ReaperDefaultImage constant. No other code changes.

Changes

Cohort / File(s) Summary
Config constant update
internal/config/config.go
Bumps ReaperDefaultImage from testcontainers/ryuk:0.13.0 to testcontainers/ryuk:0.14.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A tweak to the tag on a moonlit night,
Ryuk hops versions—new lantern, same light.
I wiggle my whiskers, configs align,
0.14.0 tastes crisp and fine.
Burrow secured, I thump with cheer—
tiny change, tidy gear!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title succinctly follows conventional commit style and accurately reflects the primary change by specifying the dependency bump of Ryuk to version v0.14.0, matching the updated constant in the code without any extraneous information.
Description Check ✅ Passed The description clearly states what the PR does by bumping Ryuk to v0.14.0 and why it is important by noting reduced pull time and footprint, and it also links to the related upstream change, making it directly relevant and sufficiently informative for this dependency update.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9f0e4cd and ede5f13.

📒 Files selected for processing (1)
  • internal/config/config.go (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (19)
  • GitHub Check: lint (modules/redis) / lint: modules/redis
  • GitHub Check: lint (modules/compose) / lint: modules/compose
  • GitHub Check: lint (modules/toxiproxy) / lint: modules/toxiproxy
  • GitHub Check: lint (modules/yugabytedb) / lint: modules/yugabytedb
  • GitHub Check: lint (modules/pulsar) / lint: modules/pulsar
  • GitHub Check: lint (modules/dockermodelrunner) / lint: modules/dockermodelrunner
  • GitHub Check: lint (modules/gcloud) / lint: modules/gcloud
  • GitHub Check: lint (modules/registry) / lint: modules/registry
  • GitHub Check: lint (modules/localstack) / lint: modules/localstack
  • GitHub Check: lint (modules/openfga) / lint: modules/openfga
  • GitHub Check: lint (modules/k3s) / lint: modules/k3s
  • GitHub Check: lint (modules/dockermcpgateway) / lint: modules/dockermcpgateway
  • GitHub Check: lint (modules/grafana-lgtm) / lint: modules/grafana-lgtm
  • GitHub Check: lint (modules/arangodb) / lint: modules/arangodb
  • GitHub Check: lint (modules/azurite) / lint: modules/azurite
  • GitHub Check: lint (modules/dind) / lint: modules/dind
  • GitHub Check: lint (modules/vault) / lint: modules/vault
  • GitHub Check: update_release_draft
  • GitHub Check: Analyze (go)
🔇 Additional comments (1)
internal/config/config.go (1)

14-14: Ryuk default image bump looks good

The constant now matches the v0.14.0 tag; no other adjustments are required.


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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Dependencies or external services
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant