Skip to content

[v17] [reversetunnel] Use access point for cert auths#61318

Merged
okraport merged 1 commit intobranch/v17from
bot/backport-61274-branch/v17
Nov 13, 2025
Merged

[v17] [reversetunnel] Use access point for cert auths#61318
okraport merged 1 commit intobranch/v17from
bot/backport-61274-branch/v17

Conversation

@okraport
Copy link
Copy Markdown
Contributor

Backport #61274 to branch/v17

changelog: Improve reverse tunnel dialing recovery from default route changes by 1min on average.

This change allows the agent dialer to use cached
results for hostCheckerFunc. In the case of a transient
grpc network failure, this allows the reverse tunnel
to be re-established before the grpc timeout takes place.

Test plan:

- [ ] Spawn two Linux VMs, one for auth/proxy and one for node.
      Simulate network change by toggling one of two interfaces on the node,
      measure time for the reverse tunnel to be functional again.

In local testing with 5 runs this on average brings the recovery time
to 2m39s from 3m45s.

Changelog: Improve reverse tunnel dialing recovery from default route changes by 1min on average.
@okraport okraport marked this pull request as ready for review November 13, 2025 11:26
@okraport okraport enabled auto-merge November 13, 2025 11:28
@okraport okraport added this pull request to the merge queue Nov 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 13, 2025
@okraport okraport added this pull request to the merge queue Nov 13, 2025
Merged via the queue into branch/v17 with commit 7c77034 Nov 13, 2025
43 checks passed
@okraport okraport deleted the bot/backport-61274-branch/v17 branch November 13, 2025 14:14
@aadc-dev aadc-dev mentioned this pull request Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants