Correctly deal with empty mclag values (#8)

Author: iljarotar

configdb/configdb.go

@@ -260,8 +260,8 @@ func getInterfaces(ports []values.Port, bgpPorts []string) map[string]Interface
 	return interfaces
 }
 
-func getMCLAGDomains(mclag *values.MCLAG) map[string]MCLAGDomain {
-	if mclag == nil {
+func getMCLAGDomains(mclag values.MCLAG) map[string]MCLAGDomain {
+	if mclag.KeepaliveVLAN == "" {
 		return nil
 	}
 
@@ -275,8 +275,8 @@ func getMCLAGDomains(mclag *values.MCLAG) map[string]MCLAGDomain {
 	}
 }
 
-func getMCLAGInterfaces(mclag *values.MCLAG) map[string]MCLAGInterface {
-	if mclag == nil {
+func getMCLAGInterfaces(mclag values.MCLAG) map[string]MCLAGInterface {
+	if mclag.KeepaliveVLAN == "" {
 		return nil
 	}
 
@@ -291,8 +291,8 @@ func getMCLAGInterfaces(mclag *values.MCLAG) map[string]MCLAGInterface {
 	return mclagInterfaces
 }
 
-func getMCLAGUniqueIPs(mclag *values.MCLAG) map[string]MCLAGUniqueIP {
-	if mclag == nil {
+func getMCLAGUniqueIPs(mclag values.MCLAG) map[string]MCLAGUniqueIP {
+	if mclag.KeepaliveVLAN == "" {
 		return nil
 	}
 

tests/2/sonic-config.yaml

@@ -13,6 +13,7 @@ hostname: leaf01
 lldp_hello_time: 10
 loopback_address: 10.7.7.7
 
+mclag: {}
 mgmtif_gateway: 10.7.10.1
 mgmtif_ip: 10.7.10.2
 mgmt_vrf: false
@@ -41,6 +42,7 @@ ports:
 
 ports_default_fec: none
 ports_default_mtu: 9000
+sag: {}
 
 ssh_sourceranges:
   - 10.1.23.1/30

values/values.go

@@ -67,7 +67,7 @@ type Values struct {
 	Interconnects           map[string]Interconnect `yaml:"interconnects"`
 	LLDPHelloTime           int                     `yaml:"lldp_hello_time"`
 	LoopbackAddress         string                  `yaml:"loopback_address"`
-	MCLAG                   *MCLAG                  `yaml:"mclag"`
+	MCLAG                   MCLAG                   `yaml:"mclag"`
 	MgmtIfGateway           string                  `yaml:"mgmtif_gateway"`
 	MgmtIfIP                string                  `yaml:"mgmtif_ip"`
 	MgmtVRF                 bool                    `yaml:"mgmt_vrf"`