Alex Lourie has posted comments on this change.

Change subject: packaging: Application Mode option in installer
......................................................................


Patch Set 5:

I am concerned with the implementation of the feature from the UX point of 
view. What would be the reason to hide the option from the user, and more, 
doing that with the constant in the basedefs?

If this is a viable option, we need always show it. For example, we can precede 
this with the question, such as explaining what the application mode is and 
then "would you like to configure an application mode?", and if answered yes, 
continue to parameters questions.

I highly doubt that most of our users would ever think of touching basedefs 
file (which is a "production" code) to alter the setup. I would rather believe 
that this should be set in an answer file during automated installation, 
similar to configuring or not configuring the firewall rules.

--
To view, visit http://gerrit.ovirt.org/9991
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ieba515d2baf3559bf27185c7c6432160551ef172
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Aravinda VK <avish...@redhat.com>
Gerrit-Reviewer: Alex Lourie <alou...@redhat.com>
Gerrit-Reviewer: Aravinda VK <avish...@redhat.com>
Gerrit-Reviewer: Bala.FA <barum...@redhat.com>
Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgold...@redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com>
Gerrit-Reviewer: Ohad Basan <oba...@redhat.com>
Gerrit-Reviewer: Ronen Angluster <rangl...@redhat.com>
Gerrit-Reviewer: Shireesh Anjal <san...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to