September 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: build: Sync gwt-debug target with install-dev target
by oschreib＠redhat.com 16 Sep '13

16 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: build: Sync gwt-debug target with install-dev target ...................................................................... build: Sync gwt-debug target with install-dev target This patch ensures that both gwt-debug and install-dev targets use same GWT-specific defaults, as defined by DEV_EXTRA_BUILD_FLAGS_GWT_DEFAULTS option. This relates mainly to "gwt.userAgent" property, whose value should be in sync for both targets mentione above. Change-Id: I33760b721f78ab26eeefc1ef3bd662829c49c312 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M Makefile 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19190 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I33760b721f78ab26eeefc1ef3bd662829c49c312 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: move package group name globally
by oschreib＠redhat.com 16 Sep '13

16 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: move package group name globally ...................................................................... packaging: move package group name globally This patch enables seemless use of group name for downstream products. Change-Id: I49b344a82773de57b3f8dacda12092cd3da43e30 Signed-off-by: Bala.FA <barumuga(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 12 insertions(+), 11 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19174 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I49b344a82773de57b3f8dacda12092cd3da43e30 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Bala.FA <barumuga(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[engine_3.2]: packaging: ovirt-engine-3.2.3
by oschreib＠redhat.com 16 Sep '13

16 Sep '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: ovirt-engine-3.2.3 ...................................................................... packaging: ovirt-engine-3.2.3 Change-Id: Ie7f54f9e3e151a675797112c5f8c62d36db6b502 Signed-off-by: Ofer Schreiber <oschreib(a)redhat.com> --- M version.mak 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19184 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie7f54f9e3e151a675797112c5f8c62d36db6b502 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: engine_3.2 Gerrit-Owner: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-engine-sdk-java[master]: sdk: apache PropertyUtilsBean throws NoSuchMethodException #...
by mpastern＠redhat.com 16 Sep '13

16 Sep '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: apache PropertyUtilsBean throws NoSuchMethodException #1007266 ...................................................................... sdk: apache PropertyUtilsBean throws NoSuchMethodException #1007266 Change-Id: I4465ad61b405b1b2447f7817391a286ad4f9d311 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1007266 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M ovirt-engine-sdk-java/src/main/java/org/ovirt/engine/sdk/mapping/Mapper.java 1 file changed, 10 insertions(+), 15 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19262 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4465ad61b405b1b2447f7817391a286ad4f9d311 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: gluster: VDS Cmd: Gluster volume rebalance status
by ofrenkel＠redhat.com 16 Sep '13

16 Sep '13

Omer Frenkel has submitted this change and it was merged. Change subject: gluster: VDS Cmd: Gluster volume rebalance status ...................................................................... gluster: VDS Cmd: Gluster volume rebalance status VDS Command for gluster volume rebalance status Change-Id: Ia9a5ec48feffac42ec073f2d6f9c91ecd5ef57da Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterVolumeTaskStatusEntity.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllErrors.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSCommandType.java M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/AbstractGlusterBrokerCommand.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GetGlusterVolumeRebalanceStatusVDSCommand.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeTaskReturnForXmlRpc.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/VdsmErrors.java M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors.properties 12 files changed, 116 insertions(+), 0 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18308 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia9a5ec48feffac42ec073f2d6f9c91ecd5ef57da Gerrit-PatchSet: 25 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin, userportal: do not add default nic on edit vm
by tjelinek＠redhat.com 16 Sep '13

16 Sep '13

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin,userportal: do not add default nic on edit vm ...................................................................... webadmin,userportal: do not add default nic on edit vm If the VM had no nics the edit VM added a default one and tried to edit it. It was not correct - if the VM has no nics than the edit should not add any. Change-Id: I21724b30dc1ad6ccbbe40448ad77c47c22123c06 Bug-Url: https://bugzilla.redhat.com/995048 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ExistingVmModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewVmModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModelNetworkAsyncCallbacks.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java 4 files changed, 81 insertions(+), 66 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17315 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I21724b30dc1ad6ccbbe40448ad77c47c22123c06 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine, frontend: disable new VM/pool if no DC is UP
by tjelinek＠redhat.com 16 Sep '13

16 Sep '13

Tomas Jelinek has submitted this change and it was merged. Change subject: engine,frontend: disable new VM/pool if no DC is UP ...................................................................... engine,frontend: disable new VM/pool if no DC is UP If there is no UP DC the new VM/Pool dialog will be disabled and a message explaining why will be added. Change-Id: Id537ac56bd2d963999c5c77ea3e92b3987d82525 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=990059 Signed-off-by: Tomas Jelinek <tjelinek(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/widget/uicommon/popup/AbstractVmPopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolEditPopupWidget.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewPoolModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewVmModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/PoolModelBehaviorBase.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/UserPortalNewVmModelBehavior.java 8 files changed, 115 insertions(+), 54 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18885 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id537ac56bd2d963999c5c77ea3e92b3987d82525 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: new VM guide dialog does not get updated
by tjelinek＠redhat.com 16 Sep '13

16 Sep '13

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: new VM guide dialog does not get updated ...................................................................... webadmin: new VM guide dialog does not get updated The "There are still unconfigured entities" is present after adding a new disk. The problem was that the postAction and resetData were not called from the NewDiskModel. This was partially implemented for the network interfaces by the NewGuideVmInterfaceModel - it just was not used. Fixed by creating the NewGuideDiskModel which works the same way as the network one and also to use the NewGuideVmInterfaceModel. Change-Id: I7f6c38cdc316b8f9b3501e04980d7cb2e3908136 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1005846 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewDiskModel.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewGuideDiskModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmGuideModel.java 3 files changed, 28 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19040 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7f6c38cdc316b8f9b3501e04980d7cb2e3908136 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Fix regression in copying template disk
by amureini＠redhat.com 15 Sep '13

15 Sep '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: Fix regression in copying template disk ...................................................................... core: Fix regression in copying template disk Fix regression introduced by Change-Id Id4b5ef206581e5f9a1727c85b30f39802859583c which used the disk's ID instead of the image ID as VmTemplateDao#getAllForImage(Guid)'s parameter. Change-Id: I2627741221aee980534a5cd19f408d03d23a6cc4 Bug-Url: https://bugzilla.redhat.com/1006811 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19258 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2627741221aee980534a5cd19f408d03d23a6cc4 Gerrit-PatchSet: 2 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: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Fix snapshot quota when removing a VM
by dfediuck＠redhat.com 15 Sep '13

15 Sep '13

Doron Fediuck has submitted this change and it was merged. Change subject: core: Fix snapshot quota when removing a VM ...................................................................... core: Fix snapshot quota when removing a VM When VM is removed the disks quota is released. Added a release for the snapshot's quota as well Change-Id: I5a1649e65c1a0cf4c7752dd5f6e39c9b034feecd Bug-Url: https://bugzilla.redhat.com/1001017 Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmCommand.java 1 file changed, 21 insertions(+), 9 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18741 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a1649e65c1a0cf4c7752dd5f6e39c9b034feecd Gerrit-PatchSet: 2 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: oVirt Jenkins CI Server Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0