February 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Fix Quota accounting for move disk
by gchaplik＠redhat.com 19 Feb '13

19 Feb '13

Gilad Chaplik has submitted this change and it was merged. Change subject: core: Fix Quota accounting for move disk ...................................................................... core: Fix Quota accounting for move disk When moving a disk without changing a quota the quota check is not performed. This is a problem since the quota can be specific for each storage domain. The logic was changed to support quota change even if quota id was not changed. Change-Id: I33a6a409388622bb0ab14644445a187357ce5fa2 Bug-Url: https://bugzilla.redhat.com/906499 Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateVmDisksCommand.java 2 files changed, 0 insertions(+), 8 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12159 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I33a6a409388622bb0ab14644445a187357ce5fa2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: ofri masad <omasad(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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]: packaging: engine-setup avoid traceback when loading iso
by juan.hernandez＠redhat.com 19 Feb '13

19 Feb '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: engine-setup avoid traceback when loading iso ...................................................................... packaging: engine-setup avoid traceback when loading iso In engine-setup, _loadFilesToIsoDomain now try to copy the iso files provided by possibly installed rpm into the configured ISO domain only if they exists avoiding useless traceback in the log. Change-Id: Ic995c3bee295b964d2f6a729e4d4fb750eb6b4b5 Bug-Url: https://bugzilla.redhat.com/904763 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/fedora/setup/engine-setup.py 1 file changed, 8 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified Juan Hernandez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12207 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic995c3bee295b964d2f6a729e4d4fb750eb6b4b5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine-cli[cli_3.2]: cli: release 3.2.0.10-1
by mpastern＠redhat.com 19 Feb '13

19 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: release 3.2.0.10-1 ...................................................................... cli: release 3.2.0.10-1 Change-Id: Idfe3920a2563b1d23e915e68b81e56ccfe73fe08 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M Makefile M ovirt-engine-cli.spec.in M setup.py 3 files changed, 6 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12208 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idfe3920a2563b1d23e915e68b81e56ccfe73fe08 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.2 Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: patterned pool names support
by ofrenkel＠redhat.com 19 Feb '13

19 Feb '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: patterned pool names support ...................................................................... core: patterned pool names support This patch contains the backend part of patterned pool names support. the user can specify inside a pool name a mask for the indexes that are included in the generated names of the VMs in the pool. that way, the user can determine a fixed-length to the generated VM names. The mask is a sequence of question-marks. the mask will be replaced with a zero-padded number in the length of the mask corredponding to the index of the VM in the pool, in the generate VM names. for example, for the patterned pool name 'my???pool', the generated names of the VMs will be: my01pool, my02pool, ... , my999pool (it's the user's responsibility not to set too many VMs in pool). Change-Id: I81a1ffb6324fb728b2584677dd654ac79b679cd8 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmPool.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/ValidationUtils.java A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/NameForVmInPoolGenerator.java A backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/NameForVmInPoolGeneratorTest.java 5 files changed, 74 insertions(+), 12 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11986 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81a1ffb6324fb728b2584677dd654ac79b679cd8 Gerrit-PatchSet: 6 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: 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-sdk[sdk_3.2]: sdk: release 3.2.0.9-1
by mpastern＠redhat.com 19 Feb '13

19 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: release 3.2.0.9-1 ...................................................................... sdk: release 3.2.0.9-1 Change-Id: I8edd0d7e27f00f5f8bb911572b6187012004129d Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M Makefile M ovirt-engine-sdk.spec.in M setup.py 3 files changed, 7 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12205 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8edd0d7e27f00f5f8bb911572b6187012004129d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.2 Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine-sdk[sdk_3.2]: sdk: regenerate against the latest api
by mpastern＠redhat.com 19 Feb '13

19 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: regenerate against the latest api ...................................................................... sdk: regenerate against the latest api - to DiskStatistics.list() added 'max' argument - to ClusterGlusterVolumeBrick added 'replace' action - Host can be added to cluster now either by id or name Change-Id: I8ae59ecef269bf0ac5c69283a60b20ad984ed602 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtsdk/infrastructure/brokers.py M src/ovirtsdk/xml/params.py 2 files changed, 54 insertions(+), 7 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12204 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8ae59ecef269bf0ac5c69283a60b20ad984ed602 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.2 Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: userportal: UserVmManager cannot edit Custom properties
by oourfali＠redhat.com 19 Feb '13

19 Feb '13

Oved Ourfali has submitted this change and it was merged. Change subject: userportal: UserVmManager cannot edit Custom properties ...................................................................... userportal: UserVmManager cannot edit Custom properties This patch hides the custom properties side tab on the User portal, when creating a new VM and when editing one. Also removing the Custom Properties panel from Run-Once, in the user portal. Change-Id: Icf8e0929f6969b5ca1fe592a960d2e91c83046b1 Bug-Url: https://bugzilla.redhat.com/911091 Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmRunOncePopupWidget.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/RunOnceModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UserPortalExistingVmModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UserPortalNewVmModelBehavior.java 7 files changed, 54 insertions(+), 0 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12171 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icf8e0929f6969b5ca1fe592a960d2e91c83046b1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: VmGuideModel DirectLUN disk setLunType
by derez＠redhat.com 19 Feb '13

19 Feb '13

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: VmGuideModel DirectLUN disk setLunType ...................................................................... webadmin: VmGuideModel DirectLUN disk setLunType VmGuideModel - when adding a DirectLUN disk set LunType according to the selected StorageType (as done in DiskListModel and VmDiskListModel). Change-Id: I4499ee37556885663c89a29f530ad2d6db97cd51 Bug-Url: https://bugzilla.redhat.com/910562 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmGuideModel.java 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12180 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4499ee37556885663c89a29f530ad2d6db97cd51 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: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine-cli[cli_3.2]: Use vncviewer passwordFile instead of passwdInput
by mpastern＠redhat.com 19 Feb '13

19 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: Use vncviewer passwordFile instead of passwdInput ...................................................................... Use vncviewer passwordFile instead of passwdInput Fedora 18 has dropped the patch to add passwdInput option to tigervnc Change-Id: I76178f15217c421d88879c88534d7cb3a7583426 Signed-off-by: Sander Hoentjen <sander(a)hoentjen.eu> --- M src/cli/messages.py M src/ovirtcli/platform/posix/vnc.py 2 files changed, 9 insertions(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12191 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I76178f15217c421d88879c88534d7cb3a7583426 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.2 Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Sander Hoentjen <sander(a)hoentjen.eu>

1 0

Change in ovirt-engine-cli[cli_3.2]: cli: ovirt-cli DistributionNotFound exception on f18 #881011
by mpastern＠redhat.com 19 Feb '13

19 Feb '13

Michael Pasternak has submitted this change and it was merged. Change subject: cli: ovirt-cli DistributionNotFound exception on f18 #881011 ...................................................................... cli: ovirt-cli DistributionNotFound exception on f18 #881011 https://bugzilla.redhat.com/show_bug.cgi?id=881011 Change-Id: Icbbf415b82022ec324eb9819a4166366a6318fe3 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M setup.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12190 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icbbf415b82022ec324eb9819a4166366a6318fe3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.2 Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0