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

r/state_machine: ignore broken semaphore and condition var exceptions #15426

Merged
merged 1 commit into from
Dec 14, 2023

Conversation

mmaslankaprv
Copy link
Member

@mmaslankaprv mmaslankaprv commented Dec 13, 2023

Seastar broken_semaphore and broken_condition_variable exceptions are thrown from when Redpanda is shutting down. Those exceptions doesn't have to be logged with error severity.

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

ztlpn
ztlpn previously approved these changes Dec 13, 2023
@mmaslankaprv
Copy link
Member Author

/dt

rockwotj
rockwotj previously approved these changes Dec 13, 2023
src/v/raft/state_machine.cc Show resolved Hide resolved
@vbotbuildovich
Copy link
Collaborator

new failures in https://buildkite.com/redpanda/redpanda/builds/42685#018c63f9-6cb5-4e73-99a6-59cbbba19989:

"rptest.tests.cluster_features_test.FeaturesMultiNodeUpgradeTest.test_upgrade"

Seastar `broken_semaphore` and `broken_condition_variable` exceptions
are thrown from when Redpanda is shutting down. Those exceptions doesn't
have to be logged with error severity.

Signed-off-by: Michal Maslanka <michal@redpanda.com>
@vbotbuildovich
Copy link
Collaborator

vbotbuildovich commented Dec 14, 2023

@piyushredpanda piyushredpanda merged commit 7ef7a17 into redpanda-data:dev Dec 14, 2023
17 of 20 checks passed
@vbotbuildovich
Copy link
Collaborator

/backport v23.2.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.

5 participants