Skip to content

balancer: enable GRPC_GO_EXPERIMENTAL_CASE_SENSITIVE_BALANCER_REGISTRIES by default#9017

Merged
Pranjali-2501 merged 4 commits intogrpc:masterfrom
Pranjali-2501:fixit-issue-5288
Apr 15, 2026
Merged

balancer: enable GRPC_GO_EXPERIMENTAL_CASE_SENSITIVE_BALANCER_REGISTRIES by default#9017
Pranjali-2501 merged 4 commits intogrpc:masterfrom
Pranjali-2501:fixit-issue-5288

Conversation

@Pranjali-2501
Copy link
Copy Markdown
Contributor

@Pranjali-2501 Pranjali-2501 commented Mar 25, 2026

Part of #5288

Follow up of PR: #8837

This PR enabled the env variable GRPC_GO_EXPERIMENTAL_CASE_SENSITIVE_BALANCER_REGISTRIES by default to transition balancer registries to be case-sensitive.

RELEASE NOTES:

  • balancer: balancer registries are now case-sensitive by default. Disable GRPC_GO_EXPERIMENTAL_CASE_SENSITIVE_BALANCER_REGISTRIES env var to temporarily revert this behavior.

@Pranjali-2501 Pranjali-2501 added this to the 1.81 Release milestone Mar 25, 2026
@Pranjali-2501 Pranjali-2501 added the Type: Behavior Change Behavior changes not categorized as bugs label Mar 25, 2026
@Pranjali-2501 Pranjali-2501 changed the title balancer: enabling env flag GRPC_GO_EXPERIMENTAL_CASE_SENSITIVE_BALANCER_REGISTRIES for case-sensitive registry. balancer: enabled GRPC_GO_EXPERIMENTAL_CASE_SENSITIVE_BALANCER_REGISTRIES by default for case-sensitive registry. Mar 25, 2026
@Pranjali-2501 Pranjali-2501 requested a review from easwars March 25, 2026 12:43
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 25, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.82%. Comparing base (ccc8d7b) to head (169aacb).
⚠️ Report is 34 commits behind head on master.

Files with missing lines Patch % Lines
balancer/balancer.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #9017      +/-   ##
==========================================
- Coverage   82.99%   82.82%   -0.18%     
==========================================
  Files         411      413       +2     
  Lines       32917    33404     +487     
==========================================
+ Hits        27319    27666     +347     
- Misses       4196     4253      +57     
- Partials     1402     1485      +83     
Files with missing lines Coverage Δ
internal/envconfig/envconfig.go 100.00% <ø> (ø)
balancer/balancer.go 52.38% <0.00%> (-42.86%) ⬇️

... and 53 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@easwars easwars changed the title balancer: enabled GRPC_GO_EXPERIMENTAL_CASE_SENSITIVE_BALANCER_REGISTRIES by default for case-sensitive registry. balancer: enable GRPC_GO_EXPERIMENTAL_CASE_SENSITIVE_BALANCER_REGISTRIES by default Mar 26, 2026
Comment thread balancer/balancer.go Outdated
@easwars easwars assigned Pranjali-2501 and unassigned easwars Mar 26, 2026
Comment thread balancer/balancer.go Outdated
Comment thread balancer/balancer.go Outdated
Comment thread internal/envconfig/envconfig.go Outdated
@easwars easwars assigned Pranjali-2501 and unassigned easwars Apr 13, 2026
Comment thread balancer/balancer.go Outdated
@easwars
Copy link
Copy Markdown
Contributor

easwars commented Apr 14, 2026

The release notes need to be updated. They are wrong.

@easwars easwars assigned Pranjali-2501 and unassigned easwars Apr 14, 2026
@Pranjali-2501 Pranjali-2501 merged commit c5b160e into grpc:master Apr 15, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Behavior Change Behavior changes not categorized as bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants