Replace bitnami Kafka image with soldevelo/kafka#10
Replace bitnami Kafka image with soldevelo/kafka#10igor-soldev wants to merge 1 commit intopsmon:masterfrom
Conversation
WalkthroughUpdated the Kafka Docker image reference from Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
Compose/docker-compose.yml (1)
42-42: Pin kafka image to a specific version instead of using:latest.Using the
:latesttag makes the image version unpredictable and can introduce instability if future updates to soldevelo/kafka introduce breaking changes. This is especially important when migrating to a new image provider.Apply this diff to pin the image to a stable version:
- image: 'soldevelo/kafka:latest' + image: 'soldevelo/kafka:3.7.0'Replace
3.7.0with the latest stable version of soldevelo/kafka that is compatible with your Zookeeper version and Kafka clients.DOCKER.md (1)
50-58: Kafka image change in documentation mirrors docker-compose.yml—ensure both receive the same version pinning.The kafka service image in the documentation snippet is updated consistently with the actual
Compose/docker-compose.ymlfile, which is good. However, the same verification and version pinning concerns mentioned in the docker-compose.yml review apply here.If you apply version pinning to the actual docker-compose.yml file (e.g.,
soldevelo/kafka:3.7.0), ensure this documentation is updated to match.Apply this diff to pin the kafka image version in the documentation:
- image: 'soldevelo/kafka:latest' + image: 'soldevelo/kafka:3.7.0'
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
Compose/docker-compose.yml(1 hunks)DOCKER.md(1 hunks)
🔇 Additional comments (1)
Compose/docker-compose.yml (1)
40-48: Soldevelo/kafka is confirmed as a Bitnami-compatible drop-in replacement.SolDevelo's soldevelo/kafka images accept the same Bitnami-style environment variables, including
KAFKA_ADVERTISED_HOST_NAME,KAFKA_ZOOKEEPER_CONNECT, andALLOW_PLAINTEXT_LISTENERused in this configuration. No compatibility issues are expected.
|
Hi! This is a straightforward fix that aligns both the Docker config and the documentation. Since the old Bitnami image is no longer pullable, this update is necessary to get the environment running again. Is there anything else needed to get this merged? |
Soldevelo image is a drop-in replacement that remains fully compatible with Bitnami’s configuration and environment variables.
The Bitnami Kafka image is now behind a paywall and no longer publicly available, which prevents CI/CD and local environments from functioning properly.
Switching to Soldevelo’s maintained image ensures:
Summary by CodeRabbit