Skip to content

Commit

Permalink
Remove report_deployment_logs variable from deploy module (#86)
Browse files Browse the repository at this point in the history
Signed-off-by: Jim Enright <[email protected]>
  • Loading branch information
jimright authored Nov 27, 2024
1 parent 67f713d commit bfed051
Show file tree
Hide file tree
Showing 9 changed files with 9 additions and 46 deletions.
1 change: 0 additions & 1 deletion modules/terraform-cdp-deploy/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,6 @@ No resources.
| <a name="input_multiaz"></a> [multiaz](#input\_multiaz) | Flag to specify that the FreeIPA and DataLake instances will be deployed across multi-availability zones. | `bool` | `true` | no |
| <a name="input_proxy_config_name"></a> [proxy\_config\_name](#input\_proxy\_config\_name) | Name of the proxy config to use for the environment. | `string` | `null` | no |
| <a name="input_public_key_text"></a> [public\_key\_text](#input\_public\_key\_text) | SSH Public key string for the nodes of the CDP environment. Required for CDP deployment on Azure. For CDP deployment on AWS, either 'keypair\_name' or 'public\_key\_text' needs to be set. | `string` | `null` | no |
| <a name="input_report_deployment_logs"></a> [report\_deployment\_logs](#input\_report\_deployment\_logs) | Flag to enable reporting of additional diagnostic information back to Cloudera. | `bool` | `null` | no |
| <a name="input_s3_guard_table_name"></a> [s3\_guard\_table\_name](#input\_s3\_guard\_table\_name) | Name for the DynamoDB table backing S3Guard. Only applicable for CDP deployment on AWS. | `string` | `null` | no |
| <a name="input_use_public_ips"></a> [use\_public\_ips](#input\_use\_public\_ips) | Use public ip's for the CDP resources created within the Cloud network. Required for CDP deployment on Azure and GCP. | `bool` | `null` | no |
| <a name="input_use_single_resource_group"></a> [use\_single\_resource\_group](#input\_use\_single\_resource\_group) | Use a single resource group for all provisioned CDP resources. Required for CDP deployment on Azure. | `bool` | `true` | no |
Expand Down
3 changes: 0 additions & 3 deletions modules/terraform-cdp-deploy/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,6 @@ module "cdp_on_aws" {
multiaz = var.multiaz
freeipa_instances = var.freeipa_instances
workload_analytics = var.workload_analytics
report_deployment_logs = var.report_deployment_logs
endpoint_access_scheme = local.endpoint_access_scheme

environment_async_creation = var.environment_async_creation
Expand Down Expand Up @@ -112,7 +111,6 @@ module "cdp_on_azure" {
multiaz = var.multiaz
freeipa_instances = var.freeipa_instances
workload_analytics = var.workload_analytics
report_deployment_logs = var.report_deployment_logs
endpoint_access_scheme = local.endpoint_access_scheme

environment_async_creation = var.environment_async_creation
Expand Down Expand Up @@ -205,7 +203,6 @@ module "cdp_on_gcp" {
freeipa_recipes = var.freeipa_recipes

workload_analytics = var.workload_analytics
report_deployment_logs = var.report_deployment_logs
endpoint_access_scheme = local.endpoint_access_scheme
availability_zones = var.gcp_availability_zones

Expand Down
9 changes: 4 additions & 5 deletions modules/terraform-cdp-deploy/modules/aws/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -60,11 +60,10 @@ resource "cdp_environments_aws_environment" "cdp_env" {
os = var.freeipa_os
}

proxy_config_name = var.proxy_config_name
s3_guard_table_name = var.s3_guard_table_name
workload_analytics = var.workload_analytics
report_deployment_logs = var.report_deployment_logs
enable_tunnel = var.enable_ccm_tunnel
proxy_config_name = var.proxy_config_name
s3_guard_table_name = var.s3_guard_table_name
workload_analytics = var.workload_analytics
enable_tunnel = var.enable_ccm_tunnel

encryption_key_arn = var.encryption_key_arn

Expand Down
7 changes: 0 additions & 7 deletions modules/terraform-cdp-deploy/modules/aws/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -64,13 +64,6 @@ variable "enable_ccm_tunnel" {

}

variable "report_deployment_logs" {
type = bool

description = "Flag to enable reporting of additional diagnostic information back to Cloudera"

}

variable "enable_raz" {
type = bool

Expand Down
7 changes: 3 additions & 4 deletions modules/terraform-cdp-deploy/modules/azure/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -81,10 +81,9 @@ resource "cdp_environments_azure_environment" "cdp_env" {
os = var.freeipa_os
}

proxy_config_name = var.proxy_config_name
workload_analytics = var.workload_analytics
report_deployment_logs = var.report_deployment_logs
enable_tunnel = var.enable_ccm_tunnel
proxy_config_name = var.proxy_config_name
workload_analytics = var.workload_analytics
enable_tunnel = var.enable_ccm_tunnel

enable_outbound_load_balancer = var.enable_outbound_load_balancer
encryption_key_resource_group_name = var.encryption_key_resource_group_name
Expand Down
7 changes: 0 additions & 7 deletions modules/terraform-cdp-deploy/modules/azure/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -70,13 +70,6 @@ variable "enable_ccm_tunnel" {

}

variable "report_deployment_logs" {
type = bool

description = "Flag to enable reporting of additional diagnostic information back to Cloudera"

}

variable "enable_raz" {
type = bool

Expand Down
5 changes: 2 additions & 3 deletions modules/terraform-cdp-deploy/modules/gcp/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -59,9 +59,8 @@ resource "cdp_environments_gcp_environment" "cdp_env" {
recipes = var.freeipa_recipes
}

workload_analytics = var.workload_analytics
report_deployment_logs = var.report_deployment_logs
enable_tunnel = var.enable_ccm_tunnel
workload_analytics = var.workload_analytics
enable_tunnel = var.enable_ccm_tunnel

polling_options = {
async = var.environment_async_creation
Expand Down
7 changes: 0 additions & 7 deletions modules/terraform-cdp-deploy/modules/gcp/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -64,13 +64,6 @@ variable "enable_ccm_tunnel" {

}

variable "report_deployment_logs" {
type = bool

description = "Flag to enable reporting of additional diagnostic information back to Cloudera"

}

variable "enable_raz" {
type = bool

Expand Down
9 changes: 0 additions & 9 deletions modules/terraform-cdp-deploy/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -215,15 +215,6 @@ variable "workload_analytics" {
default = true
}

variable "report_deployment_logs" {
type = bool

description = "Flag to enable reporting of additional diagnostic information back to Cloudera."

default = null

}

variable "datalake_scale" {
type = string

Expand Down

0 comments on commit bfed051

Please sign in to comment.