<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><pre style="margin-top: 1.5em; margin-bottom: 1.5em; padding: 0px; border: 0px; font-size: 12px; font-family: 'andale mono', 'lucida console', monospace; vertical-align: baseline; white-space: pre-wrap; line-height: 15px; color: rgb(83, 83, 83); background-color: rgb(255, 255, 255); position: static; z-index: auto; "></pre><blockquote type="cite"><pre style="margin-top: 1.5em; margin-bottom: 1.5em; padding: 0px; border: 0px; font-size: 12px; font-family: 'andale mono', 'lucida console', monospace; vertical-align: baseline; white-space: pre-wrap; line-height: 15px; color: rgb(83, 83, 83); background-color: rgb(255, 255, 255); position: static; z-index: auto; ">><i> Thanks Dan. Yes this should be inline with what we discussed.

</i>><i>

</i>><i> Here's the code for review:

</i>><i>

</i>><i> <a href="https://review.openstack.org/#/c/20745/" style="margin: 0px; padding: 0px; border: 0px; font-style: inherit; font-family: inherit; vertical-align: baseline; color: rgb(188, 21, 24); text-decoration: none; ">https://review.openstack.org/#/c/20745/</a>

</i>

I have taken a look at the code (and discovered that it was abandoned).

I like the idea of the driver. It may be worth considering having this 

at a layer above the plugins so that all plugins and applications (for 

example L3 agent may use this for HA) can benefit from this.

Thanks

Gary

</pre></blockquote><div>Hi Gary</div><div><br></div><div>Thanks for reviewing this. I've uploaded Patch Set 3 addressing your comments. Could you please let me know your feedback?</div><div><br></div><div>Regards,</div><div>Satish</div></body></html>