unknowntpo opened a new pull request, #22371:
URL: https://github.com/apache/kafka/pull/22371
### Summary
This is a small discussion PR for KAFKA-17689. It introduces
`TieredStorageTestPlan` as the execution entry point for migrated tiered
storage tests using the new `ClusterExtension` path.
The new pattern is:
```java
builder.build().execute(clusterInstance, groupProtocol);
```
`TieredStorageTestPlan` owns creation of `TieredStorageTestContext`,
consumer group protocol wiring, action execution, and report printing in
`finally`.
### Changes
- Add `TieredStorageTestPlan`
- Add `TieredStorageTestBuilder.build()`
- Refactor migrated tests:
- `AlterLogDirTest`
- `DeleteSegmentsTest`
- `ReassignReplicaMoveAndExpandTest`
### Validation
```bash
./gradlew :storage:compileTestJava :storage:checkstyleTest
./gradlew :storage:test --tests
org.apache.kafka.tiered.storage.integration.AlterLogDirTest --tests
org.apache.kafka.tiered.storage.integration.DeleteSegmentsTest --tests
org.apache.kafka.tiered.storage.integration.ReassignReplicaMoveAndExpandTest
```
### Notes
This PR is intended to discuss the `TieredStorageTestPlan` abstraction
before converting more migrated tiered storage tests.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
