<div dir="ltr">Hi -<div><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Yes, planning to keep the manifest, just removing the ping test and the code that is dependent on that succeeding.<br></blockquote><div><br></div><div> Would the ping test and code be suitable for a TripleO Validation?</div><div><br></div><div><a href="https://github.com/openstack/tripleo-validations">https://github.com/openstack/tripleo-validations</a><br></div><div><br></div><div>Thanks</div><div>-dant</div><div><br></div><div><br></div><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Uploading a new PS to reflect this approach.<br>
<br>
Thanks,<br>
Sandhya<br>
<div class="gmail-HOEnZb"><div class="gmail-h5"><br>
On 12/11/17, 12:22 PM, "Emilien Macchi" <<a href="mailto:emilien@redhat.com">emilien@redhat.com</a>> wrote:<br>
<br>
    Hi Sandhya,<br>
<br>
    See inline:<br>
<br>
    On Mon, Dec 11, 2017 at 8:49 AM, Sandhya Dasu (sadasu) <<a href="mailto:sadasu@cisco.com">sadasu@cisco.com</a>> wrote:<br>
    > Hi Steven and Emilien,<br>
    ><br>
    >     I need your advice on how to proceed with the fix in<br>
    > <a href="https://review.openstack.org/#/c/521632/" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/521632/</a>.<br>
    ><br>
    ><br>
    ><br>
    > The issue in question is that code in puppet-neutron for the Nexus switch,<br>
    > performs a ping test to see if all the Nexus switches specified in the<br>
    > configuration are actually reachable.<br>
    ><br>
    > After that it performs a ssh-keyscan and adds the list of Nexus switches to<br>
    > the list on known hosts on the Controllers.<br>
    ><br>
    > Code can be viewed here:<br>
    > <a href="https://github.com/openstack/puppet-neutron/blob/master/manifests/plugins/ml2/cisco/nexus_creds.pp" rel="noreferrer" target="_blank">https://github.com/openstack/<wbr>puppet-neutron/blob/master/<wbr>manifests/plugins/ml2/cisco/<wbr>nexus_creds.pp</a><br>
    > (starting from line #104)<br>
    ><br>
    > I spoke to Emilien about this during the Sydney summit.<br>
    ><br>
    ><br>
    ><br>
    > Since then I have tried a bunch of different ways to solve this problem and<br>
    > I am trying to figure out the best way to proceed:<br>
    ><br>
    ><br>
    ><br>
    > 1.       Adding retry login around the ping test:<br>
    > <a href="https://review.openstack.org/#/c/521632/2" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/521632/2</a><br>
    ><br>
    > 2.       Changing the order in which Neutron ML2 plugins/services were<br>
    > initialized in <a href="https://review.openstack.org/#/c/521632/8" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/521632/8</a> (Failed gate<br>
    > checks)<br>
    ><br>
    > 3.       I also tried to remove a dependency between the ping test and the<br>
    > ssh-keyscan steps. (code in <a href="https://review.openstack.org/#/c/521632/7" rel="noreferrer" target="_blank">https://review.openstack.org/#<wbr>/c/521632/7</a>)<br>
    ><br>
    > 4.       Finally, in the latest version of the fix I completely removed the<br>
    > ping test and ssh-keyscan steps to make progress.<br>
    > (<a href="https://review.openstack.org/#/c/521632/" rel="noreferrer" target="_blank">https://review.openstack.org/<wbr>#/c/521632/</a>)<br>
    ><br>
    ><br>
    ><br>
    > Although, the ping test and ssh-keyscan are not essential for the<br>
    > functioning of the Nexus driver, I would like to find a way to keep this<br>
    > code.<br>
    ><br>
    ><br>
    ><br>
    > Please let me know what would be the best way to proceed.<br>
<br>
    IMHO this code shouldn't exist. I've never seen any Puppet code<br>
    testing "ping" during a deployment.<br>
    You should rather make sure that Nexus switches are ready and<br>
    available before performing any deployment with puppet-neutron (with<br>
    Ansible for example).<br>
    But doing it with Puppet is kind of the wrong way I think.<br>
    Let me know if that makes sense but I would rather keep his manifest<br>
    to manage config files and remove this code in the future.<br>
    --<br>
    Emilien Macchi<br>
<br>
<br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a><br>
</div></div></blockquote></div><br></div></div></div>