imporve(mongodb): update the module to the format of the new version.

Author: lexwong

examples/main.tf

@@ -5,7 +5,7 @@ module "mongodb_instance" {
   db_instance_storage = 10
   security_ip_list    = ["10.168.1.12", "100.69.7.112"]
   name                = "my-mongodb"
-  vswitch_id          = "${module.module_vpc.vswitch_ids}"
+  vswitch_id          = module.module_vpc.vswitch_ids
 }
 
 module "module_vpc" {
@@ -19,3 +19,4 @@ module "module_vpc" {
 data "alicloud_zones" "default" {
   available_resource_creation = "MongoDB"
 }
+

examples/output.tf

@@ -1,19 +1,20 @@
 output "this_mongodb_instance_id" {
-  value = "${module.mongodb_instance.this_db_instance_id}"
+  value = module.mongodb_instance.this_db_instance_id
 }
 
 output "this_mongodb_instance_name" {
-  value = "${module.mongodb_instance.this_db_instance_name}"
+  value = module.mongodb_instance.this_db_instance_name
 }
 
 output "this_mongodb_instance_zone_id" {
-  value = "${module.mongodb_instance.this_db_instance_zone_id}"
+  value = module.mongodb_instance.this_db_instance_zone_id
 }
 
 output "this_mongodb_instance_vswitch_id" {
-  value = "${module.mongodb_instance.this_db_instance_vswitch_id}"
+  value = module.mongodb_instance.this_db_instance_vswitch_id
 }
 
 output "this_mongodb_instance_security_ip_list" {
-  value = "${module.mongodb_instance.this_db_instance_security_ip_list}"
+  value = module.mongodb_instance.this_db_instance_security_ip_list
 }
+

examples/versions.tf

@@ -0,0 +1,4 @@
+
+terraform {
+  required_version = ">= 0.12"
+}

main.tf

@@ -1,14 +1,21 @@
+provider "alicloud" {
+  version              = ">=1.56.0"
+  region               = var.region != "" ? var.region : null
+  configuration_source = "terraform-alicloud-modules/mongodb"
+}
+
 resource "alicloud_mongodb_instance" "this_instance" {
-  engine_version       = "${var.engine_version}"
-  name                 = "${var.name}"
-  instance_charge_type = "${var.instance_charge_type}"
-  db_instance_class    = "${var.db_instance_class}"
-  db_instance_storage  = "${var.db_instance_storage}"
-  period               = "${var.period}"
-  security_ip_list     = "${var.security_ip_list}"
-  replication_factor   = "${var.replication_factor}"
-  storage_engine       = "${var.storage_engine}"
-  vswitch_id           = "${var.vswitch_id}"
-  zone_id              = "${var.zone_id}"
-  account_password     = "${var.account_password}"
+  engine_version       = var.engine_version
+  name                 = var.name
+  instance_charge_type = var.instance_charge_type
+  db_instance_class    = var.db_instance_class
+  db_instance_storage  = var.db_instance_storage
+  period               = var.period
+  security_ip_list     = var.security_ip_list
+  replication_factor   = var.replication_factor
+  storage_engine       = var.storage_engine
+  vswitch_id           = var.vswitch_id
+  zone_id              = var.zone_id
+  account_password     = var.account_password
 }
+

outputs.tf

@@ -1,19 +1,20 @@
 output "this_db_instance_id" {
-  value = "${alicloud_mongodb_instance.this_instance.id}"
+  value = alicloud_mongodb_instance.this_instance.id
 }
 
 output "this_db_instance_name" {
-  value = "${alicloud_mongodb_instance.this_instance.name}"
+  value = alicloud_mongodb_instance.this_instance.name
 }
 
 output "this_db_instance_zone_id" {
-  value = "${alicloud_mongodb_instance.this_instance.zone_id}"
+  value = alicloud_mongodb_instance.this_instance.zone_id
 }
 
 output "this_db_instance_vswitch_id" {
-  value = "${alicloud_mongodb_instance.this_instance.vswitch_id}"
+  value = alicloud_mongodb_instance.this_instance.vswitch_id
 }
 
 output "this_db_instance_security_ip_list" {
-  value = "${alicloud_mongodb_instance.this_instance.security_ip_list}"
+  value = alicloud_mongodb_instance.this_instance.security_ip_list
 }
+

variables.tf

@@ -2,6 +2,11 @@
 #variables for alicloud_mongodb_instance
 ##############################################################
 
+variable "region" {
+  description = "The region used to launch this module resources."
+  default     = ""
+}
+
 variable "engine_version" {
   description = "The engine version to use"
 }
@@ -30,12 +35,12 @@ variable "instance_charge_type" {
 
 variable "period" {
   description = "The duration that you will buy DB instance (in month). It is valid when instance_charge_type is PrePaid. Valid values: [1~9], 12, 24, 36. Default to 1"
-  default=1
+  default     = 1
 }
 
 variable "zone_id" {
   description = "The Zone to launch the DB instance. "
-  default=""
+  default     = ""
 }
 
 variable "vswitch_id" {
@@ -44,7 +49,7 @@ variable "vswitch_id" {
 
 variable "security_ip_list" {
   description = " List of IP addresses allowed to access all databases of an instance. The list contains up to 1,000 IP addresses, separated by commas. Supported formats include 0.0.0.0/0, 10.23.12.24 (IP), and 10.23.12.24/24 (Classless Inter-Domain Routing (CIDR) mode. /24 represents the length of the prefix in an IP address. The range of the prefix length is [1,32])."
-  type        = "list"
+  type        = list(string)
   default     = []
 }
 
@@ -57,3 +62,4 @@ variable "replication_factor" {
   description = "Number of replica set nodes. Valid values: [3,5,7]"
   default     = 3
 }
+

versions.tf

@@ -0,0 +1,4 @@
+
+terraform {
+  required_version = ">= 0.12"
+}