Change in ovirt-log-collector[master]: inventory: Improve SQL query for vms pinned

Code Review gerrit at ovirt.org
Tue Aug 29 13:17:57 UTC 2017


>From Douglas Schilling Landgraf <dougsland at redhat.com>:

Douglas Schilling Landgraf has submitted this change and it was merged. ( https://gerrit.ovirt.org/81138 )

Change subject: inventory: Improve SQL query for vms pinned
......................................................................


inventory: Improve SQL query for vms pinned

The column dedicated_vm_for_vd from vm_static can contain a single
hypervisor entry or multiple hypervisors (using delimiter comma).
Based on that, we are extacting the hypervisors in the WITH statement
and using it in the INNER JOIN.

Change-Id: Ic6c00c794962687e886f0ab3aef11557965da21b
Signed-off-by: Douglas Schilling Landgraf <dougsland at redhat.com>
---
M src/inventory_report/produceReport/sqls/vms_query_pinned_virtual_machines.sql
1 file changed, 20 insertions(+), 3 deletions(-)

Approvals:
  Douglas Schilling Landgraf: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/81138
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic6c00c794962687e886f0ab3aef11557965da21b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-log-collector
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list