pivotal-cf / terraforming-azure

use terraform, deploy yourself a pcf
Apache License 2.0
34 stars 67 forks source link

vm_admin_password variable is unnecessary #12

Closed christianang closed 6 years ago

christianang commented 6 years ago

In vm.tf#L69 the admin_password is set to var.vm_admin_password, but this is unnecessary since password authentication is disabled at vm.tf#L73 and you provide a key to ssh onto the vm.

This var can be removed.

cf-gitbot commented 6 years ago

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

rowanjacobs commented 6 years ago

Thanks for pointing this out.