From 72324a29f9e3cf74b55af7e27197557cef82258a Mon Sep 17 00:00:00 2001
From: Josh Vasilevsky <95491250+jvasilevsky@users.noreply.github.com>
Date: Tue, 3 Dec 2024 10:32:33 -0500
Subject: [PATCH] LBAAS: increase maximum lb size_unit, add regional_network
 type support (#1284)

* LBAAS: increase maximum lb size_unit

* LBAAS: support REGIONAL_NETWORK type

* LBAAS:
- add REGIONAL_NETWORK in docs
- update max size in docs
---
 digitalocean/loadbalancer/datasource_loadbalancer.go | 2 +-
 digitalocean/loadbalancer/resource_loadbalancer.go   | 8 ++++----
 docs/resources/loadbalancer.md                       | 5 ++---
 3 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/digitalocean/loadbalancer/datasource_loadbalancer.go b/digitalocean/loadbalancer/datasource_loadbalancer.go
index b382b5098..b6df571c7 100644
--- a/digitalocean/loadbalancer/datasource_loadbalancer.go
+++ b/digitalocean/loadbalancer/datasource_loadbalancer.go
@@ -251,7 +251,7 @@ func DataSourceDigitalOceanLoadbalancer() *schema.Resource {
 			"type": {
 				Type:        schema.TypeString,
 				Computed:    true,
-				Description: "the type of the load balancer (GLOBAL or REGIONAL)",
+				Description: "the type of the load balancer (GLOBAL, REGIONAL, or REGIONAL_NETWORK)",
 			},
 			"domains": {
 				Type:        schema.TypeSet,
diff --git a/digitalocean/loadbalancer/resource_loadbalancer.go b/digitalocean/loadbalancer/resource_loadbalancer.go
index af3643a04..6b3f46027 100644
--- a/digitalocean/loadbalancer/resource_loadbalancer.go
+++ b/digitalocean/loadbalancer/resource_loadbalancer.go
@@ -132,7 +132,7 @@ func loadbalancerDiffCheck(ctx context.Context, d *schema.ResourceDiff, v interf
 		if regionSet && region.(string) != "" {
 			return fmt.Errorf("'region' must be empty or not set when 'type' is '%s'", typStr)
 		}
-	case "REGIONAL":
+	case "REGIONAL", "REGIONAL_NETWORK":
 		if !regionSet || region.(string) == "" {
 			return fmt.Errorf("'region' must be set and not be empty when 'type' is '%s'", typStr)
 		}
@@ -180,7 +180,7 @@ func resourceDigitalOceanLoadBalancerV0() *schema.Resource {
 				Type:         schema.TypeInt,
 				Optional:     true,
 				Computed:     true,
-				ValidateFunc: validation.IntBetween(1, 100),
+				ValidateFunc: validation.IntBetween(1, 200),
 			},
 			"name": {
 				Type:         schema.TypeString,
@@ -443,8 +443,8 @@ func resourceDigitalOceanLoadBalancerV0() *schema.Resource {
 				Type:         schema.TypeString,
 				Optional:     true,
 				ForceNew:     true,
-				ValidateFunc: validation.StringInSlice([]string{"REGIONAL", "GLOBAL"}, true),
-				Description:  "the type of the load balancer (GLOBAL or REGIONAL)",
+				ValidateFunc: validation.StringInSlice([]string{"REGIONAL", "GLOBAL", "REGIONAL_NETWORK"}, true),
+				Description:  "the type of the load balancer (GLOBAL, REGIONAL, or REGIONAL_NETWORK)",
 			},
 
 			"domains": {
diff --git a/docs/resources/loadbalancer.md b/docs/resources/loadbalancer.md
index 94e9f8ea8..91d341bdb 100644
--- a/docs/resources/loadbalancer.md
+++ b/docs/resources/loadbalancer.md
@@ -93,7 +93,7 @@ The following arguments are supported:
 * `name` - (Required) The Load Balancer name
 * `region` - (Required) The region to start in
 * `size` - (Optional) The size of the Load Balancer. It must be either `lb-small`, `lb-medium`, or `lb-large`. Defaults to `lb-small`. Only one of `size` or `size_unit` may be provided.
-* `size_unit` - (Optional) The size of the Load Balancer. It must be in the range (1, 100). Defaults to `1`. Only one of `size` or `size_unit` may be provided.
+* `size_unit` - (Optional) The size of the Load Balancer. It must be in the range (1, 200). Defaults to `1`. Only one of `size` or `size_unit` may be provided.
 * `algorithm` - (Optional) **Deprecated** This field has been deprecated. You can no longer specify an algorithm for load balancers.
 or `least_connections`. The default value is `round_robin`.
 * `forwarding_rule` - (Required) A list of `forwarding_rule` to be assigned to the
@@ -119,10 +119,9 @@ the backend service. Default value is `false`.
 * `domains` (Optional) - A list of `domains` required to ingress traffic to a Global Load Balancer. The `domains` block is documented below. 
 * `glb_settings` (Optional) - A block containing `glb_settings` required to define target rules for a Global Load Balancer. The `glb_settings` block is documented below.
 * `target_load_balancer_ids` (Optional) - A list of Load Balancer IDs to be attached behind a Global Load Balancer.
-* `type` - (Optional) The type of the Load Balancer. It must be either of `REGIONAL` or `GLOBAL`. Defaults to `REGIONAL`.
+* `type` - (Optional) The type of the Load Balancer. It must be either of `REGIONAL`, `REGIONAL_NETWORK`, or `GLOBAL`. Defaults to `REGIONAL`.
 **NOTE**: non-`REGIONAL/GLOBAL` type may be part of closed beta feature and not available for public use.
 * `network` - (Optional) The type of network the Load Balancer is accessible from. It must be either of `INTERNAL` or `EXTERNAL`. Defaults to `EXTERNAL`.
-**NOTE**: non-`EXTERNAL` type may be part of closed beta feature and not available for public use.
 
 `forwarding_rule` supports the following: