Skip to content

chore: support kraft migration#923

Merged
razvan merged 38 commits intomainfrom
chore/kraft-migration
Jan 23, 2026
Merged

chore: support kraft migration#923
razvan merged 38 commits intomainfrom
chore/kraft-migration

Conversation

@razvan
Copy link
Member

@razvan razvan commented Dec 18, 2025 •
edited
Loading

@razvan razvan self-assigned this Dec 18, 2025
@razvan razvan mentioned this pull request Dec 19, 2025
Closed
@razvan
Copy link
Member Author

razvan commented Jan 2, 2026

🟢 CI https://testing.stackable.tech/view/02%20Operator%20Tests%20(custom)/job/kafka-operator-it-custom/69/

@razvan razvan marked this pull request as ready for review January 5, 2026 13:27
@razvan razvan changed the title chore/kraft migration chore: support kraft migration Jan 21, 2026
@razvan razvan moved this to Development: Waiting for Review in Stackable Engineering Jan 21, 2026
@razvan razvan requested a review from a team January 21, 2026 16:45
@adwk67 adwk67 requested review from adwk67 and removed request for a team January 23, 2026 08:11
@adwk67 adwk67 moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Jan 23, 2026
adwk67
adwk67 reviewed Jan 23, 2026
View reviewed changes
Copy link
Member

@adwk67 adwk67 left a comment

Choose a reason for hiding this comment

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

LGTM code-wise, tests pass, and the code comments are really thorough! However, I couldn't get the example to work. Maybe we need to include this as an extra test so that we regularly verify it?

adwk67
adwk67 previously approved these changes Jan 23, 2026
View reviewed changes
Copy link
Member

@adwk67 adwk67 left a comment

Choose a reason for hiding this comment

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

LGTM - the example works fine now. Thanks!
Nit: just a couple of places to clarify the steps should be run from one of the broker pods.

@razvan @adwk67
Apply suggestions from code review
84f2fb2 
Co-authored-by: Andrew Kenworthy <1712947+adwk67@users.noreply.github.com>
@razvan razvan enabled auto-merge January 23, 2026 14:53
@razvan razvan added this pull request to the merge queue Jan 23, 2026
Merged via the queue into main with commit e99d0ee Jan 23, 2026
12 checks passed
@razvan razvan deleted the chore/kraft-migration branch January 23, 2026 15:07
@sbernauer sbernauer moved this from Development: In Review to Development: Done in Stackable Engineering Jan 29, 2026
@lfrancke lfrancke moved this from Development: Done to Acceptance: In Progress in Stackable Engineering Feb 4, 2026
@lfrancke
Copy link
Member

lfrancke commented Feb 4, 2026

Could you add release notes here or in the parent issue? Or am I missing them somewhere?

@razvan
Copy link
Member Author

razvan commented Feb 5, 2026

Release note

Users of the Stackable operator for Apache Kafka can now migrate existing 3.9.1 clusters from Apache ZooKeeper to the built in Kraft metadata manager.
This guide describes the necessary steps to perform the migration: https://docs.stackable.tech/home/stable/kafka/usage-guide/kraft-controller/#_kraft_migration_guide

@razvan razvan added the release-note Denotes a PR that will be considered when it comes time to generate release notes. label Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adwk67 adwk67 adwk67 approved these changes

Assignees

@razvan razvan

Labels

release-note Denotes a PR that will be considered when it comes time to generate release notes.

Projects

Stackable Engineering
Status: Acceptance: In Progress

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

KRaft migration

3 participants

@razvan @lfrancke @adwk67