Skip to content

chore: refactor ServerTailnet to use tailnet.Controllers #15408

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

spikecurtis
Copy link
Contributor

@spikecurtis spikecurtis commented Nov 6, 2024

chore of #14729

Refactors the ServerTailnet to use tailnet.Controller so that we reuse logic around reconnection and handling control messages, instead of reimplementing. This unifies our "client" use of the tailscale API across CLI, coderd, and wsproxy.

@spikecurtis spikecurtis force-pushed the spike/14729-controller branch from 791e183 to c79d83d Compare November 7, 2024 07:33
@spikecurtis spikecurtis force-pushed the spike/14729-server-tailnet branch from ed2ba52 to d9b0988 Compare November 7, 2024 07:33
@spikecurtis spikecurtis force-pushed the spike/14729-controller branch from c79d83d to e6e3c8f Compare November 7, 2024 13:10
@spikecurtis spikecurtis force-pushed the spike/14729-server-tailnet branch from d9b0988 to b8062ab Compare November 7, 2024 13:10
@spikecurtis spikecurtis force-pushed the spike/14729-controller branch 2 times, most recently from bf47e04 to abd4edd Compare November 7, 2024 13:25
@spikecurtis spikecurtis force-pushed the spike/14729-server-tailnet branch from b8062ab to 845efd2 Compare November 7, 2024 13:25
@spikecurtis spikecurtis changed the base branch from spike/14729-controller to graphite-base/15408 November 8, 2024 06:10
@spikecurtis spikecurtis force-pushed the spike/14729-server-tailnet branch from 845efd2 to b8ed4ea Compare November 8, 2024 06:11
@spikecurtis spikecurtis changed the base branch from graphite-base/15408 to main November 8, 2024 06:11
@spikecurtis spikecurtis force-pushed the spike/14729-server-tailnet branch from b8ed4ea to 2603e6d Compare November 8, 2024 06:11
@spikecurtis spikecurtis force-pushed the spike/14729-server-tailnet branch from 2603e6d to 778457b Compare November 8, 2024 09:05
@spikecurtis spikecurtis merged commit 8c00ebc into main Nov 8, 2024
29 of 30 checks passed
Copy link
Contributor Author

Merge activity

  • Nov 8, 4:18 AM EST: A user merged this pull request with Graphite.

@spikecurtis spikecurtis deleted the spike/14729-server-tailnet branch November 8, 2024 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants