Skip to content

test(inkless): Integration test for diskless + consolidate separately#639

Merged
EelisK merged 2 commits into
mainfrom
svv/ts-unification-int-test
Jun 16, 2026
Merged

test(inkless): Integration test for diskless + consolidate separately#639
EelisK merged 2 commits into
mainfrom
svv/ts-unification-int-test

Conversation

@viktorsomogyi

Copy link
Copy Markdown
Contributor

This commit introduces a test where a classic topic is created which then is migrated to diskless which then will be consolidated.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds an integration test in core to validate the end-to-end path of creating a classic topic, migrating it to diskless, and then consolidating it (enabling tiered storage) while maintaining full read availability from offset 0 across pruning.

Changes:

  • Enable classic → diskless migration in the embedded test cluster configuration.
  • Add a new integration test covering classic creation, migration to diskless, consolidation, and post-prune consumption correctness.
  • Introduce small helper methods to create a classic topic and to alter/poll topic configs during the test.

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

Comment thread core/src/test/java/kafka/server/InklessConsolidatedDisklessTopicsTest.java Outdated
Comment thread core/src/test/java/kafka/server/InklessConsolidatedDisklessTopicsTest.java Outdated
This commit introduces a test where a classic topic is created
which then is migrated to diskless which then will be consolidated.
@viktorsomogyi viktorsomogyi force-pushed the svv/ts-unification-int-test branch from 9d81fde to 9795fb9 Compare June 12, 2026 14:37
@viktorsomogyi viktorsomogyi requested a review from Copilot June 12, 2026 14:37
@viktorsomogyi viktorsomogyi marked this pull request as ready for review June 12, 2026 14:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@EelisK EelisK left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM :shipit:

@EelisK EelisK merged commit 4202a42 into main Jun 16, 2026
7 checks passed
@EelisK EelisK deleted the svv/ts-unification-int-test branch June 16, 2026 07:30
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