-
Notifications
You must be signed in to change notification settings - Fork 38.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Promote load balancer class to GA #107979
Promote load balancer class to GA #107979
Conversation
@XudongLiuHarold: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The 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/test-infra repository. |
53c0c54
to
8e81329
Compare
8e81329
to
9ce8c58
Compare
6d350e0
to
a71735c
Compare
/test pull-kubernetes-node-e2e-containerd |
/retest |
1 similar comment
/retest |
a71735c
to
3588f33
Compare
/retest |
0c17341
to
db7d7ad
Compare
/retest |
a599a7d
to
b1b2d2f
Compare
/retest |
Hi @andrewsykim , could you please review this PR again? Thank you! |
staging/src/k8s.io/cloud-provider/controllers/service/controller_test.go
Outdated
Show resolved
Hide resolved
b1b2d2f
to
0fcd225
Compare
/retest |
/test pull-kubernetes-integration |
Hi @andrewsykim , could you please review this PR again? I add the waiting time for the integration test cases and test it several times to prove the integration test can pass stably. :) thanks! /cc @andrewsykim |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall LGTM
/assign @thockin
staging/src/k8s.io/cloud-provider/controllers/service/controller_test.go
Outdated
Show resolved
Hide resolved
Signed-off-by: Xudong Liu <xudongliuharold@gmail.com>
0fcd225
to
c8a6c7f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: thockin, XudongLiuHarold The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest |
Signed-off-by: Xudong Liu xudongliuharold@gmail.com
What type of PR is this?
/kind feature
What this PR does / why we need it:
In #98277 we added a new field in Service that allows users to use a custom load balancer for Service Type=LB. This PR promotes this feature to GA via :
loadBalancerClass
feature gate to GA stageWhich issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: