feat: initialize work on issue #55

kral2 · kral2 · commit 2b814a14d469 · 2021-09-20T14:35:16.000+02:00
diff --git a/CHANGELOG.adoc b/CHANGELOG.adoc
@@ -25,6 +25,9 @@ Given a version number MAJOR.MINOR.PATCH:
 * Add support for freeform and defined tags for instances, vnics and block volumes (Fix #10, #11, #12, #13, #18, #20)
 * Add "module watermark" freeform tags: module defined and user defined freeform tags are merged on the final resource
 * Add support to provide the `ssh_authorized_keys` argument as a string or as a file (Fix #67 #70)
+* Provision instances with reserved Public IP
+* [ ] Define a backup policy for boot volume and additional block volumes
+* Add new outputs for each provisioned resources: "all_attributes" outputs have full provider coverage and are auto-updating.
 
 === Documentation
 
@@ -47,7 +50,6 @@ Given a version number MAJOR.MINOR.PATCH:
 * Outputs produces unnecessarily multidimensional objects (Issue #31)
 * Repo maintenance:
 ** add .gitattributes for consistent line ending and tab
-** add pre-commit configuration file
 
 == 2.0.4 - 2021-02-13
 
diff --git a/README.md b/README.md
@@ -1,6 +1,6 @@
 # Oracle Cloud Infrastructure Terraform Module for Compute Instance
 
-This Module provides an easy way to launch compute instances with advanced settings and good practices embedded.
+This module provides an easy way to launch compute instances with advanced settings and good practices embedded.
 
 On top of the compute instance capabilities, this module can also provision and attach additional Block Volumes to the instances.
 
@@ -10,24 +10,20 @@ On top of the compute instance capabilities, this module can also provision and
 >
 > Oracle recommends that you do not use custom images without these rules unless you understand the security risks. See [Compute Best Practices](https://docs.cloud.oracle.com/iaas/Content/Compute/References/bestpracticescompute.htm#two) for recommendations on how to manage instances.
 
-## Maintainers
-
-This module is maintained by Oracle.
-
 ## Requirements
 
 The diagram below summarizes the required components and their respective versions to use this module.
 
 ![versions](https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/blob/main/docs/diagrams/versions.svg?raw=true&sanitize=true)
 
-To enforce versions compatibility of both Terraform and the OCI provider, your root configuration should ideally include this block in `main.tf` for version pinning:
+To enforce versions compatibility of both Terraform and the OCI provider, your root configuration should ideally include this block for version pinning:
 
 ```HCL
 terraform {
-  required_version = ">= 0.12"
+  required_version = ">= 0.12.6"
   required_providers {
     oci = {
-      version = ">= 3.27"
+      version = ">= 4.0.0"
     }
   }
 }
@@ -59,11 +55,11 @@ module "instance" {
 
 ## What's coming next for this module?
 
-The current focus is to get back in close the gap between this module and the provider's capabilities. We started with a complete code base update for [HCL2 syntax compatibility](https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/releases/tag/v2.0.2), then adding support for [Regional Subnets](https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/releases/tag/v2.0.4) and now [Flexible Shapes](https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/pull/49).
+The current focus is to close the gap between this module and the provider's capabilities. We started with a complete codebase update for [HCL2 syntax compatibility](https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/releases/tag/v2.0.2), then adding support for [Regional Subnets](https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/releases/tag/v2.0.4) and lastly [Flexible Shapes](https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/pull/49).
 
 We will continue to push in that direction with the goal of [feature parity with the provider's capabilities](https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/projects/4), as well as adding more features and integration points with other OCI services: Block Volume Backups, Secondary VNICs and IPs, etc ...
 
-Compute Instances are also a perfect place to illustrate [module composition principles](https://www.terraform.io/docs/language/modules/develop/composition.html) reusing the other existing official Terraform OCI Modules
+Given the dependency to Network and Storage for Compute Instances,it is a perfect place to illustrate [module composition principles](https://www.terraform.io/docs/language/modules/develop/composition.html) and how to reuse the other official Terraform OCI modules.
 
 ## Configuring iSCSI volume attachments
 
@@ -73,7 +69,8 @@ Compute Instances are also a perfect place to illustrate [module composition pri
 
 ## Contributing
 
-This project is open source. Oracle appreciates any contributions that are made by the open source community: raising issues, improving documentation, fixing bugs, or adding new features.
+This project is open source and maintained by Oracle.  
+Oracle appreciates any contributions that are made by the open source community: raising issues, improving documentation, fixing bugs, or adding new features.
 
 Learn how to [contribute](https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/blob/main/CONTRIBUTING.adoc).
 
diff --git a/examples/instances_fixed_shape/README.md b/examples/instances_fixed_shape/README.md
@@ -13,7 +13,8 @@ You will need to collect the following information before you start:
 
 1. your OCI provider authentication values
 2. a compartment OCID in which the instances will be created
-3. a subnet OCID to which the instance's primary VNICs will be attached
+3. a source OCID to deploy the instance, usually an image ocid from [OCI Platform Images list]
+4. a subnet OCID to which the instance's primary VNICs will be attached
 
 For detailed instructions, see [docs/prerequisites.adoc]
 
@@ -33,3 +34,5 @@ Then apply the example using the following commands:
 
 [Terraform Variable Definition file]:https://www.terraform.io/docs/language/values/variables.html#variable-definitions-tfvars-files
 [docs/prerequisites.adoc]:https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/blob/main/docs/prerequisites.adoc
+[Provisioning Infrastructure with Terraform]:https://www.terraform.io/docs/cli/run/index.html
+[OCI Platform Images list]:https://docs.oracle.com/en-us/iaas/images/
diff --git a/examples/instances_fixed_shape/main.tf b/examples/instances_fixed_shape/main.tf
@@ -1,10 +1,10 @@
 // Copyright (c) 2018, 2021 Oracle and/or its affiliates.
 
 terraform {
-  required_version = ">= 0.12" // terraform version below 0.12 is not tested/supported with this module
+  required_version = ">= 0.13" // terraform version below 0.12 is not tested/supported with this module
   required_providers {
     oci = {
-      version = ">= 3.27" // force downloading oci-provider compatible with terraform v0.12
+      version = ">= 4.0.0" // force downloading oci-provider compatible with terraform v0.12
     }
   }
 }
diff --git a/examples/instances_flex_shape/README.md b/examples/instances_flex_shape/README.md
@@ -13,7 +13,8 @@ You will need to collect the following information before you start:
 
 1. your OCI provider authentication values
 2. a compartment OCID in which the instances will be created
-3. a subnet OCID to which the instance's primary VNICs will be attached
+3. a source OCID to deploy the instance, usually an image ocid from [OCI Platform Images list]
+4. a subnet OCID to which the instance's primary VNICs will be attached
 
 For detailed instructions, see [docs/prerequisites.adoc]
 
@@ -33,3 +34,5 @@ Then apply the example using the following commands:
 
 [Terraform Variable Definition file]:https://www.terraform.io/docs/language/values/variables.html#variable-definitions-tfvars-files
 [docs/prerequisites.adoc]:https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/blob/main/docs/prerequisites.adoc
+[Provisioning Infrastructure with Terraform]:https://www.terraform.io/docs/cli/run/index.html
+[OCI Platform Images list]:https://docs.oracle.com/en-us/iaas/images/
diff --git a/examples/instances_flex_shape/main.tf b/examples/instances_flex_shape/main.tf
@@ -1,10 +1,10 @@
 // Copyright (c) 2018, 2021 Oracle and/or its affiliates.
 
 terraform {
-  required_version = ">= 0.12" // terraform version below 0.12 is not tested/supported with this module
+  required_version = ">= 0.13" // terraform version below 0.12 is not tested/supported with this module
   required_providers {
     oci = {
-      version = ">= 3.27" // force downloading oci-provider compatible with terraform v0.12
+      version = ">= 4.0.0" // force downloading oci-provider compatible with terraform v0.12
     }
   }
 }
diff --git a/examples/instances_reserved_public_ip/README.md b/examples/instances_reserved_public_ip/README.md
@@ -0,0 +1,39 @@
+# Creating Compute Instances using Flex shape
+
+This example illustrates how to use this module to creates compute instances with a reserved public IP.
+
+One modules will be configured:
+
+- 1 instance (1 OCPU, 1GB RAM) with a reserved public IP attached the primary VNIC.
+
+## Prerequisites
+
+You will need to collect the following information before you start:
+
+1. your OCI provider authentication values
+2. a compartment OCID in which the instances will be created
+3. a source OCID to deploy the instance, usually an image ocid from [OCI Platform Images list]
+4. a subnet OCID to which the instance's primary VNICs will be attached
+
+For detailed instructions, see [docs/prerequisites.adoc]
+
+## Using this example with Terraform cli
+
+Prepare one [Terraform Variable Definition file] named `terraform.tfvars` with the required authentication information.
+
+*TIP: You can rename and configure `terraform.tfvars.example` from this example's folder.*
+
+Then apply the example using the following commands:
+
+```shell
+> terraform init
+> terraform plan
+> terraform apply
+```
+
+See [Provisioning Infrastructure with Terraform] for more details about Terraform CLI and the available subcommands.
+
+[Terraform Variable Definition file]:https://www.terraform.io/docs/language/values/variables.html#variable-definitions-tfvars-files
+[docs/prerequisites.adoc]:https://github.com/oracle-terraform-modules/terraform-oci-compute-instance/blob/main/docs/prerequisites.adoc
+[Provisioning Infrastructure with Terraform]:https://www.terraform.io/docs/cli/run/index.html
+[OCI Platform Images list]:https://docs.oracle.com/en-us/iaas/images/
diff --git a/examples/instances_reserved_public_ip/main.tf b/examples/instances_reserved_public_ip/main.tf
@@ -0,0 +1,51 @@
+// Copyright (c) 2018, 2021 Oracle and/or its affiliates.
+
+terraform {
+  required_version = ">= 0.13" // terraform version below 0.12 is not tested/supported with this module
+  required_providers {
+    oci = {
+      version = ">= 4.0.0" // force downloading oci-provider compatible with terraform v0.12
+    }
+  }
+}
+
+provider "oci" {
+  tenancy_ocid     = var.tenancy_ocid
+  user_ocid        = var.user_ocid
+  fingerprint      = var.fingerprint
+  private_key_path = var.private_key_path
+  region           = var.region
+}
+
+# # * This module will create 1 Flex Compute Instances, with a reserved public IP
+module "instance_reserved_ip" {
+  source = "../../"
+  # source = "oracle-terraform-modules/compute-instance/oci"
+  # general oci parameters
+  compartment_ocid = var.compartment_ocid
+  freeform_tags    = var.freeform_tags
+  defined_tags     = var.defined_tags
+  # compute instance parameters
+  ad_number                   = null
+  instance_count              = 1
+  instance_display_name       = "instance_reserved_ip"
+  shape                       = var.shape
+  source_ocid                 = var.source_ocid
+  source_type                 = var.source_type
+  instance_flex_memory_in_gbs = 1 # only used if shape is Flex type
+  instance_flex_ocpus         = 1 # only used if shape is Flex type
+  # operating system parameters
+  ssh_authorized_keys = var.ssh_authorized_keys
+  # networking parameters
+  assign_public_ip = var.assign_public_ip
+  public_ip        = var.public_ip
+  subnet_ocids     = var.subnet_ocids
+  # storage parameters
+  block_storage_sizes_in_gbs = [] # no block volume will be created
+  preserve_boot_volume       = false
+}
+
+output "instance_reserved_ip" {
+  description = "ocid of created instances."
+  value       = module.instance_reserved_ip.instances_summary
+}
diff --git a/examples/instances_reserved_public_ip/terraform.tfvars.example b/examples/instances_reserved_public_ip/terraform.tfvars.example
@@ -0,0 +1,34 @@
+# Copyright (c) 2019, 2020 Oracle Corporation and/or affiliates.  All rights reserved.
+# Licensed under the Universal Permissive License v 1.0 as shown at https://oss.oracle.com/licenses/upl
+
+/*----------------------------------------------------------------------------
+HOW TO USE THIS FILE
+
+1. keep this file in the same folder as your terraform *.tf files
+2. If your terraform config is managed with Git, add the tfvars file to your .gitignore.
+3. Keep your RSA private key outside of your terraform work folder!
+----------------------------------------------------------------------------*/
+
+# provider identity parameters
+
+tenancy_ocid     = "<tenancy OCID>"
+user_ocid        = "<user OCID>"
+fingerprint      = "<PEM key fingerprint>"
+region           = "<region in which to operate, example: us-ashburn-1, eu-frankfurt-1>"
+private_key_path = "<path to the private key that matches the fingerprint above>"
+
+# general oci parameters
+
+compartment_ocid = "<compartment OCID>"
+
+# compute instance parameters
+
+source_ocid  = "<The OCID of an image or a boot volume>"
+
+# operating system parameters
+
+ssh_authorized_keys = "<path to the instance's public key>"
+
+# networking parameters
+
+subnet_ocids = ["<a list of the subnet OCIDs which to create the VNICs in>"]
diff --git a/examples/instances_reserved_public_ip/variables.tf b/examples/instances_reserved_public_ip/variables.tf
diff --git a/main.tf b/main.tf
diff --git a/outputs.tf b/outputs.tf
diff --git a/variables.tf b/variables.tf

Original file line number	Diff line number	Diff line change
`@@ -1,10 +1,10 @@`
`1`	`1`	`// Copyright (c) 2018, 2021 Oracle and/or its affiliates.`
`2`	`2`
`3`	`3`	`terraform {`
`4`		`- required_version = ">= 0.12" // terraform version below 0.12 is not tested/supported with this module`
	`4`	`+ required_version = ">= 0.13" // terraform version below 0.12 is not tested/supported with this module`
`5`	`5`	`required_providers {`
`6`	`6`	`oci = {`
`7`		`- version = ">= 3.27" // force downloading oci-provider compatible with terraform v0.12`
	`7`	`+ version = ">= 4.0.0" // force downloading oci-provider compatible with terraform v0.12`
`8`	`8`	`}`
`9`	`9`	`}`
`10`	`10`	`}`