Skip to content

Commit 74137fc

Browse files
committed
Add validation test for cilium enableBBR requiring enableBandwidthManager
1 parent 6ccf19a commit 74137fc

1 file changed

Lines changed: 20 additions & 0 deletions

File tree

pkg/apis/kops/validation/validation_test.go

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1232,6 +1232,26 @@ func Test_Validate_Cilium(t *testing.T) {
12321232
},
12331233
ExpectedErrors: []string{"Forbidden::cilium.enableL7Proxy"},
12341234
},
1235+
{
1236+
// enableBBR together with enableBandwidthManager is valid.
1237+
Cilium: kops.CiliumNetworkingSpec{
1238+
EnableBandwidthManager: new(true),
1239+
EnableBBR: new(true),
1240+
},
1241+
},
1242+
{
1243+
// enableBandwidthManager on its own is valid.
1244+
Cilium: kops.CiliumNetworkingSpec{
1245+
EnableBandwidthManager: new(true),
1246+
},
1247+
},
1248+
{
1249+
// enableBBR without enableBandwidthManager is rejected.
1250+
Cilium: kops.CiliumNetworkingSpec{
1251+
EnableBBR: new(true),
1252+
},
1253+
ExpectedErrors: []string{"Forbidden::cilium.enableBBR"},
1254+
},
12351255
{
12361256
Cilium: kops.CiliumNetworkingSpec{
12371257
IPAM: "eni",

0 commit comments

Comments
 (0)