August 2013 - Engine-commits - oVirt List Archives

Change in ovirt-reports[master]: packaging: added needed package build dependency
by ydary＠redhat.com 04 Aug '13

04 Aug '13

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: added needed package build dependency ...................................................................... packaging: added needed package build dependency Change-Id: Ic4395330cf6ca9ae3d2dc8ad1be8f8b8bfdc9579 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M packaging/ovirt-engine-reports.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17618 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4395330cf6ca9ae3d2dc8ad1be8f8b8bfdc9579 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]: core: Add event logs for balloon issues
by dfediuck＠redhat.com 04 Aug '13

04 Aug '13

Doron Fediuck has submitted this change and it was merged. Change subject: core: Add event logs for balloon issues ...................................................................... core: Add event logs for balloon issues Added AuditLog for two cases: - balloon is requested but the balloon driver on the VM is not responding. - balloon is inflated but MOM lost control over the balloon device (caused by guest agent failure). Change-Id: I30395b90b74777eacca76419c20960eb812204ae Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmBalloonInfo.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmStatistics.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.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 M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 11 files changed, 200 insertions(+), 19 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17486 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I30395b90b74777eacca76419c20960eb812204ae Gerrit-PatchSet: 9 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: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: modify nfs parameters of iso domain
by oschreib＠redhat.com 04 Aug '13

04 Aug '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: modify nfs parameters of iso domain ...................................................................... packaging: setup: modify nfs parameters of iso domain sync with vdsm settings, avoid timeouts. Change-Id: I5cce9936e071b5f57cbbeccfbf039d946ced6a39 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/fedora/setup/nfsutils.py M packaging/setup/plugins/ovirt-engine-setup/config/iso_domain.py 2 files changed, 4 insertions(+), 4 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17533 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5cce9936e071b5f57cbbeccfbf039d946ced6a39 Gerrit-PatchSet: 2 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: Ayal Baron <abaron(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[master]: core: clean unused "main" blocks
by rgolan＠redhat.com 04 Aug '13

04 Aug '13

Roy Golan has submitted this change and it was merged. Change subject: core: clean unused "main" blocks ...................................................................... core: clean unused "main" blocks Change-Id: Id6a539d86c11a02106e10b0ad24916822bcaf664 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/osinfo/OsInfoPreferencesLoader.java 1 file changed, 0 insertions(+), 13 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16815 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id6a539d86c11a02106e10b0ad24916822bcaf664 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: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-host-deploy[master]: vdsm: packages: stop supervdsmd in addition to vdsmd
by Alon Bar-Lev 04 Aug '13

04 Aug '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: vdsm: packages: stop supervdsmd in addition to vdsmd ...................................................................... vdsm: packages: stop supervdsmd in addition to vdsmd vdsm project introduced another independent daemon supervdsmd[1]. This daemon should be stopped explicitly during upgrade to release vdsm resources. No need to start supervdsmd as it will be started by vdsmd. [1] http://www.ovirt.org/Features/Supervdsm_service Related-To-Bug-Url: https://bugzilla.redhat.com/990517 Change-Id: I92e7baa6fade0c0d111350c8e788bbefaaa3ae17 Signed-off-by: Dan Kenigsberg <danken(a)redhat.com> --- M src/plugins/ovirt-host-deploy/vdsm/packages.py 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, approved Meni Yakove: Verified -- To view, visit http://gerrit.ovirt.org/17596 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I92e7baa6fade0c0d111350c8e788bbefaaa3ae17 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Meni Yakove <myakove(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: core: offlinepackager: fake vdsm-gluster as installed
by Alon Bar-Lev 02 Aug '13

02 Aug '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: offlinepackager: fake vdsm-gluster as installed ...................................................................... core: offlinepackager: fake vdsm-gluster as installed Change-Id: I72d528b55e2bd36925c429dd62874358e2e8494e Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/plugins/ovirt-host-deploy/core/offlinepackager.py 1 file changed, 12 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, approved Sandro Bonazzola: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/17606 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I72d528b55e2bd36925c429dd62874358e2e8494e Gerrit-PatchSet: 2 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> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-scheduler-proxy[master]: Change Discover to return description and regex
by lhornyak＠redhat.com 02 Aug '13

02 Aug '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: Change Discover to return description and regex ...................................................................... Change Discover to return description and regex Discover used to return only the name of the module, now it returns the description and regex of each function type changed tests to reflect the new return value Signed-off-by: Noam Slomianko <nslomian(a)redhat.com> Change-Id: Ibe4e6a44302636c40c96453d433caf09fc5bda94 --- M plugins/dummy.py M src/loader.py M src/request_handler.py M src/request_handler_test.py M src/utils.py M tests/java/src/main/java/org/ovirt/schedulerproxy/SchedulerProxy.java M tests/java/src/test/java/org/ovirt/schedulerproxy/SchedulerProxyTest.java 7 files changed, 110 insertions(+), 34 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17576 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibe4e6a44302636c40c96453d433caf09fc5bda94 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com>

1 0

Change in ovirt-scheduler-proxy[master]: Convert to pass IDs
by lhornyak＠redhat.com 02 Aug '13

02 Aug '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: Convert to pass IDs ...................................................................... Convert to pass IDs Moved from passing Xml to passing ID lists Changed tests to work accordingly Signed-off-by: Noam Slomianko <nslomian(a)redhat.com> Change-Id: I8643456cf785bb2fc37d4ff1707de9290f330d0c --- M plugins/dummy.py M src/oschedproxyd.py M src/request_handler.py M src/runner_test.py M tests/java/src/main/java/org/ovirt/schedulerproxy/SchedulerProxy.java M tests/java/src/test/java/org/ovirt/schedulerproxy/SchedulerProxyTest.java 6 files changed, 214 insertions(+), 152 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved Noam Slomianko: Verified -- To view, visit http://gerrit.ovirt.org/17523 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8643456cf785bb2fc37d4ff1707de9290f330d0c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Improve attestation status message
by ecohen＠redhat.com 01 Aug '13

01 Aug '13

Einav Cohen has submitted this change and it was merged. Change subject: webadmin: Improve attestation status message ...................................................................... webadmin: Improve attestation status message This change improves the description of the enum value that indicates that the data required for attestation of a host hasn't been retrieved yet. Change-Id: If444cecc3393a4563b3d6d4a1676dc2db378acf6 Sgined-off-by: Yuko Katabami (方波見裕子) <ykatabam(a)redhat.com> Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Einav Cohen: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17517 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If444cecc3393a4563b3d6d4a1676dc2db378acf6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: minor refactoring in stop vm related commands
by ofrenkel＠redhat.com 01 Aug '13

01 Aug '13

Omer Frenkel has submitted this change and it was merged. Change subject: core: minor refactoring in stop vm related commands ...................................................................... core: minor refactoring in stop vm related commands 1. replace canDoAction methods in ShutdownVmCommand and StopVmCommand that were not doing anything besides setting the VAR__ACTION and VAR__TYPE with setActionMessageParameters methods 2. rename StopVmCommandBase#privateSuspendedVm to suspendedVm 3. reorganize StopVmCommandBase#canDoAction method to be more readable 4. extract the checks that determine whether the VM can be shutdown from the Perform method to an external method. Change-Id: I8af061d2805a85dadd84efa1cd2a0a290dbddbd1 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ShutdownVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/StopVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/StopVmCommandBase.java 3 files changed, 42 insertions(+), 59 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17472 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8af061d2805a85dadd84efa1cd2a0a290dbddbd1 Gerrit-PatchSet: 4 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: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0