Controller: Add traffic distribution support.#12974
Controller: Add traffic distribution support.#12974k8s-ci-robot merged 1 commit intokubernetes:mainfrom
Conversation
|
This issue is currently awaiting triage. If Ingress contributors determines this is a relevant issue, they will accept it by applying the The DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
Hi @tufitko. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
✅ Deploy Preview for kubernetes-ingress-nginx canceled.
|
678e084 to
093e95f
Compare
|
/triage accept |
|
@strongjz: The label(s) DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
/kind feature |
093e95f to
ade8e56
Compare
Gacko
left a comment
There was a problem hiding this comment.
/triage accepted
/kind feature
/priority backlog
/lgtm
Was a bit hard to understand at first, but I realized you're mostly extending existing behavior, so looks good to me! 🙂
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Gacko, tufitko The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Hey gents, any idea which release will contain this feature? controller 1.12.3 doesn't have it. |
What this PR does / why we need it:
Support trafficDistribution. If service doesn't have an endpoint with matched zone, fallbacks to all endpoints
Types of changes
Which issue/s this PR fixes
#11342
How Has This Been Tested?
writing unit tests
Checklist: