Skip to content

Conversation

@timtimb0t
Copy link
Contributor

@timtimb0t timtimb0t commented Sep 3, 2025

this is minor change that allow to avoid masking KillNemesis
in _refuse_connection_from_banned_node
fix: #11816

Testing:
https://jenkins.scylladb.com/job/scylla-staging/job/eugene_test_folder/job/longevity-twcs-48h-test/2/

PR pre-checks (self review)

  • I added the relevant backport labels
  • I didn't leave commented-out/debugging code

Reminders

  • Add New configuration option and document them (in sdcm/sct_config.py)
  • Add unit tests to cover my changes (under unit-test/ folder)
  • Update the Readme/doc folder relevant to this change (if needed)

@timtimb0t
Copy link
Contributor Author

Hi @fruch , could you please take a look? Also Im not sure of how to test it, maybe you have some suggestions?

@timtimb0t timtimb0t requested review from fruch and soyacz September 3, 2025 10:34
@timtimb0t timtimb0t force-pushed the kill_nemesis_exception branch from c7ffcc0 to 22d62de Compare September 3, 2025 10:47
@fruch
Copy link
Contributor

fruch commented Sep 3, 2025

Hi @fruch , could you please take a look? Also Im not sure of how to test it, maybe you have some suggestions?

A short run of only this specific nemesis, to validate first it doesn't break/regress it, other then that I don't see more that should be tested.

@timtimb0t timtimb0t force-pushed the kill_nemesis_exception branch from 22d62de to 27d9b35 Compare September 4, 2025 11:14
this is minor change that allow to avoid masking KillNemesis
 in _refuse_connection_from_banned_node
fix: scylladb#11816
@timtimb0t timtimb0t force-pushed the kill_nemesis_exception branch from 27d9b35 to b9e6d24 Compare September 4, 2025 14:13
@timtimb0t timtimb0t added the backport/none Backport is not required label Sep 4, 2025
@timtimb0t timtimb0t marked this pull request as ready for review September 4, 2025 14:14
Copy link
Contributor

@vponomaryov vponomaryov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@fruch fruch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fruch fruch merged commit 719aa45 into scylladb:master Sep 11, 2025
9 checks passed
@vponomaryov vponomaryov added backport/2025.3 and removed backport/none Backport is not required labels Sep 11, 2025
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.

Another exception occurred during handling sdcm.exceptions.KillNemesis in _refuse_connection_from_banned_node

4 participants