<div dir="ltr">Sheena, I still have concerns regarding #3. I am sending attachment how it's implemented. Firstly, it's bad practice to mix core and plugin functionality. Also we do not validate names. When there are several plugins it's very hard to find all of them<br><br>I am giving a sketch how it should be IMO <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 Tue, Jul 28, 2015 at 6:25 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 link="blue" vlink="purple" lang="EN-US"><div><p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Cambria",serif;color:#1f497d">Hey Sergii –</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">This is excellent feedback, thank you for taking the time to provide your thoughts.</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">#1 I agree that the documentation lag is challenging – I’m not sure how to best address this.  We could potentially prioritize updates to the Plugin SDK for soon-to-be-released features ahead of the standard release notes and user guide updates to ensure that plugin developers have access to this information earlier?  A number of the docs team members will be getting together in late August to discuss how to improve documentation, I will add this as a topic if we don’t feel there is good resolution on the mailing list.</span></p><p class="MsoNormal"><i><span style="font-size:10.0pt;font-family:"Cambria",serif;color:#1f497d">+Alexander/Evgeny to cc for their input</span></i></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">#3 Settings tab is getting a facelift in 7.0 and there are now subgroups in the tab which should make it significantly easier for a user to find plugin settings.  Each plugin will create a new sub-group in the Settings tab, like Access (and others) in the screenshot below.</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"><img src="cid:image002.jpg@01D0C928.1DC18140" height="274" width="597"></span><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"> </span></p><p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Cambria",serif;color:#1f497d">I don’t have any insight on the GitHub issues, so I will wait for others to weigh in on your concerns there.</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"> Sergii Golovatiuk [mailto:<a href="mailto:sgolovatiuk@mirantis.com" target="_blank">sgolovatiuk@mirantis.com</a>] <br><b>Sent:</b> Tuesday, July 28, 2015 9:51 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> [openstack-dev] [Fuel][Plugins] Feedback</span></p><div><div class="h5"><p class="MsoNormal"> </p><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></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>