November 2012 - Engine-commits - oVirt List Archives

Change in otopi[master]: packaging: update location of sources after server move
by Alon Bar-Lev 30 Nov '12

30 Nov '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: update location of sources after server move ...................................................................... packaging: update location of sources after server move Change-Id: If2dbd31da63801094757c6e971512897f006c451 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M otopi.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9602 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If2dbd31da63801094757c6e971512897f006c451 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: packaging: update location of sources after server move
by Alon Bar-Lev 30 Nov '12

30 Nov '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: update location of sources after server move ...................................................................... packaging: update location of sources after server move Change-Id: I486f9fdedcbc903978a2bfcdb6dc5ef3e7d8119a Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-host-deploy.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9601 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I486f9fdedcbc903978a2bfcdb6dc5ef3e7d8119a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: adding engine Maintenance and Prepare mode.
by emesika＠redhat.com 29 Nov '12

29 Nov '12

Eli Mesika has submitted this change and it was merged. Change subject: core: adding engine Maintenance and Prepare mode. ...................................................................... core: adding engine Maintenance and Prepare mode. Support for that was already merged in http://gerrit.ovirt.org/#/c/8669/10 Solution above assumed that configuration values can be reloaded while the engine is still running. While this is true when working from API, it is not working from the engine-config, therefor, we must take the config values directly from the database and not rely on the config cache. Change-Id: Iae5d1148c9c2868abd0061700bb9988242dd8faf Signed-off-by: Eli Mesika <emesika(a)redhat.com> Bug-Url:https://bugzilla.redhat.com/868639 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java 1 file changed, 12 insertions(+), 9 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9568 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae5d1148c9c2868abd0061700bb9988242dd8faf Gerrit-PatchSet: 4 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: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Fix new/edit pool dialogs for ja locale
by derez＠redhat.com 29 Nov '12

29 Nov '12

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: Fix new/edit pool dialogs for ja locale ...................................................................... webadmin: Fix new/edit pool dialogs for ja locale - Change the layout of some components in the new & edit pool dialogs to use relative values for their locations instead of absolute values that way, the dialog's layout supports long japanese terms - Rename newPoolPrestartedVmsIcon style to prestartedVmsIcon as this style is used for edit dialog also - Fix some typos (incrase -> increase) Change-Id: I5b9ead456e355f66eb4d8363fe0706ee4552c9fb Bug-Url: https://bugzilla.redhat.com/870251 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml 1 file changed, 18 insertions(+), 25 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9364 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5b9ead456e355f66eb4d8363fe0706ee4552c9fb Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0

Change in ovirt-iso-uploader[master]: cleanup: style by pep8
by kroberts＠redhat.com 29 Nov '12

29 Nov '12

Keith Robertson has submitted this change and it was merged. Change subject: cleanup: style by pep8 ...................................................................... cleanup: style by pep8 Change-Id: I0f5abd5dc6123937550300c132ffc1590a3e3851 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/engine-iso-uploader.py 1 file changed, 634 insertions(+), 294 deletions(-) Approvals: Keith Robertson: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8854 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0f5abd5dc6123937550300c132ffc1590a3e3851 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com>

1 0

Change in ovirt-iso-uploader[master]: cleanup: code warning by pyflakes
by kroberts＠redhat.com 29 Nov '12

29 Nov '12

Keith Robertson has submitted this change and it was merged. Change subject: cleanup: code warning by pyflakes ...................................................................... cleanup: code warning by pyflakes Change-Id: I012880df9e9d6b81b52cf311c24385342fe956c6 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/engine-iso-uploader.py 1 file changed, 7 insertions(+), 13 deletions(-) Approvals: Keith Robertson: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8853 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I012880df9e9d6b81b52cf311c24385342fe956c6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: bridge: temporary workaround for firewalld behaviour
by Alon Bar-Lev 29 Nov '12

29 Nov '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: bridge: temporary workaround for firewalld behaviour ...................................................................... bridge: temporary workaround for firewalld behaviour Currently, addNetwork just hangs if firewalld is up while creating the bridge. So as temporary workaround stop firewalld before creating the bridge. Change-Id: Ib6c248a7c49492f7889fc71f11ae1dad1e3434ae Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=869963 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/ovirt-host-deploy/vdsm/bridge.py 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9588 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib6c248a7c49492f7889fc71f11ae1dad1e3434ae Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in otopi[master]: iptables: disable and stop firewalld to avoid conflict
by Alon Bar-Lev 29 Nov '12

29 Nov '12

Alon Bar-Lev has submitted this change and it was merged. Change subject: iptables: disable and stop firewalld to avoid conflict ...................................................................... iptables: disable and stop firewalld to avoid conflict Change-Id: I423846092615a25352539cb0abc19088408ac93e Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/otopi/network/iptables.py 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9587 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I423846092615a25352539cb0abc19088408ac93e Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-engine[master]: bootstrap: ovirt-node upgrade: create destination directory
by yzaslavs＠redhat.com 29 Nov '12

29 Nov '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: bootstrap: ovirt-node upgrade: create destination directory ...................................................................... bootstrap: ovirt-node upgrade: create destination directory One would expect that there will be an ovirt-node upgrade script, however there is none, and the /data/updates is vdsm specific path, which was created by vdsm init script, and then creation was moved to vdsm-reg service. vdsm-reg is going to be retired soon, so move the creation of this directory to the component that sends the iso image. Better solution would be a node upgrade script that receives the image via stdin. Change-Id: I5519bd63383feba9b3bf971543b0d630c24bbca0 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/OVirtNodeUpgrade.java 1 file changed, 31 insertions(+), 0 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9524 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5519bd63383feba9b3bf971543b0d630c24bbca0 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: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: frontend: VM template change shouldn't change selected clust...
by tjelinek＠redhat.com 29 Nov '12

29 Nov '12

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: VM template change shouldn't change selected cluster ...................................................................... frontend: VM template change shouldn't change selected cluster Changing VM template using combobox should not change selected cluster inside new virtual machine dialog when multiple clusters and templates are available inside one DS. Initially, the VM template has a binding to a DS and a cluster, but the VM template should be available across clusters of the same DS now. Change-Id: I23bf4aaa9f16d23996b2a28ef778748c1e622e38 Bug-Url: https://bugzilla.redhat.com/877981 Signed-off-by: Libor Spevak <lspevak(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewVmModelBehavior.java 1 file changed, 0 insertions(+), 7 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Libor Spevak: Verified -- To view, visit http://gerrit.ovirt.org/9512 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23bf4aaa9f16d23996b2a28ef778748c1e622e38 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Libor Spevak <lspevak(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Libor Spevak <lspevak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Add Intel Haswell model
by rgolan＠redhat.com 29 Nov '12

29 Nov '12

Roy Golan has submitted this change and it was merged. Change subject: core: Add Intel Haswell model ...................................................................... core: Add Intel Haswell model Adding Intel Haswell model to the engine's config Change-Id: I269a041227453fb3af11ce82e0db9694b4be9b01 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9392 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I269a041227453fb3af11ce82e0db9694b4be9b01 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com>

1 0

Change in ovirt-engine[master]: core: add Opteron G5 model
by rgolan＠redhat.com 29 Nov '12

29 Nov '12

Roy Golan has submitted this change and it was merged. Change subject: core: add Opteron G5 model ...................................................................... core: add Opteron G5 model Adding AMD Opteron G5 model to the engine's config Change-Id: I6eb396a88898924459e6ba5ff58356adddc56707 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Itamar Heim: Looks good to me, approved Roy Golan: Verified -- To view, visit http://gerrit.ovirt.org/9389 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6eb396a88898924459e6ba5ff58356adddc56707 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: remove unneeded reflection use in SyntaxChecker
by rgolan＠redhat.com 29 Nov '12

29 Nov '12

Roy Golan has submitted this change and it was merged. Change subject: core: remove unneeded reflection use in SyntaxChecker ...................................................................... core: remove unneeded reflection use in SyntaxChecker Using the built-in valueOf method of Enum instead of fancy, expensive other ways to try to get the enum value. The former code did eventually a valueOf check which is case sensitive and then another EnumUtils method which was case *insensitive*. Since the inner EnumUtils call would only be read on a match there is no point in repeating the enum fetching twice, and in a very expensive way when we have a ready made method for it. Change-Id: Idd74aad49d292e64dec6f5925ffd186f13b9887c Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/EnumCompat.java M backend/manager/modules/searchbackend/pom.xml M backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/SyntaxChecker.java M backend/manager/modules/searchbackend/src/test/java/org/ovirt/engine/core/searchbackend/SyntaxCheckerTest.java 4 files changed, 57 insertions(+), 20 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9461 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idd74aad49d292e64dec6f5925ffd186f13b9887c Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Adding force extend support to extendVG
by derez＠redhat.com 29 Nov '12

29 Nov '12

Daniel Erez has submitted this change and it was merged. Change subject: core: Adding force extend support to extendVG ...................................................................... core: Adding force extend support to extendVG * Added 'SupportForceExtendVG' configuration value (true for version 3.2). * Passed 'force' flag to 'extendStorageDomain' iff 'SupportForceExtendVG' is true. * Added new extendStorageDomain method signatures in 'IrsServer' files. * StorageListModel - passed 'force' flag to ExtendSANStorageDomainParameters Change-Id: I3b9196333f680bfad8bb421ad0dad8843b9c8e2b Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ExtendSANStorageDomainCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ExtendSANStorageDomainParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/ExtendStorageDomainVDSCommandParameters.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/ExtendStorageDomainVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IIrsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsServerConnector.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsServerWrapper.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java 10 files changed, 79 insertions(+), 7 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9343 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3b9196333f680bfad8bb421ad0dad8843b9c8e2b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: LoginPopupView - construct form as table
by derez＠redhat.com 28 Nov '12

28 Nov '12

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: LoginPopupView - construct form as table ...................................................................... webadmin: LoginPopupView - construct form as table Construct login form as a table of two columns: labels and editors. I.e. labels column is auto expandable (required for localization). Change-Id: I346a858bcd26868cc2eb77c832cc553f415cb928 Bug-Url: https://bugzilla.redhat.com/867428 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/login/view/LoginPopupView.java M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/login/view/LoginPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/login/view/LoginPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/login/view/LoginPopupView.ui.xml 4 files changed, 71 insertions(+), 56 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9153 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I346a858bcd26868cc2eb77c832cc553f415cb928 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[master]: userportal: rephrase user-portal login dialog title
by derez＠redhat.com 28 Nov '12

28 Nov '12

Daniel Erez has submitted this change and it was merged. Change subject: userportal: rephrase user-portal login dialog title ...................................................................... userportal: rephrase user-portal login dialog title Rephrased title: "Open Virtualization User Portal" (resembles to webadmin's login title: "Open Virtualization Manager") Change-Id: I315d2dbdc5209a93a944ed24f407cbf7d18fa4e7 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/ApplicationConstants.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9143 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I315d2dbdc5209a93a944ed24f407cbf7d18fa4e7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: validate max lun disk attachment
by rgolan＠redhat.com 28 Nov '12

28 Nov '12

Roy Golan has submitted this change and it was merged. Change subject: core: validate max lun disk attachment ...................................................................... core: validate max lun disk attachment The check for the limit of PCI addresses was missing when adding a lun disk. This patch also contains some refactoring to make a clear separation between canDo's on adding disk with and without a VM and calls the limit check only when there is a VM in hand. Change-Id: I4a82b3ed85c89beb8df70315ad457f54b366d69d Bug-Url: http://bugzilla.redhat.com/876127 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddDiskCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddDiskToVmCommandTest.java 3 files changed, 107 insertions(+), 32 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9409 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a82b3ed85c89beb8df70315ad457f54b366d69d Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Improving almost all retrieves of storage_domains ob...
by mkublin＠redhat.com 28 Nov '12

28 Nov '12

Michael Kublin has submitted this change and it was merged. Change subject: engine: Improving almost all retrieves of storage_domains object in the system ...................................................................... engine: Improving almost all retrieves of storage_domains object in the system The following patch should improve all queries that retrieving a storage_domains object at the system except iso domain. The problem was in fn_get_storage_domain_shared_status_by_domain_id, all calculation should be done only for iso storage domain. The following query was also reported as problematic in bug #879930, so I added a bug number to commit message Change-Id: I617a631c63cd38f058e6c0711dea3a43062874fd Bug-Url: https://bugzilla.redhat.com/879930 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/dbscripts/create_functions.sql M backend/manager/dbscripts/create_views.sql 2 files changed, 59 insertions(+), 48 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9511 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I617a631c63cd38f058e6c0711dea3a43062874fd Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Asaf Shakarchi <asaf(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-reports[master]: reports: now using 3, 2 version views in all reports.
by ydary＠redhat.com 28 Nov '12

28 Nov '12

Yaniv Dary has submitted this change and it was merged. Change subject: reports: now using 3,2 version views in all reports. ...................................................................... reports: now using 3,2 version views in all reports. Fixed any changes that broke reports. Change-Id: I027efb65d4fffac9ca0b3f099c11d084199c1f24 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M reports/repository_files/resources/Reports/Executive/active_vms_by_os_br18_files/active_vms_by_os_br18_jrxml.data M reports/repository_files/resources/Reports/Executive/active_vms_by_os_br18_files/full details.jrxml.data M reports/repository_files/resources/Reports/Executive/cluster_capacity_vs_usage_br19_files/cluster_capacity_vs_usage_br19_jrxml.data M reports/repository_files/resources/Reports/Executive/cluster_capacity_vs_usage_br19_files/full details br19.jrxml.data M reports/repository_files/resources/Reports/Executive/host_os_break_down_BR22_files/full details.jrxml.data M reports/repository_files/resources/Reports/Executive/host_os_break_down_BR22_files/host_os_break_down_BR22_jrxml.data M reports/repository_files/resources/Reports/Executive/summary_of_host_usage_resources_br17_files/full details.jrxml.data M reports/repository_files/resources/Reports/Executive/summary_of_host_usage_resources_br17_files/summary_of_host_usage_resources_br17_jrxml.data M reports/repository_files/resources/Reports/Inventory/Hosts_Inventory_files/Hosts_Inventory_jrxml.data M reports/repository_files/resources/Reports/Inventory/Hosts_Inventory_files/full details br1.jrxml.data M reports/repository_files/resources/Reports/Inventory/Storage_Domain_Report_BR21_files/Storage_Domain_Report_BR21_jrxml.data M reports/repository_files/resources/Reports/Inventory/Storage_Domain_Report_BR21_files/full details br21.jrxml.data M reports/repository_files/resources/Reports/Inventory/VM_Inventory_files/VM_Inventory_jrxml.data M reports/repository_files/resources/Reports/Inventory/VM_Inventory_files/full details.jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/cluster_quality_of_service_hosts_br6.xml M reports/repository_files/resources/Reports/Service_level/Hosts/cluster_quality_of_service_hosts_br6_files/cluster_quality_of_service_hosts_br6_jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/cluster_quality_of_service_hosts_br6_files/full details br13.jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/cluster_quality_of_service_hosts_br6_files/header cluster name.jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/cluster_uptime_br7.xml M reports/repository_files/resources/Reports/Service_level/Hosts/cluster_uptime_br7_files/br7 full details.jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/cluster_uptime_br7_files/br7_table.jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/cluster_uptime_br7_files/cluster_uptime_br7_jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/single_host_uptime_br8.xml M reports/repository_files/resources/Reports/Service_level/Hosts/single_host_uptime_br8_files/br8 full details.jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/single_host_uptime_br8_files/single_host_uptime_br8_jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/top_10_downtime_hosts_br7b.xml M reports/repository_files/resources/Reports/Service_level/Hosts/top_10_downtime_hosts_br7b_files/br7b full details.jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/top_10_downtime_hosts_br7b_files/top_10_downtime_hosts_br7b_jrxml.data M reports/repository_files/resources/Reports/Service_level/VMs/cluster_quality_of_service_vms_br13.xml M reports/repository_files/resources/Reports/Service_level/VMs/cluster_quality_of_service_vms_br13_files/cluster_quality_of_service_vms_br13_jrxml.data M reports/repository_files/resources/Reports/Service_level/VMs/cluster_quality_of_service_vms_br13_files/full details.jrxml.data M reports/repository_files/resources/Reports/Service_level/VMs/cluster_quality_of_service_vms_br13_files/header cluster name.jrxml.data M reports/repository_files/resources/Reports/Service_level/VMs/virtual_servers_uptime_br14.xml M reports/repository_files/resources/Reports/Service_level/VMs/virtual_servers_uptime_br14_files/br14 full details.jrxml.data M reports/repository_files/resources/Reports/Service_level/VMs/virtual_servers_uptime_br14_files/br14_table.jrxml.data M reports/repository_files/resources/Reports/Service_level/VMs/virtual_servers_uptime_br14_files/virtual_servers_uptime_br14_jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/Five_most_utilized_hosts_over_time.xml M reports/repository_files/resources/Reports/Trend/Hosts/Five_most_utilized_hosts_over_time_files/Five_most_utilized_hosts_over_time_jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/Five_most_utilized_hosts_over_time_files/br7 full details.jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/five_least_utilized_hosts_br5.xml M reports/repository_files/resources/Reports/Trend/Hosts/five_least_utilized_hosts_br5_files/br7 full details.jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/five_least_utilized_hosts_br5_files/five_least_utilized_hosts_br5_jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/multiple_hosts_resource_usage_br3.xml M reports/repository_files/resources/Reports/Trend/Hosts/multiple_hosts_resource_usage_br3_files/br3 full details.jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/multiple_hosts_resource_usage_br3_files/multiple_hosts_resource_usage_br3_jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/single_host_resource_br2a.xml M reports/repository_files/resources/Reports/Trend/Hosts/single_host_resource_br2a_files/br2 full details.jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/single_host_resource_br2a_files/single_host_resource_br2a_jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/single_host_resource_usage_dow_br2b.xml M reports/repository_files/resources/Reports/Trend/Hosts/single_host_resource_usage_dow_br2b_files/br2 full details.jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/single_host_resource_usage_dow_br2b_files/single_host_resource_usage_dow_br2b.data M reports/repository_files/resources/Reports/Trend/Hosts/single_host_resource_usage_hour_of_day_br2c.xml M reports/repository_files/resources/Reports/Trend/Hosts/single_host_resource_usage_hour_of_day_br2c_files/br2 full details.jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/single_host_resource_usage_hour_of_day_br2c_files/single_host_resource_usage_hour_of_day_br2c_jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/five_least_utilized_vms_Over_Time_BR12.xml M reports/repository_files/resources/Reports/Trend/Virtual_machines/five_least_utilized_vms_Over_Time_BR12_files/five_least_utilized_vms_Over_Time_BR12_jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/five_least_utilized_vms_Over_Time_BR12_files/full details.jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/five_most_utilized_vms_over_time_br11.xml M reports/repository_files/resources/Reports/Trend/Virtual_machines/five_most_utilized_vms_over_time_br11_files/five_most_utilized_vms_over_time_br11_jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/five_most_utilized_vms_over_time_br11_files/full details.jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/multiple_vms_resource_usage_over_time_br16.xml M reports/repository_files/resources/Reports/Trend/Virtual_machines/multiple_vms_resource_usage_over_time_br16_files/multiple_vms_resource_usage_over_time_br16_jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/multiple_vms_resource_usage_over_time_br16_files/report3.jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/single_vm_resources_days_week_BR10B.xml M reports/repository_files/resources/Reports/Trend/Virtual_machines/single_vm_resources_days_week_BR10B_files/br2 full details.jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/single_vm_resources_days_week_BR10B_files/single_vm_resources_days_week_BR10B_jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/single_vm_resources_hour_of_day_BR10c.xml M reports/repository_files/resources/Reports/Trend/Virtual_machines/single_vm_resources_hour_of_day_BR10c_files/br10 full details.jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/single_vm_resources_hour_of_day_BR10c_files/single_vm_resources_hour_of_day_BR10c_jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/single_vm_resources_over_time_BR10A.xml M reports/repository_files/resources/Reports/Trend/Virtual_machines/single_vm_resources_over_time_BR10A_files/br10 full details.jrxml.data M reports/repository_files/resources/Reports/Trend/Virtual_machines/single_vm_resources_over_time_BR10A_files/single_vm_resources_over_time_BR10A_jrxml.data M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/non-002dquery_IC/P_Period.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/non-002dquery_IC/is_deleted.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/cluster_ic/all_cluster_with_vms/P_Cluster_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/cluster_ic/all_clusters_with_hosts/P_Cluster_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/datacenter_ic/all_dcs_with_entities/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/datacenter_ic/all_dcs_with_hosts/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/datacenter_ic/all_dcs_with_storage/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/datacenter_ic/all_dcs_with_vms/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/host_ic/Hosts_List.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/host_ic/P_Host_Type.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/storage_ic/P_StorageDomain_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/storage_ic/P_Storage_Type.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/vm_ic/P_VM_Type.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Period_Affected/single_select/vm_ic/VM_List.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_Cluster_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_Host_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_Host_ListID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/P_Host_Type.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Hosts/active_hosts_select.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Storage/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Storage/P_StorageDomain_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Storage/P_Storage_Type.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_Storage/active_storage.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_VMs/Active_VMs.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_VMs/P_Cluster_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_VMs/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_VMs/P_VM_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_VMs/P_VM_ListID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/Regular_Input_Controls/Only_VMs/P_VM_Type.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/webadmin_dashboards_ic/P_Cluster_ID.xml M reports/repository_files/resources/reports_resources/JDBC/Input_Controls/webadmin_dashboards_ic/P_DataCenter_ID.xml M reports/repository_files/resources/reports_resources/embedded_reports/Inventory_Dashboard/Host_OS_Break_Down_BR22B_for_inventory_files/Host_OS_Break_Down_BR22B_for_inventory_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/Inventory_Dashboard/active_entities_by_os_BR18A_files/active_entities_by_os_BR18A_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/Inventory_Dashboard/storage_domain_by_Datacenter_BR23_files/storage_domain_by_Datacenter_BR23_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/System_Overview_Dashboard/datacenter_quality_of_service_hosts_br6b_files/datacenter_quality_of_service_hosts_br6b_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/System_Overview_Dashboard/summary_of_host_usage_resources_br17_files/summary_of_host_usage_resources_br17_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/Trends_Dashboard/Five_most_utilized_hosts_over_time_files/Five_most_utilized_hosts_over_time_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/Trends_Dashboard/five_most_utilized_vms_over_time_br11_files/five_most_utilized_vms_over_time_br11_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/Uptime_Dashboard/cluster_uptime_by_datacenter_br7a_files/cluster_uptime_by_datacenter_br7a_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/Uptime_Dashboard/cluster_uptime_by_datacenter_br7a_files/report5.jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/Uptime_Dashboard/virtual_servers_uptime_by_cluster_br14a_files/virtual_servers_uptime_by_cluster_br14a_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_entities_counts_list_dr3_files/cluster_entities_counts_list_dr3_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_ha_vm_uptime_status_dr13_files/cluster_ha_vm_uptime_status_dr13_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_host_resources_status_dr12_files/cluster_host_resources_status_dr12_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_host_uptime_status_dr6_files/cluster_host_uptime_status_dr6_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_summary_of_hosts_resources_usage_dr28_files/cluster_summary_of_hosts_resources_usage_dr28_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_top_five_busiest_hosts_dr23_files/cluster_top_five_busiest_hosts_dr23_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_vm_over_commit_status_dr14_files/cluster_vm_over_commit_status_dr14_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/no_used/cluster_capacity_vs_usage_cores_dr29_files/cluster_capacity_vs_usage_cores_dr29_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/no_used/cluster_capacity_vs_usage_mem_dr30_files/cluster_capacity_vs_usage_mem_dr30_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/no_used/cluster_top_five_least_busy_hosts_dr24_files/cluster_top_five_least_busy_hosts_dr24_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/no_used/top_five_ha_virtual_servers_uptime_dr19_files/top_five_ha_virtual_servers_uptime_dr19_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/no_used/top_five_hosts_uptime_dr17_files/top_five_hosts_uptime_dr17_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/top_five_ha_virtual_servers_downtime_dr20_files/top_five_ha_virtual_servers_downtime_dr20_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/top_five_hosts_downtime_dr18_files/top_five_hosts_downtime_dr18_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/datacenter_top_five_busiest_clusters_dr21_files/datacenter_top_five_busiest_clusters_dr21_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/dc_entities_counts_list_dr2_files/dc_entities_counts_list_dr2_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/dc_host_resources_status_dr11_files/dc_host_resources_status_dr11_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/dc_host_uptime_status_dr5_files/dc_host_uptime_status_dr5_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/dc_storage_space_status_dr8_files/dc_storage_space_status_dr8_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/dc_summary_of_cluster_hosts_resources_usage_dr27_files/dc_summary_of_cluster_hosts_resources_usage_dr27_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/dc_top_five_used_storage_domain_dr26_files/dc_top_five_used_storage_domain_dr26_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/not_used/datacenter_top_five_least_busy_clusters_dr22_files/datacenter_top_five_least_busy_clusters_dr22_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/not_used/dc_vm_over_commit_dr25_files/dc_vm_over_commit_dr25_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/not_used/top_five_clusters_host_uptime_dr15_files/top_five_clusters_host_uptime_dr15_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/top_five_clusters_host_downtime_dr16_files/top_five_clusters_host_downtime_dr16_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/system_dashboard/system_entities_counts_list_dr1_files/system_entities_counts_list_dr1_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/system_dashboard/system_host_resources_status_dr10_files/system_host_resources_status_dr10_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/system_dashboard/system_host_uptime_status_dr4_files/system_host_uptime_status_dr4_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_entities_counts_list_dr3_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_ha_vm_uptime_status_dr13_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_host_resources_status_dr12_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_host_uptime_status_dr6_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_summary_of_hosts_resources_usage_dr28_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_top_five_busiest_hosts_dr23_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_vm_over_commit_status_dr14_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/top_five_ha_virtual_servers_downtime_dr20_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/top_five_hosts_downtime_dr18_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/datacenter_top_five_busiest_clusters_dr21_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/dc_entities_counts_list_dr2_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/dc_host_resources_status_dr11_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/dc_host_uptime_status_dr5_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/dc_storage_space_status_dr8_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/dc_summary_of_cluster_hosts_resources_usage_dr27_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/dc_top_five_used_storage_domain_dr26_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/top_five_clusters_host_downtime_dr16_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/system_dashboard_files/system_entities_counts_list_dr1_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/system_dashboard_files/system_host_resources_status_dr10_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/system_dashboard_files/system_host_uptime_status_dr4_jrxml.data 162 files changed, 2,673 insertions(+), 2,734 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9549 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I027efb65d4fffac9ca0b3f099c11d084199c1f24 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

Change in ovirt-reports[master]: reports: added interactive tables to some of the reports
by ydary＠redhat.com 28 Nov '12

28 Nov '12

Yaniv Dary has submitted this change and it was merged. Change subject: reports: added interactive tables to some of the reports ...................................................................... reports: added interactive tables to some of the reports Inventory reports and host os break. Change-Id: Icc1047c34c27fcf48627b87c013a85845bd39e8c Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M reports/repository_files/resources/Reports/Executive/active_vms_by_os_br18.xml M reports/repository_files/resources/Reports/Executive/cluster_capacity_vs_usage_br19.xml M reports/repository_files/resources/Reports/Executive/host_os_break_down_BR22.xml M reports/repository_files/resources/Reports/Executive/host_os_break_down_BR22_files/host_os_break_down_BR22_jrxml.data M reports/repository_files/resources/Reports/Executive/summary_of_host_usage_resources_br17.xml M reports/repository_files/resources/Reports/Inventory/Hosts_Inventory.xml M reports/repository_files/resources/Reports/Inventory/Hosts_Inventory_files/Hosts_Inventory_jrxml.data M reports/repository_files/resources/Reports/Inventory/Hosts_Inventory_files/full details br1.jrxml.data D reports/repository_files/resources/Reports/Inventory/Hosts_Inventory_files/header cluster name.jrxml.data M reports/repository_files/resources/Reports/Inventory/Storage_Domain_Report_BR21.xml M reports/repository_files/resources/Reports/Inventory/VM_Inventory.xml M reports/repository_files/resources/Reports/Inventory/VM_Inventory_files/VM_Inventory_jrxml.data M reports/repository_files/resources/Reports/Inventory/VM_Inventory_files/full details.jrxml.data D reports/repository_files/resources/Reports/Inventory/VM_Inventory_files/header cluster name.jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/cluster_quality_of_service_hosts_br6_files/header cluster name.jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/cluster_uptime_br7_files/cluster_uptime_br7_jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/single_host_uptime_br8_files/single_host_uptime_br8_jrxml.data M reports/repository_files/resources/Reports/Service_level/Hosts/top_10_downtime_hosts_br7b_files/report5.jrxml.data M reports/repository_files/resources/Reports/Service_level/VMs/cluster_quality_of_service_vms_br13_files/header cluster name.jrxml.data M reports/repository_files/resources/Reports/Service_level/VMs/virtual_servers_uptime_br14_files/virtual_servers_uptime_br14_jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/Five_most_utilized_hosts_over_time_files/Five_most_utilized_hosts_over_time_jrxml.data M reports/repository_files/resources/Reports/Trend/Hosts/five_least_utilized_hosts_br5_files/five_least_utilized_hosts_br5_jrxml.data 22 files changed, 972 insertions(+), 740 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9548 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc1047c34c27fcf48627b87c013a85845bd39e8c 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

Change in ovirt-engine[master]: bootstrap: cleanup approve command
by yzaslavs＠redhat.com 28 Nov '12

28 Nov '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: bootstrap: cleanup approve command ...................................................................... bootstrap: cleanup approve command Do not use parent member if there is a method to access it. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=875528 Change-Id: I212c7b7ab58dd45c56c8c94d62c49a9d2236d863 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ApproveVdsCommand.java 1 file changed, 3 insertions(+), 6 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9452 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I212c7b7ab58dd45c56c8c94d62c49a9d2236d863 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: bootstrap: bootstrap: make approve command respect installat...
by yzaslavs＠redhat.com 28 Nov '12

28 Nov '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: bootstrap: bootstrap: make approve command respect installation failure ...................................................................... bootstrap: bootstrap: make approve command respect installation failure Current implementation assumes success at all results, which is invalid. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=875528 Change-Id: Iab010f40007ac11dd909350f60b6db1880edbf7f Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ApproveVdsCommand.java 1 file changed, 4 insertions(+), 12 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9443 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iab010f40007ac11dd909350f60b6db1880edbf7f Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: all-in-one: depend on ovirt-host-deploy-offline
by yzaslavs＠redhat.com 28 Nov '12

28 Nov '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: packaging: all-in-one: depend on ovirt-host-deploy-offline ...................................................................... packaging: all-in-one: depend on ovirt-host-deploy-offline The ovirt-host-deploy-offline configures the ovirt-host-deploy for offline mode, and contains all relevant dependencies. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=878397 Change-Id: If145380ebcd3ab114197a931d4a7ce256c3ed78d Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/fedora/spec/ovirt-engine.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9371 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If145380ebcd3ab114197a931d4a7ce256c3ed78d Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: bootstrap: rewrite bootstrap using the new ovirt-host-deploy...
by yzaslavs＠redhat.com 28 Nov '12

28 Nov '12

Yair Zaslavsky has submitted this change and it was merged. Change subject: bootstrap: rewrite bootstrap using the new ovirt-host-deploy package ...................................................................... bootstrap: rewrite bootstrap using the new ovirt-host-deploy package CURRENT IMPLEMENTATION Host bootstrap is done via full blown bootstrap process via the legacy vds_bootstrap project. It executes the vds_bootstrap scripts twice one for 99% of the setup, then issue certificate and then again for 1% of the setup. Node bootstrap is done via short circuit between the vds_bootstrap project, which at the very start checks if the host is a node and if it is configure and restart vdsm_reg with one time password for it to register into engine. [I have never seen this OTP registration working]. Node approval uses the vdsm_reg's vdsm-gen-cert, vdsm-complete scripts in order to generate keys, and vdsm-complete to configure vdsm and restart. PROBLEM IN CURRENT IMPLEMENTATION *** COMPLETXITY!!!! *** Design and actual implementation. MISSION Retire VdsInstaller, OVirtInstaller implementations. NEW IMPLEMENTATION Use the new vdsm-bootstrap package that is dialog based. Changes includes: 1. Same sequence for both host and ovirt-node bootstrap, not using vdsm-reg, always run the same bootstrap process, while the bootstrap is node aware. 2. vdsm-bootstrap log download. 3. vdsm-id is acquired from the dialog, engine should not know how vdsm-id is generated, nor assume it can perform this remotely. 4. Much more freedom for the host side, it detects the host type (host/node), it decides if reboot is required and finally it decides if there was an error. 5. Use python tar implementation instead of system tar, as fedora-18 vanilla has no tar installed. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=875528 Change-Id: If78c62601231f4729ca95da7653907b37856d672 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InstallVdsCommand.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/OVirtInstaller.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsInstaller.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/utils/pom.xml D backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/hostinstall/HostKeyVerifier.java D backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/hostinstall/IVdsInstallerCallback.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/hostinstall/OpenSslCAWrapper.java D backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/hostinstall/VdsInstallerSSH.java D backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/hostinstall/DefaultValuesConfigUtil.java D backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/hostinstall/VdsInstallerSSHTest.java M backend/manager/tools/engine-config/src/main/resources/engine-config.properties M packaging/fedora/spec/ovirt-engine.spec.in M pom.xml 16 files changed, 1,175 insertions(+), 2,141 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9175 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If78c62601231f4729ca95da7653907b37856d672 Gerrit-PatchSet: 19 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-reports[master]: reports: fixed font sizes in all webadmin dashboards (#87323...
by ydary＠redhat.com 28 Nov '12

28 Nov '12

Yaniv Dary has submitted this change and it was merged. Change subject: reports: fixed font sizes in all webadmin dashboards (#873238) ...................................................................... reports: fixed font sizes in all webadmin dashboards (#873238) bug-url: https://bugzilla.redhat.com/873238 Change-Id: I6d5356f375e3e96874bddff3c42058820652b8b5 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_ha_vm_uptime_status_dr13_files/cluster_ha_vm_uptime_status_dr13_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_host_resources_status_dr12_files/cluster_host_resources_status_dr12_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_host_uptime_status_dr6_files/cluster_host_uptime_status_dr6_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_summary_of_hosts_resources_usage_dr28_files/cluster_summary_of_hosts_resources_usage_dr28_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_top_five_busiest_hosts_dr23_files/cluster_top_five_busiest_hosts_dr23_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/cluster_vm_over_commit_status_dr14_files/cluster_vm_over_commit_status_dr14_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/top_five_ha_virtual_servers_downtime_dr20_files/top_five_ha_virtual_servers_downtime_dr20_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/cluster_dashboard/top_five_hosts_downtime_dr18_files/top_five_hosts_downtime_dr18_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/datacenter_top_five_busiest_clusters_dr21_files/datacenter_top_five_busiest_clusters_dr21_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/dc_host_resources_status_dr11_files/dc_host_resources_status_dr11_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/dc_host_uptime_status_dr5_files/dc_host_uptime_status_dr5_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/dc_storage_space_status_dr8_files/dc_storage_space_status_dr8_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/dc_summary_of_cluster_hosts_resources_usage_dr27_files/dc_summary_of_cluster_hosts_resources_usage_dr27_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/dc_dashboard/top_five_clusters_host_downtime_dr16_files/top_five_clusters_host_downtime_dr16_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/system_dashboard/system_host_resources_status_dr10_files/system_host_resources_status_dr10_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/system_dashboard/system_host_uptime_status_dr4_files/system_host_uptime_status_dr4_jrxml.data M reports/repository_files/resources/reports_resources/embedded_reports/webadmin_dashboards/system_dashboard/system_storage_space_status_dr7_files/system_storage_space_status_dr7_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_ha_vm_uptime_status_dr13_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_host_resources_status_dr12_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_host_uptime_status_dr6_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_summary_of_hosts_resources_usage_dr28_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_top_five_busiest_hosts_dr23_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/cluster_vm_over_commit_status_dr14_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/top_five_ha_virtual_servers_downtime_dr20_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard_files/top_five_hosts_downtime_dr18_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/datacenter_top_five_busiest_clusters_dr21_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/dc_host_resources_status_dr11_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/dc_host_uptime_status_dr5_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/dc_storage_space_status_dr8_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/dc_summary_of_cluster_hosts_resources_usage_dr27_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard_files/top_five_clusters_host_downtime_dr16_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/system_dashboard_files/system_host_resources_status_dr10_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/system_dashboard_files/system_host_uptime_status_dr4_jrxml.data M reports/repository_files/resources/reports_resources/webadmin_dashboards/ce/system_dashboard_files/system_storage_space_status_dr7_jrxml.data 34 files changed, 81 insertions(+), 81 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9108 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d5356f375e3e96874bddff3c42058820652b8b5 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

Change in ovirt-engine[master]: packaging: add release_suffix variable to spec
by juan.hernandez＠redhat.com 28 Nov '12

28 Nov '12

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: add release_suffix variable to spec ...................................................................... packaging: add release_suffix variable to spec This will allow simpler jenkins automatic build to be able to add the git hash as release suffix. Example: $ rpmbuild -tb -D "release_suffix ${git_head}" ovirt-engine-3.1.0.tar.gz Change-Id: I080ee4b74790fe880b32fbab48050c078698527e Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/fedora/spec/ovirt-engine.spec.in 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9444 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I080ee4b74790fe880b32fbab48050c078698527e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Query getdisksvmguid caused postmaster processes to ...
by mkublin＠redhat.com 28 Nov '12

28 Nov '12

Michael Kublin has submitted this change and it was merged. Change subject: engine: Query getdisksvmguid caused postmaster processes to consume constantly 100%cpu. ...................................................................... engine: Query getdisksvmguid caused postmaster processes to consume constantly 100%cpu. The following bug occurrs because of the following query being run from GetVmStatsVDSCommand every running VM in the system. The run is cyclic and runs in infinitive loop. I don't see reason of that query, update of dynamic data ofor disc is done under lock, in worst case the dynamic data will be updated with wrong values, in next run which occurring every 4 seconds, the disk will be updated with correct values. I think we can live with wrong values for 4 seconds in case of race The change reduced CPU of machine with DB from 100% to 30%, which is still a high, but better Change-Id: I9ef4a24908a75ab4fc1f02eed7c97a7125a5db41 Bug-Url: https://bugzilla.redhat.com/879930 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java 1 file changed, 22 insertions(+), 37 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9468 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9ef4a24908a75ab4fc1f02eed7c97a7125a5db41 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Perfromance improvements at IrsBrokerCommand
by mkublin＠redhat.com 28 Nov '12

28 Nov '12

Michael Kublin has submitted this change and it was merged. Change subject: engine: Perfromance improvements at IrsBrokerCommand ...................................................................... engine: Perfromance improvements at IrsBrokerCommand The following patch is removing unneeded query for storage pool. The objects can be passed to methods and not retrieved again from DB Change-Id: I9b6054950b45793a8f37e7f8b261dbd315ff6859 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java 1 file changed, 86 insertions(+), 90 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9469 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b6054950b45793a8f37e7f8b261dbd315ff6859 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Fix storage quota release on stopStatelessVm
by dfediuck＠redhat.com 27 Nov '12

27 Nov '12

Doron Fediuck has submitted this change and it was merged. Change subject: core: Fix storage quota release on stopStatelessVm ...................................................................... core: Fix storage quota release on stopStatelessVm StopVmBase was defined QuotaStorageDependent and the getQuotaStorageConsumptionParameters() was implemented to handle cases a stateless vm is stopped (vm images are deleted). Change-Id: I7088b94cd716bb93b2a4813963a980f4eab9e597 Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/StopVmCommandBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java 2 files changed, 22 insertions(+), 2 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9470 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7088b94cd716bb93b2a4813963a980f4eab9e597 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: ofri masad <omasad(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Quota Dependency Test + RemoveSnapshot fix
by dfediuck＠redhat.com 27 Nov '12

27 Nov '12

Doron Fediuck has submitted this change and it was merged. Change subject: core: Quota Dependency Test + RemoveSnapshot fix ...................................................................... core: Quota Dependency Test + RemoveSnapshot fix Added a test that checks that is VdcActionType is defined quota dependent - the correlating command is implementing the correct interface. RemoveSnapshotCommand did not implement QuotaStorageDependent. Fixed: implementation added. Change-Id: Ifd9e215abaa5de0fc03dae0609d9dc3ffc957f55 Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/quota/QuotaManager.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/quota/QuotaDependencyTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/quota/QuotaManagerTest.java 4 files changed, 106 insertions(+), 33 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9413 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifd9e215abaa5de0fc03dae0609d9dc3ffc957f55 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: ofri masad <omasad(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Changed edit VM disk to use run single action (#84...
by tnisan＠redhat.com 27 Nov '12

27 Nov '12

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Changed edit VM disk to use run single action (#848806) ...................................................................... webadmin: Changed edit VM disk to use run single action (#848806) The edit VM disk command in uicommon used run multiple actions for no reason due to this the VM disks list was not refreshed after the command excution and waited for the next timer interval to refresh, this caused the update data to appear only after a few seconds instead of immediately Change-Id: I75f4ebe1f20ce8e924f2d5d79ab1b1b6e3d6ef27 Bug-Url: https://bugzilla.redhat.com/848806 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmDiskListModel.java 1 file changed, 8 insertions(+), 13 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9467 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I75f4ebe1f20ce8e924f2d5d79ab1b1b6e3d6ef27 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in samples-portals[master]: typo fix: Action Faild
by shavivi＠redhat.com 27 Nov '12

27 Nov '12

Shahar Havivi has submitted this change and it was merged. Change subject: typo fix: Action Faild ...................................................................... typo fix: Action Faild Change-Id: Ibc3e1bde594301367cc4e67990e96ac249b46d7a Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M python/WebHandler.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Shahar Havivi: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9502 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibc3e1bde594301367cc4e67990e96ac249b46d7a Gerrit-PatchSet: 1 Gerrit-Project: samples-portals Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in samples-portals[master]: Python Sample: first commit
by shavivi＠redhat.com 27 Nov '12

27 Nov '12

Shahar Havivi has submitted this change and it was merged. Change subject: Python Sample: first commit ...................................................................... Python Sample: first commit Working sample of Python user portal using REST API Change-Id: I1d15dd5df8992c84b2c9529168919d05ba88bc33 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- A python/Broker.py A python/README A python/RestClient.py A python/RestCommand.py A python/WebHandler.py A python/ovirt.conf 6 files changed, 422 insertions(+), 0 deletions(-) Approvals: Shahar Havivi: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9387 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1d15dd5df8992c84b2c9529168919d05ba88bc33 Gerrit-PatchSet: 3 Gerrit-Project: samples-portals Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in samples-portals[master]: Typo fix in user VMs table
by oourfali＠redhat.com 27 Nov '12

27 Nov '12

Oved Ourfali has submitted this change and it was merged. Change subject: Typo fix in user VMs table ...................................................................... Typo fix in user VMs table Change-Id: Ic7329f6c4c2935c6ae59bc8f8fc2c811b2304384 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M python/WebHandler.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9438 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic7329f6c4c2935c6ae59bc8f8fc2c811b2304384 Gerrit-PatchSet: 1 Gerrit-Project: samples-portals Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: better general VDSM exception handling
by rgolan＠redhat.com 27 Nov '12

27 Nov '12

Roy Golan has submitted this change and it was merged. Change subject: core: better general VDSM exception handling ...................................................................... core: better general VDSM exception handling refactor error translation to separate network exceptions and protocol exception so error on XML formatting could be propagated and logged correctly. Change-Id: Id2391ec81626cd984f64b9a738adca9be4116d5a Signed-off-by: Roy Golan <rgolan(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/875725 --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllErrors.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerCommand.java 3 files changed, 14 insertions(+), 20 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9479 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id2391ec81626cd984f64b9a738adca9be4116d5a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>

1 0

Change in ovirt-engine[master]: core:End failed live snapshot with warning.
by mlipchuk＠redhat.com 27 Nov '12

27 Nov '12

Maor Lipchuk has submitted this change and it was merged. Change subject: core:End failed live snapshot with warning. ...................................................................... core:End failed live snapshot with warning. Engine fails to create Live-Snapshot during power off of VM. This is since the KVM process might be killed in the middle of the process and the synchronization between the engine VM and the KVM process in the VDSM are not reliable and could be racy. The proposed solution is to avoid the rollback of the entire operation, but instead finish the command with success, and add an appropriate audit log. That way will allow the VM to be configured to the new snapshot at the next run. Change-Id: I1b7f841d3fa02800fd58590c87d64e31227adfe5 Signed-Off-By: Maor Lipchuk <mlipchuk(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/870928 --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 4 files changed, 15 insertions(+), 7 deletions(-) Approvals: Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9390 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b7f841d3fa02800fd58590c87d64e31227adfe5 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Change "Active" vnic to "Plugged"
by alkaplan＠redhat.com 27 Nov '12

27 Nov '12

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: Change "Active" vnic to "Plugged" ...................................................................... webadmin: Change "Active" vnic to "Plugged" - VmInterface Dialog -change Activate checkobox to radio button- "Card Status". - Add plugged column to VM->VM Interface table. - Changed "Activated" property om VmInterfaceModel to "Plugged". Change-Id: Ib42f1e622ee1b55e65a6ec274e762cea9a08d531 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationResources.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationTemplates.java R frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/column/CheckboxColumn.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/networkinterface/NetworkInterfacePopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/networkinterface/NetworkInterfacePopupWidget.ui.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/template/TemplateNetworkInterfacePopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/BaseInterfaceListModelTable.java A frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/images/network/icn_plugged.png A frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/images/network/icn_un_plugged.png M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmGuideModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInterfaceListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmInterfaceModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/AbstractNetworkPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterManageNetworkPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/vm/VmInterfacePopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/network/SubTabNetworkClusterView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/virtualMachine/SubTabVirtualMachineNetworkInterfaceView.java 18 files changed, 249 insertions(+), 78 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved Gilad Chaplik: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9415 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib42f1e622ee1b55e65a6ec274e762cea9a08d531 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Fix Moving Disk message (#856082)
by amureini＠redhat.com 27 Nov '12

27 Nov '12

Allon Mureinik has submitted this change and it was merged. Change subject: core: Fix Moving Disk message (#856082) ...................................................................... core: Fix Moving Disk message (#856082) Log message notifying about moving disk said "moveing", and is now corrected to "moving". The original message used ${Action}ing. The current one uses ${Action}, and MoveOrCopyDiskCommand is replacing it with Moving or Copying according to the operation taking place. Change-Id: I53bb3238d1a5e6b3e966adf488a10946e252ffe3 Bug-Url: https://bugzilla.redhat.com/856082 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommand.java M backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties 2 files changed, 8 insertions(+), 4 deletions(-) Approvals: Allon Mureinik: Looks good to me, approved Vered Volansky: Verified -- To view, visit http://gerrit.ovirt.org/9303 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I53bb3238d1a5e6b3e966adf488a10946e252ffe3 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: DB upgrade from 3.0 to 3.1 fails(#880350)
by emesika＠redhat.com 27 Nov '12

27 Nov '12

Eli Mesika has submitted this change and it was merged. Change subject: core: DB upgrade from 3.0 to 3.1 fails(#880350) ...................................................................... core: DB upgrade from 3.0 to 3.1 fails(#880350) upgrade from 3.0.x to 3.1 failed in 2 points ; 1) 03_01_0950 Reason : script assumes that a storage is in only one DC and attempt to insert duplicate rows that violates primary key to the permission table Solution: check before inserting to permissions table that record with the same key does not exist 2) 03_01_1020 Reason :script adds a column_id column to the async_tasks table. Since this column type is UUID and is defined as NOT NULL, the addition will fail if at the point that this upgrade script is running the table is not empty Solution : Create the column with an empty UUID (00000.....000000) DEFAULT After this column was added , drop the default. Change-Id: Id9658673c16610da4a25e33c7969bcde61a64259 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/dbscripts/upgrade/03_01_0950_add_permissions_on_storage_to_users.sql M backend/manager/dbscripts/upgrade/03_01_1020_add_command_id_to_async_tasks_table.sql 2 files changed, 14 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9500 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id9658673c16610da4a25e33c7969bcde61a64259 Gerrit-PatchSet: 2 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: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: [es_ES] heading "Virtual size" is truncated in Spa...
by oourfali＠redhat.com 27 Nov '12

27 Nov '12

Oved Ourfali has submitted this change and it was merged. Change subject: webadmin: [es_ES] heading "Virtual size" is truncated in Spanish (#871379) ...................................................................... webadmin: [es_ES] heading "Virtual size" is truncated in Spanish (#871379) https://bugzilla.redhat.com/871379 Increased the size of the "Virtual Size" column to fit all locales. Verified on all locales. Change-Id: Ie71097c4fae81753135eec2b28702ca9bf822f41 Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabDiskView.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alona Kaplan: Looks good to me, approved Oved Ourfali: Verified -- To view, visit http://gerrit.ovirt.org/9505 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie71097c4fae81753135eec2b28702ca9bf822f41 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: remove unused private method from CommonVmPoolWithVmsC...
by lhornyak＠redhat.com 26 Nov '12

26 Nov '12

Laszlo Hornyak has submitted this change and it was merged. Change subject: core: remove unused private method from CommonVmPoolWithVmsCommand ...................................................................... core: remove unused private method from CommonVmPoolWithVmsCommand getQuotaId is not used Change-Id: I715d25fe1bd99a5c0af435c4ad0dcd207d1a0aa6 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommand.java 1 file changed, 0 insertions(+), 4 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9393 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I715d25fe1bd99a5c0af435c4ad0dcd207d1a0aa6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: db - fixing wrong nfs_version argument in storage_san....
by rgolan＠redhat.com 26 Nov '12

26 Nov '12

Roy Golan has submitted this change and it was merged. Change subject: core: db - fixing wrong nfs_version argument in storage_san.sp ...................................................................... core: db - fixing wrong nfs_version argument in storage_san.sp 67761e1f513bdd17915ba7a47a6597ab23c4326d refactored nfs_version type and the insert sp but missed the update. Change-Id: Id9a2f060164f17e01db30f84bdf87a91c9b25769 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/dbscripts/storages_san_sp.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9459 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id9a2f060164f17e01db30f84bdf87a91c9b25769 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>

1 0

Change in ovirt-engine[master]: upgrade from 3.0.7 to SI24.4 fail(#879918)
by emesika＠redhat.com 26 Nov '12

26 Nov '12

Eli Mesika has submitted this change and it was merged. Change subject: upgrade from 3.0.7 to SI24.4 fail(#879918) ...................................................................... upgrade from 3.0.7 to SI24.4 fail(#879918) upgrade from 3.0.7 to SI24.4 fails: ERROR: insert or update on table vm_device violates foreign key constraint fk_vm_device_vm _static This bug is a result of missing FK between vmt_guid in vm_interface and vm_guid in vm_static. This cause junk interface left in vm_interface when a template that has interfaces is removed. Since in upgrade from 3.0 to 3.1 we added the vm_device table, there is an attempt to insert those junk interfaces to vm_device that fails on the FK constraint between vm_device and vm_static. This patch fix the bug by: 1) Fixing the upgrade script to cleanup those junk interfaces before an attempt to insert it to vm_device is done. 2) Adds the missing FK between vm_interface and vm_static to prevent junk data in future. Change-Id: Iaf4af7908566b5d16e39ea5167c6d6dffc5884db Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/dbscripts/upgrade/03_01_0440_vm_device_upgrade_data.sql A backend/manager/dbscripts/upgrade/03_01_1520_add_fk_vm_interface_vm_static_template.sql 2 files changed, 9 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9462 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaf4af7908566b5d16e39ea5167c6d6dffc5884db 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: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: skip quota for flows with no storage pool
by rgolan＠redhat.com 26 Nov '12

26 Nov '12

Roy Golan has submitted this change and it was merged. Change subject: core: skip quota for flows with no storage pool ...................................................................... core: skip quota for flows with no storage pool Quota checks are being made based on a commands "quota dependency" which is being marked by an enum on the VdcActionType. The problem is that some commands, e.g. AddDisk may need quota check on regular disk but not on external floating LUN disk, mainly because this command will not have a storage pool (floating disk doesn't belong to a data center) thus - no quota could be calculated on it. As a way out, instead of breaking a command to two, (one quotable and one not) an empty list of quota params can be submitted so no quota checks will be made. Change-Id: Ie468db3709522653cd93c338564592e2ba439c4a Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/quota/QuotaConsumptionParametersWrapper.java 2 files changed, 15 insertions(+), 7 deletions(-) Approvals: ofri masad: Looks good to me, but someone else must approve Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9410 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie468db3709522653cd93c338564592e2ba439c4a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Fix NPE on Clone VM from Snapshot
by mlipchuk＠redhat.com 25 Nov '12

25 Nov '12

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Fix NPE on Clone VM from Snapshot ...................................................................... core: Fix NPE on Clone VM from Snapshot Clone VM from snapshot throw NPE while using MoveOrCopyImageGroup. This happens since AddVmFromSnapshot creates the task first with the parameters and only after that it creates the disk in the DB. The proposed fix is to get the wipeAfterDelete flag from the parameters in case the destination disk is null. This fix should be revisited, to change the AddVmFromSnapshot to create the disk before the creation of task. Change-Id: I2c18b1ac552f598fc20e74aa67f81dabb915a8c2 Signed-Off-By: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyImageGroupCommand.java 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9445 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2c18b1ac552f598fc20e74aa67f81dabb915a8c2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: OvfReader cleanup - remove unused parameter
by amureini＠redhat.com 25 Nov '12

25 Nov '12

Allon Mureinik has submitted this change and it was merged. Change subject: core: OvfReader cleanup - remove unused parameter ...................................................................... core: OvfReader cleanup - remove unused parameter Removed the vmDevice parameter from the getResourceType method. This method is private, and the parameter is not used, so we can be sure it is not required. Change-Id: Iaa9872617cc62539b2b8c33c5dbe2df21db355bb Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9307 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaa9872617cc62539b2b8c33c5dbe2df21db355bb Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: OvfReader cleanup - correct DiskImage method
by amureini＠redhat.com 25 Nov '12

25 Nov '12

Allon Mureinik has submitted this change and it was merged. Change subject: core: OvfReader cleanup - correct DiskImage method ...................................................................... core: OvfReader cleanup - correct DiskImage method Replaced the usage of the deprecated DiskImage.set_imagegroup_id(Guid) with the newer DiskImage.setId(Guid) which it delegated to. Change-Id: I9b8f472eafe2ba98e8925508e6097b7093a70764 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9306 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b8f472eafe2ba98e8925508e6097b7093a70764 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: OvfReader cleanup - remove StringHelper
by amureini＠redhat.com 25 Nov '12

25 Nov '12

Allon Mureinik has submitted this change and it was merged. Change subject: core: OvfReader cleanup - remove StringHelper ...................................................................... core: OvfReader cleanup - remove StringHelper Removed usages of the deprecated StringHelper and replaced them with standard calls. * StringHelper.isNullOrEmpty(String) was replaced with apache commons' standard StringUtils.isEmpty(String). * StringHelper.EqOp(String, constant) was replaced with Java's standard constant.equals(String) Change-Id: I22f49eb0eec5c9849a4a2242a51ba0217718f0d1 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java 1 file changed, 29 insertions(+), 30 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9305 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I22f49eb0eec5c9849a4a2242a51ba0217718f0d1 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: OvfReader cleanup - document empty blocks
by amureini＠redhat.com 25 Nov '12

25 Nov '12

Allon Mureinik has submitted this change and it was merged. Change subject: core: OvfReader cleanup - document empty blocks ...................................................................... core: OvfReader cleanup - document empty blocks Documented the empty implementation of buildNetworks() and readSnapshostsSection(String) to avoid warnings by code analysis tools (e.g., Eclipse IDE's built-in inspection feature). Change-Id: I619991e6ebbbffac6c99a1ab2a98fa27aa57dadd Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9304 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I619991e6ebbbffac6c99a1ab2a98fa27aa57dadd Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sharad Mishra <snmishra(a)linux.vnet.ibm.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Replaced quartz job to thread job at MigrateVDSComma...
by mkublin＠redhat.com 25 Nov '12

25 Nov '12

Michael Kublin has submitted this change and it was merged. Change subject: engine: Replaced quartz job to thread job at MigrateVDSCommand ...................................................................... engine: Replaced quartz job to thread job at MigrateVDSCommand Because of we have more threads at the system and it is more covinient to open a thread instead of quarzt job with zero delay Change-Id: I7699a8d8e931ff336335d476fc4fc9251715b4fd Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/MigrateVDSCommand.java 1 file changed, 41 insertions(+), 52 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9416 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7699a8d8e931ff336335d476fc4fc9251715b4fd Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0