-
Notifications
You must be signed in to change notification settings - Fork 405
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
feat: support viploadbalancer controller #2036
base: master
Are you sure you want to change the base?
Conversation
b4323a2
to
6a6e532
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2036 +/- ##
==========================================
+ Coverage 56.09% 56.57% +0.47%
==========================================
Files 186 193 +7
Lines 18092 18771 +679
==========================================
+ Hits 10149 10619 +470
- Misses 6910 7052 +142
- Partials 1033 1100 +67
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
75e4110
to
77148c3
Compare
/hold |
f156534
to
963c247
Compare
/rerun |
/unhold |
/assign @rambohe-ch @zyjhtangtang @River-sh @Rui-Gan |
pkg/yurtmanager/controller/loadbalancerset/viploadbalancer/config/types.go
Outdated
Show resolved
Hide resolved
pkg/yurtmanager/controller/loadbalancerset/viploadbalancer/viploadbalancer_controller.go
Outdated
Show resolved
Hide resolved
pkg/yurtmanager/controller/loadbalancerset/viploadbalancer/viploadbalancer_controller.go
Outdated
Show resolved
Hide resolved
pkg/yurtmanager/controller/loadbalancerset/viploadbalancer/viploadbalancer_controller.go
Outdated
Show resolved
Hide resolved
pkg/yurtmanager/controller/loadbalancerset/viploadbalancer/viploadbalancer_controller.go
Show resolved
Hide resolved
pkg/yurtmanager/webhook/poolservice/v1alpha1/poolservice_default.go
Outdated
Show resolved
Hide resolved
pkg/yurtmanager/controller/loadbalancerset/viploadbalancer/viploadbalancer_controller.go
Outdated
Show resolved
Hide resolved
I have updated the logic of syncPoolService to first synchronize the validly allocated vrids in the cluster first, which is used to support the user-specified mode of vrid deployment. In addition, we will perform unified allocation for unallocated or vrid exceptions, which include cases where the annotation of the vrid is null, the vrid is abnormal, and the vrid is out of range. This layer of logic will also be judged in reconcileDelete. |
9e138a0
to
51b2104
Compare
pkg/yurtmanager/webhook/poolservice/v1alpha1/poolservice_handler.go
Outdated
Show resolved
Hide resolved
242d2f6
to
38eba42
Compare
a904f69
to
aae406d
Compare
Signed-off-by: wangxye <[email protected]>
Signed-off-by: wangxye <[email protected]>
Signed-off-by: wangxye <[email protected]>
Signed-off-by: wangxye <[email protected]>
Signed-off-by: wangxye <[email protected]>
Signed-off-by: wangxye <[email protected]>
…offline status Signed-off-by: wangxye <[email protected]>
Signed-off-by: wangxye <[email protected]>
Signed-off-by: wangxye <[email protected]>
68803c7
to
1465fa1
Compare
Signed-off-by: wangxye <[email protected]>
1465fa1
to
98b2205
Compare
Signed-off-by: wangxye <[email protected]>
fac5c82
to
b5b012d
Compare
Signed-off-by: wangxye <[email protected]>
Signed-off-by: wangxye <[email protected]>
Signed-off-by: wangxye <[email protected]>
54ee33e
to
1f89541
Compare
Signed-off-by: wangxye <[email protected]>
7e8d0f8
to
71bf9ce
Compare
Quality Gate passedIssues Measures |
@rambohe-ch Do you have time to look at this? |
@wangxye Sorry to response very lately, because of dealing with some personal works. |
Okay, no problem, looking forward to your feedback so we can get it done early! |
What type of PR is this?
/kind feature
What this PR does / why we need it:
As the second-level controller for the multi-region load balancer, It is used to support the control of high availability of virtual IP based edge services, monitor poolservice changes, and is used to synchronize vrid changes to the vip agent at the edge.
Special notes for your reviewer:
/assign @rambohe-ch @zyjhtangtang @River-sh @Rui-Gan
Does this PR introduce a user-facing change?
Support left-closed-right-open ip assignment: