Skip to content

Commit

Permalink
add default and ignored tags for the flyte module (#252)
Browse files Browse the repository at this point in the history
  • Loading branch information
steved authored Jul 19, 2024
1 parent cd2c348 commit ae594bd
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 1 deletion.
2 changes: 2 additions & 0 deletions modules/flyte/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,12 @@ No modules.
| <a name="input_compute_namespace"></a> [compute\_namespace](#input\_compute\_namespace) | Name of Domino compute namespace for this deploy | `string` | n/a | yes |
| <a name="input_eks_info"></a> [eks\_info](#input\_eks\_info) | cluster = {<br> specs {<br> name = Cluster name.<br> account\_id = AWS account id where the cluster resides.<br> }<br> oidc = {<br> arn = OIDC provider ARN.<br> url = OIDC provider url.<br> cert = {<br> thumbprint\_list = OIDC cert thumbprints.<br> url = OIDC cert URL.<br> }<br> } | <pre>object({<br> cluster = object({<br> specs = object({<br> name = string<br> account_id = string<br> })<br> oidc = object({<br> arn = string<br> url = string<br> cert = object({<br> thumbprint_list = list(string)<br> url = string<br> })<br> })<br> })<br> })</pre> | n/a | yes |
| <a name="input_force_destroy_on_deletion"></a> [force\_destroy\_on\_deletion](#input\_force\_destroy\_on\_deletion) | Whether to force destroy flyte s3 buckets on deletion | `bool` | `true` | no |
| <a name="input_ignore_tags"></a> [ignore\_tags](#input\_ignore\_tags) | Tag keys to be ignored by the aws provider. | `list(string)` | `[]` | no |
| <a name="input_kms_info"></a> [kms\_info](#input\_kms\_info) | key\_id = KMS key id.<br> key\_arn = KMS key arn.<br> enabled = KMS key is enabled | <pre>object({<br> key_id = string<br> key_arn = string<br> enabled = bool<br> })</pre> | n/a | yes |
| <a name="input_platform_namespace"></a> [platform\_namespace](#input\_platform\_namespace) | Name of Domino platform namespace for this deploy | `string` | n/a | yes |
| <a name="input_region"></a> [region](#input\_region) | AWS region for the deployment | `string` | n/a | yes |
| <a name="input_serviceaccount_names"></a> [serviceaccount\_names](#input\_serviceaccount\_names) | Service account names for Flyte | <pre>object({<br> datacatalog = optional(string, "datacatalog")<br> flyteadmin = optional(string, "flyteadmin")<br> flytepropeller = optional(string, "flytepropeller")<br> })</pre> | `{}` | no |
| <a name="input_tags"></a> [tags](#input\_tags) | Deployment tags. | `map(string)` | `{}` | no |
| <a name="input_use_fips_endpoint"></a> [use\_fips\_endpoint](#input\_use\_fips\_endpoint) | Use aws FIPS endpoints | `bool` | `false` | no |

## Outputs
Expand Down
12 changes: 12 additions & 0 deletions modules/flyte/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,18 @@ variable "region" {
}
}

variable "tags" {
type = map(string)
description = "Deployment tags."
default = {}
}

variable "ignore_tags" {
type = list(string)
description = "Tag keys to be ignored by the aws provider."
default = []
}

variable "use_fips_endpoint" {
description = "Use aws FIPS endpoints"
type = bool
Expand Down
10 changes: 9 additions & 1 deletion modules/flyte/versions.tf
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,14 @@ terraform {
}

provider "aws" {
region = var.region
region = var.region

default_tags {
tags = var.tags
}
ignore_tags {
keys = var.ignore_tags
}

use_fips_endpoint = var.use_fips_endpoint
}

0 comments on commit ae594bd

Please sign in to comment.