Skip to content
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

Various tailcfg type fixes, expiry hardening and integration test #1195

Merged
merged 10 commits into from
Feb 3, 2023
Merged

Various tailcfg type fixes, expiry hardening and integration test #1195

merged 10 commits into from
Feb 3, 2023

Conversation

kradalby
Copy link
Collaborator

@kradalby kradalby commented Feb 2, 2023 •
edited
Loading

See individual commits.
QoL improvements.

Signed-off-by: Kristoffer Dalby kristoffer@tailscale.com

@kradalby kradalby force-pushed the expiry-hardening branch 2 times, most recently from a16ed77 to 96103e2 Compare February 2, 2023 09:48
@kradalby kradalby changed the title Various tailcfg type fixes, expiry hardening Various tailcfg type fixes, expiry hardening and integration test Feb 2, 2023
@kradalby kradalby force-pushed the expiry-hardening branch 2 times, most recently from e73f2f7 to 64336a7 Compare February 2, 2023 15:11
@kradalby kradalby marked this pull request as ready for review February 2, 2023 15:36
@kradalby kradalby requested a review from juanfont as a code owner February 2, 2023 15:37
kradalby added 10 commits February 2, 2023 17:56
@kradalby
Send control time in map response
61cc57c 
This gives all the nodes the same constant time to work from

Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com>
@kradalby
Add TODOs for only sending patch updates
fb7d5e4 
Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com>
@kradalby
Do not collect services, we dont support it
c30a351 
Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com>
@kradalby
Sort tailcfg.Node creation as upstream
a9aa41d 
Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com>
@kradalby
Populate the tags field on node
e230a82 
Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com>
@kradalby
Use pingAll helper for all integration pinging
d2daee7 
Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com>
@kradalby
Stop using deprecated string function
626ebdb 
Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com>
@kradalby
Remove unused components from old integration tests
9a91cbd 
Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com>
@kradalby
Update changelog
ef50018 
Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com>
@kradalby
Add node expiry test
36d10c0 
This commits adds a test to verify that nodes get updated if a node in
their network expires.

Signed-off-by: Kristoffer Dalby <kristoffer@tailscale.com>
@kradalby kradalby mentioned this pull request Feb 3, 2023
Merged
@kradalby kradalby merged commit c390859 into juanfont:main Feb 3, 2023
@kradalby kradalby deleted the expiry-hardening branch February 3, 2023 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juanfont juanfont juanfont approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kradalby @juanfont