<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Mar 1, 2017 at 12:39 PM Jordan Pittier <<a href="mailto:jordan.pittier@scality.com">jordan.pittier@scality.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg">On Wed, Mar 1, 2017 at 3:57 AM, Ghanshyam Mann <span dir="ltr" class="gmail_msg"><<a href="mailto:ghanshyammann@gmail.com" class="gmail_msg" target="_blank">ghanshyammann@gmail.com</a>></span> wrote:<br class="gmail_msg"><blockquote class="gmail_quote gmail_msg" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr" class="gmail_msg"><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg">Doing gradual refactoring and fixing plugins time to time needs lot of wait and sync.</div><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg"><br class="gmail_msg"></div><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg">That needs:</div><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg">1. Plugins to switch from current method usage. Plugins to have some other function or same copy paste code what current scenario base class has.</div><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg">2. Tempest patch to wait for plugin fix.</div><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg">3. Plugins to switch back to stable interface once Tempest going to provide those. <br class="gmail_msg"></div><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg"><br class="gmail_msg"></div><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg">This needs lot of sync between tempest and plugins and we have to wait for tempest refactoring patch for long. </div><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg"><br class="gmail_msg"></div><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg">To make it more efficient, how about this:</div><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg">1. Keep the scenario manger copy in tempest as it is. for plugins usage only.</div></div></blockquote></div></div></div><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><div class="gmail_msg">Given that the refactoring effort "started" a year ago, at the current speed it may take 2 years to complete. In the mean time we will have a massive code duplication and a maintenance burden. </div></div></div></div><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><blockquote class="gmail_quote gmail_msg" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr" class="gmail_msg"><div style="font-family:arial,helvetica,sans-serif;color:rgb(0,0,0)" class="gmail_msg">2. Start refactoring the scenario framework by adding more and more helper function under /common or lib.</div></div></blockquote></div></div></div><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><div class="gmail_msg">Starting a "framework" (each time I see that word, I have a bad feeling) from scratch without users and usage is very very difficult. How do we know what we need in that framework and what will be actually used in tests ?  </div></div></div></div></blockquote><div><br></div><div>Yeah +1 on that - we need to refactor scenario to fix debuggability, maintainability and ability to get contributions.</div><div>Moving helpers to lib is an option, but it's a lower priority in my view.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg"><div class="gmail_extra gmail_msg"><div class="gmail_quote gmail_msg"><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">The effort was called scenario refactoring and I think that's what we should do. We should not do "start from scratch scenarios" or "copy all the code and see what happens".</div><div class="gmail_msg"><br class="gmail_msg"></div><div class="gmail_msg">There's no problem with plugins. We committed to have a stable interface which is documented and agreed upon. It's clearly written here <a href="https://docs.openstack.org/developer/tempest/plugin.html#stable-tempest-apis-plugins-may-use" class="gmail_msg" target="_blank">https://docs.openstack.org/developer/tempest/plugin.html#stable-tempest-apis-plugins-may-use</a> The rest is private, for Tempest internal use. If Tempest cores disagree with that, then we should first of all put a spec and rewrite that document. </div></div></div></div>
__________________________________________________________________________<br class="gmail_msg">
OpenStack Development Mailing List (not for usage questions)<br class="gmail_msg">
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" class="gmail_msg" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br class="gmail_msg">
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br class="gmail_msg">
</blockquote></div></div>