Skip to content

chore: add nodeUpdater to tailnet #11539

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.

Sign up for GitHub

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

Jump to bottom
Merged
spikecurtis merged 1 commit into from
Jan 11, 2024
Merged

chore: add nodeUpdater to tailnet #11539

spikecurtis merged 1 commit into from
Jan 11, 2024

Conversation

@spikecurtis
Copy link
Contributor

@spikecurtis spikecurtis commented Jan 10, 2024

Adds a nodeUpdater component, which serves a similar role to configMaps, but tracks information from tailscale going out to the coordinator as node updates. This first PR just handles netInfo, subsequent PRs will
handle DERP forced websockets, endpoints, and addresses.

This was referenced Jan 10, 2024
Merged
Merged
Merged
Merged
@spikecurtis Graphite App
Copy link
Contributor Author

spikecurtis commented Jan 10, 2024 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @spikecurtis and the rest of your teammates on Graphite Graphite

@spikecurtis spikecurtis mentioned this pull request Jan 10, 2024
Merged
@spikecurtis spikecurtis requested a review from coadler January 10, 2024 06:14
@spikecurtis spikecurtis force-pushed the spike/10533-blockEndpoints branch from 548eb0e to 63d95ce Compare January 10, 2024 06:51
@spikecurtis spikecurtis force-pushed the spike/10533-add-nodeUpdater branch from a29d08b to 4e34b3e Compare January 10, 2024 06:51
@spikecurtis spikecurtis force-pushed the spike/10533-blockEndpoints branch from 63d95ce to 38a0415 Compare January 10, 2024 11:00
@spikecurtis spikecurtis force-pushed the spike/10533-add-nodeUpdater branch from 4e34b3e to 96c48e4 Compare January 10, 2024 11:00
@spikecurtis spikecurtis force-pushed the spike/10533-blockEndpoints branch from 38a0415 to 15eef5e Compare January 10, 2024 11:55
@spikecurtis spikecurtis force-pushed the spike/10533-add-nodeUpdater branch from 96c48e4 to f48f9db Compare January 10, 2024 11:55
coadler
coadler approved these changes Jan 11, 2024
View reviewed changes
@spikecurtis spikecurtis force-pushed the spike/10533-add-nodeUpdater branch from f48f9db to d52fab0 Compare January 11, 2024 04:53
@spikecurtis spikecurtis mentioned this pull request Jan 11, 2024
Merged
@spikecurtis spikecurtis force-pushed the spike/10533-blockEndpoints branch from 15eef5e to ec72cba Compare January 11, 2024 05:01
@spikecurtis spikecurtis force-pushed the spike/10533-add-nodeUpdater branch from d52fab0 to 2d61f6d Compare January 11, 2024 05:02
@spikecurtis spikecurtis force-pushed the spike/10533-blockEndpoints branch from ec72cba to 45ad9f5 Compare January 11, 2024 05:12
Base automatically changed from spike/10533-blockEndpoints to main January 11, 2024 05:18
@spikecurtis spikecurtis force-pushed the spike/10533-add-nodeUpdater branch from 2d61f6d to 391a882 Compare January 11, 2024 05:19
@spikecurtis Graphite App
Copy link
Contributor Author

spikecurtis commented Jan 11, 2024 •
edited
Loading

Merge activity

  • Jan 11, 12:19 AM: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.
  • Jan 11, 12:29 AM: @spikecurtis merged this pull request with Graphite.

@spikecurtis spikecurtis merged commit 8701dbc into main Jan 11, 2024
@spikecurtis spikecurtis deleted the spike/10533-add-nodeUpdater branch January 11, 2024 05:29
@github-actions github-actions bot locked and limited conversation to collaborators Jan 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@coadler coadler coadler approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@spikecurtis spikecurtis

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@spikecurtis @coadler