Skip to content

Commit 68e73f6

Browse files
committed
terraform/OCI: Use "output yaml" for the memory_in_gbs setting
Refactor: Remove the line in terraform.Makefile that sets "terraform_oci_instance_flex_memory_in_gbs" and just pass it directly from Kconfig via an "output yaml" directive. As an additional clean up, update the help text. Reviewed-by: Luis Chamberlain <[email protected]> Reviewed-by: Chandan Babu R <[email protected]> Signed-off-by: Chuck Lever <[email protected]>
1 parent 187547d commit 68e73f6

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

scripts/terraform.Makefile

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,6 @@ KDEVOPS_MRPROPER += $(KDEVOPS_NODES)
4848
DEFAULT_DEPS_REQS_EXTRA_VARS += $(KDEVOPS_TFVARS)
4949

5050
ifeq (y,$(CONFIG_TERRAFORM_OCI))
51-
TERRAFORM_EXTRA_VARS += terraform_oci_instance_flex_memory_in_gbs=$(subst ",,$(CONFIG_TERRAFORM_OCI_INSTANCE_FLEX_MEMORY_IN_GBS))
5251
TERRAFORM_EXTRA_VARS += terraform_oci_os_image_ocid=$(subst ",,$(CONFIG_TERRAFORM_OCI_OS_IMAGE_OCID))
5352
ifeq (y, $(CONFIG_TERRAFORM_OCI_ASSIGN_PUBLIC_IP))
5453
TERRAFORM_EXTRA_VARS += terraform_oci_assign_public_ip=true

terraform/oci/kconfigs/shapes/Kconfig.flex

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,10 @@ config TERRAFORM_OCI_INSTANCE_FLEX_OCPUS
9191
and AMD) and OCI Ampere Compute processors.
9292

9393
config TERRAFORM_OCI_INSTANCE_FLEX_MEMORY_IN_GBS
94-
int "How much RAM to use per flexible instance in GB"
94+
int "Instance memory size"
95+
output yaml
9596
default 4
9697
help
97-
How much GiB of RAM to use per flexible instance.
98+
Memory per instance, in GiBs. The minimum value for this
99+
setting is a multiple of the number of OCPUS in each
100+
instance.

0 commit comments

Comments
 (0)