Conversation
WalkthroughContainer images were switched from bitnami to bitnamilegacy across CI and deployment configs: Kafka and MinIO images updated in docker-compose files; Kafka image updated in the GitHub Actions workflow; and the Keycloak Dockerfile base image changed accordingly. No other configuration or logic changes were made. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
Router image scan passed✅ No security vulnerabilities found in image: |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
docker-compose.full.yml (1)
175-175: Full compose aligns with bitnamilegacy; mirror health checks and ensure MinIO path.
- Kafka here lacks a healthcheck while the dev compose has one; consider adding for parity to improve local readiness checks.
- Confirm MinIO keeps
/bitnami/minio/dataunder the legacy image.Apply if you want parity with dev compose:
kafka: image: bitnamilegacy/kafka:3.7.0 + healthcheck: + test: ['CMD-SHELL', 'kafka-broker-api-versions.sh --version'] + interval: 10s + timeout: 5s + retries: 5Also applies to: 205-205
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (4)
.github/workflows/router-ci.yaml(1 hunks)docker-compose.full.yml(2 hunks)docker-compose.yml(2 hunks)keycloak/Dockerfile(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). (1)
- GitHub Check: integration_test (./. ./fuzzquery ./lifecycle ./modules)
🔇 Additional comments (2)
keycloak/Dockerfile (1)
15-15: Verify provider path and kc.sh build flags
- Run
docker run --rm bitnamilegacy/keycloak:25.0.2 ls -d /opt/bitnami/keycloak/providersto confirm the providers directory exists.- Run or inspect
docker run --rm bitnamilegacy/keycloak:25.0.2 /opt/bitnami/keycloak/bin/kc.sh build --db=postgres --metrics-enabled true --health-enabled true --http-relative-path "/"(or checkkc.sh --help) to ensure all required build flags are supported.- Ensure these match your Helm chart/docker-compose settings to avoid startup drift.
docker-compose.yml (1)
168-168: Validate bitnamilegacy image tags & clean up stale bitnami references
- Confirmed that
bitnamilegacy/minio:2024.7.16-debian-12-r0,bitnamilegacy/kafka:3.7.0andbitnamilegacy/keycloak:25.0.2are published and pullable.- Repository still contains
bitnami/…mounts and paths:
• docker-compose.yml (lines 40, 177)
• docker-compose.full.yml (lines 165, 184)
• keycloak/Dockerfile (COPY path, build script)
• helm/cosmo/values.yaml (lines 346, 385, 461) and README.md (line 182)
Update these to match bitnamilegacy image semantics (volume mounts, import dirs, KRaft envs).- Pin CI-critical images by digest to guard against upstream tag drift.
Due to upcoming changes to the bitnami registry we need to update the image refs
Summary by CodeRabbit
Chores
Tests
Checklist