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.

Sign up for GitHub

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

Jump to bottom

[TESTING] Feature/inbound only graceful draining #38450

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

[TESTING] Feature/inbound only graceful draining #38450

wants to merge 11 commits into from

Conversation

fredyw
Copy link
Member

@fredyw fredyw commented Feb 13, 2025

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@keithmattix
Merge feature/inbound-only-graceful-draining
8b87b2e 
Signed-off-by: Keith Mattix II <[email protected]>
@keithmattix
Fixup
ed1132c 
Signed-off-by: Keith Mattix II <[email protected]>
@keithmattix
Move stopped_listener check to callback on main thread
9b00aaa 
Signed-off-by: Keith Mattix II <[email protected]>
@keithmattix
Add libatomic to jni
c7f67fd 
Signed-off-by: Keith Mattix II <[email protected]>
@repokitteh-read-only RepoKitteh - Read Only
Copy link

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #38450 was opened by fredyw.

see: more, trace.

@fredyw
Copy link
Member Author

fredyw commented Feb 13, 2025

/retest

@fredyw
Try a fix
40faf7e 
Signed-off-by: Fredy Wijaya <[email protected]>
@fredyw
More fix
a0579d2 
Signed-off-by: Fredy Wijaya <[email protected]>
@fredyw
Fix
52161e7 
Signed-off-by: Fredy Wijaya <[email protected]>
@fredyw
Typo
757d5b9 
Signed-off-by: Fredy Wijaya <[email protected]>
@fredyw
Fix
b69fd4a 
Signed-off-by: Fredy Wijaya <[email protected]>
@fredyw
More fix
a327904 
Signed-off-by: Fredy Wijaya <[email protected]>
@fredyw
Fix
763b8ae 
Signed-off-by: Fredy Wijaya <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RyanTheOptimist RyanTheOptimist Awaiting requested review from RyanTheOptimist RyanTheOptimist will be requested when the pull request is marked ready for review RyanTheOptimist is a code owner

@alyssawilk alyssawilk Awaiting requested review from alyssawilk alyssawilk will be requested when the pull request is marked ready for review alyssawilk is a code owner

@abeyad abeyad Awaiting requested review from abeyad abeyad will be requested when the pull request is marked ready for review abeyad is a code owner

@wbpcode wbpcode Awaiting requested review from wbpcode wbpcode will be requested when the pull request is marked ready for review wbpcode is a code owner

@zuercher zuercher Awaiting requested review from zuercher zuercher will be requested when the pull request is marked ready for review zuercher is a code owner

@mattklein123 mattklein123 Awaiting requested review from mattklein123 mattklein123 will be requested when the pull request is marked ready for review mattklein123 is a code owner

@adisuissa adisuissa Awaiting requested review from adisuissa adisuissa will be requested when the pull request is marked ready for review adisuissa is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fredyw @keithmattix