Hi Pattabi,<div><br></div><div>Looks like you were looking at unmerged code, which is always subject to change.  In this case, the code you're looking at has been substantially change.  The new code actually follows a model that I suspect is more inline with what you are asking about where the entire new config is passed to the driver when something changes, not just the "diff".  </div>

<div><br></div><div>Dan</div><div><br><br><div class="gmail_quote">On Wed, Feb 27, 2013 at 4:39 PM, Pattabi Ayyasami <span dir="ltr"><<a href="mailto:pattabi@brocade.com" target="_blank">pattabi@brocade.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
As part of integrating the Brocade Driver for the Load Balancing Functionality, I noticed that in the update APIs, the old object is not passed to the driver fully.<br>
Instead  only the status is passed. In our case, we need the complete old object (for example, when a pool_id is updated as part of update_vip, we need the pool members of the old pool to unbind from the vip).<br>
<br>
I noticed in the plugins/services/loadbalancer/loadbalancer_plugin.py code, only the status field is fetched for the old object. Would like to know if there is any reason behind this implementation or is this something that will be addressed.<br>


<br>
def update_vip(self, context, vip_id, vip):<br>
        LOG.debug(_('Update vip %s'), vip_id)<br>
        v_old = self.get_vip(context, vip_id, fields=['status'])<br>
       ...........<br>
       ..........<br>
<br>
<br>
Regards,<br>
Pattabi<br>
<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>Dan Wendlandt <div>Nicira, Inc: <a href="http://www.nicira.com" target="_blank">www.nicira.com</a><br><div>twitter: danwendlandt<br>

~~~~~~~~~~~~~~~~~~~~~~~~~~~<br></div></div>
</div>