Arvados

Sure. Out of curiosity, what about the log in maintainTunnel? Would that get spammy if someone was actually connected to the tunnel? I'm a little surprised it seems to log unconditionally, whether or not err != nil.

runTunnel is meant to run "forever" (until the process exits) -- if it returns at all, that means the controller end hung up, there's a network issue, or something else abnormal/unexpected, so I think it makes sense to keep logging that. I've added comments to make the code look less fishy.

22431-tunnel-log-spam @ 1d42d4b1454516431db869a87fce334dd1b6d137

Tom Clegg wrote in #note-6:

runTunnel is meant to run "forever" (until the process exits) -- if it returns at all, that means the controller end hung up, there's a network issue, or something else abnormal/unexpected, so I think it makes sense to keep logging that. I've added comments to make the code look less fishy.

That makes sense and these are helpful comments, thanks. Please merge.