Skip to content

Commit

Permalink
Merge pull request #8 from truefoundry/update-cloud-integration-refer…
Browse files Browse the repository at this point in the history
…ence

Update cloud integration reference
  • Loading branch information
DeeAjayi authored Jul 3, 2024
2 parents ef8e43b + a834b0b commit 38efaa1
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ Truefoundry Google Cloud platform features module
| <a name="input_blob_storage_override_name"></a> [blob\_storage\_override\_name](#input\_blob\_storage\_override\_name) | S3 bucket name. Only used if s3\_enable\_override is enabled | `string` | `""` | no |
| <a name="input_cluster_name"></a> [cluster\_name](#input\_cluster\_name) | Name of the cluster | `string` | n/a | yes |
| <a name="input_feature_blob_storage_enabled"></a> [feature\_blob\_storage\_enabled](#input\_feature\_blob\_storage\_enabled) | Enable blob storage feature in the platform | `bool` | `true` | no |
| <a name="input_feature_cloud_integration_enabled"></a> [feature\_cloud\_integration\_enabled](#input\_feature\_cloud\_integration\_enabled) | Enable cloud integration feature in the platform | `bool` | `true` | no |
| <a name="input_feature_cluster_integration_enabled"></a> [feature\_cluster\_integration\_enabled](#input\_feature\_cluster\_integration\_enabled) | Enable cluster integration feature in the platform | `bool` | `true` | no |
| <a name="input_feature_docker_registry_enabled"></a> [feature\_docker\_registry\_enabled](#input\_feature\_docker\_registry\_enabled) | Enable docker registry feature in the platform | `bool` | `true` | no |
| <a name="input_feature_secrets_enabled"></a> [feature\_secrets\_enabled](#input\_feature\_secrets\_enabled) | Enable secrets manager feature in the platform | `bool` | `true` | no |
| <a name="input_project"></a> [project](#input\_project) | GCP Project | `string` | n/a | yes |
Expand All @@ -63,4 +63,4 @@ Truefoundry Google Cloud platform features module
| <a name="output_bucket_url"></a> [bucket\_url](#output\_bucket\_url) | URL of the bucket |
| <a name="output_serviceaccount_key"></a> [serviceaccount\_key](#output\_serviceaccount\_key) | Service account keys |
| <a name="output_serviceaccount_name"></a> [serviceaccount\_name](#output\_serviceaccount\_name) | Name of the service account |
<!-- END_TF_DOCS -->
<!-- END_TF_DOCS -->
4 changes: 2 additions & 2 deletions iam.tf
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ resource "google_project_iam_member" "truefoundry_platform_feature_artifact_regi

// role binding container cluster viewer role to service account
resource "google_project_iam_member" "truefoundry_platform_feature_container_cluster_viewer_role_binding" {
count = var.feature_cloud_integration_enabled ? 1 : 0
count = var.feature_cluster_integration_enabled ? 1 : 0

project = var.project
role = "roles/container.clusterViewer"
Expand All @@ -113,7 +113,7 @@ resource "google_project_iam_member" "truefoundry_platform_feature_container_clu

// role binding container viewer role to service account
resource "google_project_iam_member" "truefoundry_platform_feature_container_viewer_role_binding" {
count = var.feature_cloud_integration_enabled ? 1 : 0
count = var.feature_cluster_integration_enabled ? 1 : 0

project = var.project
role = "roles/container.viewer"
Expand Down
8 changes: 4 additions & 4 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -82,10 +82,10 @@ variable "feature_docker_registry_enabled" {
}

################################################################################
# Cloud integration
# Cluster integration
################################################################################
variable "feature_cloud_integration_enabled" {
description = "Enable cloud integration feature in the platform"
variable "feature_cluster_integration_enabled" {
description = "Enable cluster integration feature in the platform"
type = bool
default = true
}
}

0 comments on commit 38efaa1

Please sign in to comment.