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