Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1542798

Summary: basenames of the disks should be used for naming lv components
Product: [oVirt] cockpit-ovirt Reporter: SATHEESARAN <sasundar>
Component: GdeployAssignee: Parth Dhanjal <dparth>
Status: CLOSED CURRENTRELEASE QA Contact: SATHEESARAN <sasundar>
Severity: medium Docs Contact:
Priority: medium    
Version: 0.11.7CC: bugs, godas, rhs-bugs, sabose
Target Milestone: ovirt-4.2.3Flags: sasundar: ovirt-4.2?
sasundar: planning_ack?
rule-engine: devel_ack+
sasundar: testing_ack+
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: 4.2.2-2 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1542797 Environment:
Last Closed: 2018-05-10 06:28:31 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Gluster RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1542797    
Attachments:
Description Flags
Generated gdeploy config file none

Description SATHEESARAN 2018-02-07 02:57:00 UTC 
+++ This bug was initially created as a clone of Bug #1542797 +++

Description of problem:
-----------------------
1. If the user provides the full disk path as /dev/sdb in cockpit UI, the gdeploy configuration file is generated with vg names, thinpool names, lv name with that full path. for example, poolname=gluster_thinpool_/dev/sdb

2. If the user provides the full disk path as /dev/sdc for lvmcache configuration, then the disk is chosen as '/dev/sdc' and not just 'sdc'

Version-Release number of selected component (if applicable):
-------------------------------------------------------------
Observed this with cockpit-ovirt-dashboard-0.11.9-0.1.el7ev.noarch
But should be reproducible with earlier versions too

How reproducible:
-----------------
Always

Steps to Reproduce:
-------------------
1. Start with RHHI installation using cockpit
2. Provide the disk as '/dev/sdb'

Actual results:
---------------
All the LVM components ( vg, lv thinpool, lv, etc ) should make use of the basename

Expected results:
-----------------
LVM components have the suffix of '/dev/sdb'
Comment 1 SATHEESARAN 2018-02-07 02:59:14 UTC 
Created attachment 1392452 [details]
Generated gdeploy config file

This gdeploy configuration file is generated with user input for device as '/dev/sdb'
Comment 2 SATHEESARAN 2018-04-25 10:43:14 UTC 
Tested with cockpit-ovirt-dashboard-0.11.22

Base names are used for creating bricks.
Even when /dev/sdb is provided from cockpit UI, the names are used as gluster_vg_sdb
Comment 3 Sandro Bonazzola 2018-05-10 06:28:31 UTC 
This bugzilla is included in oVirt 4.2.3 release, published on May 4th 2018.

Since the problem described in this bug report should be
resolved in oVirt 4.2.3 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.