Skip to content
Snippets Groups Projects

Improve error message on route creation

Merged Neil-Jocelyn Schark requested to merge improve-error-message into master
1 file
+ 12
7
Compare changes
  • Side-by-side
  • Inline
@@ -45,6 +45,8 @@ func (yh *AssignForwardingHandler) Update(c ygot.ValidatedGoStruct, jobs []*gnmi
@@ -45,6 +45,8 @@ func (yh *AssignForwardingHandler) Update(c ygot.ValidatedGoStruct, jobs []*gnmi
return fmt.Errorf("failed type assertion for newConfig %T", (*gnmitargetygot.Gnmitarget)(nil))
return fmt.Errorf("failed type assertion for newConfig %T", (*gnmitargetygot.Gnmitarget)(nil))
}
}
 
log.Debugf("Received Update within AssignForwardingHandler")
 
forwarding := config.GetOrCreateAssignForwarding()
forwarding := config.GetOrCreateAssignForwarding()
pathId := forwarding.GetPathId()
pathId := forwarding.GetPathId()
@@ -56,9 +58,11 @@ func (yh *AssignForwardingHandler) Update(c ygot.ValidatedGoStruct, jobs []*gnmi
@@ -56,9 +58,11 @@ func (yh *AssignForwardingHandler) Update(c ygot.ValidatedGoStruct, jobs []*gnmi
if nextHop != nil {
if nextHop != nil {
nextHopString = nextHop.GetNodeId()
nextHopString = nextHop.GetNodeId()
 
log.Debugf("AssignForwardingHandler - NextHopString: %s", nextHopString)
}
}
if prevHop != nil {
if prevHop != nil {
prevHopString = prevHop.GetNodeId()
prevHopString = prevHop.GetNodeId()
 
log.Debugf("AssignForwardingHandler - PrevHopString: %s", prevHopString)
}
}
var initKMS *util.RemoteKMS
var initKMS *util.RemoteKMS
@@ -69,15 +73,16 @@ func (yh *AssignForwardingHandler) Update(c ygot.ValidatedGoStruct, jobs []*gnmi
@@ -69,15 +73,16 @@ func (yh *AssignForwardingHandler) Update(c ygot.ValidatedGoStruct, jobs []*gnmi
addressPrefix = initiatingKmsAddress.GetIpAddress()
addressPrefix = initiatingKmsAddress.GetIpAddress()
} else if initiatingKmsAddress.GetHostname() != "" {
} else if initiatingKmsAddress.GetHostname() != "" {
addressPrefix = initiatingKmsAddress.GetHostname()
addressPrefix = initiatingKmsAddress.GetHostname()
} else {
log.Error("initiatingKmsAddress must have either an IP address or a hostname")
return fmt.Errorf("initiatingKmsAddress must have either an IP address or a hostname")
}
}
initKMS = &util.RemoteKMS{
log.Debugf("AssignForwardingHandler - InitiatingKMSAddress was set with the following parameters; NodeId: %s, Hostname: %s, IpAddress: %s, Port: %s", initiatingKmsAddress.GetNodeId(), initiatingKmsAddress.GetHostname(), initiatingKmsAddress.GetIpAddress(), initiatingKmsAddress.GetPort())
Id: initiatingKmsAddress.GetNodeId(),
Address: addressPrefix,
if addressPrefix != "" {
Port: initiatingKmsAddress.GetPort(),
initKMS = &util.RemoteKMS{
 
Id: initiatingKmsAddress.GetNodeId(),
 
Address: addressPrefix,
 
Port: initiatingKmsAddress.GetPort(),
 
}
}
}
}
}
Loading