<div dir="ltr">Hi Sergii, thank you for feedback,<br><div><br></div><div>>> <span style="font-size:12.8000001907349px">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></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">We had a documentation, but removed it because the newer fpb was released,</span></div><div><span style="font-size:12.8000001907349px">probably we should add this information permanently [1].</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">>> </span><span style="font-size:12.8000001907349px">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></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">These plugins are the data which are required for integration testing,</span></div><div><span style="font-size:12.8000001907349px">we test that plugin build is not broken, which we </span><span style="font-size:12.8000001907349px">run when patch gets</span></div><div><span style="font-size:12.8000001907349px">published. I see nothing wrong with having the data for integration testing</span></div><div><span style="font-size:12.8000001907349px">in the same repository with product which should be tested.</span></div><div><span style="font-size:12.8000001907349px">Also in previous release we *removed* all the plugins which are not</span></div><div><span style="font-size:12.8000001907349px">related to the builder itself, lbaas and glusterfs.</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">>> </span><span style="font-size:12.8000001907349px">This breaks a couple of things</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">Having data for testing in the repository doesn't break anything.</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">>> </span><span style="font-size:12.8000001907349px">b. I cannot build fpm with simple</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">That is a good point, we should move code from fuel_plugin_builder directory</span></div><div><span style="font-size:12.8000001907349px">on top level, and move data for testing into examples directory.</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">>> </span><span style="font-size:12.8000001907349px">c. There is no tags as I can see only stable/6.0</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">Correct, tags should be added.</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">>> </span><span style="font-size:12.8000001907349px">d. There are no tests to improve code quality pep8 flask8, code coverage</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">That is not true, there are more then one hundreds unit tests which we run</span></div><div><span style="font-size:12.8000001907349px">for each patch with python 2.6 and python 2.7, also there are integration tests</span></div><div><span style="font-size:12.8000001907349px">which check that for each patch we don't break validation and that we can</span></div><div><span style="font-size:12.8000001907349px">build plugins for previous versions. Plus there are functional tests which are</span></div><div><span style="font-size:12.8000001907349px">written by fuel-qa team, those tests check that we perform deployment</span></div><div><span style="font-size:12.8000001907349px">with plugins and required functionality works correctly. Also there *is* pep8</span></div><div><span style="font-size:12.8000001907349px">check [2].</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div><span style="font-size:12.8000001907349px">>> </span><span style="font-size:12.8000001907349px">e. Repository doesn't follow community standards.</span></div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">I think this issue should be resolved with moving fuel_plugin_builder directory</div><div style="font-size:12.8000001907349px">on level higher, if not, please provide more specific description what is wrong.</div><div style="font-size:12.8000001907349px"><br></div><div style="font-size:12.8000001907349px">>> <span style="font-size:12.8000001907349px">3. Setting tab ...</span></div><div style="font-size:12.8000001907349px"><span style="font-size:12.8000001907349px"><br></span></div><div style="font-size:12.8000001907349px"><span style="font-size:12.8000001907349px">Agree.</span></div><div><span style="font-size:12.8000001907349px"><br></span></div><div>[1] <a href="https://wiki.openstack.org/w/index.php?title=Fuel%2FPlugins&diff=78677&oldid=78204">https://wiki.openstack.org/w/index.php?title=Fuel%2FPlugins&diff=78677&oldid=78204</a></div><div>[2] <a href="https://github.com/stackforge/fuel-plugins/blob/master/fuel_plugin_builder/tox.ini#L17-L21">https://github.com/stackforge/fuel-plugins/blob/master/fuel_plugin_builder/tox.ini#L17-L21</a></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 28, 2015 at 5:51 PM, Sergii Golovatiuk <span dir="ltr"><<a href="mailto:sgolovatiuk@mirantis.com" target="_blank">sgolovatiuk@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 dir="ltr"><div><div><div><div><div>Hi,<br><br></div>I have started digging into plugins recently. There are many positive things though I would like to point to some problem areas<br><br></div>1. Documentation<br><br></div>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.<br><br></div>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<br><br></div><div>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<br><br></div>2. Github repository [2] is messed up<br><div>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<br><br></div><div>b. I cannot build fpm with simple <br><br>pip install git+https://<br><br></div><div>Instead I am forced to do<br><br></div><div>git clone https://<br></div><div>cd fuel-plugins<br></div><div>pip install .<br></div><br><div>c. There is no tags as I can see only stable/6.0<br></div><div>d. There are no tests to improve code quality pep8 flask8, code coverage<br></div><div>e. Repository doesn't follow community standards.<br></div><div><br></div><div>3. Setting tab<br><br></div><div>When plugin is installed, it's very hard to find in. In setting tab it's somewhere between A and Z<br></div><div>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.<br><br></div><div>P.S. I am asking everyone to add own concerns so we'll be able to make a plan how to address them.<br><br></div><div>Thank you in advance.<br></div><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
</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>