August 2013 - Engine-commits - oVirt List Archives

Change in ovirt-scheduler-proxy[master]: packaging: fixed deployment related issues
by lhornyak＠redhat.com 14 Aug '13

14 Aug '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: packaging: fixed deployment related issues ...................................................................... packaging: fixed deployment related issues This patch fixes the runner execution by adding the site-packages path. Additionally, added logging to provide better information on startup. Change-Id: Ie3be5b94abbc71d841b0b278e5a893127c150b15 Signed-off-by: Doron Fediuck <dfediuck(a)redhat.com> --- M setup.py M src/ovirtscheduler/loader.py M src/ovirtscheduler/oschedproxyd.py M src/ovirtscheduler/request_handler.py M src/ovirtscheduler/runner.py 5 files changed, 50 insertions(+), 30 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18010 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie3be5b94abbc71d841b0b278e5a893127c150b15 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-scheduler-proxy Gerrit-Branch: master Gerrit-Owner: 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]: tools: Fix engine-config logging config
by yzaslavs＠redhat.com 14 Aug '13

14 Aug '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Fix engine-config logging config ...................................................................... tools: Fix engine-config logging config Fixes engine-config logging config under JBoss EAP. The problem started when JBoss EAP reimplemented log4j classes in JBoss Log Manager, but logging configuration initialization from system property log4j.configuration has not been included in those reimplemented classes. This patch initializes log4j configuration for engine-config. Change-Id: I7a125f58bf6450c5d0a1ff4d998fb0d1865da991 Bug-Url: https://bugzilla.redhat.com/996962 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/config/EngineConfig.java 1 file changed, 24 insertions(+), 0 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18073 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7a125f58bf6450c5d0a1ff4d998fb0d1865da991 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: tools: Fix engine-config logging config
by yzaslavs＠redhat.com 14 Aug '13

14 Aug '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Fix engine-config logging config ...................................................................... tools: Fix engine-config logging config Fixes engine-config logging config under JBoss EAP. The problem started when JBoss EAP reimplemented log4j classes in JBoss Log Manager, but logging configuration initialization from system property log4j.configuration has not been included in those reimplemented classes. This patch initializes log4j configuration for engine-config. Change-Id: I6141d73902d2c5943de3f322996aeedc39e0f151 Bug-Url: https://bugzilla.redhat.com/996962 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/config/EngineConfig.java 1 file changed, 24 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18075 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6141d73902d2c5943de3f322996aeedc39e0f151 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: fix task revert for ImportRepoImageCommand
by amureini＠redhat.com 14 Aug '13

14 Aug '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: fix task revert for ImportRepoImageCommand ...................................................................... core: fix task revert for ImportRepoImageCommand ImportRepoImageCreateTaskHandler needs to define getRevertTaskType in order to have the proper task handling in CommandBase during the task revert process. The compensation implementation that was previously using: RunVdsCommand(VDSCommandType.DeleteImageGroup, ...) has been replaced with the more complete: runInternalAction(VdcActionType.RemoveDisk, ...) which handles the entire removal of the disk (database entry included). Change-Id: I2fea15129eb9e1b0dc1ed6d7d09d8dd63cddd5db Signed-off-by: Federico Simoncelli <fsimonce(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCreateTaskHandler.java 1 file changed, 5 insertions(+), 29 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17756 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2fea15129eb9e1b0dc1ed6d7d09d8dd63cddd5db Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-guest-agent[master]: agent: ovirt-guest-agent version 1.0.8
by bazulay＠redhat.com 14 Aug '13

14 Aug '13

Barak Azulay has submitted this change and it was merged. Change subject: agent: ovirt-guest-agent version 1.0.8 ...................................................................... agent: ovirt-guest-agent version 1.0.8 ChangeLog updated Change-Id: I71b6165b799a730bfe1b3559b8840b0457af997a Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ChangeLog M configure.ac M debian/changelog M debian/copyright M debian/files M ovirt-guest-agent.rhel6.spec M ovirt-guest-agent.spec 7 files changed, 944 insertions(+), 49 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, but someone else must approve Barak Azulay: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11302 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I71b6165b799a730bfe1b3559b8840b0457af997a Gerrit-PatchSet: 21 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>

1 0

Change in ovirt-engine-sdk[master]: sdk: encapsulate context removal
by mpastern＠redhat.com 14 Aug '13

14 Aug '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: encapsulate context removal ...................................................................... sdk: encapsulate context removal this patch encapsulate context removal and among removing cache content, removes the cache instance as well Change-Id: I5a4560c17f7d0123e5cc6c33a1b2fc393849e302 Signed-off-by: Michael pasternak <mpastern(a)redhat.com> --- M src/codegen/templates/entrypointmethodstemplate M src/ovirtsdk/api.py M src/ovirtsdk/infrastructure/contextmanager.py 3 files changed, 14 insertions(+), 4 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18066 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a4560c17f7d0123e5cc6c33a1b2fc393849e302 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk 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]: core: scheduling: handle external scheduler failure
by gchaplik＠redhat.com 14 Aug '13

14 Aug '13

Gilad Chaplik has submitted this change and it was merged. Change subject: core: scheduling: handle external scheduler failure ...................................................................... core: scheduling: handle external scheduler failure in case the external scheduler fails, continue with scheduling process, 'skipping' external scheduler (no) results Change-Id: I11cc70260e99cee7c563cb727274608e325da354 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/external/ExternalSchedulerBrokerImpl.java 2 files changed, 8 insertions(+), 18 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18037 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I11cc70260e99cee7c563cb727274608e325da354 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: scheduling: don't run externl scheduler on !empty data
by gchaplik＠redhat.com 14 Aug '13

14 Aug '13

Gilad Chaplik has submitted this change and it was merged. Change subject: core: scheduling: don't run externl scheduler on !empty data ...................................................................... core: scheduling: don't run externl scheduler on !empty data * Cluster policy contains external filters or weight modules. * All hosts filtered out in internal filtering. Change-Id: Ib77bb37590153802939bc1e2d2a9a5ee198fbce6 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18035 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib77bb37590153802939bc1e2d2a9a5ee198fbce6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: scheduling: indication for external plugin
by gchaplik＠redhat.com 14 Aug '13

14 Aug '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: scheduling: indication for external plugin ...................................................................... webadmin: scheduling: indication for external plugin the text '(EXT)' will be added next to each module that is loaded by the external proxy. Change-Id: Ica7522ed5a46012c3c1205044c57723d4067aa11 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/scheduling/ClusterPolicyPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/scheduling/ClusterPolicyPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/scheduling/panels/FunctionPolicyUnitPanel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/scheduling/panels/PolicyUnitPanel.java 5 files changed, 42 insertions(+), 7 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18034 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica7522ed5a46012c3c1205044c57723d4067aa11 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Missing properties for DAO tests
by amureini＠redhat.com 14 Aug '13

14 Aug '13

Allon Mureinik has submitted this change and it was merged. Change subject: core: Missing properties for DAO tests ...................................................................... core: Missing properties for DAO tests Added missing properties to the tests' engine.conf to avoid annoying stack traces being printed out when running with -Penable-dao-tests. Change-Id: I371aa390a380447014825f5197661a3516beff8d Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/dal/src/test/resources/engine.conf 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Allon Mureinik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18044 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I371aa390a380447014825f5197661a3516beff8d 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: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0