Merge pull request #2 from clouddrove/CD-147

fix labels managedby variables

Author: anmolnagpal

README.md

@@ -37,7 +37,7 @@ usage : |-
   ### Basic Example
   ```hcl
     module "api-gateway" {
-      source                          = "git::https://github.com/clouddrove/terraform-aws-api-gateway.git?ref=tags/0.12.0"
+      source                          = "git::https://github.com/clouddrove/terraform-aws-api-gateway.git?ref=tags/0.12.2"
       name        = "api-gateway"
       application = "clouddrove"
       environment = "test"
@@ -95,7 +95,7 @@ usage : |-
   ### Complete Example
   ```hcl
     module "api-gateway" {
-      source                          = "git::https://github.com/clouddrove/terraform-aws-api-gateway.git?ref=tags/0.12.0"
+      source                          = "git::https://github.com/clouddrove/terraform-aws-api-gateway.git?ref=tags/0.12.2"
       name        = "api-gateway"
       application = "clouddrove"
       environment = "test"

README.yaml

@@ -3,7 +3,7 @@ provider "aws" {
 }
 
 module "api-gateway" {
-  source      = "git::https://github.com/clouddrove/terraform-aws-api-gateway.git?ref=tags/0.12.0"
+  source      = "../../"
   name        = "api-gateway"
   application = "clouddrove"
   environment = "test"

_example/basic/example.tf

@@ -3,7 +3,7 @@ provider "aws" {
 }
 
 module "api-gateway" {
-  source = "git::https://github.com/clouddrove/terraform-aws-api-gateway.git?ref=tags/0.12.0"
+  source = "../../"
 
   name        = "api-gateway"
   application = "clouddrove"

_example/complete/example.tf

@@ -11,6 +11,7 @@ module "labels" {
   name        = var.name
   application = var.application
   environment = var.environment
+  managedby   = var.managedby
   label_order = var.label_order
 }
 
@@ -144,7 +145,7 @@ resource "aws_api_gateway_method_response" "options_200" {
     "method.response.header.Access-Control-Allow-Credentials" = true
   }
 
-  depends_on = ["aws_api_gateway_method.options_method"]
+  depends_on = [aws_api_gateway_method.options_method]
 }
 
 resource "aws_api_gateway_integration" "options_integration" {
@@ -156,7 +157,7 @@ resource "aws_api_gateway_integration" "options_integration" {
   type             = "MOCK"
   content_handling = "CONVERT_TO_TEXT"
 
-  depends_on = ["aws_api_gateway_method.options_method"]
+  depends_on = [aws_api_gateway_method.options_method]
 }
 
 resource "aws_api_gateway_integration_response" "options_integration_response" {
@@ -173,8 +174,8 @@ resource "aws_api_gateway_integration_response" "options_integration_response" {
   }
 
   depends_on = [
-    "aws_api_gateway_method_response.options_200",
-    "aws_api_gateway_integration.options_integration",
+    aws_api_gateway_method_response.options_200,
+    aws_api_gateway_integration.options_integration,
   ]
 }
 

main.tf

@@ -42,6 +42,12 @@ variable "tags" {
   description = "Additional tags (e.g. map(`BusinessUnit`,`XYZ`)."
 }
 
+variable "managedby" {
+  type        = string
+  default     = "anmol@clouddrove.com"
+  description = "ManagedBy, eg 'CloudDrove' or 'AnmolNagpal'."
+}
+
 # Module      : Api Gateway
 # Description : Terraform Api Gateway module variables.
 variable "enabled" {