api: Added support for weighted field in BackendRef#7452
api: Added support for weighted field in BackendRef#7452arkodg merged 18 commits intoenvoyproxy:mainfrom
Conversation
…process Signed-off-by: Nayan <nayanpahuja25@gmail.com>
Signed-off-by: Nayan <nayanpahuja25@gmail.com>
Signed-off-by: Nayan <nayanpahuja25@gmail.com>
Signed-off-by: Nayan <nayanpahuja25@gmail.com>
Signed-off-by: Nayan <nayanpahuja25@gmail.com>
|
cc: @arkodg Will need your help on what tests should I exactly write for this PR. I will make the necessary changes to the same PR. Edit: Also I see that since BackendRef is referenced other places such as the down below. Let me know if any subsequent changes are required since these might be impacted as well. ALSEnvoyProxyAccessLog |
Signed-off-by: Nayan <nayanpahuja25@gmail.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7452 +/- ##
==========================================
+ Coverage 72.27% 72.30% +0.03%
==========================================
Files 232 232
Lines 34109 34114 +5
==========================================
+ Hits 24653 24667 +14
+ Misses 7680 7673 -7
+ Partials 1776 1774 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: Nayan <nayanpahuja25@gmail.com>
Signed-off-by: Nayan <nayanpahuja25@gmail.com>
|
hey @NayanPahuja thanks for filling this gap, the code LGTM |
Signed-off-by: Nayan <nayanpahuja25@gmail.com>
Signed-off-by: Nayan <nayanpahuja25@gmail.com>
What type of PR is this?
feat(api): Added support for weighted field in BackendRef
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes Support weight field for non x-routes #7449
Release Notes: Yes