Skip to content

chore: replaced kafka images with internal icr registry#2372

Open
abhilash-sivan wants to merge 5 commits intomainfrom
chore-replace-kafka-images
Open

chore: replaced kafka images with internal icr registry#2372
abhilash-sivan wants to merge 5 commits intomainfrom
chore-replace-kafka-images

Conversation

@abhilash-sivan
Copy link
Contributor

@abhilash-sivan abhilash-sivan commented Feb 24, 2026

@abhilash-sivan abhilash-sivan changed the title chore: replaced kafka image chore: replaced kafka images with internal icr registry Feb 24, 2026

zookeeper:
image: confluentinc/cp-zookeeper:7.9.0
image: confluentinc/cp-zookeeper:7.9.1
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Aligned the image versions used in the sidecar for local development.

Since this Docker Compose setup is intended for developers and the project is open source, I have not changed the image reference in the Docker Compose file. Keeping the public image avoids forcing open-source contributors (who don’t have access to our internal registry) to manually locate and replace the image for their local usage.

I strongly believe this is the correct approach. The only required change was in the sidecars, and that has been completed.

Copy link
Contributor

Choose a reason for hiding this comment

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

Agree.

Copy link
Contributor

Choose a reason for hiding this comment

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

Oki.

Did you try these btw?

public.ecr.aws/aktosecurity/confluentinc-cp-kafka:7.9.1-1-ubi8
public.ecr.aws/aktosecurity/confluentinc-cp-zookeeper:7.9.2-1-ubi8

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, haven't tried. any other non-official images
Can we use this in sidecars as well ? or just for dev use

@abhilash-sivan abhilash-sivan marked this pull request as ready for review February 24, 2026 14:55
@abhilash-sivan abhilash-sivan requested a review from a team as a code owner February 24, 2026 14:55
Copy link
Contributor

@aryamohanan aryamohanan left a comment

Choose a reason for hiding this comment

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

The Kafka image update also requires updating the image references in the Tekton tasks too

@abhilash-sivan
Copy link
Contributor Author

The Kafka image update also requires updating the image references in the Tekton tasks too

Yes, updated now! Let the pipeline run

@abhilash-sivan abhilash-sivan force-pushed the chore-replace-kafka-images branch from 253a2ef to a716de4 Compare February 25, 2026 07:01
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants