Eli Mesika has posted comments on this change.

Change subject: core: GraphicsInfo db persistence part
......................................................................


Patch Set 31:

(1 comment)

http://gerrit.ovirt.org/#/c/28570/31/packaging/dbscripts/upgrade/03_06_0140_add_graphics_info_columns.sql
File packaging/dbscripts/upgrade/03_06_0140_add_graphics_info_columns.sql:

Line 87: update vm_dynamic set spice_port = display where display_type = 0;
Line 88: update vm_dynamic set spice_tls_port = display_secure_port where 
display_type = 0;
Line 89: update vm_dynamic set spice_ip = display_ip where display_type = 0;
Line 90: update vm_dynamic set vnc_port = display where display_type = 1;
Line 91: update vm_dynamic set vnc_ip = display_ip where display_type = 1;
please do all those 5 updates two statement

update vm_dynamic set spice_port = display, spice_tls_port = 
display_secure_port, spice_ip = display_ip where display_type = 0;

update vm_dynamic set vnc_port = display, vnc_ip = display_ip where 
display_type = 1;


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5a129454fc64aa34493b0495fb76f22f3d667e14
Gerrit-PatchSet: 31
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Frank Kobzik <fkob...@redhat.com>
Gerrit-Reviewer: Arik Hadas <aha...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Frank Kobzik <fkob...@redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to