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

Bug 1460512

Summary:	Useless log about removed luns is printed although no luns were removed
Product:	[oVirt] ovirt-engine	Reporter:	Idan Shaby <ishaby>
Component:	BLL.Storage	Assignee:	Idan Shaby <ishaby>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Kevin Alon Goldblatt <kgoldbla>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	future	CC:	amureini, bugs, stirabos
Target Milestone:	ovirt-4.1.3	Flags:	rule-engine: ovirt-4.1+
Target Release:	4.1.3.3
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:	2017-07-06 13:35:51 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 Idan Shaby 2017-06-11 15:31:02 UTC

Description of problem:
When synchronizing a storage domain, the following log is printed: "Removed LUNs, IDs ''" (with an empty list of luns' IDs).

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

How reproducible:
100%

Steps to Reproduce:
1. Activate a block domain that was previously on maintenance.
2. Watch the log and see that as a part of the execution of SyncLunsInfoForBlockStorageDomainCommand, the useless log is printed.

Actual results:
"Removed LUNs, IDs ''"

Expected results:
No log about removed luns should be printed when no luns were removed.

Comment 1 Allon Mureinik 2017-06-11 15:36:30 UTC

Low hanging fruit in a new feature - let's backport this to 4.1.z too please.

Comment 2 Allon Mureinik 2017-06-20 10:02:02 UTC

This patch is included in the ovirt-engine-4.1.3.3 tag.

Comment 3 Kevin Alon Goldblatt 2017-06-20 15:51:58 UTC

Verified with the following code:
------------------------------------
ovirt-engine-4.1.3.4-0.1.el7.noarch
rhevm-4.1.3.4-0.1.el7.noarch
vdsm-4.19.19-1.el7ev

Steps to Reproduce:
-----------------------
1. Activate a block domain that was previously on maintenance.
2. Watch the log and see that as a part of the execution of SyncLunsInfoForBlockStorageDomainCommand, the useless log is printed.

Actual results:
No log about removed luns should be printed when no luns were removed.


Moving to VERIFIED!