Skip to content

Commit

Permalink
Merge pull request #107 from shiftstack/fix-port-security
Browse files Browse the repository at this point in the history
OCPBUGS-30855: Ensure portSecurity is correctly set in the Port
  • Loading branch information
openshift-merge-bot[bot] committed Mar 13, 2024
2 parents d1d44b4 + 555ef8d commit bf45c1f
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions pkg/machine/convert.go
Expand Up @@ -355,12 +355,18 @@ func MachineToInstanceSpec(machine *machinev1beta1.Machine, apiVIPs, ingressVIPs
if port.SecurityGroups != nil {
portSecurityGroupParams = securityGroupsToSecurityGroupParams(*port.SecurityGroups)
}
disablePortSecurity := port.PortSecurity
if disablePortSecurity != nil {
ps := !*disablePortSecurity
disablePortSecurity = &ps
}
capoPort := capov1.PortOpts{
Network: &capov1.NetworkFilter{ID: port.NetworkID},
NameSuffix: port.NameSuffix,
Description: port.Description,
AdminStateUp: port.AdminStateUp,
MACAddress: port.MACAddress,
DisablePortSecurity: disablePortSecurity,
FixedIPs: make([]capov1.FixedIP, len(port.FixedIPs)),
SecurityGroupFilters: securityGroupParamToCapov1SecurityGroupFilter(portSecurityGroupParams),
VNICType: port.VNICType,
Expand Down

0 comments on commit bf45c1f

Please sign in to comment.