Bug 1914125 - Still using /dev/vde as default device path when create localvolume
Summary: Still using /dev/vde as default device path when create localvolume
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Storage
Version: 4.7
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: 4.7.0
Assignee: Santosh Pillai
QA Contact: Chao Yang
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-01-08 07:55 UTC by Chao Yang
Modified: 2021-02-24 15:51 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-02-24 15:51:24 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github openshift local-storage-operator pull 199 0 None closed Bug 1914125: updated localVolume CRD and CSV files examples to encourage using /dev/disk/by-id... 2021-01-18 05:07:19 UTC
Red Hat Product Errata RHSA-2020:5633 0 None None None 2021-02-24 15:51:39 UTC

Description Chao Yang 2021-01-08 07:55:26 UTC
Description of problem:
Users should not use /dev/sda, they should use /dev/disk/by-id names in LSO CRs
So we should point it on the web console

Version-Release number of selected component (if applicable):
4.7.0-202101062238.p0 

How reproducible:
Always

Steps to Reproduce:
1.Deploy LSO
2.Create Localvolume on web console
3.Click StorageclassDevice-->DevicePath

Actual results:
Should update /dev/vde to /dev/disk/by-id/ata-crucial

Expected results:
Users should not use /dev/sda, they should use /dev/disk/by-id names in LSO CRs
So we should point it on the web console

Master Log:

Node Log (of failed PODs):

PV Dump:

PVC Dump:

StorageClass Dump (if StorageClass used by PV/PVC):

Additional info:

Comment 1 Jan Safranek 2021-01-08 15:27:17 UTC
Santosh, can you please take a look? It could be as simple as changing the LocalVolume template in CRD.

Comment 2 Santosh Pillai 2021-01-11 16:18:30 UTC
checking

Comment 4 Chao Yang 2021-01-15 05:30:18 UTC
              storageClassDevices:
                description: List of storage class and devices they can match
                items:
                  properties:
                    devicePaths:
                      description: A list of devices which would be chosen for local storage. For example - ["/dev/sda", "/dev/sdb", "/dev/disk/by-id/ata-crucial"]. It is strongly recommended to use the unique /dev/disk/by-id/... name if available, as these identifiers persist across node restarts.
                      items:
                        type: string
                      type: array

oc get csv -n openshift-local-storage
NAME                                           DISPLAY         VERSION                 REPLACES   PHASE
local-storage-operator.4.7.0-202101141359.p0   Local Storage   4.7.0-202101141359.p0              Succeeded

Comment 7 errata-xmlrpc 2021-02-24 15:51:24 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Moderate: OpenShift Container Platform 4.7.0 security, bug fix, and enhancement update), and where to find the updated
files, follow the link below.

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

https://access.redhat.com/errata/RHSA-2020:5633


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