Skip to content

Use generated Kafka image in CI tests#2330

Open
delthas wants to merge 1 commit intoimprovement/ZENKO-5194/fix-ci-cgroupv2-upgrade-zookeeperfrom
improvement/ZENKO-5196/use-generated-kafka-image-ci
Open

Use generated Kafka image in CI tests#2330
delthas wants to merge 1 commit intoimprovement/ZENKO-5194/fix-ci-cgroupv2-upgrade-zookeeperfrom
improvement/ZENKO-5196/use-generated-kafka-image-ci

Conversation

@delthas
Copy link
Contributor

@delthas delthas commented Feb 19, 2026

Issue: ZENKO-5196

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR modifies the CI configuration to use generated Kafka images with content-addressable tags based on a git tree hash. The changes enable using Kafka and Kafka Connect images that are built specifically for the current codebase state, ensuring that tests run against the correct image versions.

Changes:

  • Added BUILD_TREE_HASH suffix to Kafka cluster and Kafka Connect image tags in the version specification
  • Added BUILD_TREE_HASH environment variable computation in the end-to-end deployment script

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
solution/zenkoversion.yaml Modified Kafka cluster and Kafka Connect tags to include BUILD_TREE_HASH suffix for content-addressable tagging
.github/scripts/end2end/deploy-zenko.sh Added BUILD_TREE_HASH computation using git tree hash of solution/kafka directory to dependencies_env function

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@delthas delthas force-pushed the improvement/ZENKO-5196/use-generated-kafka-image-ci branch from e51a0a2 to cf3ed9d Compare February 19, 2026 18:44
Copy link
Contributor

@francoisferrand francoisferrand left a comment

Choose a reason for hiding this comment

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

Does not embed the correct Kafka, ISO would not work (in artesca).
Fixed in #2331

@francoisferrand francoisferrand force-pushed the improvement/ZENKO-5196/use-generated-kafka-image-ci branch from cf3ed9d to 2322a73 Compare February 20, 2026 00:06
@francoisferrand francoisferrand dismissed their stale review February 20, 2026 00:14

actually, image really not embedded in ISO

@francoisferrand francoisferrand force-pushed the improvement/ZENKO-5196/use-generated-kafka-image-ci branch from 2322a73 to cf3ed9d Compare February 20, 2026 00:15
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.

2 participants

Comments