gateway icon indicating copy to clipboard operation
gateway copied to clipboard

feat(translator): Set zone for endpoint and arrange endpoints by zone in EDS

Open aoledk opened this issue 1 year ago • 2 comments

What this PR does / why we need it:

Set zone for endpoint and arrange endpoints by zone in EDS if zone is available in EndpointSlice. This is required before users can apply priority levels or zone aware routing load balancing strategy.

Which issue(s) this PR fixes:

Fixes #3538

aoledk avatar Jun 05 '24 08:06 aoledk

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 68.34%. Comparing base (4a74e60) to head (026f29f).

:exclamation: Current head 026f29f differs from pull request most recent head eba8f1c

Please upload reports for the commit eba8f1c to get more accurate results.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3542      +/-   ##
==========================================
- Coverage   68.81%   68.34%   -0.47%     
==========================================
  Files         175      170       -5     
  Lines       21525    20770     -755     
==========================================
- Hits        14812    14196     -616     
+ Misses       5636     5556      -80     
+ Partials     1077     1018      -59     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jun 06 '24 04:06 codecov[bot]

/retest

aoledk avatar Jun 06 '24 10:06 aoledk

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. Please feel free to give a status update now, ping for review, when it's ready. Thank you for your contributions!

github-actions[bot] avatar Aug 02 '24 08:08 github-actions[bot]

closing this PR since its become inactive, Zone aware routing is now available on v1.4.0

arkodg avatar May 23 '25 02:05 arkodg