From b269b417b0374f608ad2aa4f3422cc48b1d95208 Mon Sep 17 00:00:00 2001
From: Joe Topjian <joe@topjian.net>
Date: Tue, 1 Nov 2016 04:25:18 -0600
Subject: [PATCH] provider/openstack: LoadBalancer v2 VIP Port ID (#9727)

This commit adds vip_port_id as an exported attribute to the
lb_loadbalancer_v2 resource.
---
 .../openstack/resource_openstack_lb_loadbalancer_v2.go   | 6 ++++++
 .../resource_openstack_lb_loadbalancer_v2_test.go        | 2 ++
 .../openstack/r/lb_loadbalancer_v2.html.markdown         | 9 +++++----
 3 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/builtin/providers/openstack/resource_openstack_lb_loadbalancer_v2.go b/builtin/providers/openstack/resource_openstack_lb_loadbalancer_v2.go
index 0e2fb7dfc..358d7fc75 100644
--- a/builtin/providers/openstack/resource_openstack_lb_loadbalancer_v2.go
+++ b/builtin/providers/openstack/resource_openstack_lb_loadbalancer_v2.go
@@ -57,6 +57,11 @@ func resourceLoadBalancerV2() *schema.Resource {
 				ForceNew: true,
 			},
 
+			"vip_port_id": &schema.Schema{
+				Type:     schema.TypeString,
+				Computed: true,
+			},
+
 			"admin_state_up": &schema.Schema{
 				Type:     schema.TypeBool,
 				Default:  true,
@@ -145,6 +150,7 @@ func resourceLoadBalancerV2Read(d *schema.ResourceData, meta interface{}) error
 	d.Set("vip_subnet_id", lb.VipSubnetID)
 	d.Set("tenant_id", lb.TenantID)
 	d.Set("vip_address", lb.VipAddress)
+	d.Set("vip_port_id", lb.VipPortID)
 	d.Set("admin_state_up", lb.AdminStateUp)
 	d.Set("flavor", lb.Flavor)
 	d.Set("provider", lb.Provider)
diff --git a/builtin/providers/openstack/resource_openstack_lb_loadbalancer_v2_test.go b/builtin/providers/openstack/resource_openstack_lb_loadbalancer_v2_test.go
index 9d45469c7..06fc1a603 100644
--- a/builtin/providers/openstack/resource_openstack_lb_loadbalancer_v2_test.go
+++ b/builtin/providers/openstack/resource_openstack_lb_loadbalancer_v2_test.go
@@ -2,6 +2,7 @@ package openstack
 
 import (
 	"fmt"
+	"regexp"
 	"testing"
 
 	"github.com/gophercloud/gophercloud/openstack/networking/v2/extensions/lbaas_v2/loadbalancers"
@@ -27,6 +28,7 @@ func TestAccLBV2LoadBalancer_basic(t *testing.T) {
 				Config: TestAccLBV2LoadBalancerConfig_update,
 				Check: resource.ComposeTestCheckFunc(
 					resource.TestCheckResourceAttr("openstack_lb_loadbalancer_v2.loadbalancer_1", "name", "tf_test_loadbalancer_v2_updated"),
+					resource.TestMatchResourceAttr("openstack_lb_loadbalancer_v2.loadbalancer_1", "vip_port_id", regexp.MustCompile("^[a-f0-9-]+")),
 				),
 			},
 		},
diff --git a/website/source/docs/providers/openstack/r/lb_loadbalancer_v2.html.markdown b/website/source/docs/providers/openstack/r/lb_loadbalancer_v2.html.markdown
index 8e3bfcb8f..f861a4f50 100644
--- a/website/source/docs/providers/openstack/r/lb_loadbalancer_v2.html.markdown
+++ b/website/source/docs/providers/openstack/r/lb_loadbalancer_v2.html.markdown
@@ -27,7 +27,7 @@ The following arguments are supported:
     `OS_REGION_NAME` environment variable is used. Changing this creates a new
     LB member.
 
-* `vip_subnet_id` - (Required) The network on which to allocate the 
+* `vip_subnet_id` - (Required) The network on which to allocate the
     Loadbalancer's address. A tenant can only create Loadbalancers on networks
     authorized by policy (e.g. networks that belong to them or networks that
     are shared).  Changing this creates a new loadbalancer.
@@ -40,13 +40,13 @@ The following arguments are supported:
 * `tenant_id` - (Optional) Required for admins. The UUID of the tenant who owns
     the Loadbalancer.  Only administrative users can specify a tenant UUID
     other than their own.  Changing this creates a new loadbalancer.
-    
+
 * `vip_address` - (Optional) The ip address of the load balancer.
     Changing this creates a new loadbalancer.
-    
+
 * `admin_state_up` - (Optional) The administrative state of the Loadbalancer.
     A valid value is true (UP) or false (DOWN).
-    
+
 * `flavor` - (Optional) The UUID of a flavor. Changing this creates a new
     loadbalancer.
 
@@ -66,3 +66,4 @@ The following attributes are exported:
 * `admin_state_up` - See Argument Reference above.
 * `flavor` - See Argument Reference above.
 * `provider` - See Argument Reference above.
+* `vip_port_id` - The Port ID of the Load Balancer IP.