Skip to content

Add enable_lf_grants for opt-in LF TableWildcard grants #104

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

Open
drernie wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add enable_lf_grants for opt-in LF TableWildcard grants #104

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ae652a5
Add enable_lf_grants variable for opt-in Lake Formation TableWildcard…
drernie Apr 12, 2026
0c00b6e
fix: use null to omit EnableLakeFormationGrants when disabled
drernie Apr 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions VARIABLES.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,12 @@ This document provides comprehensive documentation for all variables available i
| `search_volume_throughput` | `number` | `null` | EBS throughput (MiB/s, for some gp3 volumes) | 125-1000 |
| `search_auto_tune_desired_state` | `string` | `"DISABLED"` | ElasticSearch Auto-Tune state | `"ENABLED"`, `"DISABLED"` |

### Lake Formation Variables

| Variable | Type | Default | Description |
|---------------------|--------|---------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
| `enable_lf_grants` | `bool` | `false` | Enable per-role Lake Formation TableWildcard grants. Requires the CloudFormation execution role to be a Lake Formation administrator. Leave `false` unless LF enforcement is active and the deploying role is on the LF admin list. |

### CloudFormation Stack Variables

| Variable | Type | Default | Description |
Expand Down
2 changes: 2 additions & 0 deletions modules/quilt/main.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,8 @@ resource "aws_cloudformation_stack" "stack" {
parameters = merge(
var.parameters,
{
EnableLakeFormationGrants = var.enable_lf_grants ? "Enabled" : null

VPC = module.vpc.vpc_id
Subnets = join(",", module.vpc.private_subnets)
PublicSubnets = var.internal ? null : join(",", module.vpc.public_subnets)
Expand Down
7 changes: 7 additions & 0 deletions modules/quilt/variables.tf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,13 @@ variable "search_volume_type" {
description = "Type of EBS volumes attached to data nodes in the ElasticSearch cluster"
}

variable "enable_lf_grants" {
type = bool
nullable = false
default = false
description = "Enable per-role Lake Formation TableWildcard grants. Requires the CloudFormation execution role to be a Lake Formation administrator. Leave false unless LF enforcement is active and the deploying role is on the LF admin list."
}

variable "template_file" {
type = string
nullable = true
Expand Down