Skip to content

Commit 850179b

Browse files
committed
Cleanup with another round of go mod tidy and some minor fixes from rebasing
1 parent e2d2568 commit 850179b

File tree

3 files changed

+3
-10
lines changed

3 files changed

+3
-10
lines changed

go.mod

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ require (
66
github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.18.6
77
github.com/aws/aws-sdk-go-v2/service/ec2 v1.249.0
88
github.com/aws/smithy-go v1.23.0
9-
github.com/ccoveille/go-safecast v1.6.1
9+
github.com/ccoveille/go-safecast/v2 v2.0.0
1010
github.com/coreos/go-iptables v0.8.0
1111
github.com/docker/docker v28.4.0+incompatible
1212
github.com/goccy/go-yaml v1.18.0
@@ -127,7 +127,6 @@ require (
127127
sigs.k8s.io/json v0.0.0-20241014173422-cfa47c3a1cc8 // indirect
128128
sigs.k8s.io/randfill v1.0.0 // indirect
129129
sigs.k8s.io/structured-merge-diff/v6 v6.3.0 // indirect
130-
sigs.k8s.io/yaml v1.6.0 // indirect
131130
)
132131

133132
go 1.24.0

go.sum

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@ github.com/aws/smithy-go v1.23.0 h1:8n6I3gXzWJB2DxBDnfxgBaSX6oe0d/t10qGz7OKqMCE=
3434
github.com/aws/smithy-go v1.23.0/go.mod h1:t1ufH5HMublsJYulve2RKmHDC15xu1f26kHCp/HgceI=
3535
github.com/beorn7/perks v1.0.1 h1:VlbKKnNfV8bJzeqoa4cOKqO6bYr3WgKZxO8Z16+hsOM=
3636
github.com/beorn7/perks v1.0.1/go.mod h1:G2ZrVWU2WbWT9wwq4/hrbKbnv/1ERSJQ0ibhJ6rlkpw=
37-
github.com/ccoveille/go-safecast v1.6.1 h1:Nb9WMDR8PqhnKCVs2sCB+OqhohwO5qaXtCviZkIff5Q=
38-
github.com/ccoveille/go-safecast v1.6.1/go.mod h1:QqwNjxQ7DAqY0C721OIO9InMk9zCwcsO7tnRuHytad8=
37+
github.com/ccoveille/go-safecast/v2 v2.0.0 h1:+5eyITXAUj3wMjad6cRVJKGnC7vDS55zk0INzJagub0=
38+
github.com/ccoveille/go-safecast/v2 v2.0.0/go.mod h1:JIYA4CAR33blIDuE6fSwCp2sz1oOBahXnvmdBhOAABs=
3939
github.com/cenkalti/backoff/v4 v4.2.1 h1:y4OZtCnogmCPw98Zjyt5a6+QwPLGkiQsYW5oUqylYbM=
4040
github.com/cenkalti/backoff/v4 v4.2.1/go.mod h1:Y3VNntkOUPxTVeUxJ/G5vcM//AlwfmyYozVcomhLiZE=
4141
github.com/cespare/xxhash/v2 v2.3.0 h1:UL815xU9SqsFlibzuggzjXhog7bL6oX9BbNZnL2UFvs=

pkg/controllers/routing/network_routes_controller.go

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -333,10 +333,6 @@ func (nrc *NetworkRoutingController) Run(
333333
"not work: %s", err.Error())
334334
}
335335
}()
336-
if _, err := exec.Command("modprobe", "br_netfilter").CombinedOutput(); err != nil {
337-
klog.Errorf("Failed to enable netfilter for bridge. Network policies and service proxy may "+
338-
"not work: %s", err.Error())
339-
}
340336
sysctlErr := utils.SetSysctl(utils.BridgeNFCallIPTables, 1)
341337
if sysctlErr != nil {
342338
klog.Errorf("Failed to enable iptables for bridge. Network policies and service proxy may "+
@@ -473,7 +469,6 @@ func (nrc *NetworkRoutingController) updateCNIConfig() {
473469
err = cniNetConf.InsertPodCIDRIntoIPAM(ipv6CIDR)
474470
if err != nil {
475471
klog.Fatalf("failed to insert IPv6 `subnet`(pod CIDR) '%s' into CNI conf file: %v", ipv6CIDR, err)
476-
)
477472
}
478473
}
479474

@@ -1293,7 +1288,6 @@ func NewNetworkRoutingController(clientset kubernetes.Interface,
12931288
if nrc.bgpHoldtime > 65536 || nrc.bgpHoldtime < 3 {
12941289
return nil, errors.New("this is an incorrect BGP holdtime range, holdtime must be in the range " +
12951290
"3s to 18h12m16s")
1296-
)
12971291
}
12981292

12991293
nrc.hostnameOverride = kubeRouterConfig.HostnameOverride

0 commit comments

Comments
 (0)