December 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: api: fixing a typo in api.xsd
by emesika＠redhat.com 24 Dec '13

24 Dec '13

Eli Mesika has submitted this change and it was merged. Change subject: api: fixing a typo in api.xsd ...................................................................... api: fixing a typo in api.xsd Change-Id: Icd7b7aab9a126d16a3575184b329ada77672151e Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22226 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icd7b7aab9a126d16a3575184b329ada77672151e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Delete jobs that their steps have no async-tasks
by emesika＠redhat.com 24 Dec '13

24 Dec '13

Eli Mesika has submitted this change and it was merged. Change subject: engine: Delete jobs that their steps have no async-tasks ...................................................................... engine: Delete jobs that their steps have no async-tasks Sometimes async tasks complete but there were database errors when closing the steps and jobs. This patch updates the periodic job deletion to delete such jobs. Change-Id: Iae00d615bff79f42d8428b867f1816d6e0aa9a72 Bug-Url: https://bugzilla.redhat.com/1040952 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M packaging/dbscripts/job_sp.sql 1 file changed, 11 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22474 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae00d615bff79f42d8428b867f1816d6e0aa9a72 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Anonymous Coward #1000524 Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Hosts SearchBox doesn't filter correctly
by emesika＠redhat.com 24 Dec '13

24 Dec '13

Eli Mesika has submitted this change and it was merged. Change subject: engine: Hosts SearchBox doesn't filter correctly ...................................................................... engine: Hosts SearchBox doesn't filter correctly Hosts - SearchBox "Host:user.usrname =*" doesn't filter correctly Change-Id: I420b7705596335b9be6d6d8374a11f7542a03db9 Bug-Url: https://bugzilla.redhat.com/1006231 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SearchObjectAutoCompleter.java M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SyntaxChecker.java 2 files changed, 5 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22438 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I420b7705596335b9be6d6d8374a11f7542a03db9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: build: validate shell scripts
by sbonazzo＠redhat.com 24 Dec '13

24 Dec '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: build: validate shell scripts ...................................................................... packaging: build: validate shell scripts Change-Id: Ib16140e4abcb3740a51f4c3b77b9eee3a6e5cffd Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile A build/shell-check.sh M ovirt-engine.spec.in M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.sysv.in M packaging/services/ovirt-engine/ovirt-engine.sysv.in M packaging/services/ovirt-websocket-proxy/ovirt-websocket-proxy.sysv.in M packaging/setup/dbutils/encodingvalidator.sh 7 files changed, 54 insertions(+), 15 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22332 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib16140e4abcb3740a51f4c3b77b9eee3a6e5cffd Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(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-engine[master]: core: [Fix] NPE in UpdateVmTemplateCommand
by ofrenkel＠redhat.com 24 Dec '13

24 Dec '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: [Fix] NPE in UpdateVmTemplateCommand ...................................................................... core: [Fix] NPE in UpdateVmTemplateCommand Fix NPE in the UpdateVmTemplateCommand when checking if the watchdog model is supported. Change-Id: I73f74f9899fa39c2b651a6f1b83376ddade6bf11 Signed-off-by: Gustavo Pedrosa <gustavo.pedrosa(a)eldorado.org.br> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmTemplateCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22574 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I73f74f9899fa39c2b651a6f1b83376ddade6bf11 Gerrit-PatchSet: 2 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: Omer Frenkel <ofrenkel(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[ovirt-engine-3.3]: engine: Vnic live update should be blocked if pm enabled
by iheim＠redhat.com 24 Dec '13

24 Dec '13

Itamar Heim has submitted this change and it was merged. Change subject: engine: Vnic live update should be blocked if pm enabled ...................................................................... engine: Vnic live update should be blocked if pm enabled Any vnic live update action should be blocked either if the new profile is marked for port mirroring or if the existing profile is configured with port mirroring. Such change requires unplugging and plugging the vnic. Change-Id: I5101e769d33db8b1fa2cfcd0ab0bd766fd9ac13f Bug-Url: https://bugzilla.redhat.com/1045079 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/UpdateVmInterfaceCommand.java 1 file changed, 7 insertions(+), 13 deletions(-) Approvals: Mike Kolesnik: Looks good to me, approved Moti Asayag: Verified -- To view, visit http://gerrit.ovirt.org/22666 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5101e769d33db8b1fa2cfcd0ab0bd766fd9ac13f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Vnic live update should be blocked if pm enabled
by masayag＠redhat.com 23 Dec '13

23 Dec '13

Moti Asayag has submitted this change and it was merged. Change subject: engine: Vnic live update should be blocked if pm enabled ...................................................................... engine: Vnic live update should be blocked if pm enabled Any vnic live update action should be blocked either if the new profile is marked for port mirroring or if the existing profile is configured with port mirroring. Such change requires unplugging and plugging the vnic. Change-Id: I5101e769d33db8b1fa2cfcd0ab0bd766fd9ac13f Bug-Url: https://bugzilla.redhat.com/1045079 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/vm/UpdateVmInterfaceCommand.java 1 file changed, 7 insertions(+), 13 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22660 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5101e769d33db8b1fa2cfcd0ab0bd766fd9ac13f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Basic cleanup in InterfaceDaoDbFacadeImpl
by lvernia＠redhat.com 23 Dec '13

23 Dec '13

Lior Vernia has submitted this change and it was merged. Change subject: core: Basic cleanup in InterfaceDaoDbFacadeImpl ...................................................................... core: Basic cleanup in InterfaceDaoDbFacadeImpl Removed a warning suppression that was no longer required, and changed a variable name from "stats" to "entity" where it was more fitting. Change-Id: Id08a742d431232097a6c2a9556b3aab6c872d336 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/InterfaceDaoDbFacadeImpl.java 1 file changed, 38 insertions(+), 39 deletions(-) Approvals: Lior Vernia: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22598 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id08a742d431232097a6c2a9556b3aab6c872d336 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: [Fix] Remove HotPlugUnsupportedOsList config value
by emesika＠redhat.com 23 Dec '13

23 Dec '13

Eli Mesika has submitted this change and it was merged. Change subject: engine: [Fix] Remove HotPlugUnsupportedOsList config value ...................................................................... engine: [Fix] Remove HotPlugUnsupportedOsList config value The property HotPlugUnsupportedOsList was remove after the migration to the osinfo file, but the name is wrong in this script. Change-Id: I67a79cff58983540fa22b60501c5d390a3972b2a Signed-off-by: Gustavo Pedrosa <gustavo.pedrosa(a)eldorado.org.br> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22588 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I67a79cff58983540fa22b60501c5d390a3972b2a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa(a)eldorado.org.br> Gerrit-Reviewer: Eli Mesika <elimesika(a)gmail.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa(a)eldorado.org.br> Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(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]: webadmin: update hash name for remove brick status
by kmayilsa＠redhat.com 23 Dec '13

23 Dec '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin: update hash name for remove brick status ...................................................................... webadmin: update hash name for remove brick status Updating the hash name for remove brick and rebalance status pop ups. Change-Id: Ia7b359a9a0f9c004b1ab6b654d77bd28a08027c6 Bug-Url: https://bugzilla.redhat.com/1044932 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java 2 files changed, 4 insertions(+), 3 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved Ramesh N: Verified -- To view, visit http://gerrit.ovirt.org/22578 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia7b359a9a0f9c004b1ab6b654d77bd28a08027c6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0