Sahina Bose has posted comments on this change.

Change subject: engine: Do not access storage domain for gluster only hosts
......................................................................


Patch Set 4:

(1 comment)

Query is used in StorageHandlingCommandBase.
In ActivateStorageDomainCommand::refreshAllVdssInPool - all vds attached to the 
storage pool are moved to non-operational. this should happen only for vds that 
supports virt service.

http://gerrit.ovirt.org/#/c/28434/4/packaging/dbscripts/vds_sp.sql
File packaging/dbscripts/vds_sp.sql:

Line 887:     AS $procedure$
Line 888: BEGIN
Line 889:     BEGIN
Line 890:         RETURN QUERY SELECT vds.*
Line 891:         FROM vds 
> please remove whitespace
Done
Line 892:         INNER JOIN vds_groups vdsgroup ON vds.vds_group_id = 
vdsgroup.vds_group_id
Line 893:         WHERE (vds.status = v_status) AND (vds.storage_pool_id = 
v_storage_pool_id)
Line 894:         AND vdsgroup.virt_service = true;
Line 895:     END;


-- 
To view, visit http://gerrit.ovirt.org/28434
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic6e9c02bf68146975987c6cb48cccf5c198a2e5b
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to