Skip to content

[v25.3.x] cluster/feature_manager: use insert_or_assign for version updates#29880

Merged
pgellert merged 1 commit intoredpanda-data:v25.3.xfrom
pgellert:manual-backport-29824-v25.3.x-902
Mar 19, 2026
Merged

[v25.3.x] cluster/feature_manager: use insert_or_assign for version updates#29880
pgellert merged 1 commit intoredpanda-data:v25.3.xfrom
pgellert:manual-backport-29824-v25.3.x-902

Conversation

@pgellert
Copy link
Copy Markdown
Contributor

@pgellert pgellert commented Mar 19, 2026

Backport of PR #29824

The second commit (dt: increase alter_cluster_quotas timeout during upgrade) was
dropped because the QuotaManagementUtils mixin and QuotaManagementUpgradeTest
it modifies do not exist on v25.3.x.

Fixes #29855

emplace is a no-op when the key already exists, so a stale version
entry can shadow a newer one queued before the loop drains _updates.
This delays feature activation by an extra health monitor tick (~10s)
because the controller must wait for the next health collection to
re-learn the node's version.

(cherry picked from commit 1ea6e4e)
@pgellert pgellert added this to the v25.3.x-next milestone Mar 19, 2026
@pgellert pgellert added the kind/backport PRs targeting a stable branch label Mar 19, 2026
@pgellert pgellert self-assigned this Mar 19, 2026
@pgellert pgellert requested review from a team and nguyen-andrew and removed request for a team March 19, 2026 14:22
@pgellert pgellert marked this pull request as ready for review March 19, 2026 14:30
@pgellert pgellert enabled auto-merge March 19, 2026 14:37
@vbotbuildovich
Copy link
Copy Markdown
Collaborator

CI test results

test results on build#82003
test_class test_method test_arguments test_kind job_url test_status passed reason test_history
RpkRedpandaStartTest test_rpc_tls_start null integration https://buildkite.com/redpanda/redpanda/builds/82003#019d069d-f282-4c81-adfb-881b21416d10 FLAKY 10/11 Test PASSES after retries.No significant increase in flaky rate(baseline=0.0000, p0=1.0000, reject_threshold=0.0100. adj_baseline=0.1000, p1=0.3487, trust_threshold=0.5000) https://redpanda.metabaseapp.com/dashboard/87-tests?tab=142-dt-individual-test-history&test_class=RpkRedpandaStartTest&test_method=test_rpc_tls_start
WriteCachingFailureInjectionE2ETest test_crash_all {"use_transactions": false} integration https://buildkite.com/redpanda/redpanda/builds/82003#019d069d-f282-4c81-adfb-881b21416d10 FLAKY 10/11 Test PASSES after retries.No significant increase in flaky rate(baseline=0.1250, p0=1.0000, reject_threshold=0.0100. adj_baseline=0.3301, p1=0.0182, trust_threshold=0.5000) https://redpanda.metabaseapp.com/dashboard/87-tests?tab=142-dt-individual-test-history&test_class=WriteCachingFailureInjectionE2ETest&test_method=test_crash_all

@pgellert pgellert merged commit d856db9 into redpanda-data:v25.3.x Mar 19, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/redpanda kind/backport PRs targeting a stable branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants