Octavia LBaaSv2: Allow setting an empty members list

kayrus · kayrus · commit 4ca539c0f592 · 2019-10-10T16:22:40.000+02:00
diff --git a/openstack/loadbalancer/v2/pools/requests.go b/openstack/loadbalancer/v2/pools/requests.go
@@ -354,7 +354,7 @@ func (opts BatchUpdateMemberOpts) ToBatchMemberUpdateMap() (map[string]interface
 
 // BatchUpdateMembers updates the pool members in batch
 func BatchUpdateMembers(c *gophercloud.ServiceClient, poolID string, opts []BatchUpdateMemberOpts) (r UpdateMembersResult) {
-	var members []map[string]interface{}
+	members := []map[string]interface{}{}
 	for _, opt := range opts {
 		b, err := opt.ToBatchMemberUpdateMap()
 		if err != nil {
diff --git a/openstack/loadbalancer/v2/pools/testing/fixtures.go b/openstack/loadbalancer/v2/pools/testing/fixtures.go
@@ -438,3 +438,18 @@ func HandleMembersUpdateSuccessfully(t *testing.T) {
 		w.WriteHeader(http.StatusAccepted)
 	})
 }
+
+// HandleEmptyMembersUpdateSuccessfully sets up the test server to respond to an empty batch member Update request.
+func HandleEmptyMembersUpdateSuccessfully(t *testing.T) {
+	th.Mux.HandleFunc("/v2.0/lbaas/pools/332abe93-f488-41ba-870b-2ac66be7f853/members", func(w http.ResponseWriter, r *http.Request) {
+		th.TestMethod(t, r, "PUT")
+		th.TestHeader(t, r, "X-Auth-Token", client.TokenID)
+		th.TestHeader(t, r, "Accept", "application/json")
+		th.TestHeader(t, r, "Content-Type", "application/json")
+		th.TestJSONRequest(t, r, `{
+			"members": []
+		}`)
+
+		w.WriteHeader(http.StatusAccepted)
+	})
+}
diff --git a/openstack/loadbalancer/v2/pools/testing/requests_test.go b/openstack/loadbalancer/v2/pools/testing/requests_test.go
@@ -293,6 +293,15 @@ func TestBatchUpdateMembers(t *testing.T) {
 	th.AssertNoErr(t, res.Err)
 }
 
+func TestEmptyBatchUpdateMembers(t *testing.T) {
+	th.SetupHTTP()
+	defer th.TeardownHTTP()
+	HandleEmptyMembersUpdateSuccessfully(t)
+
+	res := pools.BatchUpdateMembers(fake.ServiceClient(), "332abe93-f488-41ba-870b-2ac66be7f853", []pools.BatchUpdateMemberOpts{})
+	th.AssertNoErr(t, res.Err)
+}
+
 func TestRequiredBatchUpdateMemberOpts(t *testing.T) {
 	th.SetupHTTP()
 	defer th.TeardownHTTP()
