Skip to content

Conversation

@jankalinic
Copy link
Contributor

This PR adds a systemtest for a rebalance feature, deploying Kafka with CC and KafkaRebalance CRs, and using Console UI to verify data and approval of the generated proposals.

@jankalinic jankalinic self-assigned this Dec 10, 2025
@jankalinic jankalinic added do not merge Do not merge just yet. Work is still in progress ready for review Pull request is ready for a static review labels Dec 10, 2025
@jankalinic
Copy link
Contributor Author

/systemtests run --profile=regression --retry-count=0

@MikeEdgar MikeEdgar added this to the 0.11.0 milestone Dec 10, 2025
Copy link
Member

@MikeEdgar MikeEdgar left a comment

Choose a reason for hiding this comment

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

LGTM, just the Sonar nit. I assume you're still looking at the test failures.

import static org.junit.jupiter.api.Assertions.assertTrue;

@Tag(TestTags.REGRESSION)
public class RebalanceST extends AbstractST {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
public class RebalanceST extends AbstractST {
class RebalanceST extends AbstractST {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry for the delay.

I shall keep the public modifier there, as TestBuckets extension reads the classes, orders the methods and invokes them. Maybe i could add an exception rule for this?

Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
Signed-off-by: jkalinic <jkalinic@redhat.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 6, 2026

@jankalinic
Copy link
Contributor Author

/systemtests run --profile=regression --retry-count=1

@streamshub-bot
Copy link

⚠️ Systemtests run finished - failure ⚠️

Test Summary:

  • TOTAL: 0
  • PASS: 0
  • FAIL: 0
  • SKIP: 0

Used parameters:

  • TEST_CASE:
  • PROFILE: regression
  • INSTALL_TYPE: olm
  • RETRY_COUNT: 1
  • ENVS:

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

Labels

do not merge Do not merge just yet. Work is still in progress ready for review Pull request is ready for a static review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants