Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

c/topic_table: do not log duplicated lifecycle marker command #15407

Merged
merged 1 commit into from
Dec 20, 2023

Conversation

mmaslankaprv
Copy link
Member

@mmaslankaprv mmaslankaprv commented Dec 12, 2023

It is possible that the drop topic lifecycle marker replicate request would timeout. In this case requester has no other option than to retry dropping lifecycle marker. Logic in cluster::topic_table is idempotent and duplicate marker drop command does not harm to Redpanda. Changed log level of an entry notifying about duplicated drop command from an error to info level.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.2.x
  • v23.1.x
  • v22.3.x

Release Notes

  • none

@mmaslankaprv
Copy link
Member Author

/dt

@vbotbuildovich
Copy link
Collaborator

vbotbuildovich commented Dec 12, 2023

new failures in https://buildkite.com/redpanda/redpanda/builds/42618#018c5f37-fac9-4baa-95e3-1ea007e87cdb:

"rptest.tests.partition_balancer_test.PartitionBalancerTest.test_fuzz_admin_ops"

new failures in https://buildkite.com/redpanda/redpanda/builds/42618#018c5f10-2446-4630-94d8-ab10b32f43da:

"rptest.tests.data_transforms_test.DataTransformsTest.test_identity.transactional=True"

@mmaslankaprv
Copy link
Member Author

/ci-repeat 1

ztlpn
ztlpn previously approved these changes Dec 15, 2023
src/v/cluster/topic_table.cc Outdated Show resolved Hide resolved
It is possible that the drop topic lifecycle marker replicate request
would timeout. In this case requester has no other option than to retry
dropping lifecycle marker. Logic in `cluster::topic_table` is idempotent
and duplicate marker drop command does not harm to Redpanda. Changed log
level of an entry notifying about duplicated drop command from an error
to info level.

Signed-off-by: Michal Maslanka <michal@redpanda.com>
ztlpn
ztlpn previously approved these changes Dec 15, 2023
@mmaslankaprv
Copy link
Member Author

/ci-repeat 1

@mmaslankaprv mmaslankaprv merged commit 4275dcb into redpanda-data:dev Dec 20, 2023
23 checks passed
@mmaslankaprv mmaslankaprv deleted the fix-error-topic-table branch December 20, 2023 07:30
@vbotbuildovich
Copy link
Collaborator

/backport v23.2.x

@mmaslankaprv
Copy link
Member Author

/backport v23.3.x

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.

None yet

4 participants