November 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin : mouse hover in volumes Activities column shows a box
by kmayilsa＠redhat.com 20 Nov '13

20 Nov '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : mouse hover in volumes Activities column shows a box ...................................................................... webadmin : mouse hover in volumes Activities column shows a box Drop down menu in Activities column for volumes should have a box on mouse hover Change-Id: I807ac3fd92ef91f5524148f768cf176f3e811614 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1019559 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationTemplates.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/table/column/MenuCell.java 2 files changed, 14 insertions(+), 2 deletions(-) Approvals: anmolbabu: Verified Kanagaraj M: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20251 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I807ac3fd92ef91f5524148f768cf176f3e811614 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: same exception is logged multiple times in engine log
by amureini＠redhat.com 20 Nov '13

20 Nov '13

Allon Mureinik has submitted this change and it was merged. Change subject: engine: same exception is logged multiple times in engine log ...................................................................... engine: same exception is logged multiple times in engine log In order to remove multiple places where information are logged new runtime exception was introduced and VDSCommandBase do not log any information for it in the catch block. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=982686 Change-Id: Icbc476f1f5799967e5776ede3dcfd462eada2ce9 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/BrokerCommandBase.java 1 file changed, 2 insertions(+), 3 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21159 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icbc476f1f5799967e5776ede3dcfd462eada2ce9 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Add ppc64 CPUs to the CPU list
by rgolan＠redhat.com 20 Nov '13

20 Nov '13

Roy Golan has submitted this change and it was merged. Change subject: core: Add ppc64 CPUs to the CPU list ...................................................................... core: Add ppc64 CPUs to the CPU list * This introduces some ppc64 CPUs to the ServerCPUList. * The ClusterEmulatedMachines configuration value was changed to include the "pseries" emulated machine. * The ArchitectureType enum (and its correspoding REST API type) was changed to include the ppc64 architecture * The CpuFlagsManagerHandler class was modified to handle the IBM processor vendor Change-Id: Id56ef93cd377bdadc10db008618ab4bbb8ed7170 Signed-off-by: Vitor de Lima <vitor.lima(a)eldorado.org.br> Signed-off-by: Gustavo Pedrosa <gustavo.pedrosa(a)eldorado.org.br> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CpuFlagsManagerHandler.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ArchitectureType.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/Architecture.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/CPUMapper.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 5 files changed, 39 insertions(+), 7 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20614 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id56ef93cd377bdadc10db008618ab4bbb8ed7170 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa(a)eldorado.org.br> Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa(a)eldorado.org.br> Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vitor.lima(a)eldorado.org.br> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core:there are dos end of lines in engine.log
by emesika＠redhat.com 20 Nov '13

20 Nov '13

Eli Mesika has submitted this change and it was merged. Change subject: core:there are dos end of lines in engine.log ...................................................................... core:there are dos end of lines in engine.log Removing the '\r\n' from the message. Change-Id: I73eae1a3d76ffca00089ace3e9fc371950da30ff Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1015052 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/CreateImageVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/CreateSnapshotVDSCommand.java 2 files changed, 0 insertions(+), 30 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20983 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I73eae1a3d76ffca00089ace3e9fc371950da30ff Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: utils: ForwardServlet: enable accepting components before qu...
by mpastern＠redhat.com 20 Nov '13

20 Nov '13

Michael Pasternak has submitted this change and it was merged. Change subject: utils: ForwardServlet: enable accepting components before query parameters ...................................................................... utils: ForwardServlet: enable accepting components before query parameters Example: http://host/api/events;max=2 As more edge conditions, apply exceptional style. Change-Id: Icfc5fd7bf98d3461ce8251c301de310237e0507c Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/root/src/main/webapp/WEB-INF/web.xml M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/servlet/ForwardServlet.java 2 files changed, 26 insertions(+), 20 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21335 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icfc5fd7bf98d3461ce8251c301de310237e0507c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: avoid confusing messages on db credentials
by iheim＠redhat.com 19 Nov '13

19 Nov '13

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: avoid confusing messages on db credentials ...................................................................... packaging: setup: avoid confusing messages on db credentials removed confusing message: Using existing credentials If we're not asking for credentials, if we're not provisioning it's implicit and if we're provisioning it's not true. Change-Id: I0a83fb945c49c7c2c2b4ad49d0dbd87e42144d54 Bug-Url: https://bugzilla.redhat.com/1030829 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/db/connection.py 1 file changed, 0 insertions(+), 4 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21402 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0a83fb945c49c7c2c2b4ad49d0dbd87e42144d54 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core: support pin-to-any
by iheim＠redhat.com 19 Nov '13

19 Nov '13

Itamar Heim has submitted this change and it was merged. Change subject: core: support pin-to-any ...................................................................... core: support pin-to-any Fixing a regression that forces to specify a host for non-migratable VMs. Change-Id: I10d60bb9db2506a63a19f5e75446ddac32b8dc3e Bug-Url: https://bugzilla.redhat.com/1021574 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/policyunits/PinToHostPolicyUnit.java 1 file changed, 11 insertions(+), 3 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21324 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I10d60bb9db2506a63a19f5e75446ddac32b8dc3e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: gluster: Remove virt related mange event options for gluster...
by ofrenkel＠redhat.com 19 Nov '13

19 Nov '13

Omer Frenkel has submitted this change and it was merged. Change subject: gluster: Remove virt related mange event options for gluster only mode. ...................................................................... gluster: Remove virt related mange event options for gluster only mode. Currently "Add event notification" dialog for user Event Notifier displays virt specific event options also for the gluster only application mode.This patch hides virt related event options from "Manage Events" for gluster only mode. Change-Id: I7fec833cc9322ad0ee6cd5c48ee54fceafc00f21 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=973638 Signed-off-by: Timothy Asir <tjeyasin(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/EventNotificationEntity.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/VdcEventNotificationUtils.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/LocalizedEnums.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 4 files changed, 8 insertions(+), 5 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20468 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7fec833cc9322ad0ee6cd5c48ee54fceafc00f21 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Timothy Asir <tjeyasin(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Timothy Asir <tjeyasin(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: build: ovirt-hosted-engine-ha-0.1.0-0.6.beta1
by gpadgett＠redhat.com 19 Nov '13

19 Nov '13

Greg Padgett has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-ha-0.1.0-0.6.beta1 ...................................................................... build: ovirt-hosted-engine-ha-0.1.0-0.6.beta1 Change-Id: If1d835181eb5aa1800eca496a19a8433f9896273 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M .gitignore M configure.ac M ovirt-hosted-engine-ha.spec.in 3 files changed, 6 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified Martin Sivák: Looks good to me, but someone else must approve Greg Padgett: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21410 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If1d835181eb5aa1800eca496a19a8433f9896273 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: reports: enable full text in ods export format
by ydary＠redhat.com 19 Nov '13

19 Nov '13

Yaniv Dary has submitted this change and it was merged. Change subject: reports: enable full text in ods export format ...................................................................... reports: enable full text in ods export format Change-Id: Ia4ccea7cf83f7a475f8a29928f5d71866f310be2 Bug-Url: https://bugzilla.redhat.com/722819 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M server-customizations/WEB-INF/classes/jasperreports.properties 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21419 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia4ccea7cf83f7a475f8a29928f5d71866f310be2 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0