Skip to content

Backport of #3676 - #3800

Merged
dotnwat merged 1 commit into
redpanda-data:v21.11.xfrom
mmaslankaprv:v21.11.x
Feb 17, 2022
Merged

Backport of #3676#3800
dotnwat merged 1 commit into
redpanda-data:v21.11.xfrom
mmaslankaprv:v21.11.x

Conversation

@mmaslankaprv

@mmaslankaprv mmaslankaprv commented Feb 15, 2022

Copy link
Copy Markdown
Member

Previously metadata dissemination service used cluster health snapshot
to update partition leaders metadata. This might lead to situation in
which cluster health report in never requested and eventually metadata
information is not updated. This may lead to situation in which
different subsystems querying cluster::partitions_leaders_table
directly would receive stale leadership information. (if dissemination
request was lost)
Changed the metadata dissemination to request cluster health report
refresh. This way leadership information will always be updated and not
older than health monitor max metadata age.

backport #3676

Previously metadata dissemination service used cluster health snapshot
to update partition leaders metadata. This might lead to situation in
which cluster health report in never requested and eventually metadata
information is not updated. This may lead to situation in which
different subsystems querying `cluster::partitions_leaders_table`
directly would receive stale leadership information. (if dissemination
request was lost)
Changed the metadata dissemination to request cluster health report
refresh. This way leadership information will always be updated and not
older than health monitor max metadata age.

Signed-off-by: Michal Maslanka <michal@vectorized.io>
(cherry picked from commit e7e7634)
@mmaslankaprv mmaslankaprv added this to the v21.11.5 milestone Feb 15, 2022
@mmaslankaprv mmaslankaprv modified the milestones: v21.11.5, v21.11.6 Feb 15, 2022
@dotnwat
dotnwat merged commit 7e55a85 into redpanda-data:v21.11.x Feb 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants