Skip to content

Commit ef713aa

Browse files
Merge pull request openshift#7985 from vr4manta/OCPBUGS-29114
OCPBUGS-29114: Fixed control plane machine set handling of static IPs when AddressesFromPools is not in use.
2 parents d7742ee + 4f7bae9 commit ef713aa

File tree

1 file changed

+12
-5
lines changed

1 file changed

+12
-5
lines changed

pkg/asset/machines/vsphere/machines.go

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -136,12 +136,19 @@ func Machines(clusterID string, config *types.InstallConfig, pool *types.Machine
136136
vsphereMachineProvider.Template = ""
137137
}
138138

139+
// Only set AddressesFromPools and Nameservers if AddressesFromPools is > 0, else revert to
140+
// the older static IP manifest way.
139141
if len(hosts) > 0 {
140-
vsphereMachineProvider.Network.Devices = []machineapi.NetworkDeviceSpec{
141-
{
142-
AddressesFromPools: origProv.Network.Devices[0].AddressesFromPools,
143-
Nameservers: origProv.Network.Devices[0].Nameservers,
144-
},
142+
if len(origProv.Network.Devices[0].AddressesFromPools) > 0 {
143+
vsphereMachineProvider.Network.Devices = []machineapi.NetworkDeviceSpec{
144+
{
145+
AddressesFromPools: origProv.Network.Devices[0].AddressesFromPools,
146+
Nameservers: origProv.Network.Devices[0].Nameservers,
147+
},
148+
}
149+
} else {
150+
// Older static IP config, lets remove network since it'll come from FD
151+
vsphereMachineProvider.Network = machineapi.NetworkSpec{}
145152
}
146153
}
147154

0 commit comments

Comments
 (0)