July 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: setup: do not log sensitive information
by oschreib＠redhat.com 16 Jul '13

16 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: do not log sensitive information ...................................................................... packaging: setup: do not log sensitive information Change-Id: Ic5e00da7d95e284c77f82b49de8bea6a13427f8a Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/pki/ssh.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16783 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic5e00da7d95e284c77f82b49de8bea6a13427f8a 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> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: build: remove debug
by oschreib＠redhat.com 16 Jul '13

16 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: build: remove debug ...................................................................... packaging: build: remove debug Change-Id: I9fc45bca214783a5e3afb6ca1472d7e7a88fd79f Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16782 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9fc45bca214783a5e3afb6ca1472d7e7a88fd79f 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> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: Revert "packaging: setup: add branding support"
by oschreib＠redhat.com 16 Jul '13

16 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: Revert "packaging: setup: add branding support" ...................................................................... Revert "packaging: setup: add branding support" This reverts commit e1c1fa6a058edc8e070e77c23384b6533594e512 because of versionlock, we cannot share the same package between setup and backend. Change-Id: Ib1e9933cf2fa5b589e5bd26ea8bdfdb6551b2c8e --- M packaging/branding/ovirt.brand/messages.properties D packaging/setup/ovirt_engine_setup/branding.py M packaging/setup/ovirt_engine_setup/constants.py 3 files changed, 2 insertions(+), 96 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16773 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib1e9933cf2fa5b589e5bd26ea8bdfdb6551b2c8e 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> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: fix ovirt-engine-notifier jboss configurat...
by oschreib＠redhat.com 16 Jul '13

16 Jul '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: fix ovirt-engine-notifier jboss configuration ...................................................................... packaging: setup: fix ovirt-engine-notifier jboss configuration Change-Id: I6b750e75e9dcf62b2eff644ae57ff7289b86abd5 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/conf/notifier.conf.defaults.in M packaging/fedora/setup/basedefs.py 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16762 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b750e75e9dcf62b2eff644ae57ff7289b86abd5 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> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Preventing host activation when host is non responsive
by yzaslavs＠redhat.com 16 Jul '13

16 Jul '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Preventing host activation when host is non responsive ...................................................................... core: Preventing host activation when host is non responsive The following introduces proper error messages + logic to the VdsValidator to prevent non responsive host from being activated Bug-Url: https://bugzilla.redhat.com/669475 Change-Id: I02846c6e1bc780f5b80a9e99c167f49d24c6bc57 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsValidator.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 6 files changed, 10 insertions(+), 0 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16650 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I02846c6e1bc780f5b80a9e99c167f49d24c6bc57 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Use VdsValidator in canDoAction of ActivateVds
by yzaslavs＠redhat.com 16 Jul '13

16 Jul '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Use VdsValidator in canDoAction of ActivateVds ...................................................................... core: Use VdsValidator in canDoAction of ActivateVds Introduce usage of VdsValidator to ActivateVds (as exists in other commands) Change-Id: Ie9a887de27757cd8ff89c8d68eb916be68d327fd Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ActivateVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsValidator.java 2 files changed, 38 insertions(+), 19 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16649 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie9a887de27757cd8ff89c8d68eb916be68d327fd Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Fix some Error constants of Host Activation
by yzaslavs＠redhat.com 16 Jul '13

16 Jul '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Fix some Error constants of Host Activation ...................................................................... core: Fix some Error constants of Host Activation There is no reason the ACTIVATE word will be part of the constants. As the $action and $type exist in these two messages, they can be reused if needed for other operations on Host. Change-Id: I65d5cd35c8ef4a1a318670a90ffdfb9bfce9ad8c Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ActivateVdsCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 6 files changed, 18 insertions(+), 20 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16648 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I65d5cd35c8ef4a1a318670a90ffdfb9bfce9ad8c Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in otopi[master]: packaging: post otopi-1.1.0_beta
by Alon Bar-Lev 16 Jul '13

16 Jul '13

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

1 0

Change in otopi[master]: packaging: otopi-1.1.0_beta
by Alon Bar-Lev 16 Jul '13

16 Jul '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: otopi-1.1.0_beta ...................................................................... packaging: otopi-1.1.0_beta Change-Id: I3d6105339c9ca741565ccd21f3a85d6cad7c7b0f Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M configure.ac 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16922 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3d6105339c9ca741565ccd21f3a85d6cad7c7b0f Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master 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: support soon-to-be-released vdsm-4.12
by dfediuck＠redhat.com 16 Jul '13

16 Jul '13

Doron Fediuck has submitted this change and it was merged. Change subject: core: support soon-to-be-released vdsm-4.12 ...................................................................... core: support soon-to-be-released vdsm-4.12 vdsm-4.12 would be part of oVirt-3.3. Change-Id: Icc7692bc6e8373cb854443dbeaea08bcd43fe30d Signed-off-by: Dan Kenigsberg <danken(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/16911 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc7692bc6e8373cb854443dbeaea08bcd43fe30d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com>

1 0