Skip to content

types/netmap,wgengine/magicsock: propagate CapVer to magicsock.endpoint#16244

Merged
jwhited merged 1 commit intomainfrom
jwhited/relay-manager-set-servers
Jun 11, 2025
Merged

types/netmap,wgengine/magicsock: propagate CapVer to magicsock.endpoint#16244
jwhited merged 1 commit intomainfrom
jwhited/relay-manager-set-servers

Conversation

@jwhited
Copy link
Copy Markdown
Member

@jwhited jwhited commented Jun 11, 2025

This enables us to mark nodes as relay capable or not. We don't actually do that yet, as we haven't established a relay CapVer.

Updates tailscale/corp#27502

This enables us to mark nodes as relay capable or not. We don't actually
do that yet, as we haven't established a relay CapVer.

Updates tailscale/corp#27502

Signed-off-by: Jordan Whited <[email protected]>
@jwhited jwhited requested a review from danderson June 11, 2025 00:09
@ghost
Copy link
Copy Markdown

ghost commented Jun 11, 2025

Pull Request Revisions

RevisionDescription
r1
Added capability version node mutationIntroduced NodeMutationCap to handle capability version changes in network map mutations, with placeholder relay capability check

✅ AI review completed for r1
Help React with emojis to give feedback on AI-generated reviews:
  • 👍 means the feedback was helpful and actionable
  • 👎 means the feedback was incorrect or unhelpful
💬 Replying to feedback with a comment helps us improve the system. Your input also contributes to shaping future interactions with the AI reviewer.

We'd love to hear from you—reach out anytime at [email protected].

@bradfitz
Copy link
Copy Markdown
Member

keep in mind that these aren’t plumbed to server side yet either, so you’ll have to do that as well. The current ones on the wire are faked to be close enough for old needs.

@jwhited jwhited merged commit 6a93b17 into main Jun 11, 2025
91 of 93 checks passed
@jwhited jwhited deleted the jwhited/relay-manager-set-servers branch June 11, 2025 00:31
jwhited added a commit that referenced this pull request Jun 19, 2025
…k.endpoint (#16244)"

This reverts commit 6a93b17.

The reverted commit added more complexity than it was worth at the
current stage. Handling delta CapVer changes requires extensive changes
to relayManager datastructures in order to also support delta updates of
relay servers.

Updates tailscale/corp#27502
jwhited added a commit that referenced this pull request Jun 19, 2025
…k.endpoint (#16244)"

This reverts commit 6a93b17.

The reverted commit added more complexity than it was worth at the
current stage. Handling delta CapVer changes requires extensive changes
to relayManager datastructures in order to also support delta updates of
relay servers.

Updates tailscale/corp#27502

Signed-off-by: Jordan Whited <[email protected]>
jwhited added a commit that referenced this pull request Jun 19, 2025
…k.endpoint (#16244)" (#16322)

This reverts commit 6a93b17.

The reverted commit added more complexity than it was worth at the
current stage. Handling delta CapVer changes requires extensive changes
to relayManager datastructures in order to also support delta updates of
relay servers.

Updates tailscale/corp#27502

Signed-off-by: Jordan Whited <[email protected]>
davidsbond pushed a commit that referenced this pull request Jun 30, 2025
…k.endpoint (#16244)" (#16322)

This reverts commit 6a93b17.

The reverted commit added more complexity than it was worth at the
current stage. Handling delta CapVer changes requires extensive changes
to relayManager datastructures in order to also support delta updates of
relay servers.

Updates tailscale/corp#27502

Signed-off-by: Jordan Whited <[email protected]>
ChandonPierre pushed a commit to coreweave/tailscale that referenced this pull request Sep 3, 2025
…nt (tailscale#16244)

This enables us to mark nodes as relay capable or not. We don't actually
do that yet, as we haven't established a relay CapVer.

Updates tailscale/corp#27502

Signed-off-by: Jordan Whited <[email protected]>
ChandonPierre pushed a commit to coreweave/tailscale that referenced this pull request Sep 3, 2025
…k.endpoint (tailscale#16244)" (tailscale#16322)

This reverts commit 6a93b17.

The reverted commit added more complexity than it was worth at the
current stage. Handling delta CapVer changes requires extensive changes
to relayManager datastructures in order to also support delta updates of
relay servers.

Updates tailscale/corp#27502

Signed-off-by: Jordan Whited <[email protected]>
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.

3 participants