<div dir="ltr">Hi Sheena,<div><br></div><div>Created ticket to change the structure of the directories [1].</div><div>And as far as I know any core can push tags into the repository,</div><div>Sebastian, Igor and I.</div><div><br></div><div>[1] <a href="https://bugs.launchpad.net/fuel/+bug/1479785">https://bugs.launchpad.net/fuel/+bug/1479785</a></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 28, 2015 at 8:44 PM, Sheena Gregson <span dir="ltr"><<a href="mailto:sgregson@mirantis.com" target="_blank">sgregson@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Cambria",serif;color:#1f497d">Evgeniy –</span></p><p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Cambria",serif;color:#1f497d"> </span></p><p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Cambria",serif;color:#1f497d">For the items which you have listed actions, who should be responsible for next steps?</span></p><p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Cambria",serif;color:#1f497d"> </span></p><p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Cambria",serif;color:#1f497d">Sheena</span></p><p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Cambria",serif;color:#1f497d"> </span></p><p class="MsoNormal"><b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif">From:</span></b><span style="font-size:11.0pt;font-family:"Calibri",sans-serif"> Evgeniy L [mailto:<a href="mailto:eli@mirantis.com" target="_blank">eli@mirantis.com</a>] <br><b>Sent:</b> Tuesday, July 28, 2015 11:54 AM<br><b>To:</b> OpenStack Development Mailing List (not for usage questions) <<a href="mailto:openstack-dev@lists.openstack.org" target="_blank">openstack-dev@lists.openstack.org</a>><br><b>Subject:</b> Re: [openstack-dev] [Fuel][Plugins] Feedback</span></p><div><div class="h5"><p class="MsoNormal"> </p><div><p class="MsoNormal">Hi Sergii, thank you for feedback,</p><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal">>> <span style="font-size:9.5pt">c. There is no documentation how to install fpb from github master branch. It's very useful for developers who want to use latest version. We should add something</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">We had a documentation, but removed it because the newer fpb was released,</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">probably we should add this information permanently [1].</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">>> a. We are doing the same mistake putting all things into one basket. There should be 2 repositories. One for examples and one for fpb. What's the goal of keeping fpb in directory and examples on top? </span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">These plugins are the data which are required for integration testing,</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">we test that plugin build is not broken, which we run when patch gets</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">published. I see nothing wrong with having the data for integration testing</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">in the same repository with product which should be tested.</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">Also in previous release we *removed* all the plugins which are not</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">related to the builder itself, lbaas and glusterfs.</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">>> This breaks a couple of things</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">Having data for testing in the repository doesn't break anything.</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">>> b. I cannot build fpm with simple</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">That is a good point, we should move code from fuel_plugin_builder directory</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">on top level, and move data for testing into examples directory.</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">>> c. There is no tags as I can see only stable/6.0</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">Correct, tags should be added.</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">>> d. There are no tests to improve code quality pep8 flask8, code coverage</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">That is not true, there are more then one hundreds unit tests which we run</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">for each patch with python 2.6 and python 2.7, also there are integration tests</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">which check that for each patch we don't break validation and that we can</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">build plugins for previous versions. Plus there are functional tests which are</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">written by fuel-qa team, those tests check that we perform deployment</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">with plugins and required functionality works correctly. Also there *is* pep8</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">check [2].</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">>> e. Repository doesn't follow community standards.</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt"> </span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">I think this issue should be resolved with moving fuel_plugin_builder directory</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">on level higher, if not, please provide more specific description what is wrong.</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt"> </span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">>> 3. Setting tab ...</span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt"> </span></p></div><div><p class="MsoNormal"><span style="font-size:9.5pt">Agree.</span></p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal">[1] <a href="https://wiki.openstack.org/w/index.php?title=Fuel%2FPlugins&diff=78677&oldid=78204" target="_blank">https://wiki.openstack.org/w/index.php?title=Fuel%2FPlugins&diff=78677&oldid=78204</a></p></div><div><p class="MsoNormal">[2] <a href="https://github.com/stackforge/fuel-plugins/blob/master/fuel_plugin_builder/tox.ini#L17-L21" target="_blank">https://github.com/stackforge/fuel-plugins/blob/master/fuel_plugin_builder/tox.ini#L17-L21</a></p></div></div><div><p class="MsoNormal"> </p><div><p class="MsoNormal">On Tue, Jul 28, 2015 at 5:51 PM, Sergii Golovatiuk <<a href="mailto:sgolovatiuk@mirantis.com" target="_blank">sgolovatiuk@mirantis.com</a>> wrote:</p><blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in"><div><div><div><div><div><div><p class="MsoNormal" style="margin-bottom:12.0pt">Hi,</p></div><p class="MsoNormal" style="margin-bottom:12.0pt">I have started digging into plugins recently. There are many positive things though I would like to point to some problem areas</p></div><p class="MsoNormal" style="margin-bottom:12.0pt">1. Documentation</p></div><p class="MsoNormal" style="margin-bottom:12.0pt">a. It doesn't include the features of 7.0. There are many outstanding features, though I needed to ping the developers to ask how these features work. It means that it's almost impossible to develop plugins for upcoming releases. The external developer needs to wait for documentation so it creates a lag between release and plugin release.</p></div><p class="MsoNormal" style="margin-bottom:12.0pt">b. in [1] the statement about 'For Ubuntu 12.04.2 LTS' should be extended to 14.04. Also we don't need to add PATCH version as 12.04.2 is equivalent to 12.04</p></div><div><p class="MsoNormal" style="margin-bottom:12.0pt">c. There is no documentation how to install fpb from github master branch. It's very useful for developers who want to use latest version. We should add something</p></div><p class="MsoNormal">2. Github repository [2] is messed up</p><div><p class="MsoNormal" style="margin-bottom:12.0pt">a. We are doing the same mistake putting all things into one basket. There should be 2 repositories. One for examples and one for fpb. What's the goal of keeping fpb in directory and examples on top? This breaks a couple of things</p></div><div><p class="MsoNormal" style="margin-bottom:12.0pt">b. I cannot build fpm with simple <br><br>pip install git+https://</p></div><div><p class="MsoNormal" style="margin-bottom:12.0pt">Instead I am forced to do</p></div><div><p class="MsoNormal">git clone https://</p></div><div><p class="MsoNormal">cd fuel-plugins</p></div><div><p class="MsoNormal">pip install .</p></div><p class="MsoNormal"> </p><div><p class="MsoNormal">c. There is no tags as I can see only stable/6.0</p></div><div><p class="MsoNormal">d. There are no tests to improve code quality pep8 flask8, code coverage</p></div><div><p class="MsoNormal">e. Repository doesn't follow community standards.</p></div><div><p class="MsoNormal"> </p></div><div><p class="MsoNormal" style="margin-bottom:12.0pt">3. Setting tab</p></div><div><p class="MsoNormal">When plugin is installed, it's very hard to find in. In setting tab it's somewhere between A and Z</p></div><div><p class="MsoNormal" style="margin-bottom:12.0pt">How is user supposed to find it? There should be a separator between Core features and plugins. User must easily find, configure, enable/disable them.</p></div><div><p class="MsoNormal" style="margin-bottom:12.0pt">P.S. I am asking everyone to add own concerns so we'll be able to make a plan how to address them.</p></div><div><p class="MsoNormal">Thank you in advance.</p></div><p class="MsoNormal"><br>[1] <a href="https://wiki.openstack.org/wiki/Fuel/Plugins#Installation" target="_blank">https://wiki.openstack.org/wiki/Fuel/Plugins#Installation</a><br>[2] <a href="https://github.com/stackforge/fuel-plugins" target="_blank">https://github.com/stackforge/fuel-plugins</a><br>--<br>Best regards,<br>Sergii Golovatiuk,<br>Skype #golserge<br>IRC #holser </p></div><p class="MsoNormal" style="margin-bottom:12.0pt"><br>__________________________________________________________________________<br>OpenStack Development Mailing List (not for usage questions)<br>Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</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></p></blockquote></div><p class="MsoNormal"> </p></div></div></div></div></div>
<br>__________________________________________________________________________<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.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div>