Darshan N has posted comments on this change.

Change subject: packaging: gluster:Hide irrelevant fields in configuration 
preview during setup
......................................................................


Patch Set 6:

Thanks a lot Sandro and Alon.

Regarding hiding application mode: In downstream (rhsc) the default application 
mode is gluster and input is not asked to the user, so displaying application 
mode is not useful. So we thought of checking if default application mode is 
gluster  only then we hide application mode.In upstream it will be displayed.

Regarding storage type: I will try and implement your suggestion.

NFS setup: This wont be prompted to user during setup if application mode is 
gluster. So we feel hiding it would be better.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9e4b3fffd37a62d9fc551dff178d5662d610ddd4
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Darshan N <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Aravinda VK <[email protected]>
Gerrit-Reviewer: Darshan N <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to