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

Bug 1755829

Summary:	One of the 'HSM.moveImage' exception handlers refers to non-existing members on the exception instance
Product:	[oVirt] vdsm	Reporter:	Marcin Sobczyk <msobczyk>
Component:	Core	Assignee:	Nir Soffer <nsoffer>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Yosi Ben Shimon <ybenshim>
Severity:	high	Docs Contact:
Priority:	high
Version:	4.40.0	CC:	aefrat, aoconnor, bugs, dfediuck, frolland, nsoffer
Target Milestone:	ovirt-4.4.0	Flags:	dfediuck: ovirt-4.4+
Target Release:	4.40.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2019-12-30 12:18:19 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Storage	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Marcin Sobczyk 2019-09-26 09:57:44 UTC

Description of problem:

Py3-based pylint run has reported an issue with 'HSM.moveImage' method. It seems that one of the exception handlers [1] tries to refer to members that don't exist on the exception instance [2]. Unfortunately this piece of code is not covered by tests nor by basic suite. The pylint errors will be silenced for now, but this isuse should probably be investigated by the storage team.

[1] https://github.com/oVirt/vdsm/blob/ec3aa6746a4ce0510fa314cb62056bf32ce6ab6d/lib/vdsm/storage/hsm.py#L1671
[2] https://github.com/oVirt/vdsm/blob/ec3aa6746a4ce0510fa314cb62056bf32ce6ab6d/lib/vdsm/storage/exception.py#L471

Comment 2 Yosi Ben Shimon 2019-12-09 15:23:00 UTC

Can I have a scenario for reproduction/verification, please?

Comment 3 Nir Soffer 2019-12-30 12:17:28 UTC

This is internal issue that was relevant during 4.4 development, and no was
never released. We don't need to verify this.

Comment 4 Nir Soffer 2020-05-20 23:59:50 UTC

Removing stale need infos.