fix(VSECPC-11449): Updated management_nic variable description for accuracy and clarity (#8)

chkp-eddiek · chkp-natanelm · web-flow · commit c601d8d9c0a4 · 2025-08-26T12:40:29.000+03:00
Co-authored-by: natanelm &lt;natanelm@checkpoint.com&gt;
diff --git a/modules/common/network-security-integration-common/variables.tf b/modules/common/network-security-integration-common/variables.tf
@@ -40,7 +40,7 @@ variable "os_version" {
 }
 variable "management_nic" {
   type = string
-  description = "Management Interface - Autoscaling Security Gateways in GCP can be managed by an ephemeral public IP or using the private IP of the internal interface (eth1)."
+  description = "Management Interface - Autoscaling Security Gateways in GCP can be managed by the ephemeral public IP or by the private IP of the Management interface (eth0)."
   default = "Ephemeral Public IP (eth0)"
 }
 variable "management_name" {
diff --git a/modules/network-security-integration/README.md b/modules/network-security-integration/README.md
@@ -231,7 +231,7 @@ Leave an empty list for a protocol if you want to disable traffic for it.
 | license | Check Point license (BYOL).                                                                                                                                                                                                                                                                                                                                    | string | BYOL <br/> | "BYOL" | no |
 | image_name | The NSI image name (for example, check-point-r8120-gw-byol-nsi-631-991001866-v20250731).                                                                                                | string | N/A | N/A | yes |
 | os_version | Gaia OS Version                                                                                                                                                                                                                                                                                                                                                       | string | R8110;<br/> R8120;<br/> R82; | "R8120" | yes
-| management_nic | Management Interface - Autoscaling Security Gateways in GCP can be managed by an ephemeral public IP or using the private IP of the internal interface (eth1).                                                                                                                                                                                                        | string | Ephemeral Public IP (eth0) <br/> Private IP (eth1) | "Ephemeral Public IP (eth0)" | no |
+| management_nic | Management Interface - Autoscaling Security Gateways in GCP can be managed by the ephemeral public IP or by the private IP of the Management interface (eth0).                                                                                                                                                                                                        | string | Ephemeral Public IP (eth0) <br/> Private IP (eth0) | "Ephemeral Public IP (eth0)" | no |
 | management_name | The name of the Security Management Server as it appears in the autoprovisioning configuration. (Enter a valid Security Management name including lowercase letters, digits and hyphens only).                                                                                                                                                                        | string | N/A | "checkpoint-management" | no |
 | configuration_template_name | Specify the provisioning configuration template name (for autoprovisioning). (Enter a valid autoprovisioning configuration template name including lowercase letters, digits, and hyphens only).                                                                                                                                                                 | string | N/A | "gcp-asg-autoprov-tmplt" | no |
 | generate_password  | Automatically generate an administrator password.                                                                                                                                                                                                                                                                                                                     | bool | true <br/>false | false | no |
diff --git a/modules/network-security-integration/variables.tf b/modules/network-security-integration/variables.tf
@@ -48,7 +48,7 @@ variable "os_version" {
 }
 variable "management_nic" {
   type = string
-  description = "Management Interface - Autoscaling Security Gateways in GCP can be managed by an ephemeral public IP or using the private IP of the internal interface (eth1)."
+  description = "Management Interface - Autoscaling Security Gateways in GCP can be managed by the ephemeral public IP or by the private IP of the Management interface (eth0)."
   default = "Ephemeral Public IP (eth0)"
   validation {
     condition = contains(["Ephemeral Public IP (eth0)", "Private IP (eth0)"], var.management_nic)

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ variable "os_version" {`
`40`	`40`	`}`
`41`	`41`	`variable "management_nic" {`
`42`	`42`	`type = string`
`43`		`- description = "Management Interface - Autoscaling Security Gateways in GCP can be managed by an ephemeral public IP or using the private IP of the internal interface (eth1)."`
	`43`	`+ description = "Management Interface - Autoscaling Security Gateways in GCP can be managed by the ephemeral public IP or by the private IP of the Management interface (eth0)."`
`44`	`44`	`default = "Ephemeral Public IP (eth0)"`
`45`	`45`	`}`
`46`	`46`	`variable "management_name" {`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ variable "os_version" {`
`48`	`48`	`}`
`49`	`49`	`variable "management_nic" {`
`50`	`50`	`type = string`
`51`		`- description = "Management Interface - Autoscaling Security Gateways in GCP can be managed by an ephemeral public IP or using the private IP of the internal interface (eth1)."`
	`51`	`+ description = "Management Interface - Autoscaling Security Gateways in GCP can be managed by the ephemeral public IP or by the private IP of the Management interface (eth0)."`
`52`	`52`	`default = "Ephemeral Public IP (eth0)"`
`53`	`53`	`validation {`
`54`	`54`	`condition = contains(["Ephemeral Public IP (eth0)", "Private IP (eth0)"], var.management_nic)`