October 2013 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-ha[master]: agent: enable local maintenance mode
by gpadgett＠redhat.com 29 Oct '13

29 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: agent: enable local maintenance mode ...................................................................... agent: enable local maintenance mode The HA agent needs to be aware of local maintenance mode in order to not run the engine VM on the local host. This provides a method of setting a local maintenance flag which is stored on the host and then read back by the agent, which takes appropriate action to stop the VM. A client method has been provided to set this mode: HAClient.set_maintenance_mode(mode, value) Where mode is a constant in HAClient.MaintenanceMode, and value is a value parsable as a boolean (True, 'true', 'yes', 1, False, etc.) Change-Id: I60e62cd03daec812e16a3a5bbd2d3d896e41402d Bug-Url: https://bugzilla.redhat.com/1015724 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M build/var_subst.inc M configure.ac M ovirt-hosted-engine-ha.spec.in M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/client/client.py M ovirt_hosted_engine_ha/env/Makefile.am M ovirt_hosted_engine_ha/env/config.py M ovirt_hosted_engine_ha/env/constants.py.in A ovirt_hosted_engine_ha/env/ha.conf M ovirt_hosted_engine_ha/lib/util.py 10 files changed, 139 insertions(+), 15 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20278 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I60e62cd03daec812e16a3a5bbd2d3d896e41402d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: agent: enable global maintenance mode
by gpadgett＠redhat.com 29 Oct '13

29 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: agent: enable global maintenance mode ...................................................................... agent: enable global maintenance mode Setting the maintenance flag ("maintenance=1") in the cluster-wide shared metadata block will now cause all aware HA agents to ignore the HA VM state, allowing the admin to perform arbitrary operations on/with the VM. To facilitate setting/unsetting maintenance mode, the HAClient class has a new method: HAClient.set_global_md_flag('maintenance', <true|false>) The boolean argument can be anything typically parsed as a boolean, such as True, 'true', 'yes', 1, False, 'false', 'no', 0. Change-Id: I796d768e197c5fe73646e67297714392bc6e8201 Bug-Url: https://bugzilla.redhat.com/1015724 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M doc/agent-fsm.gv M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/broker/constants.py.in M ovirt_hosted_engine_ha/broker/storage_broker.py M ovirt_hosted_engine_ha/client/client.py M ovirt_hosted_engine_ha/env/constants.py.in M ovirt_hosted_engine_ha/lib/metadata.py 7 files changed, 241 insertions(+), 30 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/19711 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I796d768e197c5fe73646e67297714392bc6e8201 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[master]: history: resynced the create db scripts
by ydary＠redhat.com 29 Oct '13

29 Oct '13

Yaniv Dary has submitted this change and it was merged. Change subject: history: resynced the create db scripts ...................................................................... history: resynced the create db scripts Change-Id: I09136453a3b50ca34b29e9d26912cbb10e2f4998 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M data-warehouse/historydbscripts_postgres/common_sp.sql M data-warehouse/historydbscripts_postgres/create_db.sh M data-warehouse/historydbscripts_postgres/create_schema.sh M data-warehouse/historydbscripts_postgres/dbcustomfunctions.sh M data-warehouse/historydbscripts_postgres/dbfunctions.sh 5 files changed, 463 insertions(+), 163 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20653 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I09136453a3b50ca34b29e9d26912cbb10e2f4998 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh 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]: webadmin : rebalance file size bug fixed
by kmayilsa＠redhat.com 29 Oct '13

29 Oct '13

Kanagaraj M has submitted this change and it was merged. Change subject: webadmin : rebalance file size bug fixed ...................................................................... webadmin : rebalance file size bug fixed Rebalance file size in rebalance status was getting rounded down to nearest integeral value now it is modified to show a floating value. Change-Id: I04862c25ee0549eb7faa84170a11ae71fb12cf45 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationMessages.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/renderer/RebalanceFileSizeRenderer.java 2 files changed, 16 insertions(+), 11 deletions(-) Approvals: Kanagaraj M: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20045 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I04862c25ee0549eb7faa84170a11ae71fb12cf45 Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-ha[master]: env: fix path to glusterfs storage domains
by gpadgett＠redhat.com 29 Oct '13

29 Oct '13

Greg Padgett has submitted this change and it was merged. Change subject: env: fix path to glusterfs storage domains ...................................................................... env: fix path to glusterfs storage domains If the hosted engine config specifies a GlusterFS storage domain, look in the appropriate path on the mounted domain to find it. The agent will now use the existing env.path module to get pathnames rather than having its own (duplicated) code for this. Change-Id: I7838e373d1cbe0682bfbffa697e6cd5eff819308 Bug-Url: https://bugzilla.redhat.com/1023965 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/env/config.py M ovirt_hosted_engine_ha/env/path.py 3 files changed, 7 insertions(+), 16 deletions(-) Approvals: Greg Padgett: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20631 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7838e373d1cbe0682bfbffa697e6cd5eff819308 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> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: adding utility for db objects owner change
by emesika＠redhat.com 29 Oct '13

29 Oct '13

Eli Mesika has submitted this change and it was merged. Change subject: core: adding utility for db objects owner change ...................................................................... core: adding utility for db objects owner change This utility change objects in the engine database to be owned by the engine user. Change-Id: I56c59c0fe749389bd110bcd1a39faae74e71174b Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M packaging/dbscripts/common_sp.sql M packaging/dbscripts/dbfunctions.sh A packaging/setup/dbutils/changedbowner.sh 3 files changed, 65 insertions(+), 3 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18682 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I56c59c0fe749389bd110bcd1a39faae74e71174b Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Hide virt specific fields in gluster only mode
by lvernia＠redhat.com 29 Oct '13

29 Oct '13

Lior Vernia has submitted this change and it was merged. Change subject: webadmin: Hide virt specific fields in gluster only mode ...................................................................... webadmin: Hide virt specific fields in gluster only mode Hiding following Virt related fields in gluster only mode. 1. Select as SPM action under host 2. Network Provider option in New Host Popup 3. Network Provider tab in Install Host Popup 4. Emulated machine under General details tab in Cluster tab Change-Id: Ibe932731d7ec1913569e7f2f54c270b174eb6e36 Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1019570 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/NewHostModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostInstallPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostInstallPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/ClusterGeneralModelForm.java 6 files changed, 21 insertions(+), 3 deletions(-) Approvals: Lior Vernia: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20539 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibe932731d7ec1913569e7f2f54c270b174eb6e36 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: host-deploy: always register as standard host
by emesika＠redhat.com 29 Oct '13

29 Oct '13

Eli Mesika has submitted this change and it was merged. Change subject: host-deploy: always register as standard host ...................................................................... host-deploy: always register as standard host during host-deploy (post approve) we detect if host is node and set type. this will enable for regular hosts to be registered as well. Change-Id: Ie2449527d7dba92179e22ddc97521841b3e02010 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ApproveVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RegisterVdsQuery.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/RegisterVdsParameters.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M backend/manager/modules/root/src/main/java/org/ovirt/engine/core/register/RegisterServlet.java 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/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_de_DE.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_es_ES.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_fr_FR.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_ja_JP.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_ko_KR.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_pt_BR.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_zh_CN.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_de_DE.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_es_ES.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_fr_FR.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_ja_JP.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_ko_KR.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_pt_BR.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_zh_CN.properties 24 files changed, 12 insertions(+), 63 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20611 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie2449527d7dba92179e22ddc97521841b3e02010 Gerrit-PatchSet: 2 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: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: host-deploy: open correct vdsm port in iptables
by emesika＠redhat.com 29 Oct '13

29 Oct '13

Eli Mesika has submitted this change and it was merged. Change subject: host-deploy: open correct vdsm port in iptables ...................................................................... host-deploy: open correct vdsm port in iptables Change-Id: I17c26f0c87ef8e49b842909103127cb675247d89 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 2 files changed, 6 insertions(+), 2 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20600 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I17c26f0c87ef8e49b842909103127cb675247d89 Gerrit-PatchSet: 2 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: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: host-deploy: cleanup: move ssh registration parameters to pa...
by emesika＠redhat.com 29 Oct '13

29 Oct '13

Eli Mesika has submitted this change and it was merged. Change subject: host-deploy: cleanup: move ssh registration parameters to parameters class ...................................................................... host-deploy: cleanup: move ssh registration parameters to parameters class Change-Id: Iae82caad3b9d7c56e197e8978dc7ffe59e961344 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RegisterVdsQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/RegisterVdsParameters.java M backend/manager/modules/root/src/main/java/org/ovirt/engine/core/register/RegisterServlet.java 3 files changed, 28 insertions(+), 8 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/20599 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae82caad3b9d7c56e197e8978dc7ffe59e961344 Gerrit-PatchSet: 2 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: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0