Skip to content

Commit ccaf487

Browse files
committed
Fix the various tests that all used '.' separation.
Since we don't put '.' in the bridge names anymore, quite a few tests were still referencing the old style.
1 parent bbec16b commit ccaf487

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

network/containerizer/bridgepolicy_test.go

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@ func (s *bridgePolicyStateSuite) TestPopulateContainerLinkLayerDevicesCorrectlyP
235235
Type: state.BridgeDevice,
236236
},
237237
state.LinkLayerDeviceArgs{
238-
Name: "br-eth10.100",
238+
Name: "br-eth10-100",
239239
Type: state.BridgeDevice,
240240
},
241241
state.LinkLayerDeviceArgs{
@@ -282,7 +282,7 @@ func (s *bridgePolicyStateSuite) TestPopulateContainerLinkLayerDevicesCorrectlyP
282282
"br-eth3",
283283
"br-eth4",
284284
"br-eth10",
285-
"br-eth10.100",
285+
"br-eth10-100",
286286
}
287287

288288
err = s.machine.SetParentLinkLayerDevicesBeforeTheirChildren(devicesArgs[:])
@@ -865,7 +865,7 @@ func (s *bridgePolicyStateSuite) TestFindMissingBridgesForContainerOneOfTwoBridg
865865
// Only the first device (by sort order) should be selected
866866
c.Check(missing, gc.DeepEquals, []network.DeviceToBridge{{
867867
DeviceName: "ens2.1",
868-
BridgeName: "br-ens2.1",
868+
BridgeName: "br-ens2-1",
869869
}})
870870
c.Check(reconfigureDelay, gc.Equals, 0)
871871
}
@@ -944,7 +944,7 @@ func (s *bridgePolicyStateSuite) TestFindMissingBridgesForContainerMultipleSpace
944944
BridgeName: "br-eth0",
945945
}, {
946946
DeviceName: "eth0.1",
947-
BridgeName: "br-eth0.1",
947+
BridgeName: "br-eth0-1",
948948
}, {
949949
DeviceName: "eth1",
950950
BridgeName: "br-eth1",
@@ -1056,7 +1056,7 @@ func (s *bridgePolicyStateSuite) TestFindMissingBridgesForContainerVLAN(c *gc.C)
10561056
BridgeName: "br-eth0",
10571057
}, {
10581058
DeviceName: "eth0.100",
1059-
BridgeName: "br-eth0.100",
1059+
BridgeName: "br-eth0-100",
10601060
}})
10611061
c.Check(reconfigureDelay, gc.Equals, 0)
10621062
}
@@ -1122,7 +1122,7 @@ func (s *bridgePolicyStateSuite) TestFindMissingBridgesForContainerVLANOnBond(c
11221122
BridgeName: "br-bond0",
11231123
}, {
11241124
DeviceName: "bond0.100",
1125-
BridgeName: "br-bond0.100",
1125+
BridgeName: "br-bond0-100",
11261126
}})
11271127
c.Check(reconfigureDelay, gc.Equals, 13)
11281128
}
@@ -1151,7 +1151,7 @@ var bridgeNames = map[string]string{
11511151
"enfourteenchar": "b-fourteenchar",
11521152
"enfifteenchars0": "b-fifteenchars0",
11531153
"fourteenchars1": "b-5590a4-chars1",
1154-
"fifteenchars.12": "b-7e0acf-ars.12",
1154+
"fifteenchars.12": "b-38b496-ars-12",
11551155
"zeros0526193032": "b-000000-193032",
11561156
"enx00e07cc81e1d": "b-x00e07cc81e1d",
11571157
}

0 commit comments

Comments
 (0)