Description of problem: I see that user is able to proceed further with out giving any input. Do not allow the user to proceed further in case he does not fill the mandatory fileds required for deployment. Version-Release number of selected component (if applicable): cockpit-ovirt-dashboard-0.10.7-0.0.3.el7ev.noarch How reproducible: Always Steps to Reproduce: 1. install latest cockpit-ovirt-dashboard 2. Login to the UI 3. Now select "Hosted Engine with gluster" and click on next with out providing the hosts. Actual results: I see that user is able to proceed further with out providing the inputs for mandatory fields. Expected results: user should not be allowed further if he does not provide the mandatory fields required for deployment. Additional info:
While verifying this bug i see that when mount point is not specified Gluster Deployment throws a error which reads "Brick Directory cannot be empty".should not it be "Mont Point cannot be empty" ? In the same lines, if LV Name is not specified i see that "Brick name cannot be empty". Should not it be "LV Name cannot be empty".
(In reply to RamaKasturi from comment #1) > While verifying this bug i see that when mount point is not specified > Gluster Deployment throws a error which reads "Brick Directory cannot be > empty".should not it be "Mont Point cannot be empty" ? > > In the same lines, if LV Name is not specified i see that "Brick name cannot > be empty". Should not it be "LV Name cannot be empty". Both the suggestions are valid. This bug was to add validations for all the steps. Can you raise a separate bug for these text changes?.
(In reply to Ramesh N from comment #2) > (In reply to RamaKasturi from comment #1) > > While verifying this bug i see that when mount point is not specified > > Gluster Deployment throws a error which reads "Brick Directory cannot be > > empty".should not it be "Mont Point cannot be empty" ? > > > > In the same lines, if LV Name is not specified i see that "Brick name cannot > > be empty". Should not it be "LV Name cannot be empty". > > Both the suggestions are valid. This bug was to add validations for all the > steps. Can you raise a separate bug for these text changes?. The new bug[1] is created to take care of this issue [1] - https://bugzilla.redhat.com/show_bug.cgi?id=1431094
Verified and works fine with build cockpit-ovirt-dashboard-0.10.7-0.0.15.el7ev.noarch 1) when there are no gluster IP specified and if user clicks on 'Next' a message is shown to user 'Host address cannot be empty' 2) If volume name is not specified a message is shown to the user "volume name cannot be empty". 3) If brick directory is not specified a message is shown to the user " Brick directory cannot be empty" 4) If stripe size is not specified a message is shown to the user to"Enter stripe size". 5) If data disk count is not specified a message is shown to the user to "Enter data disk count" 6) If device name is not specified a message is shown to the user to "Enter correct device name for brick" 7) If brick size is not specified a message is shown to the user to "Brick Size cannot be empty" Raised a bug to change the text messages when lv and mount points are empty https://bugzilla.redhat.com/show_bug.cgi?id=1431094