Skip to content

Commit b7495a8

Browse files
committed
terraform/OCI: Use "output yaml" for the subnet_ocid setting
Refactor: Remove the line in terraform.Makefile that sets "terraform_oci_subnet_ocid" 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 f712659 commit b7495a8

File tree

2 files changed

+8
-7
lines changed

2 files changed

+8
-7
lines changed

scripts/terraform.Makefile

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -47,10 +47,6 @@ KDEVOPS_MRPROPER += $(KDEVOPS_NODES)
4747

4848
DEFAULT_DEPS_REQS_EXTRA_VARS += $(KDEVOPS_TFVARS)
4949

50-
ifeq (y,$(CONFIG_TERRAFORM_OCI))
51-
TERRAFORM_EXTRA_VARS += terraform_oci_subnet_ocid=$(subst ",,$(CONFIG_TERRAFORM_OCI_SUBNET_OCID))
52-
endif
53-
5450
ifeq (y,$(CONFIG_TERRAFORM_OPENSTACK))
5551
TERRAFORM_EXTRA_VARS += terraform_openstack_cloud_name=$(subst ",,$(CONFIG_TERRAFORM_TERRAFORM_OPENSTACK_CLOUD_NAME))
5652
TERRAFORM_EXTRA_VARS += terraform_openstack_instance_prefix=$(subst ",,$(CONFIG_TERRAFORM_TERRAFORM_OPENSTACK_INSTANCE_PREFIX))

terraform/oci/kconfigs/Kconfig.network

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,12 @@ config TERRAFORM_OCI_ASSIGN_PUBLIC_IP
99

1010
config TERRAFORM_OCI_SUBNET_OCID
1111
string "OCI Subnet OCID"
12+
output yaml
1213
help
13-
The OCI ID of the subnet.
14-
Read this:
15-
https://docs.oracle.com/en-us/iaas/Content/API/SDKDocs/terraformproviderconfiguration.htm
14+
If there is a long-lived VNC and subnet already available
15+
to which to attach kdevops compute instances, specify the
16+
OCID of this subnet via this option.
17+
18+
kdevops does not manage this resource. Before running
19+
"make bringup", the subnet must already exist and your OCI
20+
user must have permission to attach to it.

0 commit comments

Comments
 (0)