Skip to content

Conversation

@He-Pin
Copy link
Member

@He-Pin He-Pin commented Aug 27, 2025

backport add non-default config that allows InboundQuarantineCheck to ignore 'harmless' quarantine events (#1555)

cherry picks ec5e33f

…harmless' quarantine events (apache#1555)

* stub test for harmless=true

Update OutboundIdleShutdownSpec.scala

Update OutboundIdleShutdownSpec.scala

Update OutboundIdleShutdownSpec.scala

* add quarantinedButHarmless check for tests

* new test case

* Update OutboundIdleShutdownSpec.scala

* try to not shutdown when quarantine is harmless

* Update OutboundIdleShutdownSpec.scala

* Create quarantine.backwards.excludes

* Update quarantine.backwards.excludes

* update log message

* try to add config

* Update ArterySettings.scala

* add tests

* Update OutboundIdleShutdownSpec.scala

* rework test

(cherry picked from commit ec5e33f)
@He-Pin He-Pin requested a review from pjfanning August 27, 2025 10:45
@He-Pin He-Pin added this to the 1.1.6 milestone Aug 27, 2025
Copy link
Member

@pjfanning pjfanning left a comment

Choose a reason for hiding this comment

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

lgtm

@He-Pin He-Pin merged commit 85291e0 into apache:1.1.x Aug 27, 2025
9 of 10 checks passed
@He-Pin He-Pin deleted the harmless branch August 27, 2025 12:13
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