Fixes for reference-architectures (#68)

* Add missing makefile

* Make SMTP credentials optional

* use simpler makefile pattern

* Standardize interface to be compatible with other modules

* remove debug typo

Author: osterman

aws/organization/Makefile

@@ -0,0 +1,8 @@
+init:
+	init-terraform
+
+%:
+	terraform $@
+
+clean:
+	rm -rf .terraform

aws/tfstate-backend/Makefile

@@ -1,6 +1,8 @@
-## Initialize the configuration (should only be run once)
+## Initialize the terraform state backend
 init:
-	@scripts/$@.sh
+	@aws s3 ls s3://${TF_BUCKET}/tfstate-backend/terraform.tfstate > /dev/null || \
+		scripts/$@.sh
+	@[ -d .terraform ] || init-terraform
 
 ## Destroy the configuration (only works if `force_destroy=true`)
 destroy:
@@ -9,3 +11,7 @@ destroy:
 ## Force destroy the bucket (not recommended!)
 force-destroy:
 	@scripts/$@.sh
+
+## Pass target through to terraform
+%:
+	terraform $@

aws/users/Makefile

@@ -1,10 +1,7 @@
 init:
 	init-terraform
 
-plan:
-	terraform $@
-
-apply:
+%:
 	terraform $@
 
 clean:

aws/users/variables.tf

@@ -19,16 +19,19 @@ variable "name" {
 variable "smtp_username" {
   description = "Username to authenticate with the SMTP server"
   type        = "string"
+  default     = ""
 }
 
 variable "smtp_password" {
   description = "Password to authenticate with the SMTP server"
   type        = "string"
+  default     = ""
 }
 
 variable "smtp_host" {
   description = "SMTP Host"
   default     = "smtp.mailgun.org"
+  default     = ""
 }
 
 variable "smtp_port" {