Bug 1542798 - basenames of the disks should be used for naming lv components
Summary: basenames of the disks should be used for naming lv components
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: cockpit-ovirt
Classification: oVirt
Component: Gdeploy
Version: 0.11.7
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: ovirt-4.2.3
: ---
Assignee: Parth Dhanjal
QA Contact: SATHEESARAN
URL:
Whiteboard:
Depends On:
Blocks: 1542797
TreeView+ depends on / blocked
 
Reported: 2018-02-07 02:57 UTC by SATHEESARAN
Modified: 2018-05-10 06:28 UTC (History)
4 users (show)

Fixed In Version: 4.2.2-2
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1542797
Environment:
Last Closed: 2018-05-10 06:28:31 UTC
oVirt Team: Gluster
Embargoed:
sasundar: ovirt-4.2?
sasundar: planning_ack?
rule-engine: devel_ack+
sasundar: testing_ack+


Attachments (Terms of Use)
Generated gdeploy config file (2.69 KB, text/plain)
2018-02-07 02:59 UTC, SATHEESARAN
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 89100 0 master MERGED Basenames of the disks used to retrieve LV components 2018-03-19 11:53:19 UTC
oVirt gerrit 89199 0 ovirt-4.2 MERGED Basenames of the disks used to retrieve LV components 2018-03-19 15:53:30 UTC

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.


Note You need to log in before you can comment on or make changes to this bug.