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

Bug 909838

Summary:	Reduce engine -> vdsm communication at ConnectHostToStoragePoolServersCommand
Product:	Red Hat Enterprise Virtualization Manager	Reporter:	Yair Zaslavsky <yzaslavs>
Component:	ovirt-engine	Assignee:	mkublin <mkublin>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Haim <hateya>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	3.2.0	CC:	acathrow, bazulay, dyasny, iheim, lpeer, oourfali, Rhev-m-bugs, yeylon, ykaul
Target Milestone:	---	Keywords:	CodeChange
Target Release:	3.2.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	storage
Fixed In Version:	sf9	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		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:
Bug Depends On:
Bug Blocks:	917401

Description Yair Zaslavsky 2013-02-11 09:47:17 UTC

Description of problem:

The check of storage connection validation should be removed from
ConnectHostToStoagePoolServerCommand.



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


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Dan Yasny 2013-02-14 09:51:39 UTC

Details from gerrit:
The following change is removes canDoAction from ConnectHostToStoragePoolServersCommand.
The purpose of canDoAction was to validate storage connections at VDSM, the following API
is depreacted and not useful at vdsm side, and doesn't provide any benefit to engine, also
the following command is internal so no reason to perfrom additional xml rpc call in order to
validate storage server connection.
Benefit, reduced number of XML RPC calls via InitVdsOnUp, meaning vds should be moved to status UP
much faster.
Additional benefit, less code, less bugs

Comment 3 Haim 2013-03-18 18:32:11 UTC

verified on SF10.1.

check was removed from engine hence vdsm is no longer called with validateStorageServer.

Comment 4 Itamar Heim 2013-06-11 09:34:21 UTC

3.2 has been released

Comment 5 Itamar Heim 2013-06-11 09:34:33 UTC

3.2 has been released

Comment 6 Itamar Heim 2013-06-11 09:49:59 UTC

3.2 has been released