TUN-6778: Cleanup logs about ICMP

Author: chungthuang

ingress/icmp_darwin.go

@@ -202,7 +202,7 @@ func (ip *icmpProxy) Serve(ctx context.Context) error {
 			// In unit test, we found out when the listener listens on 0.0.0.0, the socket reads the full packet after
 			// the second reply
 			if err := ip.handleFullPacket(icmpDecoder, buf[:n]); err != nil {
-				ip.logger.Err(err).Str("dst", from.String()).Msg("Failed to parse ICMP reply as full packet")
+				ip.logger.Debug().Err(err).Str("dst", from.String()).Msg("Failed to parse ICMP reply as full packet")
 			}
 			continue
 		}

supervisor/supervisor.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"errors"
 	"fmt"
+	"net"
 	"strings"
 	"time"
 
@@ -155,7 +156,11 @@ func (s *Supervisor) Run(
 	if s.edgeTunnelServer.icmpRouter != nil {
 		go func() {
 			if err := s.edgeTunnelServer.icmpRouter.Serve(ctx); err != nil {
-				s.log.Logger().Err(err).Msg("icmp router terminated")
+				if errors.Is(err, net.ErrClosed) {
+					s.log.Logger().Info().Err(err).Msg("icmp router terminated")
+				} else {
+					s.log.Logger().Err(err).Msg("icmp router terminated")
+				}
 			}
 		}()
 	}