- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: Adding quartz configuration to standalone.xml
by mkublin＠redhat.com 07 May '13

07 May '13

Michael Kublin has submitted this change and it was merged. Change subject: engine: Adding quartz configuration to standalone.xml ...................................................................... engine: Adding quartz configuration to standalone.xml In order to configure a quartz scheduler his configuration values should be inside standalone.xml Removing old configuration file, which is not needed anymore Change-Id: I3e6f881ff7a8f080de5f18d295b72ceb0d478347 Signed-off-by: Michael Kublin <mkublin(a)redhat.com> --- M backend/manager/conf/standalone.xml D backend/manager/modules/utils/src/main/resources/quartz.properties M packaging/services/ovirt-engine.xml.in 3 files changed, 12 insertions(+), 22 deletions(-) Approvals: Michael Kublin: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14504 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3e6f881ff7a8f080de5f18d295b72ceb0d478347 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Michael Kublin <mkublin(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(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]: notifier: perserve notifier.conf
by oschreib＠redhat.com 07 May '13

07 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: notifier: perserve notifier.conf ...................................................................... notifier: perserve notifier.conf at the end there is no significant configuration modification, leaving the file as-is will ease migration. Change-Id: I9546d1e279538641630d139c1c80b1e2ea6b0fde Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14519 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9546d1e279538641630d139c1c80b1e2ea6b0fde 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: Ofer Schreiber <oschreib(a)redhat.com>

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: post ovirt-host-deploy-1.0.2
by Alon Bar-Lev 06 May '13

06 May '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: post ovirt-host-deploy-1.0.2 ...................................................................... packaging: post ovirt-host-deploy-1.0.2 Change-Id: I6350a7b9a766d6926facf9d8252c6ef55ca062f4 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14524 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6350a7b9a766d6926facf9d8252c6ef55ca062f4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.0 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[ovirt-host-deploy-1.0]: packaging: ovirt-host-deploy-1.0.2
by Alon Bar-Lev 06 May '13

06 May '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: ovirt-host-deploy-1.0.2 ...................................................................... packaging: ovirt-host-deploy-1.0.2 Change-Id: I1bd545c7425c72fac45e42c849015096746792fd Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac M ovirt-host-deploy.spec.in 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14523 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1bd545c7425c72fac45e42c849015096746792fd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.0 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in otopi[otopi-1.0]: packaging: post otopi-1.0.2
by Alon Bar-Lev 06 May '13

06 May '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: post otopi-1.0.2 ...................................................................... packaging: post otopi-1.0.2 Change-Id: Id1c675eeb0a1d3bd43d6403a5d8cb81d779de6f3 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14522 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id1c675eeb0a1d3bd43d6403a5d8cb81d779de6f3 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: otopi-1.0 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in otopi[otopi-1.0]: packaging: otopi-1.0.2
by Alon Bar-Lev 06 May '13

06 May '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: otopi-1.0.2 ...................................................................... packaging: otopi-1.0.2 Change-Id: Ifdb304559d1571d5dfcd5bff40cbe9237860f21c Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac M otopi.spec.in 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14521 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifdb304559d1571d5dfcd5bff40cbe9237860f21c Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: otopi-1.0 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: live migrate - log error on failure
by derez＠redhat.com 06 May '13

06 May '13

Daniel Erez has submitted this change and it was merged. Change subject: core: live migrate - log error on failure ...................................................................... core: live migrate - log error on failure In case of shutting down the vm during snapshot creation part of live migrate, an appropriate failure AuditLog should be displayed (in addition, the error should be logged). Change-Id: Icae61b1c156e2bbb542a8790b310686028cc7a82 Bug-Url: https://bugzilla.redhat.com/957498 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateDiskCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/lsm/LiveMigrateDisksTaskHandler.java 2 files changed, 13 insertions(+), 1 deletion(-) Approvals: Daniel Erez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14440 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icae61b1c156e2bbb542a8790b310686028cc7a82 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>

1 0

Change in ovirt-engine[master]: notifier: move to the new service infrastructure
by yzaslavs＠redhat.com 06 May '13

06 May '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: notifier: move to the new service infrastructure ...................................................................... notifier: move to the new service infrastructure rename service to ovirt-engine-notifier to match other services. remove redundant parameter validations in shell script as there is duplicate java logic. Change-Id: I2aadeca8b3017dad4107939c7c624ef020347b94 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M .gitignore M Makefile M ovirt-engine.spec.in D packaging/bin/engine-notifier-service.sh D packaging/bin/engine-notifier.sh M packaging/conf/notifier.conf.defaults.in M packaging/fedora/setup/basedefs.py M packaging/fedora/setup/common_utils.py M packaging/fedora/setup/engine-cleanup.py M packaging/fedora/setup/engine-upgrade.py M packaging/services/config.py.in A packaging/services/ovirt-engine-notifier.py A packaging/services/ovirt-engine-notifier.systemd.in A packaging/services/ovirt-engine-notifier.sysv.in 14 files changed, 424 insertions(+), 426 deletions(-) Approvals: Alon Bar-Lev: Verified Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14378 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2aadeca8b3017dad4107939c7c624ef020347b94 Gerrit-PatchSet: 8 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: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core:unlock_entity is not locating locked snapshots
by yzaslavs＠redhat.com 06 May '13

06 May '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core:unlock_entity is not locating locked snapshots ...................................................................... core:unlock_entity is not locating locked snapshots Adding handling of 1) -t snapshot flag to display locked snapshots where -q is used 2) -t snapshot to remove a specific snapshot 3) -r if run with -t vm then all vm snapshots are removed as well Change-Id: I415be6fbe99a97740c9d4e9c55c1ad63bc689531 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=958370 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/dbscripts/common_sp.sql M backend/manager/dbscripts/dbfunctions.sh M backend/manager/dbscripts/unlock_entity.sh 3 files changed, 33 insertions(+), 2 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14494 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I415be6fbe99a97740c9d4e9c55c1ad63bc689531 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]: notifier: use LocalConfig as configuration file
by Gerrit Code Review 06 May '13

06 May '13

Anonymous Coward has submitted this change and it was merged. Change subject: notifier: use LocalConfig as configuration file ...................................................................... notifier: use LocalConfig as configuration file gives the advantage of defaults and conf.d structure. Change-Id: Iea3d8a9887d76920c51d26467c634912e64242dd Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/LocalConfig.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/Notifier.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/NotificationConfigurator.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/NotificationProperties.java M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/utils/sender/mail/JavaMailSender.java M backend/manager/tools/src/test/java/org/ovirt/engine/core/notifier/EngineMonitorServiceTest.java M backend/manager/tools/src/test/java/org/ovirt/engine/core/notifier/NotificationServiceTest.java M backend/manager/tools/src/test/java/org/ovirt/engine/core/notifier/utils/NotificationConfiguratorTest.java M ovirt-engine.spec.in M packaging/bin/engine-notifier.sh R packaging/conf/notifier.conf.defaults 12 files changed, 100 insertions(+), 144 deletions(-) -- To view, visit http://gerrit.ovirt.org/14376 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iea3d8a9887d76920c51d26467c634912e64242dd Gerrit-PatchSet: 6 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: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0