diff options
| author | Adrian Dewhurst <adrian@tailscale.com> | 2024-05-31 09:54:46 -0400 |
|---|---|---|
| committer | Adrian Dewhurst <sailor@sailorfrag.net> | 2024-06-06 20:26:14 -0400 |
| commit | 0219317372b7a96de9377dd1f91626e2ed3cedc9 (patch) | |
| tree | b45948241673cabd849c8cc6b5b4a9987d6afc20 /control/controlknobs | |
| parent | 7a7e314096919a235165622ad488cef81daed20e (diff) | |
| download | tailscale-0219317372b7a96de9377dd1f91626e2ed3cedc9.tar.xz tailscale-0219317372b7a96de9377dd1f91626e2ed3cedc9.zip | |
ipn/ipnlocal: improve sticky last suggestion
The last suggested exit node needs to be incorporated in the decision
making process when a new suggestion is requested, but currently it is
not quite right: it'll be used if the suggestion code has an error or a
netmap is unavailable, but it won't be used otherwise.
Instead, this makes the last suggestion into a tiebreaker when making a
random selection between equally-good options. If the last suggestion
does not make it to the final selection pool, then a different
suggestion will be made.
Since LocalBackend.SuggestExitNode is back to being a thin shim that
sets up the parameters to suggestExitNode, it no longer needs a test.
Its test was unable to be comprehensive anyway as the code being tested
contains an uncontrolled random number generator.
Updates tailscale/corp#19681
Change-Id: I94ecc9a0d1b622de3df4ef90523f1d3e67b4bfba
Signed-off-by: Adrian Dewhurst <adrian@tailscale.com>
Diffstat (limited to 'control/controlknobs')
0 files changed, 0 insertions, 0 deletions