From 23c6af45cea19e1769b203cb5111b3fc1735722b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Conall=20=C3=93=20Cofaigh?= <ocofaigh@ie.ibm.com>
Date: Wed, 27 Sep 2023 20:23:09 +0100
Subject: [PATCH] fix: update the ssh key related variable descriptions for the
 OCP pattern (#586)

---
 ibm_catalog.json                  | 4 ++++
 patterns/roks/module/variables.tf | 4 ++--
 patterns/roks/variables.tf        | 4 ++--
 3 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/ibm_catalog.json b/ibm_catalog.json
index 04e9040a0..99c752de0 100644
--- a/ibm_catalog.json
+++ b/ibm_catalog.json
@@ -1002,6 +1002,10 @@
                             "hidden": true,
                             "key": "ssh_public_key"
                         },
+                        {
+                            "hidden": true,
+                            "key": "existing_ssh_key_name"
+                        },
                         {
                             "custom_config": {
                                 "config_constraints": {
diff --git a/patterns/roks/module/variables.tf b/patterns/roks/module/variables.tf
index 1a9a05c5e..789f12fbe 100644
--- a/patterns/roks/module/variables.tf
+++ b/patterns/roks/module/variables.tf
@@ -207,7 +207,7 @@ variable "vpn_firewall_type" {
 }
 
 variable "ssh_public_key" {
-  description = "Public SSH Key. Must be an RSA key with a key size of either 2048 bits or 4096 bits (recommended) - See https://cloud.ibm.com/docs/vpc?topic=vpc-ssh-keys. Must not already exists in the deployment region. Use only if provisioning F5 or Bastion Host."
+  description = "A public SSH key that does not exist in the deployment region. Used only if you provision F5 or Bastion Host. Must be an RSA key with a key size of either 2048 or 4096 bits (recommended). See https://cloud.ibm.com/docs/vpc?topic=vpc-ssh-keys. To use an existing key, specify a value in the `existing_ssh_key_name` variable instead."
   type        = string
   default     = null
   validation {
@@ -217,7 +217,7 @@ variable "ssh_public_key" {
 }
 
 variable "existing_ssh_key_name" {
-  description = "The name of the public ssh key which already exists."
+  description = "The name of a public SSH key that exists in the deployment region. Used only if you provision F5 or Bastion Host. To add a SSH key, use the `ssh_public_key` variable instead."
   type        = string
   default     = null
 }
diff --git a/patterns/roks/variables.tf b/patterns/roks/variables.tf
index bb9be1f9d..a473746ef 100644
--- a/patterns/roks/variables.tf
+++ b/patterns/roks/variables.tf
@@ -221,7 +221,7 @@ variable "vpn_firewall_type" {
 }
 
 variable "ssh_public_key" {
-  description = "Public SSH Key. Must be an RSA key with a key size of either 2048 bits or 4096 bits (recommended) - See https://cloud.ibm.com/docs/vpc?topic=vpc-ssh-keys. Must not already exists in the deployment region. Use only if provisioning F5 or Bastion Host."
+  description = "A public SSH key that does not exist in the deployment region. Used only if you provision F5 or Bastion Host. Must be an RSA key with a key size of either 2048 or 4096 bits (recommended). See https://cloud.ibm.com/docs/vpc?topic=vpc-ssh-keys. To use an existing key, specify a value in the `existing_ssh_key_name` variable instead."
   type        = string
   default     = null
   validation {
@@ -231,7 +231,7 @@ variable "ssh_public_key" {
 }
 
 variable "existing_ssh_key_name" {
-  description = "The name of the public ssh key which already exists."
+  description = "The name of a public SSH key that exists in the deployment region. Used only if you provision F5 or Bastion Host. To add a SSH key, use the `ssh_public_key` variable instead."
   type        = string
   default     = null
 }