Skip to content

Conversation

@kradalby
Copy link
Collaborator

@kradalby kradalby commented Nov 13, 2025

This PR changes tags to be something that exists on nodes in addition to users, to being its own thing. It is part of moving our tags support towards the correct tailscale compatible implementation.

There are probably rough edges in this PR, but the intention is to get it in, and then start fixing bugs from 0.28.0 milestone (long standing tags issue) to discover what works and what doesnt.

Updates #2417
Closes #2619

@kradalby kradalby force-pushed the kradalby/tags-reboot branch from fe1266c to 2a8ca3e Compare November 14, 2025 14:05
Copy link
Collaborator Author

@kradalby kradalby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Round 1.

@kradalby kradalby force-pushed the kradalby/tags-reboot branch 2 times, most recently from 45bdb84 to 6d9d63d Compare November 28, 2025 09:55
@kradalby kradalby changed the title wip tag take 2 make tags first class node owner Nov 30, 2025
@kradalby kradalby marked this pull request as ready for review November 30, 2025 18:45
@kradalby kradalby force-pushed the kradalby/tags-reboot branch from c089a04 to ef7b0a9 Compare December 2, 2025 08:52
kradalby and others added 3 commits December 2, 2025 09:54
This commit changes tags to be something that exists on nodes in addition to users, to being its own thing. It is part
of moving our tags support towards the correct tailscale compatible implementation.

There are probably rough edges in this commit, but the intention is to get it in, and then start fixing bugs from 0.28.0
milestone (long standing tags issue) to discover what works and what doesnt.

Updates juanfont#2417
Closes juanfont#2619
Signed-off-by: Kristoffer Dalby <[email protected]>
Signed-off-by: Kristoffer Dalby <[email protected]>
@kradalby kradalby force-pushed the kradalby/tags-reboot branch from ef7b0a9 to 467036f Compare December 2, 2025 08:55
@kradalby kradalby merged commit eb788cd into juanfont:main Dec 2, 2025
97 of 99 checks passed
kradalby added a commit to kradalby/headscale that referenced this pull request Dec 11, 2025
Update TestTaildrop to validate that juanfont#2462 is fixed
through juanfont#2885 and juanfont#2931.

Fixes juanfont#2462

Signed-off-by: Kristoffer Dalby <[email protected]>
kradalby added a commit to kradalby/headscale that referenced this pull request Dec 11, 2025
Update TestTaildrop to validate that juanfont#2462 is fixed
through juanfont#2885 and juanfont#2931.

Fixes juanfont#2462

Signed-off-by: Kristoffer Dalby <[email protected]>
kradalby added a commit to kradalby/headscale that referenced this pull request Dec 11, 2025
Update TestTaildrop to validate that juanfont#2462 is fixed
through juanfont#2885 and juanfont#2931.

Fixes juanfont#2462

Signed-off-by: Kristoffer Dalby <[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.

2 participants