Description of problem: value of disk type is set as 'raid6' in the generated gdeploy config file even if the value selected using gui is 'raid5' Version-Release number of selected component (if applicable): cockpit-ovirt-dashboard-0.10.7-0.0.20.el7ev.noarch How reproducible: Always Steps to Reproduce: 1. Install latest ovirt-cockpit-dashboard 2. select 'configure gluster' 3. Now select the disk type as 'raid5' 4. proceed till generation of conf file. Actual results: Even thought the disk type is selected as 'raid5' in the generated gdeploy conf it is set as 'raid6' Expected results: When the disk type is selected as 'raid5' generated gdeploy conf file should also have the disk type as 'raid5' Additional info: This issue was found by one of the community user and i am logging it on behalf of him.
comments from him: ======================== I had to manually change the install script to RAID5 even if it is chosen in the gui (sets RAID6 in the script).
disktype as 'raid5' or 'raid5' has the same effect on the calculation of LVM parameters, as the 'diskcount' always amounts to data disks only ( which are non parity ones ) So technically disktype as 'raid5' or 'raid6' doesn't have any difference at all , and that's the reason even when disktype is selected as 'raid5' from cockpit, the generated gdeploy conf file always has 'raid6'.
(In reply to RamaKasturi from comment #1) > comments from him: > ======================== > I had to manually change the install script to RAID5 even if it is chosen in > the gui (sets RAID6 in the script). He need not change it to raid5 in the install script, he can just carry on with raid6 and that will not cause him any problem.
(In reply to SATHEESARAN from comment #2) > disktype as 'raid5' or 'raid5' Typo here, I meant that LVM parameters are agnostic to RAID types ( raid5 or raid6 )
(In reply to RamaKasturi from comment #1) > comments from user: > ======================== > I had to manually change the install script to RAID5 even if it is chosen in > the gui (sets RAID6 in the script).
(In reply to SATHEESARAN from comment #4) > (In reply to SATHEESARAN from comment #2) > > disktype as 'raid5' or 'raid5' > > Typo here, I meant that LVM parameters are agnostic to RAID types ( raid5 or > raid6 ) Agree with you on this. That is the same thing which was explained to the user as well. I feel that when raid5 is selected in the UI, irrespective of what we do underneath we should always put it as 'raid5' in the conf file too.
Agree with Kasturi; leaving this as raid6 in the config file is confusing for users.
(In reply to RamaKasturi from comment #6) > (In reply to SATHEESARAN from comment #4) > > (In reply to SATHEESARAN from comment #2) > > > disktype as 'raid5' or 'raid5' > > > > Typo here, I meant that LVM parameters are agnostic to RAID types ( raid5 or > > raid6 ) > > Agree with you on this. That is the same thing which was explained to the > user as well. > > I feel that when raid5 is selected in the UI, irrespective of what we do > underneath we should always put it as 'raid5' in the conf file too. The reason why RAID5 was not passed to gdeploy is because gdeploy did not have support for this RAID type. Has this changed?
Clearing needinfo as Sac confirmed that gdeploy now supports raid5 disktype
Should be in oVirt 4.1.7 RC3
Verified and works fine with build cockpit-ovirt-dashboard-0.10.8-2.2.ovirt41.el7ev.noarch. When the Raid Type is chosen as 'RAID5' in gdeploy wizard, generated gdeploy conf file has the same value.