ipn/ipnlocal: disable sockstats on (unstable) mobile by default

We're tracking down a new instance of memory usage, and excessive memory usage
from sockstats is definitely not going to help with debugging, so disable it by
default on mobile.

Updates tailscale/corp#18514

Signed-off-by: James Tucker <james@tailscale.com>
pull/11584/head
James Tucker 8 months ago committed by James Tucker
parent e20ce7bf0c
commit 7558a1d594

@ -405,8 +405,8 @@ func NewLocalBackend(logf logger.Logf, logID logid.PublicID, sys *tsd.System, lo
if err != nil { if err != nil {
log.Printf("error setting up sockstat logger: %v", err) log.Printf("error setting up sockstat logger: %v", err)
} }
// Enable sockstats logs only on unstable builds // Enable sockstats logs only on non-mobile unstable builds
if version.IsUnstableBuild() && b.sockstatLogger != nil { if version.IsUnstableBuild() && !version.IsMobile() && b.sockstatLogger != nil {
b.sockstatLogger.SetLoggingEnabled(true) b.sockstatLogger.SetLoggingEnabled(true)
} }

Loading…
Cancel
Save