Skip to content

Commit

Permalink
[DPE-6247] fix: test-balancer-single integration test timeout
Browse files Browse the repository at this point in the history
  • Loading branch information
Iman Enami committed Dec 19, 2024
1 parent 1c6b435 commit e6fd583
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions src/managers/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,8 @@
partition.metric.sample.store.topic=__KafkaCruiseControlPartitionMetricSamples
broker.metric.sample.store.topic=__KafkaCruiseControlModelTrainingSamples
max.active.user.tasks=10
default.api.timeout.ms=20000
request.timeout.ms=10000
"""
# Divided periods by 10
CRUISE_CONTROL_TESTING_OPTIONS = """
Expand All @@ -72,6 +74,8 @@
min.samples.per.partition.metrics.window=1
num.partition.metrics.windows=3
num.broker.metrics.windows=10
default.api.timeout.ms=10000
request.timeout.ms=5000
"""
SERVER_PROPERTIES_BLACKLIST = [
"profile",
Expand Down
2 changes: 1 addition & 1 deletion tests/integration/test_balancer.py
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ async def test_relate_not_enough_brokers(self, ops_test: OpsTest):
)

await ops_test.model.wait_for_idle(
apps=list({APP_NAME, ZK_NAME, self.balancer_app}), idle_period=30
apps=list({APP_NAME, ZK_NAME, self.balancer_app}), idle_period=30, timeout=1800
)

async with ops_test.fast_forward(fast_interval="20s"):
Expand Down

0 comments on commit e6fd583

Please sign in to comment.