August 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: start discovery thread only if enabled
by lhornyak＠redhat.com 09 Aug '13

09 Aug '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: start discovery thread only if enabled ...................................................................... engine: start discovery thread only if enabled Checks the external scheduler configuration before starting the external scheduler discovery thread. Added logging for better tracing. Change-Id: Ibac067131d3725d39afaf2f4941165eccdb60372 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java 1 file changed, 7 insertions(+), 2 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17703 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibac067131d3725d39afaf2f4941165eccdb60372 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(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 <drekmastermind(a)gmail.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: external scheduler integration
by lhornyak＠redhat.com 09 Aug '13

09 Aug '13

Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: external scheduler integration ...................................................................... engine: external scheduler integration This patch adds a new ExternalSchedulingManager class to handle communication with the external scheduler proxy. Other changes: - added configuration values (scheduler proxy url, enable/disable) - added external scheduler invocation to vds filtering - error messages Change-Id: I95b6660beb319df0afe569f31c16033bf6192df9 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/bll/pom.xml M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/external/ExternalSchedulerBroker.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/external/ExternalSchedulerBrokerImpl.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/external/ExternalSchedulerDiscoveryResult.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/external/ExternalSchedulerDiscoveryThread.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/external/ExternalSchedulerDiscoveryUnit.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/external/ExternalSchedulerFactory.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/etc/engine-config/engine-config.properties 12 files changed, 757 insertions(+), 20 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17501 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I95b6660beb319df0afe569f31c16033bf6192df9 Gerrit-PatchSet: 16 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Laszlo Hornyak <lhornyak(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: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Noam Slomianko <nslomian(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: spec: requires ovirt-hosted-engine-ha
by sbonazzo＠redhat.com 09 Aug '13

09 Aug '13

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: spec: requires ovirt-hosted-engine-ha ...................................................................... packaging: spec: requires ovirt-hosted-engine-ha Adding ovirt-hosted-engine-ha requirement. Needed for having ha services to be enabled by --deploy. Change-Id: Ib4a22fa7413fec280374ff26ebf61bf1fce18c4c Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-hosted-engine-setup.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/17835 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib4a22fa7413fec280374ff26ebf61bf1fce18c4c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)gmail.com> Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com> Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-host-deploy[master]: core: add virt flag
by Alon Bar-Lev 09 Aug '13

09 Aug '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: add virt flag ...................................................................... core: add virt flag this will enable us in future to provide virt specific behaviour. Change-Id: I18f57bd601a4abfdaab26e7906181a2801c1d29f Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M README.environment M src/ovirt_host_deploy/constants.py 2 files changed, 11 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17857 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I18f57bd601a4abfdaab26e7906181a2801c1d29f Gerrit-PatchSet: 1 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>

1 0

Change in ovirt-engine[master]: engine:Trusted Compute Pools - Open Attestation integration ...
by dfediuck＠redhat.com 09 Aug '13

09 Aug '13

Doron Fediuck has submitted this change and it was merged. Change subject: engine:Trusted Compute Pools - Open Attestation integration with oVirt engine ...................................................................... engine:Trusted Compute Pools - Open Attestation integration with oVirt engine Detailed description: http://wiki.ovirt.org/Trusted_compute_pools Add rule to canDoAction to block setting of cluster as trusted when Attestation Server is not defined properly. Change-Id: I1dd7da6ed6ff33cb8927fb8853eb9799cf178190 Signed-off-by: Jimmy <gang.wei(a)intel.com> Signed-off-by: Emily <lijuan.zhang(a)intel.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsGroupCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/ErrorType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/VdcBllMessages.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/ConfigurationValues.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/ErrorMessageHelper.java M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java A packaging/dbscripts/upgrade/03_03_0730_update_attestation_server.sql M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/etc/engine-config/engine-config.properties 13 files changed, 42 insertions(+), 4 deletions(-) Approvals: Doron Fediuck: Looks good to me, approved Emily Zhang: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/16987 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1dd7da6ed6ff33cb8927fb8853eb9799cf178190 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gang Wei <gang.wei(a)intel.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Dave Chen <wei.d.chen(a)intel.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Emily Zhang <lijuan.zhang(a)intel.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0

Change in ovirt-hosted-engine-ha[master]: agent, broker: run daemons as vdsm:kvm
by gpadgett＠redhat.com 09 Aug '13

09 Aug '13

Greg Padgett has submitted this change and it was merged. Change subject: agent, broker: run daemons as vdsm:kvm ...................................................................... agent, broker: run daemons as vdsm:kvm Run the daemons as the vdsm user and kvm group in order to avoid permissions issues writing to nfs shares. Change-Id: Ib031ea5e0ffe365af3fdd323e303c1bdbc2d1ce9 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M Makefile.am M configure.ac M ovirt-hosted-engine-ha.spec.in M ovirt_hosted_engine_ha/agent/agent.py M ovirt_hosted_engine_ha/agent/constants.py.in M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/broker/broker.py M ovirt_hosted_engine_ha/broker/constants.py.in A sudoers/Makefile.am A sudoers/sudoers.in 10 files changed, 121 insertions(+), 6 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17800 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib031ea5e0ffe365af3fdd323e303c1bdbc2d1ce9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>

1 0

Change in ovirt-hosted-engine-ha[master]: build: regenerate spec file after configure
by gpadgett＠redhat.com 09 Aug '13

09 Aug '13

Greg Padgett has submitted this change and it was merged. Change subject: build: regenerate spec file after configure ...................................................................... build: regenerate spec file after configure Configure options may change pathnames that affect compilation, but the files that are affected aren't always regenerated. This is especially relevant to files processed by var_subst.inc, such as the rpm spec file. To prevent issues including build failures, go ahead and `make clean` in autogen.sh after running `configure`. Change-Id: I26ecbe06e29e9c1821cb0d623ef0783709eefcbc Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M autogen.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17856 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I26ecbe06e29e9c1821cb0d623ef0783709eefcbc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>

1 0

Change in ovirt-hosted-engine-ha[master]: build: fix build for new autoconf
by gpadgett＠redhat.com 09 Aug '13

09 Aug '13

Greg Padgett has submitted this change and it was merged. Change subject: build: fix build for new autoconf ...................................................................... build: fix build for new autoconf The path for the m4 directory wasn't correct, causing autoreconf to fail for automake 1.13, which doesn't create AC_CONFIG_MACRO_DIR automatically. See https://bugzilla.redhat.com/show_bug.cgi?id=901333 for details. Change-Id: I33ddf7bcf950411ddc89e88b3ebc3d99df5ff617 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M Makefile.am R build/m4/ax_python_module.m4 2 files changed, 1 insertion(+), 1 deletion(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17799 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I33ddf7bcf950411ddc89e88b3ebc3d99df5ff617 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>

1 0

Change in ovirt-hosted-engine-ha[master]: broker: fix SSL vdsClient connections
by gpadgett＠redhat.com 09 Aug '13

09 Aug '13

Greg Padgett has submitted this change and it was merged. Change subject: broker: fix SSL vdsClient connections ...................................................................... broker: fix SSL vdsClient connections Set correct variable in vdscli for SSL. Change-Id: Ic127be901965e541dbd72655c9b509d308565cfe Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M ovirt_hosted_engine_ha/broker/submonitor_util.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17798 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic127be901965e541dbd72655c9b509d308565cfe Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>

1 0

Change in ovirt-hosted-engine-ha[master]: agent: don't spam /var/log/messages
by gpadgett＠redhat.com 09 Aug '13

09 Aug '13

Greg Padgett has submitted this change and it was merged. Change subject: agent: don't spam /var/log/messages ...................................................................... agent: don't spam /var/log/messages The "Host x metadata updated" message occurs frequently and should be at INFO level so that it's not written to /var/log/messages. Change-Id: I3df3cc61c74953cc345fed84c70e67f5a961d83a Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17797 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3df3cc61c74953cc345fed84c70e67f5a961d83a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>

1 0