Skip to content

[outlier detection] fix crash with pick_first and add tests#33069

Merged
markdroth merged 3 commits into
grpc:masterfrom
markdroth:outlier_detection_pick_first_crash
May 10, 2023
Merged

[outlier detection] fix crash with pick_first and add tests#33069
markdroth merged 3 commits into
grpc:masterfrom
markdroth:outlier_detection_pick_first_crash

Conversation

@markdroth

Copy link
Copy Markdown
Member

Fixes #32967.

Also fix incorrect defaults for enforcementPercentage fields.

@markdroth markdroth added lang/core release notes: yes Indicates if PR needs to be in release notes labels May 10, 2023
@markdroth markdroth requested a review from eugeneo May 10, 2023 16:19
@markdroth markdroth merged commit 2c423d2 into grpc:master May 10, 2023
@markdroth markdroth deleted the outlier_detection_pick_first_crash branch May 10, 2023 20:33
@copybara-service copybara-service Bot added the imported Specifies if the PR has been imported to the internal repository label May 10, 2023
eugeneo pushed a commit to eugeneo/grpc that referenced this pull request May 17, 2023
Fixes grpc#32967.

Also fix incorrect defaults for `enforcementPercentage` fields.
wanlin31 pushed a commit that referenced this pull request May 18, 2023
Fixes #32967.

Also fix incorrect defaults for `enforcementPercentage` fields.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bloat/none imported Specifies if the PR has been imported to the internal repository lang/core per-call-memory/neutral per-channel-memory/neutral release notes: yes Indicates if PR needs to be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Segfault when using outlier detection LB with pick_first

2 participants