Skip to content

Commit f66bf5b

Browse files
committed
configuration: introduced tunable controlling partition shutdown timeout
Introduced property that is going to be used by `cluster::partition_manager` shutdown watchdog mechanism. Signed-off-by: Michal Maslanka <[email protected]>
1 parent 166dda6 commit f66bf5b

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

src/v/config/configuration.cc

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -261,6 +261,14 @@ configuration::configuration()
261261
.min = 0, // It is not mandatory to reserve any capacity
262262
.max = 131072 // Same max as topic_partitions_per_shard
263263
})
264+
, partition_manager_shutdown_watchdog_timeout(
265+
*this,
266+
"partition_manager_shutdown_watchdog_timeout",
267+
"A threshold value to detect partitions which shutdown might have been "
268+
"stuck. After this threshold a watchdog in partition manager will log "
269+
"information about partition shutdown not making progress",
270+
{.needs_restart = needs_restart::no, .visibility = visibility::tunable},
271+
30s)
264272
, admin_api_require_auth(
265273
*this,
266274
"admin_api_require_auth",

src/v/config/configuration.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,8 @@ struct configuration final : public config_store {
8080
bounded_property<std::optional<int32_t>> topic_fds_per_partition;
8181
bounded_property<uint32_t> topic_partitions_per_shard;
8282
bounded_property<uint32_t> topic_partitions_reserve_shard0;
83+
property<std::chrono::milliseconds>
84+
partition_manager_shutdown_watchdog_timeout;
8385

8486
// Admin API
8587
property<bool> admin_api_require_auth;

0 commit comments

Comments
 (0)