Tested with ARM (#31)

* tested with ARM, changed default shape to E4.Flex, added ability to control state of operator host

* added deprecation notice in changelog

* Update variables.tf

Changed default shape in variables to E4

Author: hyder

CHANGELOG.adoc

@@ -7,6 +7,20 @@ All notable changes to this project are documented in this file.
 
 The format is based on {uri-changelog}[Keep a Changelog].
 
+= Unreleased
+
+== New features
+* New variable (`operator_state`) to specify state of operator host
+
+== Changes
+* Set default shape to E4.Flex
+
+== Deprecation notice
+
+The following variables will be renamed at the next major release of this module:
+
+* var.operator_enabled --> var.create_operator
+
 == v2.1.0 (March 22,2021)
 * Upgrade default version of operator host to Oracle Linux 8 (#26)
 

compute.tf

@@ -46,12 +46,13 @@ resource "oci_core_instance" "operator" {
     }
   }
 
-
   source_details {
     source_type = "image"
     source_id   = local.operator_image_id
   }
 
+  state = var.operator_state
+
   timeouts {
     create = "60m"
   }

docs/terraformoptions.adoc

@@ -141,17 +141,23 @@ Ensure you review the {uri-terraform-dependencies}[dependencies].
 |`operator_shape`
 |The shape of operator instance. This is now specified as a map and supports E3.Flex. If a non-Flex shape is specified, then the other parameters are ignored.
 |e.g. `operator_shape = {
-  shape="VM.Standard.E3.Flex",
+  shape="VM.Standard.E4.Flex",
   ocpus=1,
   memory=4,
   boot_volume_size=50
 }`
 |`operator_shape = {
-  shape="VM.Standard.E3.Flex",
+  shape="VM.Standard.E4.Flex",
   ocpus=1,
   memory=4,
   boot_volume_size=50
 }`
+
+|`operator_state`
+|The target state for the instance. Could be set to RUNNING or STOPPED. (Updatable)
+|RUNNING|STOPPED
+|RUNNING
+
 |`operator_upgrade`
 |Whether to upgrade the operator host packages after provisioning. It's useful to set this to false during development/testing so the operator is provisioned faster.
 |true/false

terraform.tfvars.example

@@ -33,6 +33,8 @@ nsg_ids = []
 
 vcn_id = ""
 
+operating_system_version = "8"
+
 operator_enabled = true
 
 operator_image_id = "Oracle"
@@ -41,13 +43,13 @@ operator_instance_principal = true
 
 operator_shape = {
   # shape = "VM.Standard.E2.2"
-  shape="VM.Standard.E3.Flex",
+  shape="VM.Standard.E4.Flex",
   ocpus=1,
   memory=4,
   boot_volume_size=50
 }
 
-operating_system_version = "8"
+operator_state= "RUNNING"
 
 operator_upgrade = false
 

variables.tf

@@ -83,14 +83,21 @@ variable "vcn_id" {
 
 # operator host parameters
 
+variable "operating_system_version" {
+  description = "The version of the Oracle Linux to use."
+  default     = "8"
+  type        = string
+}
+
 variable "operator_enabled" {
+  #! Deprecation notice: will be renamed to create_operator at next major release
   description = "whether to create the operator"
   default     = false
   type        = bool
 }
 
 variable "operator_image_id" {
-  description = "Provide a custom image id for the operator host or leave as Autonomous."
+  description = "Provide a custom image id for the operator host or leave as Oracle."
   default     = "Oracle"
   type        = string
 }
@@ -103,17 +110,18 @@ variable "operator_instance_principal" {
 
 variable "operator_shape" {
   description = "The shape of the operator instance."
-  default     = {
-   shape = "VM.Standard.E3.Flex", ocpus = 1, memory = 4, boot_volume_size = 50
+  default = {
+    shape = "VM.Standard.E4.Flex", ocpus = 1, memory = 4, boot_volume_size = 50
   }
-  type        = map(any)
+  type = map(any)
 }
 
-variable "operating_system_version" {
-  description = "The version of the Oracle Linux to use."
-  default     = "8"
+variable "operator_state" {
+  description = "The target state for the instance. Could be set to RUNNING or STOPPED. (Updatable)"
+  default     = "RUNNING"
   type        = string
 }
+
 variable "operator_upgrade" {
   description = "Whether to upgrade the operator host packages after provisioning. It's useful to set this to false during development/testing so the operator is provisioned faster."
   default     = false