a10networks / a10-neutron-lbaas

A10 Networks, Openstack Neutron LBaaS Driver
Apache License 2.0
9 stars 19 forks source link

Ensure admin_state is propogated to members. #250

Open ghost opened 8 years ago

ghost commented 8 years ago

admin_state not working with members. We calculate status in a10-neutron-lbaas, but do not pass it down.

dougwig commented 8 years ago

ml patch:

@@ -34,13 +35,15 @@
         server_name = self._meta_name(member, server_ip)

         status = c.client.slb.UP
+        admin_state = 'enable'
         if not member['admin_state_up']:
             status = c.client.slb.DOWN
+            admin_state = 'disable'

         try:
             server_args = {'server': self.meta(member, 'server', {})}
             c.client.slb.server.create(server_name, server_ip,
-                                       axapi_args=server_args)
+                                       axapi_args=server_args,admin_state=admin_state)
         except (acos_errors.Exists, acos_errors.AddressSpecifiedIsInUse):
             pass