From d7d7a109067d02cba862c5c6d8e0cb114e37cfec Mon Sep 17 00:00:00 2001 From: Kenneth Bingham Date: Tue, 21 Nov 2023 16:26:25 -0500 Subject: [PATCH 1/2] demote and clarify a log warning message about mismatched advertised and listening ports, which is expected in deployed envs where exposed, advertised ports are forwarded to listening, server ports --- router/internal/edgerouter/config.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/router/internal/edgerouter/config.go b/router/internal/edgerouter/config.go index 3d759f303..5550ad0a8 100644 --- a/router/internal/edgerouter/config.go +++ b/router/internal/edgerouter/config.go @@ -310,7 +310,7 @@ func parseEdgeListenerOptions(index int, address string, edgeListenerMap map[int } if advertisePort != addressPort { - pfxlog.Logger().Warnf("port in [listeners[%d].options.advertise] must equal port in [listeners[%d].address] for edge binding but did not. Got [%d] [%d]", index, index, advertisePort, addressPort) + pfxlog.Logger().Debugf("advertised port [%d] in [listeners[%d].options.advertise] does not match the listening port [%d] in [listeners[%d].address].", index, advertisePort, index, addressPort) } return &edge_ctrl_pb.Listener{ From 22b54e8558ecb5ef16d9b7db9dda72b49a8832d9 Mon Sep 17 00:00:00 2001 From: Kenneth Bingham Date: Wed, 22 Nov 2023 12:12:38 -0500 Subject: [PATCH 2/2] promote log from debug to info --- router/internal/edgerouter/config.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/router/internal/edgerouter/config.go b/router/internal/edgerouter/config.go index 5550ad0a8..604a2d2e9 100644 --- a/router/internal/edgerouter/config.go +++ b/router/internal/edgerouter/config.go @@ -310,7 +310,7 @@ func parseEdgeListenerOptions(index int, address string, edgeListenerMap map[int } if advertisePort != addressPort { - pfxlog.Logger().Debugf("advertised port [%d] in [listeners[%d].options.advertise] does not match the listening port [%d] in [listeners[%d].address].", index, advertisePort, index, addressPort) + pfxlog.Logger().Infof("advertised port [%d] in [listeners[%d].options.advertise] does not match the listening port [%d] in [listeners[%d].address].", index, advertisePort, index, addressPort) } return &edge_ctrl_pb.Listener{