Skip to content

ringhash: enable behaviors of A76 by default#8922

Merged
arjan-bal merged 1 commit intogrpc:masterfrom
atollena:enable-a76-by-default
Feb 23, 2026
Merged

ringhash: enable behaviors of A76 by default#8922
arjan-bal merged 1 commit intogrpc:masterfrom
atollena:enable-a76-by-default

Conversation

@atollena
Copy link
Copy Markdown
Collaborator

@atollena atollena commented Feb 20, 2026

Those features have been gated by environment variables since 1.73.0, and used in production at Datadog. They can be considered stable.

Change the defaults to enable using EDS hash_key lb metadata and requestHashHeader LB config field.

RELEASE NOTES:

  • ringhash: Enable A76 ring hash improvements by default: EDS endpoint hash key parsing from LbEndpoint metadata and request hash header support via the "requestHashHeader" field. The previous behavior can be restored by setting GRPC_XDS_ENDPOINT_HASH_KEY_BACKWARD_COMPAT=true or GRPC_EXPERIMENTAL_RING_HASH_SET_REQUEST_HASH_KEY=false.

Those features have been gated by environment variables since 1.73.0, and used
in production at Datadog. They can be considered stable.

Change the defaults to enable using eds hash_key lb metadata and
set_request_hash key.

RELEASE NOTES:
- ringhash: Enable A76 ring hash improvements by default: EDS endpoint hash
  key parsing from LbEndpoint metadata and request hash header support via
  the "requestHashHeader" field. The previous behavior can be restored by
  setting GRPC_XDS_ENDPOINT_HASH_KEY_BACKWARD_COMPAT=true or
  GRPC_EXPERIMENTAL_RING_HASH_SET_REQUEST_HASH_KEY=false.
@atollena atollena added Type: Feature New features or improvements in behavior Area: Resolvers/Balancers Includes LB policy & NR APIs, resolver/balancer/picker wrappers, LB policy impls and utilities. labels Feb 20, 2026
@atollena atollena added this to the 1.80 Release milestone Feb 20, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.20%. Comparing base (19e4128) to head (f47dce7).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8922      +/-   ##
==========================================
- Coverage   83.30%   83.20%   -0.11%     
==========================================
  Files         417      417              
  Lines       32940    32940              
==========================================
- Hits        27442    27407      -35     
- Misses       4082     4111      +29     
- Partials     1416     1422       +6     
Files with missing lines Coverage Δ
internal/envconfig/envconfig.go 100.00% <ø> (ø)

... and 29 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
Copy link
Copy Markdown
Contributor

easwars commented Feb 20, 2026

@arjan-bal : For second set of eyes

Copy link
Copy Markdown
Contributor

@arjan-bal arjan-bal left a comment

Choose a reason for hiding this comment

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

LGTM

@arjan-bal arjan-bal merged commit 4a83bf9 into grpc:master Feb 23, 2026
14 of 18 checks passed
This was referenced Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Resolvers/Balancers Includes LB policy & NR APIs, resolver/balancer/picker wrappers, LB policy impls and utilities. Type: Feature New features or improvements in behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants