Skip to content

Conversation

@mpminardi
Copy link
Member

Update the documentation for the ACL resource to clarify that it controls the entire policy file for a tailnet and not just the ACLs section of the policy file.

Update the example usage to show usage of grants over the deprecated acl syntax.

Updates #514

@mpminardi mpminardi self-assigned this Oct 14, 2025
@mpminardi mpminardi force-pushed the mpminardi/acl-resource-docs-update branch from 272c6eb to 79646ae Compare December 15, 2025 21:00
@mpminardi mpminardi requested a review from oxtoacart December 15, 2025 23:22
Copy link
Contributor

@mcoulombe mcoulombe left a comment

Choose a reason for hiding this comment

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

Nice. Food for thought, what's our opinion on replacing the tailscale_acl resource/source altogether with tailscale_policy_file?

Iiuc the same resource implementation can be added multiple times in the provider's ResourceMap, effectively making aliases. So we could mark the entire tailscale_acl as deprecated and have new projects use tailscale_policy_file.

@mpminardi
Copy link
Member Author

Nice. Food for thought, what's our opinion on replacing the tailscale_acl resource/source altogether with tailscale_policy_file?

Iiuc the same resource implementation can be added multiple times in the provider's ResourceMap, effectively making aliases. So we could mark the entire tailscale_acl as deprecated and have new projects use tailscale_policy_file.

Good call! Will look at getting this done as a follow-up to this, should be pretty quick to do.

Update the documentation for the ACL resource and data source to clarify
that they control / read the entire policy file for a tailnet and not
just the ACLs section of the policy file.

Update the example usage for the resource to show usage of grants over
the deprecated acl syntax.

Updates #514

Signed-off-by: Mario Minardi <mario@tailscale.com>
@mpminardi mpminardi force-pushed the mpminardi/acl-resource-docs-update branch from 79646ae to 0cfee31 Compare December 16, 2025 15:57
@mpminardi mpminardi merged commit 73e7c60 into main Dec 16, 2025
5 checks passed
@mpminardi mpminardi deleted the mpminardi/acl-resource-docs-update branch December 16, 2025 20:48
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