Neutron LBaaS - VIP Resource Modify create/delete/update methods

Bug #1280167 reported by Rabi Mishra
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Heat
Invalid
Medium
Unassigned

Bug Description

All life-cycle methods for the new VIP Resource (refer bug #1280165) need to be implemented differently.

Without the existence of the neutron VIP resource earlier; 'handle_create','handle_update','handle_delete' handle methods use heat 'db_api' and 'nova_client' to implement this. This should be changed.

Current implementation below:
-----------------------------------------------
 def handle_create(self):
        pool = self.properties[self.POOL_ID]
        client = self.neutron()
        nova_client = self.nova()
        protocol_port = self.properties[self.PROTOCOL_PORT]

        for member in self.properties.get(self.MEMBERS):
            address = nova_utils.server_to_ipaddress(nova_client, member)
            lb_member = client.create_member({
                'member': {
                    'pool_id': pool,
                    'address': address,
                    'protocol_port': protocol_port}})['member']
            db_api.resource_data_set(self, member, lb_member['id'])

Rabi Mishra (rabi)
Changed in heat:
assignee: nobody → Rabi Mishra (ramishra)
Changed in heat:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Anne Gentle (annegentle) wrote :

Wanted to note that Heat documentation is somewhat lacking for the VIP resource. In the Pool list, there's an attribute vip which is the "Vip associated with the pool." No mention on what properties are available within the VIP though. It would be great to address the gap when working on this bug.

Revision history for this message
Sergey Kraynev (skraynev) wrote :

Anne, thank you for the good point. You are right it's really missed. Unfortunatelly the works is blocked by changes in neutron. AFAIK, they have new model and the reason, why we did not fix it yet, is we wait changes in neutron for improving or reworking current implemenation in heat. I hope when we start change LB in heat we update all documentation and make it more similar on neutron. Thanks.

Angus Salkeld (asalkeld)
Changed in heat:
assignee: Rabi Mishra (rabi) → nobody
Thomas Herve (therve)
Changed in heat:
status: Triaged → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.