<div dir="ltr"><font color="#ff9900">Hi Trinath,</font><div><font color="#ff9900"><br></font></div><div><font color="#ff9900">I think the vendor company has many experts to review their codes. They can do it well.</font></div><div><font color="#ff9900"><br></font></div><div><font color="#ff9900">But I still have some comments inline.<br><br></font></div><div class="gmail_extra"><font color="#ff9900">Germy</font></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 18, 2014 at 1:42 PM, <a href="mailto:trinath.somanchi@freescale.com">trinath.somanchi@freescale.com</a> <span dir="ltr"><<a href="mailto:trinath.somanchi@freescale.com" target="_blank">trinath.somanchi@freescale.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">Though Code reviews for vendor code takes more time, I feel it must go through Core reviews.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">Since, Vendors might submit the code that is working fine within their third party CI environment but the Code review make it more efficient with respect to
 the coding standards followed in the community.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">Also, for all the vendor plugins/drivers the code reviews (+1s and +2s) give a feedback on the quality they must be in to be with Neutron.</span></p></div></div></blockquote><div><font color="#ff9900">I think the quality of a software mainly lies on developers, otherwise reviewers will be very very busy.</font></div><div><font color="#ff9900">We suppose that all core members reviewed your plugin and gave feedback many +, so can you guarantee the plugin high quality? even no BUGs? </font></div><div><font color="#ff9900">I think only the vendor, cooperating with customer and providing plugin and driver, can and must guarantee the quality. But those *private* releases only exist in vendor's disk and running in customer's machine. It cannot be updated to community because of approving waiting, because of not efficient enough, because of the coding standards, ....</font></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">But one suggestion I want to put forward, when an -1 or -2 is given to the code, Reviewers might give a brief comment on why this was given, what might be preferred
 solution and Is there any reference implementation that can be considered for the code in review to move away from these errors. This can help the developers.</span></p></div></div></blockquote><div><font color="#ff9900">If core members prefer Cisco's implementation, all the other vendors follow it? Why different plugins? Only one is enough.</font></div><div><font color="#ff9900">Of course, this is a very extreme assumption. We just discuss a problem.</font></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">--<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">Trinath Somanchi - B39208<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(132,150,176)">trinath.somanchi@</span><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(132,150,176)">freescale</span><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(132,150,176)">.com</span><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">
 | extn: 4048</span></p></div></div>_______________________________________________<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>
<br></blockquote></div><br></div></div>