Skip to content
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.

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

Remove report_deployment_logs variable from deploy module #86

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
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