diff options
author | ISHIDA Wataru <ishida.wataru@lab.ntt.co.jp> | 2015-08-25 11:29:24 +0900 |
---|---|---|
committer | ISHIDA Wataru <ishida.wataru@lab.ntt.co.jp> | 2015-08-25 11:45:23 +0900 |
commit | 1d8c71e22b9c974db66ff41cba81ae601ea14dd7 (patch) | |
tree | 25e458da32ddc543352452741119aa295dd9a5f5 /table | |
parent | 64ddb52890451da4c9877f8b3273fb9fe35f878c (diff) |
*: better log msg
Signed-off-by: ISHIDA Wataru <ishida.wataru@lab.ntt.co.jp>
Diffstat (limited to 'table')
-rw-r--r-- | table/destination.go | 9 | ||||
-rw-r--r-- | table/table.go | 4 |
2 files changed, 7 insertions, 6 deletions
diff --git a/table/destination.go b/table/destination.go index 96979a78..290830a4 100644 --- a/table/destination.go +++ b/table/destination.go @@ -251,15 +251,16 @@ func (dest *Destination) Calculate() (*Path, string, error) { //""" func (dest *Destination) removeWithdrawals() { + // If we have no withdrawals, we have nothing to do. + if len(dest.withdrawList) == 0 { + return + } + log.WithFields(log.Fields{ "Topic": "Table", "Key": dest.GetNlri().String(), "Length": len(dest.withdrawList), }).Debug("Removing withdrawals") - // If we have no withdrawals, we have nothing to do. - if len(dest.withdrawList) == 0 { - return - } // If we have some withdrawals and no know-paths, it means it is safe to // delete these withdraws. diff --git a/table/table.go b/table/table.go index 0d6abd29..d40ac9c4 100644 --- a/table/table.go +++ b/table/table.go @@ -182,8 +182,8 @@ func (t *Table) getOrCreateDest(nlri bgp.AddrPrefixInterface) *Destination { if dest == nil { log.WithFields(log.Fields{ "Topic": "Table", - "Key": t.routeFamily, - }).Debugf("create Destination with key %s", tableKey) + "Key": tableKey, + }).Debugf("create Destination") dest = NewDestination(nlri) t.setDestination(tableKey, dest) } |