• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

kubeovn / kube-ovn / 21935848371

12 Feb 2026 06:18AM UTC coverage: 23.003% (-0.008%) from 23.011%
21935848371

push

github

oilbeater
remove unnecessary logs

Signed-off-by: Mengxin Liu <liumengxinfly@gmail.com>

12504 of 54358 relevant lines covered (23.0%)

0.27 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

16.36
/pkg/controller/vpc.go
1
package controller
2

3
import (
4
        "context"
5
        "encoding/json"
6
        "errors"
7
        "fmt"
8
        "maps"
9
        "math"
10
        "net"
11
        "reflect"
12
        "slices"
13
        "sort"
14
        "strings"
15
        "time"
16

17
        v1 "k8s.io/api/core/v1"
18
        k8serrors "k8s.io/apimachinery/pkg/api/errors"
19
        metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
20
        "k8s.io/apimachinery/pkg/labels"
21
        "k8s.io/apimachinery/pkg/types"
22
        "k8s.io/client-go/tools/cache"
23
        "k8s.io/klog/v2"
24
        "sigs.k8s.io/controller-runtime/pkg/controller/controllerutil"
25

26
        kubeovnv1 "github.com/kubeovn/kube-ovn/pkg/apis/kubeovn/v1"
27
        "github.com/kubeovn/kube-ovn/pkg/ovsdb/ovnnb"
28
        "github.com/kubeovn/kube-ovn/pkg/util"
29
)
30

31
func (c *Controller) enqueueAddVpc(obj any) {
×
32
        vpc := obj.(*kubeovnv1.Vpc)
×
33
        key := cache.MetaObjectToName(vpc).String()
×
34
        if _, ok := vpc.Labels[util.VpcExternalLabel]; !ok {
×
35
                klog.V(3).Infof("enqueue add vpc %s", key)
×
36
                c.addOrUpdateVpcQueue.Add(key)
×
37
        }
×
38
}
39

40
func vpcBFDPortChanged(oldObj, newObj *kubeovnv1.BFDPort) bool {
×
41
        if oldObj == nil && newObj == nil {
×
42
                return false
×
43
        }
×
44
        if oldObj == nil || newObj == nil {
×
45
                return true
×
46
        }
×
47
        return oldObj.Enabled != newObj.Enabled || oldObj.IP != newObj.IP || !reflect.DeepEqual(oldObj.NodeSelector, newObj.NodeSelector)
×
48
}
49

50
func (c *Controller) enqueueUpdateVpc(oldObj, newObj any) {
×
51
        oldVpc := oldObj.(*kubeovnv1.Vpc)
×
52
        newVpc := newObj.(*kubeovnv1.Vpc)
×
53

×
54
        if newVpc.Labels != nil && newVpc.Labels[util.VpcExternalLabel] == "true" {
×
55
                return
×
56
        }
×
57

58
        if !newVpc.DeletionTimestamp.IsZero() ||
×
59
                !slices.Equal(oldVpc.Spec.Namespaces, newVpc.Spec.Namespaces) ||
×
60
                !reflect.DeepEqual(oldVpc.Spec.StaticRoutes, newVpc.Spec.StaticRoutes) ||
×
61
                !reflect.DeepEqual(oldVpc.Spec.PolicyRoutes, newVpc.Spec.PolicyRoutes) ||
×
62
                !reflect.DeepEqual(oldVpc.Spec.VpcPeerings, newVpc.Spec.VpcPeerings) ||
×
63
                kubeOvnAnnotationsChanged(oldVpc.Annotations, newVpc.Annotations) ||
×
64
                !slices.Equal(oldVpc.Spec.ExtraExternalSubnets, newVpc.Spec.ExtraExternalSubnets) ||
×
65
                oldVpc.Spec.EnableExternal != newVpc.Spec.EnableExternal ||
×
66
                oldVpc.Spec.EnableBfd != newVpc.Spec.EnableBfd ||
×
67
                vpcBFDPortChanged(oldVpc.Spec.BFDPort, newVpc.Spec.BFDPort) ||
×
68
                oldVpc.Labels[util.VpcExternalLabel] != newVpc.Labels[util.VpcExternalLabel] ||
×
69
                !slices.Equal(oldVpc.Status.Subnets, newVpc.Status.Subnets) {
×
70
                // TODO:// label VpcExternalLabel replace with spec enable external
×
71

×
72
                // recode last policies
×
73
                c.vpcLastPoliciesMap.Store(newVpc.Name, convertPolicies(oldVpc.Spec.PolicyRoutes))
×
74

×
75
                key := cache.MetaObjectToName(newVpc).String()
×
76
                klog.Infof("enqueue update vpc %s", key)
×
77
                c.addOrUpdateVpcQueue.Add(key)
×
78
        }
×
79
}
80

81
func (c *Controller) enqueueDelVpc(obj any) {
×
82
        var vpc *kubeovnv1.Vpc
×
83
        switch t := obj.(type) {
×
84
        case *kubeovnv1.Vpc:
×
85
                vpc = t
×
86
        case cache.DeletedFinalStateUnknown:
×
87
                v, ok := t.Obj.(*kubeovnv1.Vpc)
×
88
                if !ok {
×
89
                        klog.Warningf("unexpected object type: %T", t.Obj)
×
90
                        return
×
91
                }
×
92
                vpc = v
×
93
        default:
×
94
                klog.Warningf("unexpected type: %T", obj)
×
95
                return
×
96
        }
97

98
        if _, ok := vpc.Labels[util.VpcExternalLabel]; !vpc.Status.Default || !ok {
×
99
                klog.V(3).Infof("enqueue delete vpc %s", vpc.Name)
×
100
                c.delVpcQueue.Add(vpc)
×
101
        }
×
102
}
103

104
func (c *Controller) handleDelVpc(vpc *kubeovnv1.Vpc) error {
×
105
        c.vpcKeyMutex.LockKey(vpc.Name)
×
106
        defer func() { _ = c.vpcKeyMutex.UnlockKey(vpc.Name) }()
×
107
        klog.Infof("handle delete vpc %s", vpc.Name)
×
108

×
109
        // should delete vpc subnets first
×
110
        var err error
×
111
        for _, subnet := range vpc.Status.Subnets {
×
112
                if _, err = c.subnetsLister.Get(subnet); err != nil {
×
113
                        if k8serrors.IsNotFound(err) {
×
114
                                continue
×
115
                        }
116
                        err = fmt.Errorf("failed to get subnet %s for vpc %s: %w", subnet, vpc.Name, err)
×
117
                } else {
×
118
                        err = fmt.Errorf("failed to delete vpc %s, please delete subnet %s first", vpc.Name, subnet)
×
119
                }
×
120
                klog.Error(err)
×
121
                return err
×
122
        }
123

124
        // clean up vpc last policies cached
125
        c.vpcLastPoliciesMap.Delete(vpc.Name)
×
126

×
127
        if err := c.deleteVpcLb(vpc); err != nil {
×
128
                klog.Error(err)
×
129
                return err
×
130
        }
×
131

132
        if err := c.handleDelVpcExternalSubnet(vpc.Name, c.config.ExternalGatewaySwitch); err != nil {
×
133
                klog.Errorf("failed to delete external connection for vpc %s, error %v", vpc.Name, err)
×
134
                return err
×
135
        }
×
136

137
        for _, subnet := range vpc.Status.ExtraExternalSubnets {
×
138
                klog.Infof("disconnect external network %s to vpc %s", subnet, vpc.Name)
×
139
                if err := c.handleDelVpcExternalSubnet(vpc.Name, subnet); err != nil {
×
140
                        klog.Error(err)
×
141
                        return err
×
142
                }
×
143
        }
144

145
        if err := c.deleteVpcRouter(vpc.Status.Router); err != nil {
×
146
                klog.Error(err)
×
147
                return err
×
148
        }
×
149

150
        return nil
×
151
}
152

153
func (c *Controller) handleUpdateVpcStatus(key string) error {
×
154
        c.vpcKeyMutex.LockKey(key)
×
155
        defer func() { _ = c.vpcKeyMutex.UnlockKey(key) }()
×
156
        klog.Infof("handle status update for vpc %s", key)
×
157

×
158
        cachedVpc, err := c.vpcsLister.Get(key)
×
159
        if err != nil {
×
160
                if k8serrors.IsNotFound(err) {
×
161
                        return nil
×
162
                }
×
163
                klog.Error(err)
×
164
                return err
×
165
        }
166
        vpc := cachedVpc.DeepCopy()
×
167

×
168
        subnets, defaultSubnet, err := c.getVpcSubnets(vpc)
×
169
        if err != nil {
×
170
                klog.Error(err)
×
171
                return err
×
172
        }
×
173

174
        change := vpc.Status.DefaultLogicalSwitch != defaultSubnet
×
175

×
176
        vpc.Status.DefaultLogicalSwitch = defaultSubnet
×
177
        vpc.Status.Subnets = subnets
×
178

×
179
        if !vpc.Spec.BFDPort.IsEnabled() && !vpc.Status.BFDPort.IsEmpty() {
×
180
                vpc.Status.BFDPort.Clear()
×
181
        }
×
182
        bytes, err := vpc.Status.Bytes()
×
183
        if err != nil {
×
184
                klog.Error(err)
×
185
                return err
×
186
        }
×
187

188
        vpc, err = c.config.KubeOvnClient.KubeovnV1().Vpcs().Patch(context.Background(), vpc.Name, types.MergePatchType, bytes, metav1.PatchOptions{}, "status")
×
189
        if err != nil {
×
190
                klog.Error(err)
×
191
                return err
×
192
        }
×
193

194
        if len(vpc.Status.Subnets) == 0 {
×
195
                klog.Infof("vpc %s has no subnets, add to queue", vpc.Name)
×
196
                c.addOrUpdateVpcQueue.AddAfter(vpc.Name, 5*time.Second)
×
197
        }
×
198

199
        if change {
×
200
                for _, ns := range vpc.Spec.Namespaces {
×
201
                        c.addNamespaceQueue.Add(ns)
×
202
                }
×
203
        }
204

205
        natGws, err := c.vpcNatGatewayLister.List(labels.Everything())
×
206
        if err != nil {
×
207
                klog.Error(err)
×
208
                return err
×
209
        }
×
210
        for _, gw := range natGws {
×
211
                if key == gw.Spec.Vpc {
×
212
                        c.updateVpcSubnetQueue.Add(gw.Name)
×
213
                }
×
214
        }
215
        return nil
×
216
}
217

218
type VpcLoadBalancer struct {
219
        TCPLoadBalancer      string
220
        TCPSessLoadBalancer  string
221
        UDPLoadBalancer      string
222
        UDPSessLoadBalancer  string
223
        SctpLoadBalancer     string
224
        SctpSessLoadBalancer string
225
}
226

227
func (c *Controller) GenVpcLoadBalancer(vpcKey string) *VpcLoadBalancer {
×
228
        if vpcKey == c.config.ClusterRouter || vpcKey == "" {
×
229
                return &VpcLoadBalancer{
×
230
                        TCPLoadBalancer:      c.config.ClusterTCPLoadBalancer,
×
231
                        TCPSessLoadBalancer:  c.config.ClusterTCPSessionLoadBalancer,
×
232
                        UDPLoadBalancer:      c.config.ClusterUDPLoadBalancer,
×
233
                        UDPSessLoadBalancer:  c.config.ClusterUDPSessionLoadBalancer,
×
234
                        SctpLoadBalancer:     c.config.ClusterSctpLoadBalancer,
×
235
                        SctpSessLoadBalancer: c.config.ClusterSctpSessionLoadBalancer,
×
236
                }
×
237
        }
×
238
        return &VpcLoadBalancer{
×
239
                TCPLoadBalancer:      fmt.Sprintf("vpc-%s-tcp-load", vpcKey),
×
240
                TCPSessLoadBalancer:  fmt.Sprintf("vpc-%s-tcp-sess-load", vpcKey),
×
241
                UDPLoadBalancer:      fmt.Sprintf("vpc-%s-udp-load", vpcKey),
×
242
                UDPSessLoadBalancer:  fmt.Sprintf("vpc-%s-udp-sess-load", vpcKey),
×
243
                SctpLoadBalancer:     fmt.Sprintf("vpc-%s-sctp-load", vpcKey),
×
244
                SctpSessLoadBalancer: fmt.Sprintf("vpc-%s-sctp-sess-load", vpcKey),
×
245
        }
×
246
}
247

248
func (c *Controller) addLoadBalancer(vpc string) (*VpcLoadBalancer, error) {
×
249
        vpcLbConfig := c.GenVpcLoadBalancer(vpc)
×
250
        if err := c.initLB(vpcLbConfig.TCPLoadBalancer, string(v1.ProtocolTCP), false); err != nil {
×
251
                return nil, err
×
252
        }
×
253
        if err := c.initLB(vpcLbConfig.TCPSessLoadBalancer, string(v1.ProtocolTCP), true); err != nil {
×
254
                return nil, err
×
255
        }
×
256
        if err := c.initLB(vpcLbConfig.UDPLoadBalancer, string(v1.ProtocolUDP), false); err != nil {
×
257
                return nil, err
×
258
        }
×
259
        if err := c.initLB(vpcLbConfig.UDPSessLoadBalancer, string(v1.ProtocolUDP), true); err != nil {
×
260
                return nil, err
×
261
        }
×
262
        if err := c.initLB(vpcLbConfig.SctpLoadBalancer, string(v1.ProtocolSCTP), false); err != nil {
×
263
                return nil, err
×
264
        }
×
265
        if err := c.initLB(vpcLbConfig.SctpSessLoadBalancer, string(v1.ProtocolSCTP), true); err != nil {
×
266
                return nil, err
×
267
        }
×
268

269
        return vpcLbConfig, nil
×
270
}
271

272
func (c *Controller) handleAddOrUpdateVpc(key string) error {
1✔
273
        c.vpcKeyMutex.LockKey(key)
1✔
274
        defer func() { _ = c.vpcKeyMutex.UnlockKey(key) }()
2✔
275
        klog.Infof("handle add/update vpc %s", key)
1✔
276

1✔
277
        cachedVpc, err := c.vpcsLister.Get(key)
1✔
278
        if err != nil {
1✔
279
                if k8serrors.IsNotFound(err) {
×
280
                        return nil
×
281
                }
×
282
                klog.Error(err)
×
283
                return err
×
284
        }
285

286
        vpc, err := c.formatVpc(cachedVpc.DeepCopy())
1✔
287
        if err != nil {
1✔
288
                klog.Errorf("failed to format vpc %s: %v", key, err)
×
289
                return err
×
290
        }
×
291

292
        learnFromARPRequest := vpc.Spec.EnableExternal
1✔
293
        if !learnFromARPRequest {
2✔
294
                for _, subnetName := range vpc.Status.Subnets {
1✔
295
                        subnet, err := c.subnetsLister.Get(subnetName)
×
296
                        if err != nil {
×
297
                                if k8serrors.IsNotFound(err) {
×
298
                                        continue
×
299
                                }
300
                                klog.Errorf("failed to get subnet %s for vpc %s: %v", subnetName, key, err)
×
301
                                return err
×
302
                        }
303
                        if subnet.Spec.Vlan != "" && subnet.Spec.U2OInterconnection {
×
304
                                learnFromARPRequest = true
×
305
                                break
×
306
                        }
307
                }
308
        }
309

310
        if err = c.createVpcRouter(key, learnFromARPRequest); err != nil {
1✔
311
                klog.Errorf("failed to create vpc router for vpc %s: %v", key, err)
×
312
                return err
×
313
        }
×
314

315
        var newPeers []string
1✔
316
        for _, peering := range vpc.Spec.VpcPeerings {
1✔
317
                if err = util.CheckCidrs(peering.LocalConnectIP); err != nil {
×
318
                        klog.Errorf("invalid cidr %s", peering.LocalConnectIP)
×
319
                        return err
×
320
                }
×
321

322
                newPeers = append(newPeers, peering.RemoteVpc)
×
323
                if err := c.OVNNbClient.CreatePeerRouterPort(vpc.Name, peering.RemoteVpc, peering.LocalConnectIP); err != nil {
×
324
                        klog.Errorf("create peer router port for vpc %s, %v", vpc.Name, err)
×
325
                        return err
×
326
                }
×
327
        }
328
        for _, oldPeer := range vpc.Status.VpcPeerings {
1✔
329
                if !slices.Contains(newPeers, oldPeer) {
×
330
                        if err = c.OVNNbClient.DeleteLogicalRouterPort(fmt.Sprintf("%s-%s", vpc.Name, oldPeer)); err != nil {
×
331
                                klog.Errorf("delete peer router port for vpc %s, %v", vpc.Name, err)
×
332
                                return err
×
333
                        }
×
334
                }
335
        }
336

337
        // handle static route
338
        var (
1✔
339
                staticExistedRoutes []*ovnnb.LogicalRouterStaticRoute
1✔
340
                staticTargetRoutes  []*kubeovnv1.StaticRoute
1✔
341
                staticRouteMapping  map[string][]*kubeovnv1.StaticRoute
1✔
342
                externalIDs         = map[string]string{"vendor": util.CniTypeName}
1✔
343
        )
1✔
344

1✔
345
        // only manage static routes which are kube-ovn managed, by filtering for vendor util.CniTypeName
1✔
346
        staticExistedRoutes, err = c.OVNNbClient.ListLogicalRouterStaticRoutes(vpc.Name, nil, nil, "", externalIDs)
1✔
347
        if err != nil {
1✔
348
                klog.Errorf("failed to get vpc %s static route list, %v", vpc.Name, err)
×
349
                return err
×
350
        }
×
351

352
        var externalSubnet *kubeovnv1.Subnet
1✔
353
        externalSubnetExist := false
1✔
354
        externalSubnetGW := ""
1✔
355
        if c.config.EnableEipSnat {
1✔
356
                externalSubnet, err = c.subnetsLister.Get(c.config.ExternalGatewaySwitch)
×
357
                if err != nil {
×
358
                        klog.Warningf("enable-eip-snat need external subnet %s to be exist: %v", c.config.ExternalGatewaySwitch, err)
×
359
                } else {
×
360
                        if !externalSubnet.Spec.LogicalGateway {
×
361
                                // logical gw external subnet can not access external
×
362
                                externalSubnetExist = true
×
363
                                externalSubnetGW = externalSubnet.Spec.Gateway
×
364
                        } else {
×
365
                                klog.Infof("default external subnet %s using logical gw", c.config.ExternalGatewaySwitch)
×
366
                        }
×
367
                }
368
        }
369

370
        staticRouteMapping = c.getRouteTablesByVpc(vpc)
1✔
371
        staticTargetRoutes = vpc.Spec.StaticRoutes
1✔
372
        if vpc.Name == c.config.ClusterRouter {
1✔
373
                if _, ok := staticRouteMapping[util.MainRouteTable]; !ok {
×
374
                        staticRouteMapping[util.MainRouteTable] = nil
×
375
                }
×
376

377
                joinSubnet, err := c.subnetsLister.Get(c.config.NodeSwitch)
×
378
                if err != nil {
×
379
                        if !k8serrors.IsNotFound(err) {
×
380
                                klog.Errorf("failed to get node switch subnet %s: %v", c.config.NodeSwitch, err)
×
381
                                return err
×
382
                        }
×
383
                        c.addOrUpdateVpcQueue.Add(vpc.Name)
×
384
                        return nil
×
385
                }
386
                gatewayV4, gatewayV6 := util.SplitStringIP(joinSubnet.Spec.Gateway)
×
387
                if gatewayV4 != "" {
×
388
                        for table := range staticRouteMapping {
×
389
                                staticTargetRoutes = append(
×
390
                                        staticTargetRoutes,
×
391
                                        &kubeovnv1.StaticRoute{
×
392
                                                Policy:     kubeovnv1.PolicyDst,
×
393
                                                CIDR:       "0.0.0.0/0",
×
394
                                                NextHopIP:  gatewayV4,
×
395
                                                RouteTable: table,
×
396
                                        },
×
397
                                )
×
398
                        }
×
399
                }
400
                if gatewayV6 != "" {
×
401
                        for table := range staticRouteMapping {
×
402
                                staticTargetRoutes = append(
×
403
                                        staticTargetRoutes,
×
404
                                        &kubeovnv1.StaticRoute{
×
405
                                                Policy:     kubeovnv1.PolicyDst,
×
406
                                                CIDR:       "::/0",
×
407
                                                NextHopIP:  gatewayV6,
×
408
                                                RouteTable: table,
×
409
                                        },
×
410
                                )
×
411
                        }
×
412
                }
413
                if c.config.EnableEipSnat {
×
414
                        cm, err := c.configMapsLister.ConfigMaps(c.config.ExternalGatewayConfigNS).Get(util.ExternalGatewayConfig)
×
415
                        if err == nil {
×
416
                                nextHop := cm.Data["external-gw-addr"]
×
417
                                if nextHop == "" {
×
418
                                        if !externalSubnetExist {
×
419
                                                err = fmt.Errorf("failed to get external subnet %s", c.config.ExternalGatewaySwitch)
×
420
                                                klog.Error(err)
×
421
                                                return err
×
422
                                        }
×
423
                                        nextHop = externalSubnet.Spec.Gateway
×
424
                                        if nextHop == "" {
×
425
                                                err := fmt.Errorf("subnet %s has no gateway configuration", externalSubnet.Name)
×
426
                                                klog.Error(err)
×
427
                                                return err
×
428
                                        }
×
429
                                }
430
                                if strings.Contains(nextHop, "/") {
×
431
                                        nextHop = strings.Split(nextHop, "/")[0]
×
432
                                }
×
433

434
                                lr, err := c.OVNNbClient.GetLogicalRouter(vpc.Name, false)
×
435
                                if err != nil {
×
436
                                        klog.Errorf("failed to get logical router %s: %v", vpc.Name, err)
×
437
                                        return err
×
438
                                }
×
439

440
                                for _, nat := range lr.Nat {
×
441
                                        info, err := c.OVNNbClient.GetNATByUUID(nat)
×
442
                                        if err != nil {
×
443
                                                klog.Errorf("failed to get nat ip info for vpc %s, %v", vpc.Name, err)
×
444
                                                return err
×
445
                                        }
×
446
                                        if info.LogicalIP != "" {
×
447
                                                for table := range staticRouteMapping {
×
448
                                                        staticTargetRoutes = append(
×
449
                                                                staticTargetRoutes,
×
450
                                                                &kubeovnv1.StaticRoute{
×
451
                                                                        Policy:     kubeovnv1.PolicySrc,
×
452
                                                                        CIDR:       info.LogicalIP,
×
453
                                                                        NextHopIP:  nextHop,
×
454
                                                                        RouteTable: table,
×
455
                                                                },
×
456
                                                        )
×
457
                                                }
×
458
                                        }
459
                                }
460
                        }
461
                }
462
        } else {
1✔
463
                subnets, err := c.subnetsLister.List(labels.Everything())
1✔
464
                if err != nil {
1✔
465
                        klog.Error(err)
×
466
                        return err
×
467
                }
×
468
                // Add static routes created by addCustomVPCStaticRouteForSubnet
469
                for _, subnet := range subnets {
1✔
470
                        if subnet.Spec.Vpc == key {
×
471
                                v4Gw, v6Gw := util.SplitStringIP(subnet.Spec.Gateway)
×
472
                                v4Cidr, v6Cidr := util.SplitStringIP(subnet.Spec.CIDRBlock)
×
473
                                if v4Gw != "" && v4Cidr != "" {
×
474
                                        staticTargetRoutes = append(staticTargetRoutes, &kubeovnv1.StaticRoute{
×
475
                                                Policy:     kubeovnv1.PolicySrc,
×
476
                                                CIDR:       v4Cidr,
×
477
                                                NextHopIP:  v4Gw,
×
478
                                                RouteTable: subnet.Spec.RouteTable,
×
479
                                        })
×
480
                                }
×
481
                                if v6Gw != "" && v6Cidr != "" {
×
482
                                        staticTargetRoutes = append(staticTargetRoutes, &kubeovnv1.StaticRoute{
×
483
                                                Policy:     kubeovnv1.PolicySrc,
×
484
                                                CIDR:       v6Cidr,
×
485
                                                NextHopIP:  v6Gw,
×
486
                                                RouteTable: subnet.Spec.RouteTable,
×
487
                                        })
×
488
                                }
×
489
                        }
490
                }
491
        }
492

493
        routeNeedDel, routeNeedAdd := diffStaticRoute(staticExistedRoutes, staticTargetRoutes)
1✔
494

1✔
495
        for _, item := range routeNeedDel {
2✔
496
                klog.Infof("vpc %s del static route: %+v", vpc.Name, item)
1✔
497
                policy := convertPolicy(item.Policy)
1✔
498
                if err = c.OVNNbClient.DeleteLogicalRouterStaticRoute(vpc.Name, &item.RouteTable, &policy, item.CIDR, item.NextHopIP); err != nil {
1✔
499
                        klog.Errorf("del vpc %s static route failed, %v", vpc.Name, err)
×
500
                        return err
×
501
                }
×
502
        }
503

504
        for _, item := range routeNeedAdd {
2✔
505
                if item.BfdID != "" {
1✔
506
                        klog.Infof("vpc %s add static ecmp route: %+v", vpc.Name, item)
×
507
                        if err = c.OVNNbClient.AddLogicalRouterStaticRoute(
×
508
                                vpc.Name, item.RouteTable, convertPolicy(item.Policy), item.CIDR, &item.BfdID, externalIDs, item.NextHopIP,
×
509
                        ); err != nil {
×
510
                                klog.Errorf("failed to add bfd static route to vpc %s , %v", vpc.Name, err)
×
511
                                return err
×
512
                        }
×
513
                } else {
1✔
514
                        klog.Infof("vpc %s add static route: %+v", vpc.Name, item)
1✔
515
                        if err = c.OVNNbClient.AddLogicalRouterStaticRoute(
1✔
516
                                vpc.Name, item.RouteTable, convertPolicy(item.Policy), item.CIDR, nil, externalIDs, item.NextHopIP,
1✔
517
                        ); err != nil {
1✔
518
                                klog.Errorf("failed to add normal static route to vpc %s , %v", vpc.Name, err)
×
519
                                return err
×
520
                        }
×
521
                }
522
        }
523

524
        // handle policy route
525
        var (
1✔
526
                policyRouteExisted, policyRouteNeedDel, policyRouteNeedAdd []*kubeovnv1.PolicyRoute
1✔
527
                policyRouteLogical                                         []*ovnnb.LogicalRouterPolicy
1✔
528
        )
1✔
529

1✔
530
        if vpc.Name == c.config.ClusterRouter {
1✔
531
                lastPolicies, _ := c.vpcLastPoliciesMap.Load(vpc.Name)
×
532
                policyRouteExisted = reversePolicies(lastPolicies)
×
533
                // diff list
×
534
                policyRouteNeedDel, policyRouteNeedAdd = diffPolicyRouteWithExisted(policyRouteExisted, vpc.Spec.PolicyRoutes)
×
535
        } else {
1✔
536
                policyRouteLogical, err = c.OVNNbClient.ListLogicalRouterPolicies(vpc.Name, -1, nil, true)
1✔
537
                if err != nil {
1✔
538
                        klog.Errorf("failed to get vpc %s policy route list, %v", vpc.Name, err)
×
539
                        return err
×
540
                }
×
541
                // diff vpc policy route
542
                policyRouteNeedDel, policyRouteNeedAdd = diffPolicyRouteWithLogical(policyRouteLogical, vpc.Spec.PolicyRoutes)
1✔
543
        }
544
        // delete policies non-exist
545
        for _, item := range policyRouteNeedDel {
1✔
546
                klog.Infof("delete policy route for router: %s, priority: %d, match %s", vpc.Name, item.Priority, item.Match)
×
547
                if err = c.OVNNbClient.DeleteLogicalRouterPolicy(vpc.Name, item.Priority, item.Match); err != nil {
×
548
                        klog.Errorf("del vpc %s policy route failed, %v", vpc.Name, err)
×
549
                        return err
×
550
                }
×
551
        }
552
        // add new policies
553
        for _, item := range policyRouteNeedAdd {
1✔
554
                klog.Infof("add policy route for router: %s, match %s, action %s, nexthop %s, externalID %v", c.config.ClusterRouter, item.Match, string(item.Action), item.NextHopIP, externalIDs)
×
555
                if err = c.OVNNbClient.AddLogicalRouterPolicy(vpc.Name, item.Priority, item.Match, string(item.Action), []string{item.NextHopIP}, nil, externalIDs); err != nil {
×
556
                        klog.Errorf("add policy route to vpc %s failed, %v", vpc.Name, err)
×
557
                        return err
×
558
                }
×
559
        }
560

561
        vpcSubnets, defaultSubnet, err := c.getVpcSubnets(vpc)
1✔
562
        if err != nil {
1✔
563
                klog.Error(err)
×
564
                return err
×
565
        }
×
566

567
        vpc.Status.Subnets = vpcSubnets
1✔
568
        vpc.Status.DefaultLogicalSwitch = defaultSubnet
1✔
569
        vpc.Status.Router = key
1✔
570
        vpc.Status.Standby = true
1✔
571
        vpc.Status.VpcPeerings = newPeers
1✔
572
        if c.config.EnableLb {
1✔
573
                vpcLb, err := c.addLoadBalancer(key)
×
574
                if err != nil {
×
575
                        klog.Error(err)
×
576
                        return err
×
577
                }
×
578
                vpc.Status.TCPLoadBalancer = vpcLb.TCPLoadBalancer
×
579
                vpc.Status.TCPSessionLoadBalancer = vpcLb.TCPSessLoadBalancer
×
580
                vpc.Status.UDPLoadBalancer = vpcLb.UDPLoadBalancer
×
581
                vpc.Status.UDPSessionLoadBalancer = vpcLb.UDPSessLoadBalancer
×
582
                vpc.Status.SctpLoadBalancer = vpcLb.SctpLoadBalancer
×
583
                vpc.Status.SctpSessionLoadBalancer = vpcLb.SctpSessLoadBalancer
×
584
        }
585
        bytes, err := vpc.Status.Bytes()
1✔
586
        if err != nil {
1✔
587
                klog.Error(err)
×
588
                return err
×
589
        }
×
590
        vpc, err = c.config.KubeOvnClient.KubeovnV1().Vpcs().Patch(context.Background(), vpc.Name, types.MergePatchType, bytes, metav1.PatchOptions{}, "status")
1✔
591
        if err != nil {
1✔
592
                klog.Error(err)
×
593
                return err
×
594
        }
×
595

596
        if len(vpc.Annotations) != 0 && strings.ToLower(vpc.Annotations[util.VpcLbAnnotation]) == "on" {
1✔
597
                if err = c.createVpcLb(vpc); err != nil {
×
598
                        klog.Error(err)
×
599
                        return err
×
600
                }
×
601
        } else if err = c.deleteVpcLb(vpc); err != nil {
1✔
602
                klog.Error(err)
×
603
                return err
×
604
        }
×
605

606
        subnets, err := c.subnetsLister.List(labels.Everything())
1✔
607
        if err != nil {
1✔
608
                klog.Error(err)
×
609
                return err
×
610
        }
×
611
        custVpcEnableExternalEcmp := false
1✔
612
        for _, subnet := range subnets {
1✔
613
                if subnet.Spec.Vpc == key {
×
614
                        // Accelerate subnet update when vpc config is updated.
×
615
                        // In case VPC not set namespaces, subnet will backoff and may take long time to back to ready.
×
616
                        if subnet.Status.IsNotReady() {
×
617
                                c.addOrUpdateSubnetQueue.Add(subnet.Name)
×
618
                        }
×
619
                        if vpc.Name != util.DefaultVpc && vpc.Spec.EnableBfd && subnet.Spec.EnableEcmp {
×
620
                                custVpcEnableExternalEcmp = true
×
621
                        }
×
622
                }
623
        }
624

625
        if vpc.Spec.EnableExternal || vpc.Status.EnableExternal {
1✔
626
                if err = c.handleUpdateVpcExternal(cachedVpc, custVpcEnableExternalEcmp, externalSubnetExist, externalSubnetGW); err != nil {
×
627
                        klog.Errorf("failed to handle update external subnet for vpc %s, %v", key, err)
×
628
                        return err
×
629
                }
×
630
        }
631

632
        bfdPortName, bfdPortNodes, err := c.reconcileVpcBfdLRP(vpc)
1✔
633
        if err != nil {
1✔
634
                klog.Error(err)
×
635
                return err
×
636
        }
×
637
        if vpc.Spec.BFDPort == nil || !vpc.Spec.BFDPort.Enabled {
2✔
638
                vpc.Status.BFDPort = kubeovnv1.BFDPortStatus{}
1✔
639
        } else {
1✔
640
                vpc.Status.BFDPort = kubeovnv1.BFDPortStatus{
×
641
                        Name:  bfdPortName,
×
642
                        IP:    vpc.Spec.BFDPort.IP,
×
643
                        Nodes: bfdPortNodes,
×
644
                }
×
645
        }
×
646
        if _, err = c.config.KubeOvnClient.KubeovnV1().Vpcs().
1✔
647
                UpdateStatus(context.Background(), vpc, metav1.UpdateOptions{}); err != nil {
1✔
648
                klog.Error(err)
×
649
                return err
×
650
        }
×
651

652
        return nil
1✔
653
}
654

655
func (c *Controller) handleUpdateVpcExternal(vpc *kubeovnv1.Vpc, custVpcEnableExternalEcmp, defaultExternalSubnetExist bool, externalSubnetGW string) error {
×
656
        if c.config.EnableEipSnat && vpc.Name == util.DefaultVpc {
×
657
                klog.Infof("external_gw handle ovn default external gw %s", vpc.Name)
×
658
                return nil
×
659
        }
×
660

661
        if !vpc.Spec.EnableExternal && !vpc.Status.EnableExternal {
×
662
                // no need to handle external connection
×
663
                return nil
×
664
        }
×
665

666
        // handle any vpc external
667
        if vpc.Spec.EnableExternal && !defaultExternalSubnetExist && vpc.Spec.ExtraExternalSubnets == nil {
×
668
                // at least have a external subnet
×
669
                err := fmt.Errorf("failed to get external subnet for enable external vpc %s", vpc.Name)
×
670
                klog.Error(err)
×
671
                return err
×
672
        }
×
673

674
        if !vpc.Spec.EnableExternal && vpc.Status.EnableExternal {
×
675
                // just del all external subnets connection
×
676
                klog.Infof("disconnect default external subnet %s to vpc %s", c.config.ExternalGatewaySwitch, vpc.Name)
×
677
                if err := c.handleDelVpcExternalSubnet(vpc.Name, c.config.ExternalGatewaySwitch); err != nil {
×
678
                        klog.Errorf("failed to delete external subnet %s connection for vpc %s, error %v", c.config.ExternalGatewaySwitch, vpc.Name, err)
×
679
                        return err
×
680
                }
×
681
                for _, subnet := range vpc.Status.ExtraExternalSubnets {
×
682
                        klog.Infof("disconnect external subnet %s to vpc %s", subnet, vpc.Name)
×
683
                        if err := c.handleDelVpcExternalSubnet(vpc.Name, subnet); err != nil {
×
684
                                klog.Errorf("failed to delete external subnet %s connection for vpc %s, error %v", subnet, vpc.Name, err)
×
685
                                return err
×
686
                        }
×
687
                }
688
        }
689

690
        if vpc.Spec.EnableExternal {
×
691
                if !vpc.Status.EnableExternal {
×
692
                        // just add external connection
×
693
                        if vpc.Spec.ExtraExternalSubnets == nil && defaultExternalSubnetExist {
×
694
                                // only connect default external subnet
×
695
                                klog.Infof("connect default external subnet %s with vpc %s", c.config.ExternalGatewaySwitch, vpc.Name)
×
696
                                if err := c.handleAddVpcExternalSubnet(vpc.Name, c.config.ExternalGatewaySwitch); err != nil {
×
697
                                        klog.Errorf("failed to add external subnet %s connection for vpc %s, error %v", c.config.ExternalGatewaySwitch, vpc.Name, err)
×
698
                                        return err
×
699
                                }
×
700
                        }
701

702
                        // only connect provider network vlan external subnet
703
                        for _, subnet := range vpc.Spec.ExtraExternalSubnets {
×
704
                                klog.Infof("connect external subnet %s with vpc %s", subnet, vpc.Name)
×
705
                                if err := c.handleAddVpcExternalSubnet(vpc.Name, subnet); err != nil {
×
706
                                        klog.Errorf("failed to add external subnet %s connection for vpc %s, error %v", subnet, vpc.Name, err)
×
707
                                        return err
×
708
                                }
×
709
                        }
710
                }
711

712
                // diff to add
713
                for _, subnet := range vpc.Spec.ExtraExternalSubnets {
×
714
                        if !slices.Contains(vpc.Status.ExtraExternalSubnets, subnet) {
×
715
                                klog.Infof("connect external subnet %s with vpc %s", subnet, vpc.Name)
×
716
                                if err := c.handleAddVpcExternalSubnet(vpc.Name, subnet); err != nil {
×
717
                                        klog.Errorf("failed to add external subnet %s connection for vpc %s, error %v", subnet, vpc.Name, err)
×
718
                                        return err
×
719
                                }
×
720
                        }
721
                }
722

723
                // diff to del
724
                for _, subnet := range vpc.Status.ExtraExternalSubnets {
×
725
                        if !slices.Contains(vpc.Spec.ExtraExternalSubnets, subnet) {
×
726
                                klog.Infof("disconnect external subnet %s to vpc %s", subnet, vpc.Name)
×
727
                                if err := c.handleDelVpcExternalSubnet(vpc.Name, subnet); err != nil {
×
728
                                        klog.Errorf("failed to delete external subnet %s connection for vpc %s, error %v", subnet, vpc.Name, err)
×
729
                                        return err
×
730
                                }
×
731
                        }
732
                }
733
        }
734

735
        // custom vpc enable bfd
736
        if vpc.Spec.EnableBfd && vpc.Name != util.DefaultVpc && defaultExternalSubnetExist {
×
737
                // create bfd between lrp and physical switch gw
×
738
                // bfd status down means current lrp binding chassis node external nic lost external network connectivity
×
739
                // should switch lrp to another node
×
740
                lrpEipName := fmt.Sprintf("%s-%s", vpc.Name, c.config.ExternalGatewaySwitch)
×
741
                v4ExtGw, _ := util.SplitStringIP(externalSubnetGW)
×
742
                // TODO: dualstack
×
743
                if _, err := c.OVNNbClient.CreateBFD(lrpEipName, v4ExtGw, c.config.BfdMinRx, c.config.BfdMinTx, c.config.BfdDetectMult, nil); err != nil {
×
744
                        klog.Error(err)
×
745
                        return err
×
746
                }
×
747
                // TODO: support multi external nic
748
                if custVpcEnableExternalEcmp {
×
749
                        klog.Infof("remove normal static ecmp route for vpc %s", vpc.Name)
×
750
                        // auto remove normal type static route, if using ecmp based bfd
×
751
                        if err := c.reconcileCustomVpcDelNormalStaticRoute(vpc.Name); err != nil {
×
752
                                klog.Errorf("failed to reconcile del vpc %q normal static route", vpc.Name)
×
753
                                return err
×
754
                        }
×
755
                }
756
        }
757

758
        if !vpc.Spec.EnableBfd && vpc.Status.EnableBfd {
×
759
                lrpEipName := fmt.Sprintf("%s-%s", vpc.Name, c.config.ExternalGatewaySwitch)
×
760
                if err := c.OVNNbClient.DeleteBFDByDstIP(lrpEipName, ""); err != nil {
×
761
                        klog.Error(err)
×
762
                        return err
×
763
                }
×
764
                if err := c.handleDeleteVpcStaticRoute(vpc.Name); err != nil {
×
765
                        klog.Errorf("failed to delete bfd route for vpc %s, error %v", vpc.Name, err)
×
766
                        return err
×
767
                }
×
768
        }
769

770
        if err := c.updateVpcExternalStatus(vpc.Name); err != nil {
×
771
                klog.Errorf("failed to update vpc external subnets status, %v", err)
×
772
                return err
×
773
        }
×
774
        return nil
×
775
}
776

777
func (c *Controller) reconcileVpcBfdLRP(vpc *kubeovnv1.Vpc) (string, []string, error) {
1✔
778
        portName := "bfd@" + vpc.Name
1✔
779
        if vpc.Spec.BFDPort == nil || !vpc.Spec.BFDPort.Enabled {
2✔
780
                if err := c.OVNNbClient.DeleteLogicalRouterPort(portName); err != nil {
1✔
781
                        err = fmt.Errorf("failed to delete BFD LRP %s: %w", portName, err)
×
782
                        klog.Error(err)
×
783
                        return portName, nil, err
×
784
                }
×
785
                if err := c.OVNNbClient.DeleteHAChassisGroup(portName); err != nil {
1✔
786
                        err = fmt.Errorf("failed to delete HA chassis group %s: %w", portName, err)
×
787
                        klog.Error(err)
×
788
                        return portName, nil, err
×
789
                }
×
790
                return portName, nil, nil
1✔
791
        }
792

793
        var err error
×
794
        chassisCount := 3
×
795
        selector := labels.Everything()
×
796
        if vpc.Spec.BFDPort.NodeSelector != nil {
×
797
                chassisCount = math.MaxInt
×
798
                if selector, err = metav1.LabelSelectorAsSelector(vpc.Spec.BFDPort.NodeSelector); err != nil {
×
799
                        err = fmt.Errorf("failed to parse node selector %q: %w", vpc.Spec.BFDPort.NodeSelector.String(), err)
×
800
                        klog.Error(err)
×
801
                        return portName, nil, err
×
802
                }
×
803
        }
804

805
        nodes, err := c.nodesLister.List(selector)
×
806
        if err != nil {
×
807
                err = fmt.Errorf("failed to list nodes with selector %q: %w", vpc.Spec.BFDPort.NodeSelector, err)
×
808
                klog.Error(err)
×
809
                return portName, nil, err
×
810
        }
×
811
        if len(nodes) == 0 {
×
812
                err = fmt.Errorf("no nodes found by selector %q", selector.String())
×
813
                klog.Error(err)
×
814
                return portName, nil, err
×
815
        }
×
816

817
        nodeNames := make([]string, 0, len(nodes))
×
818
        chassisCount = min(chassisCount, len(nodes))
×
819
        chassisNames := make([]string, 0, chassisCount)
×
820
        for _, node := range nodes[:chassisCount] {
×
821
                chassis, err := c.OVNSbClient.GetChassisByHost(node.Name)
×
822
                if err != nil {
×
823
                        err = fmt.Errorf("failed to get chassis of node %s: %w", node.Name, err)
×
824
                        klog.Error(err)
×
825
                        return portName, nil, err
×
826
                }
×
827
                chassisNames = append(chassisNames, chassis.Name)
×
828
                nodeNames = append(nodeNames, node.Name)
×
829
        }
830

831
        networks := strings.Split(vpc.Spec.BFDPort.IP, ",")
×
832
        if err = c.OVNNbClient.CreateLogicalRouterPort(vpc.Name, portName, "", networks); err != nil {
×
833
                klog.Error(err)
×
834
                return portName, nil, err
×
835
        }
×
836
        if err = c.OVNNbClient.UpdateLogicalRouterPortNetworks(portName, networks); err != nil {
×
837
                klog.Error(err)
×
838
                return portName, nil, err
×
839
        }
×
840
        if err = c.OVNNbClient.UpdateLogicalRouterPortOptions(portName, map[string]string{"bfd-only": "true"}); err != nil {
×
841
                klog.Error(err)
×
842
                return portName, nil, err
×
843
        }
×
844
        if err = c.OVNNbClient.CreateHAChassisGroup(portName, chassisNames, map[string]string{"lrp": portName}); err != nil {
×
845
                klog.Error(err)
×
846
                return portName, nil, err
×
847
        }
×
848
        if err = c.OVNNbClient.SetLogicalRouterPortHAChassisGroup(portName, portName); err != nil {
×
849
                klog.Error(err)
×
850
                return portName, nil, err
×
851
        }
×
852

853
        return portName, nodeNames, nil
×
854
}
855

856
func (c *Controller) addPolicyRouteToVpc(vpcName string, policy *kubeovnv1.PolicyRoute, externalIDs map[string]string) error {
×
857
        var (
×
858
                nextHops []string
×
859
                err      error
×
860
        )
×
861

×
862
        if policy.NextHopIP != "" {
×
863
                nextHops = strings.Split(policy.NextHopIP, ",")
×
864
        }
×
865

866
        if err = c.OVNNbClient.AddLogicalRouterPolicy(vpcName, policy.Priority, policy.Match, string(policy.Action), nextHops, nil, externalIDs); err != nil {
×
867
                klog.Errorf("add policy route to vpc %s failed, %v", vpcName, err)
×
868
                return err
×
869
        }
×
870
        return nil
×
871
}
872

873
func buildExternalIDsMapKey(match, action string, priority int) string {
×
874
        return fmt.Sprintf("%s-%s-%d", match, action, priority)
×
875
}
×
876

877
func (c *Controller) batchAddPolicyRouteToVpc(name string, policies []*kubeovnv1.PolicyRoute, externalIDs map[string]map[string]string) error {
×
878
        if len(policies) == 0 {
×
879
                return nil
×
880
        }
×
881
        start := time.Now()
×
882
        routerPolicies := make([]*ovnnb.LogicalRouterPolicy, 0, len(policies))
×
883
        for _, policy := range policies {
×
884
                var nextHops []string
×
885
                if policy.NextHopIP != "" {
×
886
                        nextHops = strings.Split(policy.NextHopIP, ",")
×
887
                }
×
888
                routerPolicies = append(routerPolicies, &ovnnb.LogicalRouterPolicy{
×
889
                        Priority:    policy.Priority,
×
890
                        Nexthops:    nextHops,
×
891
                        Action:      string(policy.Action),
×
892
                        Match:       policy.Match,
×
893
                        ExternalIDs: externalIDs[buildExternalIDsMapKey(policy.Match, string(policy.Action), policy.Priority)],
×
894
                })
×
895
        }
896

897
        if err := c.OVNNbClient.BatchAddLogicalRouterPolicy(name, routerPolicies...); err != nil {
×
898
                klog.Errorf("batch add policy route to vpc %s failed, %v", name, err)
×
899
                return err
×
900
        }
×
901
        klog.Infof("take to %v batch add policy route to vpc %s policies %d", time.Since(start), name, len(policies))
×
902
        return nil
×
903
}
904

905
func (c *Controller) deletePolicyRouteFromVpc(vpcName string, priority int, match string) error {
×
906
        if err := c.OVNNbClient.DeleteLogicalRouterPolicy(vpcName, priority, match); err != nil {
×
907
                klog.Error(err)
×
908
                return err
×
909
        }
×
910
        return nil
×
911
}
912

913
func (c *Controller) batchDeletePolicyRouteFromVpc(name string, policies []*kubeovnv1.PolicyRoute) error {
×
914
        start := time.Now()
×
915
        routerPolicies := make([]*ovnnb.LogicalRouterPolicy, 0, len(policies))
×
916
        for _, policy := range policies {
×
917
                routerPolicies = append(routerPolicies, &ovnnb.LogicalRouterPolicy{
×
918
                        Priority: policy.Priority,
×
919
                        Match:    policy.Match,
×
920
                })
×
921
        }
×
922

923
        if err := c.OVNNbClient.BatchDeleteLogicalRouterPolicy(name, routerPolicies); err != nil {
×
924
                return err
×
925
        }
×
926
        klog.V(3).Infof("take to %v batch delete policy route from vpc %s policies %d", time.Since(start), name, len(policies))
×
927
        return nil
×
928
}
929

930
func (c *Controller) addStaticRouteToVpc(name string, route *kubeovnv1.StaticRoute) error {
×
931
        externalIDs := map[string]string{"vendor": util.CniTypeName}
×
932
        if route.BfdID != "" {
×
933
                klog.Infof("vpc %s add static ecmp route: %+v", name, route)
×
934
                if err := c.OVNNbClient.AddLogicalRouterStaticRoute(
×
935
                        name, route.RouteTable, convertPolicy(route.Policy), route.CIDR, &route.BfdID, externalIDs, route.NextHopIP,
×
936
                ); err != nil {
×
937
                        klog.Errorf("failed to add bfd static route to vpc %s , %v", name, err)
×
938
                        return err
×
939
                }
×
940
        } else {
×
941
                klog.Infof("vpc %s add static route: %+v", name, route)
×
942
                if err := c.OVNNbClient.AddLogicalRouterStaticRoute(
×
943
                        name, route.RouteTable, convertPolicy(route.Policy), route.CIDR, nil, externalIDs, route.NextHopIP,
×
944
                ); err != nil {
×
945
                        klog.Errorf("failed to add normal static route to vpc %s , %v", name, err)
×
946
                        return err
×
947
                }
×
948
        }
949
        return nil
×
950
}
951

952
func (c *Controller) deleteStaticRouteFromVpc(name, table, cidr, nextHop string, policy kubeovnv1.RoutePolicy) error {
×
953
        var (
×
954
                policyStr string
×
955
                err       error
×
956
        )
×
957

×
958
        policyStr = convertPolicy(policy)
×
959
        if err = c.OVNNbClient.DeleteLogicalRouterStaticRoute(name, &table, &policyStr, cidr, nextHop); err != nil {
×
960
                klog.Errorf("del vpc %s static route failed, %v", name, err)
×
961
                return err
×
962
        }
×
963

964
        return nil
×
965
}
966

967
func (c *Controller) batchDeleteStaticRouteFromVpc(name string, staticRoutes []*kubeovnv1.StaticRoute) error {
×
968
        start := time.Now()
×
969
        routeCount := len(staticRoutes)
×
970
        delRoutes := make([]*ovnnb.LogicalRouterStaticRoute, 0, routeCount)
×
971
        for _, sr := range staticRoutes {
×
972
                policyStr := convertPolicy(sr.Policy)
×
973
                newRoute := &ovnnb.LogicalRouterStaticRoute{
×
974
                        RouteTable: sr.RouteTable,
×
975
                        Nexthop:    sr.NextHopIP,
×
976
                        Policy:     &policyStr,
×
977
                        IPPrefix:   sr.CIDR,
×
978
                }
×
979
                delRoutes = append(delRoutes, newRoute)
×
980
        }
×
981
        if err := c.OVNNbClient.BatchDeleteLogicalRouterStaticRoute(name, delRoutes); err != nil {
×
982
                klog.Errorf("batch del vpc %s static route %d failed, %v", name, routeCount, err)
×
983
                return err
×
984
        }
×
985
        klog.V(3).Infof("take to %v batch delete static route from vpc %s static routes %d", time.Since(start), name, len(delRoutes))
×
986
        return nil
×
987
}
988

989
func diffPolicyRouteWithExisted(exists, target []*kubeovnv1.PolicyRoute) ([]*kubeovnv1.PolicyRoute, []*kubeovnv1.PolicyRoute) {
×
990
        var (
×
991
                dels, adds []*kubeovnv1.PolicyRoute
×
992
                existsMap  map[string]*kubeovnv1.PolicyRoute
×
993
                key        string
×
994
                ok         bool
×
995
        )
×
996

×
997
        existsMap = make(map[string]*kubeovnv1.PolicyRoute, len(exists))
×
998
        for _, item := range exists {
×
999
                existsMap[getPolicyRouteItemKey(item)] = item
×
1000
        }
×
1001
        // load policies to add
1002
        for _, item := range target {
×
1003
                key = getPolicyRouteItemKey(item)
×
1004

×
1005
                if _, ok = existsMap[key]; ok {
×
1006
                        delete(existsMap, key)
×
1007
                } else {
×
1008
                        adds = append(adds, item)
×
1009
                }
×
1010
        }
1011
        // load policies to delete
1012
        for _, item := range existsMap {
×
1013
                dels = append(dels, item)
×
1014
        }
×
1015
        return dels, adds
×
1016
}
1017

1018
func diffPolicyRouteWithLogical(exists []*ovnnb.LogicalRouterPolicy, target []*kubeovnv1.PolicyRoute) ([]*kubeovnv1.PolicyRoute, []*kubeovnv1.PolicyRoute) {
1✔
1019
        var (
1✔
1020
                dels, adds []*kubeovnv1.PolicyRoute
1✔
1021
                existsMap  map[string]*kubeovnv1.PolicyRoute
1✔
1022
                key        string
1✔
1023
                ok         bool
1✔
1024
        )
1✔
1025
        existsMap = make(map[string]*kubeovnv1.PolicyRoute, len(exists))
1✔
1026

1✔
1027
        for _, item := range exists {
1✔
1028
                if item.ExternalIDs["vpc-egress-gateway"] != "" || item.ExternalIDs["subnet"] != "" ||
×
1029
                        item.ExternalIDs["isU2ORoutePolicy"] == "true" {
×
1030
                        continue
×
1031
                }
1032
                policy := &kubeovnv1.PolicyRoute{
×
1033
                        Priority: item.Priority,
×
1034
                        Match:    item.Match,
×
1035
                        Action:   kubeovnv1.PolicyRouteAction(item.Action),
×
1036
                }
×
1037
                existsMap[getPolicyRouteItemKey(policy)] = policy
×
1038
        }
1039

1040
        for _, item := range target {
1✔
1041
                key = getPolicyRouteItemKey(item)
×
1042

×
1043
                if _, ok = existsMap[key]; ok {
×
1044
                        delete(existsMap, key)
×
1045
                } else {
×
1046
                        adds = append(adds, item)
×
1047
                }
×
1048
        }
1049

1050
        for _, item := range existsMap {
1✔
1051
                dels = append(dels, item)
×
1052
        }
×
1053
        return dels, adds
1✔
1054
}
1055

1056
func getPolicyRouteItemKey(item *kubeovnv1.PolicyRoute) (key string) {
×
1057
        return fmt.Sprintf("%d:%s:%s:%s", item.Priority, item.Match, item.Action, item.NextHopIP)
×
1058
}
×
1059

1060
func diffStaticRoute(exist []*ovnnb.LogicalRouterStaticRoute, target []*kubeovnv1.StaticRoute) (routeNeedDel, routeNeedAdd []*kubeovnv1.StaticRoute) {
1✔
1061
        existRouteMap := make(map[string]*kubeovnv1.StaticRoute, len(exist))
1✔
1062
        for _, item := range exist {
2✔
1063
                policy := kubeovnv1.PolicyDst
1✔
1064
                if item.Policy != nil && *item.Policy == ovnnb.LogicalRouterStaticRoutePolicySrcIP {
2✔
1065
                        policy = kubeovnv1.PolicySrc
1✔
1066
                }
1✔
1067
                route := &kubeovnv1.StaticRoute{
1✔
1068
                        Policy:     policy,
1✔
1069
                        CIDR:       item.IPPrefix,
1✔
1070
                        NextHopIP:  item.Nexthop,
1✔
1071
                        RouteTable: item.RouteTable,
1✔
1072
                        ECMPMode:   util.StaticRouteBfdEcmp,
1✔
1073
                }
1✔
1074
                if item.BFD != nil {
1✔
1075
                        route.BfdID = *item.BFD
×
1076
                }
×
1077
                existRouteMap[getStaticRouteItemKey(route)] = route
1✔
1078
        }
1079

1080
        for _, item := range target {
2✔
1081
                key := getStaticRouteItemKey(item)
1✔
1082
                if _, ok := existRouteMap[key]; ok {
2✔
1083
                        delete(existRouteMap, key)
1✔
1084
                } else {
2✔
1085
                        routeNeedAdd = append(routeNeedAdd, item)
1✔
1086
                }
1✔
1087
        }
1088
        for _, item := range existRouteMap {
2✔
1089
                routeNeedDel = append(routeNeedDel, item)
1✔
1090
        }
1✔
1091
        return routeNeedDel, routeNeedAdd
1✔
1092
}
1093

1094
func getStaticRouteItemKey(item *kubeovnv1.StaticRoute) string {
1✔
1095
        var key string
1✔
1096
        if item.Policy == kubeovnv1.PolicyDst {
2✔
1097
                key = fmt.Sprintf("%s:dst:%s=>%s", item.RouteTable, item.CIDR, item.NextHopIP)
1✔
1098
        } else {
2✔
1099
                key = fmt.Sprintf("%s:src:%s=>%s", item.RouteTable, item.CIDR, item.NextHopIP)
1✔
1100
        }
1✔
1101
        return key
1✔
1102
}
1103

1104
func (c *Controller) formatVpc(vpc *kubeovnv1.Vpc) (*kubeovnv1.Vpc, error) {
1✔
1105
        var changed bool
1✔
1106
        for _, item := range vpc.Spec.StaticRoutes {
2✔
1107
                // check policy
1✔
1108
                if item.Policy == "" {
1✔
1109
                        item.Policy = kubeovnv1.PolicyDst
×
1110
                        changed = true
×
1111
                }
×
1112
                if item.Policy != kubeovnv1.PolicyDst && item.Policy != kubeovnv1.PolicySrc {
1✔
1113
                        return nil, fmt.Errorf("unknown policy type: %q", item.Policy)
×
1114
                }
×
1115
                // check cidr
1116
                if strings.Contains(item.CIDR, "/") {
2✔
1117
                        if _, _, err := net.ParseCIDR(item.CIDR); err != nil {
1✔
1118
                                return nil, fmt.Errorf("invalid cidr %q: %w", item.CIDR, err)
×
1119
                        }
×
1120
                } else if ip := net.ParseIP(item.CIDR); ip == nil {
×
1121
                        return nil, fmt.Errorf("invalid ip %q", item.CIDR)
×
1122
                }
×
1123
                // check next hop ip
1124
                if ip := net.ParseIP(item.NextHopIP); ip == nil {
1✔
1125
                        return nil, fmt.Errorf("invalid next hop ip %q", item.NextHopIP)
×
1126
                }
×
1127
        }
1128

1129
        for _, route := range vpc.Spec.PolicyRoutes {
1✔
1130
                if route.Action != kubeovnv1.PolicyRouteActionReroute {
×
1131
                        if route.NextHopIP != "" {
×
1132
                                route.NextHopIP = ""
×
1133
                                changed = true
×
1134
                        }
×
1135
                } else {
×
1136
                        // ecmp policy route may reroute to multiple next hop ips
×
1137
                        for ipStr := range strings.SplitSeq(route.NextHopIP, ",") {
×
1138
                                if ip := net.ParseIP(ipStr); ip == nil {
×
1139
                                        err := fmt.Errorf("invalid next hop ips: %s", route.NextHopIP)
×
1140
                                        klog.Error(err)
×
1141
                                        return nil, err
×
1142
                                }
×
1143
                        }
1144
                }
1145
        }
1146

1147
        if vpc.DeletionTimestamp.IsZero() && !slices.Contains(vpc.GetFinalizers(), util.KubeOVNControllerFinalizer) {
2✔
1148
                controllerutil.AddFinalizer(vpc, util.KubeOVNControllerFinalizer)
1✔
1149
                changed = true
1✔
1150
        }
1✔
1151

1152
        if !vpc.DeletionTimestamp.IsZero() && len(vpc.Status.Subnets) == 0 {
1✔
1153
                controllerutil.RemoveFinalizer(vpc, util.KubeOVNControllerFinalizer)
×
1154
                changed = true
×
1155
        }
×
1156

1157
        if changed {
2✔
1158
                newVpc, err := c.config.KubeOvnClient.KubeovnV1().Vpcs().Update(context.Background(), vpc, metav1.UpdateOptions{})
1✔
1159
                if err != nil {
1✔
1160
                        klog.Errorf("failed to update vpc %s: %v", vpc.Name, err)
×
1161
                        return nil, err
×
1162
                }
×
1163
                return newVpc, nil
1✔
1164
        }
1165

1166
        return vpc, nil
×
1167
}
1168

1169
func convertPolicies(list []*kubeovnv1.PolicyRoute) string {
×
1170
        if list == nil {
×
1171
                return ""
×
1172
        }
×
1173

1174
        var (
×
1175
                res []byte
×
1176
                err error
×
1177
        )
×
1178

×
1179
        if res, err = json.Marshal(list); err != nil {
×
1180
                klog.Errorf("failed to serialize policy routes %v , reason : %v", list, err)
×
1181
                return ""
×
1182
        }
×
1183
        return string(res)
×
1184
}
1185

1186
func reversePolicies(origin string) []*kubeovnv1.PolicyRoute {
×
1187
        if origin == "" {
×
1188
                return nil
×
1189
        }
×
1190

1191
        var (
×
1192
                list []*kubeovnv1.PolicyRoute
×
1193
                err  error
×
1194
        )
×
1195

×
1196
        if err = json.Unmarshal([]byte(origin), &list); err != nil {
×
1197
                klog.Errorf("failed to deserialize policy routes %v , reason : %v", list, err)
×
1198
                return nil
×
1199
        }
×
1200
        return list
×
1201
}
1202

1203
func convertPolicy(origin kubeovnv1.RoutePolicy) string {
1✔
1204
        if origin == kubeovnv1.PolicyDst {
2✔
1205
                return ovnnb.LogicalRouterStaticRoutePolicyDstIP
1✔
1206
        }
1✔
1207
        return ovnnb.LogicalRouterStaticRoutePolicySrcIP
1✔
1208
}
1209

1210
func reversePolicy(origin ovnnb.LogicalRouterStaticRoutePolicy) kubeovnv1.RoutePolicy {
×
1211
        if origin == ovnnb.LogicalRouterStaticRoutePolicyDstIP {
×
1212
                return kubeovnv1.PolicyDst
×
1213
        }
×
1214
        return kubeovnv1.PolicySrc
×
1215
}
1216

1217
func (c *Controller) getVpcSubnets(vpc *kubeovnv1.Vpc) (subnets []string, defaultSubnet string, err error) {
1✔
1218
        subnets = []string{}
1✔
1219
        allSubnets, err := c.subnetsLister.List(labels.Everything())
1✔
1220
        if err != nil {
1✔
1221
                klog.Error(err)
×
1222
                return nil, "", err
×
1223
        }
×
1224

1225
        for _, subnet := range allSubnets {
1✔
1226
                if subnet.Spec.Vpc != vpc.Name || !subnet.DeletionTimestamp.IsZero() || !isOvnSubnet(subnet) {
×
1227
                        continue
×
1228
                }
1229

1230
                subnets = append(subnets, subnet.Name)
×
1231
                if subnet.Spec.Default {
×
1232
                        defaultSubnet = subnet.Name
×
1233
                }
×
1234

1235
                if vpc.Name != util.DefaultVpc && vpc.Spec.DefaultSubnet != "" && vpc.Spec.DefaultSubnet == subnet.Name {
×
1236
                        defaultSubnet = vpc.Spec.DefaultSubnet
×
1237
                }
×
1238
        }
1239
        sort.Strings(subnets)
1✔
1240
        return subnets, defaultSubnet, err
1✔
1241
}
1242

1243
// createVpcRouter create router to connect logical switches in vpc
1244
func (c *Controller) createVpcRouter(lr string, learnFromARPRequest bool) error {
1✔
1245
        if err := c.OVNNbClient.CreateLogicalRouter(lr); err != nil {
1✔
1246
                klog.Errorf("create logical router %s failed: %v", lr, err)
×
1247
                return err
×
1248
        }
×
1249

1250
        vpcRouter, err := c.OVNNbClient.GetLogicalRouter(lr, false)
1✔
1251
        if err != nil {
1✔
1252
                klog.Errorf("get logical router %s failed: %v", lr, err)
×
1253
                return err
×
1254
        }
×
1255

1256
        lrOptions := map[string]string{
1✔
1257
                "mac_binding_age_threshold": "300",
1✔
1258
                "dynamic_neigh_routers":     "true",
1✔
1259
        }
1✔
1260
        if !learnFromARPRequest {
2✔
1261
                lrOptions["always_learn_from_arp_request"] = "false"
1✔
1262
        }
1✔
1263
        if !maps.Equal(vpcRouter.Options, lrOptions) {
2✔
1264
                vpcRouter.Options = lrOptions
1✔
1265
                if err = c.OVNNbClient.UpdateLogicalRouter(vpcRouter, &vpcRouter.Options); err != nil {
1✔
1266
                        klog.Errorf("failed to update options of logical router %s: %v", lr, err)
×
1267
                        return err
×
1268
                }
×
1269
        }
1270

1271
        return nil
1✔
1272
}
1273

1274
// deleteVpcRouter delete router to connect logical switches in vpc
1275
func (c *Controller) deleteVpcRouter(lr string) error {
×
1276
        return c.OVNNbClient.DeleteLogicalRouter(lr)
×
1277
}
×
1278

1279
func (c *Controller) handleAddVpcExternalSubnet(key, subnet string) error {
×
1280
        cachedSubnet, err := c.subnetsLister.Get(subnet)
×
1281
        if err != nil {
×
1282
                klog.Error(err)
×
1283
                return err
×
1284
        }
×
1285
        lrpEipName := fmt.Sprintf("%s-%s", key, subnet)
×
1286
        cachedEip, err := c.ovnEipsLister.Get(lrpEipName)
×
1287
        var needCreateEip bool
×
1288
        if err != nil {
×
1289
                if !k8serrors.IsNotFound(err) {
×
1290
                        klog.Error(err)
×
1291
                        return err
×
1292
                }
×
1293
                needCreateEip = true
×
1294
        }
1295
        var v4ip, v6ip, mac string
×
1296
        klog.V(3).Infof("create vpc lrp eip %s", lrpEipName)
×
1297
        if needCreateEip {
×
1298
                if v4ip, v6ip, mac, err = c.acquireIPAddress(subnet, lrpEipName, lrpEipName); err != nil {
×
1299
                        klog.Errorf("failed to acquire ip address for lrp eip %s, %v", lrpEipName, err)
×
1300
                        return err
×
1301
                }
×
1302
                if err := c.createOrUpdateOvnEipCR(lrpEipName, subnet, v4ip, v6ip, mac, util.OvnEipTypeLRP); err != nil {
×
1303
                        klog.Errorf("failed to create ovn eip for lrp %s: %v", lrpEipName, err)
×
1304
                        return err
×
1305
                }
×
1306
        } else {
×
1307
                v4ip = cachedEip.Spec.V4Ip
×
1308
                mac = cachedEip.Spec.MacAddress
×
1309
        }
×
1310
        if v4ip == "" || mac == "" {
×
1311
                err := fmt.Errorf("lrp '%s' ip or mac should not be empty", lrpEipName)
×
1312
                klog.Error(err)
×
1313
                return err
×
1314
        }
×
1315
        // init lrp gw chassis group
1316
        chassises := []string{}
×
1317
        gwNodes, err := c.nodesLister.List(externalGatewayNodeSelector)
×
1318
        if err != nil {
×
1319
                klog.Errorf("failed to list external gw nodes, %v", err)
×
1320
                return err
×
1321
        }
×
1322
        for _, gwNode := range gwNodes {
×
1323
                annoChassisName := gwNode.Annotations[util.ChassisAnnotation]
×
1324
                if annoChassisName == "" {
×
1325
                        err := fmt.Errorf("node %s has no chassis annotation, kube-ovn-cni not ready", gwNode.Name)
×
1326
                        klog.Error(err)
×
1327
                        return err
×
1328
                }
×
1329
                klog.Infof("get node %s chassis: %s", gwNode.Name, annoChassisName)
×
1330
                chassis, err := c.OVNSbClient.GetChassis(annoChassisName, false)
×
1331
                if err != nil {
×
1332
                        klog.Errorf("failed to get node %s chassis: %s, %v", gwNode.Name, annoChassisName, err)
×
1333
                        return err
×
1334
                }
×
1335
                chassises = append(chassises, chassis.Name)
×
1336
        }
1337

1338
        if len(chassises) == 0 {
×
1339
                err := errors.New("no external gw nodes")
×
1340
                klog.Error(err)
×
1341
                return err
×
1342
        }
×
1343

1344
        v4ipCidr, err := util.GetIPAddrWithMask(v4ip, cachedSubnet.Spec.CIDRBlock)
×
1345
        if err != nil {
×
1346
                klog.Error(err)
×
1347
                return err
×
1348
        }
×
1349
        lspName := fmt.Sprintf("%s-%s", subnet, key)
×
1350
        lrpName := fmt.Sprintf("%s-%s", key, subnet)
×
1351

×
1352
        if err := c.OVNNbClient.CreateLogicalPatchPort(subnet, key, lspName, lrpName, v4ipCidr, mac, chassises...); err != nil {
×
1353
                klog.Errorf("failed to connect router '%s' to external: %v", key, err)
×
1354
                return err
×
1355
        }
×
1356
        return nil
×
1357
}
1358

1359
func (c *Controller) handleDeleteVpcStaticRoute(key string) error {
×
1360
        vpc, err := c.vpcsLister.Get(key)
×
1361
        if err != nil {
×
1362
                if k8serrors.IsNotFound(err) {
×
1363
                        return nil
×
1364
                }
×
1365
                klog.Errorf("failed to get vpc %s, %v", key, err)
×
1366
                return err
×
1367
        }
1368
        needUpdate := false
×
1369
        newStaticRoutes := make([]*kubeovnv1.StaticRoute, 0, len(vpc.Spec.StaticRoutes))
×
1370
        for _, route := range vpc.Spec.StaticRoutes {
×
1371
                if route.ECMPMode == util.StaticRouteBfdEcmp {
×
1372
                        needUpdate = true
×
1373
                        continue
×
1374
                }
1375
                newStaticRoutes = append(newStaticRoutes, route)
×
1376
        }
1377
        // keep routes except bfd ecmp routes
1378
        if needUpdate {
×
1379
                vpc.Spec.StaticRoutes = newStaticRoutes
×
1380
                if _, err = c.config.KubeOvnClient.KubeovnV1().Vpcs().Update(context.Background(), vpc, metav1.UpdateOptions{}); err != nil {
×
1381
                        klog.Errorf("failed to update vpc spec static route %s, %v", vpc.Name, err)
×
1382
                        return err
×
1383
                }
×
1384
        }
1385
        if err = c.patchVpcBfdStatus(vpc.Name); err != nil {
×
1386
                klog.Errorf("failed to patch vpc %s, %v", vpc.Name, err)
×
1387
                return err
×
1388
        }
×
1389
        return nil
×
1390
}
1391

1392
func (c *Controller) handleDelVpcExternalSubnet(key, subnet string) error {
×
1393
        lspName := fmt.Sprintf("%s-%s", subnet, key)
×
1394
        lrpName := fmt.Sprintf("%s-%s", key, subnet)
×
1395
        klog.Infof("delete vpc lrp %s", lrpName)
×
1396
        if err := c.OVNNbClient.RemoveLogicalPatchPort(lspName, lrpName); err != nil {
×
1397
                klog.Errorf("failed to disconnect router '%s' to external, %v", key, err)
×
1398
                return err
×
1399
        }
×
1400
        if err := c.config.KubeOvnClient.KubeovnV1().OvnEips().Delete(context.Background(), lrpName, metav1.DeleteOptions{}); err != nil {
×
1401
                if !k8serrors.IsNotFound(err) {
×
1402
                        klog.Errorf("failed to delete ovn eip %s, %v", lrpName, err)
×
1403
                        return err
×
1404
                }
×
1405
        }
1406
        if err := c.OVNNbClient.DeleteBFDByDstIP(lrpName, ""); err != nil {
×
1407
                klog.Error(err)
×
1408
                return err
×
1409
        }
×
1410
        return nil
×
1411
}
1412

1413
func (c *Controller) patchVpcBfdStatus(key string) error {
×
1414
        cachedVpc, err := c.vpcsLister.Get(key)
×
1415
        if err != nil {
×
1416
                if k8serrors.IsNotFound(err) {
×
1417
                        return nil
×
1418
                }
×
1419
                klog.Errorf("failed to get vpc %s, %v", key, err)
×
1420
                return err
×
1421
        }
1422

1423
        if cachedVpc.Status.EnableBfd != cachedVpc.Spec.EnableBfd {
×
1424
                status := cachedVpc.Status.DeepCopy()
×
1425
                status.EnableExternal = cachedVpc.Spec.EnableExternal
×
1426
                status.EnableBfd = cachedVpc.Spec.EnableBfd
×
1427
                bytes, err := status.Bytes()
×
1428
                if err != nil {
×
1429
                        klog.Errorf("failed to marshal vpc status: %v", err)
×
1430
                        return err
×
1431
                }
×
1432
                if _, err = c.config.KubeOvnClient.KubeovnV1().Vpcs().Patch(context.Background(),
×
1433
                        cachedVpc.Name, types.MergePatchType, bytes, metav1.PatchOptions{}, "status"); err != nil {
×
1434
                        klog.Error(err)
×
1435
                        return err
×
1436
                }
×
1437
        }
1438
        return nil
×
1439
}
1440

1441
func (c *Controller) getRouteTablesByVpc(vpc *kubeovnv1.Vpc) map[string][]*kubeovnv1.StaticRoute {
1✔
1442
        rtbs := make(map[string][]*kubeovnv1.StaticRoute)
1✔
1443
        for _, route := range vpc.Spec.StaticRoutes {
2✔
1444
                rtbs[route.RouteTable] = append(rtbs[route.RouteTable], route)
1✔
1445
        }
1✔
1446
        return rtbs
1✔
1447
}
1448

1449
func (c *Controller) updateVpcExternalStatus(key string) error {
×
1450
        cachedVpc, err := c.vpcsLister.Get(key)
×
1451
        if err != nil {
×
1452
                klog.Errorf("failed to get vpc %s, %v", key, err)
×
1453
                return err
×
1454
        }
×
1455
        vpc := cachedVpc.DeepCopy()
×
1456
        vpc.Status.EnableExternal = vpc.Spec.EnableExternal
×
1457
        vpc.Status.EnableBfd = vpc.Spec.EnableBfd
×
1458

×
1459
        if vpc.Spec.EnableExternal {
×
1460
                sort.Strings(vpc.Spec.ExtraExternalSubnets)
×
1461
                vpc.Status.ExtraExternalSubnets = vpc.Spec.ExtraExternalSubnets
×
1462
        } else {
×
1463
                vpc.Status.ExtraExternalSubnets = nil
×
1464
        }
×
1465

1466
        bytes, err := vpc.Status.Bytes()
×
1467
        if err != nil {
×
1468
                klog.Errorf("failed to get vpc bytes, %v", err)
×
1469
                return err
×
1470
        }
×
1471
        if _, err = c.config.KubeOvnClient.KubeovnV1().Vpcs().Patch(context.Background(),
×
1472
                vpc.Name, types.MergePatchType, bytes, metav1.PatchOptions{}, "status"); err != nil {
×
1473
                klog.Errorf("failed to patch vpc %s, %v", key, err)
×
1474
                return err
×
1475
        }
×
1476

1477
        return nil
×
1478
}
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc