<div dir="ltr"><div>Hi,<br><br></div>I would go with the 1st approach. The thing I don't like in the 2nd approach is that we have to make the user enable plugin twice. For example, we have to enable Ceph as a plugin and then add Ceph role to nodes and choose what we want to store in Ceph (images, objects). Why we would need to explicitly enable Ceph plugin? Let's always show plugin options in wizard and settings tab, and if the user just doesn't want to enable Ceph, he would just leave all the checkboxes unchecked. The 2nd approach would also lead to some kind of inconsistency in case the user enabled Ceph plugin but left all the Ceph-related checkboxes unchecked and didn't add Ceph nodes.<br></div><div class="gmail_extra"><br><div class="gmail_quote">2014-10-07 21:17 GMT+07:00 Evgeniy L <span dir="ltr"><<a href="mailto:eli@mirantis.com" target="_blank">eli@mirantis.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>We had a meeting today about plugins on UI, as result of the meeting</div><div>we have two approaches and this approaches affect not only UX but</div><div>plugins itself.</div><div><br></div><div><b>1st - disable/enable plugin on settings tab</b></div><div><ol><li>user installs the plugin</li><li>creates a cluster</li><li>configures and enables/disables plugins on settings tab</li></ol><div>For user it will look like Ceph plugin checkboxes on settings tab,</div></div><div>if he enables checkbox, then we pass the parameter to orchestrator</div><div>as `true`.</div><div><br></div><div>Cons:</div><div><ul><li>plugin developer should define a checkbox in each plugin (for plugin disabling/enabling)</li><li>on the backend we have to enable all of the plugins for environment, because user can define any name for his checkbox and we won't be able to find it and make appropriate mapping plugin <-> env</li><li>since all of the plugins are always "enabled" we have to run tasks for all of the plugins, and each plugin should parse astute.yaml in order to figure out if it's required to run task current script</li></ul></div><div>Pros:</div><div><ul><li>it won't require additional setting or step for wizard</li><li>user will be able to disable plugin after environment creation</li></ul></div><div><b>2nd - enable plugins in wizard</b></div><div><ol><li>user installs the plugin</li><li>now he can choose specific plugins for his environment in wizard</li><li>after cluster is created, he can configure additional parameters on settings tab, if plugin provides any</li></ol><div>Cons:</div></div><div><ul><li>user won't be able to disable plugin after cluster is created</li><li>additional step or configuration subcategory in wizard</li></ul></div><div>Pros:</div><div><br></div><div>On backend we always know which plugin is disabled and which is enabled.</div><div><ul><ul></ul></ul></div><div><ul><li>it means we don't provide settings for plugins which are disabled<br></li><li>we don't run tasks on slaves if it's not required</li></ul></div><div>Thanks,</div></div>
<br>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</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><br>
<br></blockquote></div><br><br clear="all"><br>-- <br><div dir="ltr">Vitaly Kramskikh,<br>Software Engineer,<br>Mirantis, Inc.</div>
</div>