Skip to content

Commit 66c0fa0

Browse files
authored
feat: the default value of ocp_entitlement has been removed as not everyone will have an entitlement that can be used (#454)
1 parent 099b94b commit 66c0fa0

File tree

20 files changed

+60
-7
lines changed

20 files changed

+60
-7
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,7 @@ Optionally, you need the following permissions to attach Access Management tags
258258
| <a name="input_kms_config"></a> [kms\_config](#input\_kms\_config) | Use to attach a KMS instance to the cluster. If account\_id is not provided, defaults to the account in use. | <pre>object({<br> crk_id = string<br> instance_id = string<br> private_endpoint = optional(bool, true) # defaults to true<br> account_id = optional(string) # To attach KMS instance from another account<br> wait_for_apply = optional(bool, true) # defaults to true so terraform will wait until the KMS is applied to the master, ready and deployed<br> })</pre> | `null` | no |
259259
| <a name="input_manage_all_addons"></a> [manage\_all\_addons](#input\_manage\_all\_addons) | Instructs Terraform to manage all cluster addons, even if addons were installed outside of the module. If set to 'true' this module will destroy any addons that were installed by other sources. | `bool` | `false` | no |
260260
| <a name="input_number_of_lbs"></a> [number\_of\_lbs](#input\_number\_of\_lbs) | The number of LBs to associated the additional\_lb\_security\_group\_names security group with. | `number` | `1` | no |
261-
| <a name="input_ocp_entitlement"></a> [ocp\_entitlement](#input\_ocp\_entitlement) | Value that is applied to the entitlements for OCP cluster provisioning | `string` | `"cloud_pak"` | no |
261+
| <a name="input_ocp_entitlement"></a> [ocp\_entitlement](#input\_ocp\_entitlement) | Value that is applied to the entitlements for OCP cluster provisioning | `string` | `null` | no |
262262
| <a name="input_ocp_version"></a> [ocp\_version](#input\_ocp\_version) | The version of the OpenShift cluster that should be provisioned (format 4.x). If no value is specified, the current default version is used. You can also specify `default`. This input is used only during initial cluster provisioning and is ignored for updates. To prevent possible destructive changes, update the cluster version outside of Terraform. | `string` | `null` | no |
263263
| <a name="input_operating_system"></a> [operating\_system](#input\_operating\_system) | The operating system of the workers in the default worker pool. If no value is specified, the current default version OS will be used. See https://cloud.ibm.com/docs/openshift?topic=openshift-openshift_versions#openshift_versions_available . | `string` | `null` | no |
264264
| <a name="input_region"></a> [region](#input\_region) | The IBM Cloud region where the cluster will be provisioned. | `string` | n/a | yes |

examples/add_rules_to_sg/main.tf

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ resource "ibm_is_security_group_rule" "kube_cluster_rules" {
111111
direction = each.value.direction
112112
remote = each.value.remote
113113

114+
114115
dynamic "tcp" {
115116
for_each = each.value.tcp == null ? [] : [each.value]
116117
content {
@@ -174,4 +175,5 @@ module "ocp_base" {
174175
worker_pools = local.worker_pools
175176
ocp_version = var.ocp_version
176177
tags = var.resource_tags
178+
ocp_entitlement = var.ocp_entitlement
177179
}

examples/add_rules_to_sg/variables.tf

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,3 +39,9 @@ variable "ocp_version" {
3939
description = "Version of the OCP cluster to provision"
4040
default = null
4141
}
42+
43+
variable "ocp_entitlement" {
44+
type = string
45+
description = "Value that is applied to the entitlements for OCP cluster provisioning"
46+
default = null
47+
}

examples/advanced/main.tf

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -160,6 +160,7 @@ module "ocp_base" {
160160
tags = var.resource_tags
161161
access_tags = var.access_tags
162162
worker_pools_taints = local.worker_pools_taints
163+
ocp_entitlement = var.ocp_entitlement
163164
# Enable if using worker autoscaling. Stops Terraform managing worker count.
164165
ignore_worker_pool_size_changes = true
165166
addons = {

examples/advanced/variables.tf

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,3 +45,9 @@ variable "access_tags" {
4545
description = "A list of access tags to apply"
4646
default = []
4747
}
48+
49+
variable "ocp_entitlement" {
50+
type = string
51+
description = "Value that is applied to the entitlements for OCP cluster provisioning"
52+
default = null
53+
}

examples/basic/main.tf

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,4 +79,5 @@ module "ocp_base" {
7979
ocp_version = var.ocp_version
8080
worker_pools = local.worker_pools
8181
access_tags = var.access_tags
82+
ocp_entitlement = var.ocp_entitlement
8283
}

examples/basic/variables.tf

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,3 +45,9 @@ variable "access_tags" {
4545
description = "A list of access tags to apply to the resources created by the module."
4646
default = []
4747
}
48+
49+
variable "ocp_entitlement" {
50+
type = string
51+
description = "Value that is applied to the entitlements for OCP cluster provisioning"
52+
default = null
53+
}

examples/basic/version.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
terraform {
2-
required_version = ">= 1.3.0"
2+
required_version = ">=1.3.0"
33

44
# Ensure that there is always 1 example locked into the lowest provider version of the range defined in the main
55
# module's version.tf (basic and add_rules_to_sg), and 1 example that will always use the latest provider version (advanced, fscloud and multiple mzr).

examples/cross_kms_support/main.tf

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ module "ocp_base" {
8585
ocp_version = var.ocp_version
8686
worker_pools = local.worker_pools
8787
access_tags = var.access_tags
88+
ocp_entitlement = var.ocp_entitlement
8889

8990
kms_config = {
9091
instance_id = var.kms_instance_guid

examples/cross_kms_support/variables.tf

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,3 +60,9 @@ variable "kms_cross_account_id" {
6060
type = string
6161
description = "Id of the account that owns the KMS instance to encrypt the cluster"
6262
}
63+
64+
variable "ocp_entitlement" {
65+
type = string
66+
description = "Value that is applied to the entitlements for OCP cluster provisioning"
67+
default = null
68+
}

0 commit comments

Comments
 (0)