[kiview]

OceanBaseCEContainer waits for boot success! with the default 60s startup timeout. On GitHub runners, boot takes ~55–60s, so the module's tests fail intermittently with Timed out waiting for log output matching '.*boot success!.*'. The main CI run on 2026-10-05 failed 7 of 10 attempts; the passing ones took 57.6s and 60.3s.

The log message is the right readiness signal: the image's start.sh only prints it after the tenant accepts SQL connections and init SQL has run. Boot is just slow. This PR raises the wait strategy's startup timeout to 5 minutes, following the image README ("The bootstrap procedure may take up to five minutes"). The MINI and NORMAL modes are slower than SLIM, which the tests use.

User-visible change: a container that hangs during boot now fails after 5 minutes instead of 60s. Failed boots still fail fast because the container exits (EXIT_WHILE_ERROR defaults to true). The timeout can still be overridden via withStartupTimeout(...).

Local cold boots (SLIM mode, Docker Desktop) took 42–44s to boot success!. A SELECT 1 succeeded ~0.7s earlier, so an extra SQL readiness check would add nothing.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Bug Fixes
  • OceanBase containers now allow up to five minutes for the database to report successful startup before timing out.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 3e703da8-6bd1-4b14-b905-4526a6ad64a3

📥 Commits

Reviewing files that changed from the base of the PR and between f829405 and 615b686.

📒 Files selected for processing: 1

• modules/oceanbase/src/main/java/org/testcontainers/oceanbase/OceanBaseCEContainer.java

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 4 remain after this review.

📝 Walkthrough

The OceanBase container’s log-based readiness strategy now has a five-minute startup timeout. It still waits for one boot success! message.

Changes: OceanBase container startup

Layer / File(s)	Summary
Configure readiness timeout modules/oceanbase/src/main/java/org/testcontainers/oceanbase/OceanBaseCEContainer.java	The log-based readiness strategy now has a five-minute startup timeout. It still waits for one boot success! message.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Bug fix

Suggested reviewers: whhe

Merge Risk: ⚪ Minimal · up to 615b6

The longer startup allowance addresses the reported slow boots while preserving the existing readiness check. No actionable merge-blocking risk is established by the supplied review context.

Pre-merge checks: 4 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 1 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 4 passed

Check name	Status	Explanation
Title check		The title clearly and concisely describes the main change: increasing the OceanBase CE startup timeout to five minutes.
Description check		The description explains the intermittent timeout, identifies the readiness signal, gives the rationale for the five-minute timeout, describes failure behavior, and notes override support.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

• Fix all pre-merge checks with AI

✨ Finishing Touches: 💡 1

📝 Generate docstrings: 💡

• Commit to this branch
• Create a new PR

• 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

• X
• Mastodon
• Reddit
• LinkedIn

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