ibm-cloud-architecture / terraform-icp-vmware

Terraform recipe to deploy IBM Cloud Private on VMware
10 stars 15 forks source link

icp-deploy.tf uses the old vulnerability advisor configuration in disabled management service section #10

Closed gangchen03 closed 5 years ago

gangchen03 commented 6 years ago

The latest configuration for ICP changed the variable to "vulnerability-advisor" This will cause the installation ignore the fact we want to disable vulnerability advisor. Suggest to change icp-deploy.tf line 57 from

    "disabled_management_services"    = [ "${var.va["nodes"] == 0 ? "va" : "" }" ]

To:

    "disabled_management_services"    = [ "${var.va["nodes"] == 0 ? "vulnerability-advisor" : "" }" ]
jesusmah commented 5 years ago

I believe this might be addressed in one of the commits for a pending PR: https://github.com/ibm-cloud-architecture/terraform-icp-vmware/pull/19/commits/f7c78c8d06b4b7f07d16646350e634d5ce3c5f3b

hassenius commented 5 years ago

Fixed by @jesusmah