Skip to content

Commit

Permalink
Update comment on L2 connectivity checks and MachineNetworks
Browse files Browse the repository at this point in the history
OpenShift has ~always supported having machines in multiple
machineNetworks, so update the TODO comment to reflect that accounting
for this is already something we need to do to fully support
non-UserManagedNetworking clusters. (UserManagedNetworking clusters use
only the L3 connectivity check.)

See https://issues.redhat.com/browse/OCPBUGS-30730 for more details.
  • Loading branch information
zaneb committed Nov 19, 2024
1 parent 430b55b commit 924e8a1
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions internal/host/validator.go
Original file line number Diff line number Diff line change
Expand Up @@ -675,8 +675,9 @@ func (v *validator) belongsToL2MajorityGroup(c *validationContext, majorityGroup
return ValidationFailure
}

// TODO(mko) This rule should be revised as soon as OCP supports multiple machineNetwork
// entries using the same IP stack.
// TODO(mko) This rule must be revised to support multiple machineNetwork
// entries using the same IP stack on clusters without
// user-managed networking. (OCPBUGS-30730)
areNetworksEqual := func(ipnet1, ipnet2 *net.IPNet) bool {
return ipnet1.IP.Equal(ipnet2.IP) && bytes.Equal(ipnet1.Mask, ipnet2.Mask)
}
Expand Down

0 comments on commit 924e8a1

Please sign in to comment.