Bug 1985572

Summary: storage - covscan error - DEADCODE - vdopool if create_vdo else parent
Product: Red Hat Enterprise Linux 8 Reporter: Rich Megginson <rmeggins>
Component: rhel-system-rolesAssignee: Rich Megginson <rmeggins>
Status: CLOSED ERRATA QA Contact: Zhang Yi <yizhan>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 8.5CC: djez, nhosoi, pkettman, qe-baseos-apps, spetrosi, yizhan
Target Milestone: betaKeywords: Triaged
Target Release: 8.5   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: role:storage
Fixed In Version: rhel-system-roles-1.6.4-1.el8 Doc Type: No Doc Update
Doc Text:
Story Points: ---
Clone Of: 1985571 Environment:
Last Closed: 2021-11-09 17:46:02 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1985571    
Bug Blocks:    

Description Rich Megginson 2021-07-24 00:16:53 UTC 
+++ This bug was initially created as a clone of Bug #1985571 +++

Description of problem:

There are two covscan reports in storage code:

 8. auto-maintenance-ec364d2e0bd422b69f34871a1188114f368097a0/storage/library/blivet.py:701: dead_error_line: Execution cannot reach the sub-expression "vdopool" inside this expression: "create_vdo ? vdopool : parent".
#   699|                   # This is here for backwards compatibility. Until 8.4 blivet does not support
#   700|                   # vdo_lv optional parameter
#   701|->                 device = self._blivet.new_lv(name=self._volume['name'],
#   702|                                                parents=[vdopool if create_vdo else parent],
#   703|                                                size=size, fmt=fmt)


and

 11. auto-maintenance-ec364d2e0bd422b69f34871a1188114f368097a0/storage/library/blivet.py:695: dead_error_line: Execution cannot reach the sub-expression "parent" inside this expression: "create_vdo ? vdopool : parent".
#   693|           try:
#   694|               if create_vdo:
#   695|->                 device = self._blivet.new_lv(name=self._volume['name'], vdo_lv=create_vdo,
#   696|                                                parents=[vdopool if create_vdo else parent],
#   697|                                                size=size, fmt=fmt)


the one on 695 should just be [vdopool] and the other one should just be [parent]

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:
Comment 10 errata-xmlrpc 2021-11-09 17:46:02 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 (rhel-system-roles 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/RHBA-2021:4159