March 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: setup: unlock all the entities
by sbonazzo＠redhat.com 26 Mar '15

26 Mar '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: unlock all the entities ...................................................................... packaging: setup: unlock all the entities On upgrades unlock all the entities Change-Id: I1ae892524931e66cd50e2b34a9754035447a6a95 Bug-Url: https://bugzilla.redhat.com/1204673 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/engine/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/upgrade/asynctasks.py 2 files changed, 98 insertions(+), 61 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified oVirt Jenkins CI Server: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/39035 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ae892524931e66cd50e2b34a9754035447a6a95 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Restore host status if pm action failed
by emesika＠redhat.com 26 Mar '15

26 Mar '15

Eli Mesika has submitted this change and it was merged. Change subject: core: Restore host status if pm action failed ...................................................................... core: Restore host status if pm action failed Fixes restoring previous host status if power management start/stop action failed. Change-Id: I96fc99e2ffad639508a5923f268fa57ae312daa3 Bug-Url: https://bugzilla.redhat.com/1205589 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/FenceVdsBaseCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/StartVdsCommandTest.java 2 files changed, 18 insertions(+), 2 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39133 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I96fc99e2ffad639508a5923f268fa57ae312daa3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: webadmin: Fix NPE in VdsArchitectureHelper
by emesika＠redhat.com 26 Mar '15

26 Mar '15

Eli Mesika has submitted this change and it was merged. Change subject: core: webadmin: Fix NPE in VdsArchitectureHelper ...................................................................... core: webadmin: Fix NPE in VdsArchitectureHelper Fixes NullPointerException in VdsArchitectureHelper which can occur if we try to test fence agent settings for a new host which was not yet saved in database. Change-Id: I361c930d01a17da8bc1085286fdb845af0b5c237 Bug-Url: https://bugzilla.redhat.com/1205584 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetFenceAgentStatusQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsArchitectureHelper.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetFenceAgentStatusParameters.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java 4 files changed, 19 insertions(+), 5 deletions(-) Approvals: Martin Peřina: Verified Eli Mesika: Looks good to me, approved Yair Zaslavsky: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/38963 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I361c930d01a17da8bc1085286fdb845af0b5c237 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <wallaroo1976(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: gluster: set iptables rules for HC ports
by sbonazzo＠redhat.com 26 Mar '15

26 Mar '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: core: gluster: set iptables rules for HC ports ...................................................................... core: gluster: set iptables rules for HC ports Firewall rules passed to oivrt-host-deploy must open the ports for brick usage starting at base-port 49217 for avoiding to interfere with libvirt migration. Change-Id: Iaeda3f4310d7f3be69abf47758ceae629682feb7 Bug-Url: https://bugzilla.redhat.com/1202768 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 12 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Sahina Bose: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/38815 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaeda3f4310d7f3be69abf47758ceae629682feb7 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Vijay Bellur <vbellur(a)gmail.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: remove non-existing resource DiskType
by ahadas＠redhat.com 26 Mar '15

26 Mar '15

Arik Hadas has submitted this change and it was merged. Change subject: webadmin: remove non-existing resource DiskType ...................................................................... webadmin: remove non-existing resource DiskType Remove DiskType from Common.gwt.xml Change-Id: I30e5e31cbca9a2ad5557e374a9012be21a278f94 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Daniel Erez: Looks good to me, approved Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/39142 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I30e5e31cbca9a2ad5557e374a9012be21a278f94 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: gluster: Dont append timestamp to volume snapshot name
by kmayilsa＠redhat.com 26 Mar '15

26 Mar '15

Kanagaraj M has submitted this change and it was merged. Change subject: gluster: Dont append timestamp to volume snapshot name ...................................................................... gluster: Dont append timestamp to volume snapshot name Modified to not append the time stamp to the created gluster volume snapshot name. The same is done from gluster side. Change-Id: I206fb257fedd08a264e7e104af67e68f4b05652b Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/CreateGlusterVolumeSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSnapshotScheduleJob.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/CreateGlusterVolumeSnapshotVDSCommand.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumeSnapshotCreateReturnForXmlRpc.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java 7 files changed, 78 insertions(+), 27 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Shubhendu Tripathi: Verified Sahina Bose: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39086 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I206fb257fedd08a264e7e104af67e68f4b05652b Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: build: pack gwt-symbols within jar
by gshereme＠redhat.com 26 Mar '15

26 Mar '15

Greg Sheremeta has submitted this change and it was merged. Change subject: build: pack gwt-symbols within jar ...................................................................... build: pack gwt-symbols within jar due to limitation of rhel build, gwt target files are not available to the make process. packing these as maven artifacts will enable to reference them. somewhat more complex, until we drop el6. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1182000 Change-Id: I489c154859d2092c844d6798a339959baa63a0ce Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile A frontend/webadmin/modules/frontend-symbols/pom.xml A frontend/webadmin/modules/frontend-symbols/resources/userportal/.keep A frontend/webadmin/modules/frontend-symbols/resources/webadmin/.keep M frontend/webadmin/modules/pom.xml 5 files changed, 60 insertions(+), 9 deletions(-) Approvals: Alon Bar-Lev: Verified Greg Sheremeta: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39118 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I489c154859d2092c844d6798a339959baa63a0ce Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: Adapting to latest jjb (1.1.0/1.1.1)
by dcaroest＠redhat.com 25 Mar '15

25 Mar '15

David Caro has submitted this change and it was merged. Change subject: Adapting to latest jjb (1.1.0/1.1.1) ...................................................................... Adapting to latest jjb (1.1.0/1.1.1) Change-Id: I8e5b6c346653ad0e65b819e70225020db31711ca Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/shell-scripts/jenkins_check_yaml.sh M jobs/confs/shell-scripts/jenkins_deploy_yamls.sh M jobs/confs/yaml/jobs/ebay-cors-filter/ebay-cors-filter.yaml M jobs/confs/yaml/jobs/infra-docs/infra-docs.yaml M jobs/confs/yaml/jobs/ioprocess/ioprocess_create-rpms.yaml M jobs/confs/yaml/jobs/jenkins/jenkins_check_yaml.yaml M jobs/confs/yaml/jobs/jenkins/jenkins_deploy_yamls.yaml M jobs/confs/yaml/jobs/kexec-tools/kexec-tools_create-rpms.yaml M jobs/confs/yaml/jobs/mom/mom_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-engine-extension-aaa-ldap/ovirt-engine-extension-aaa-ldap_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-engine-extension-aaa-misc/ovirt-engine-extension-aaa-misc_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-engine-extension-logger-log4j/ovirt-engine-extension-logger-log4j_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-engine-sdk-java/ovirt-engine-sdk-java_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-engine-wildfly/ovirt-engine-wildfly.yaml M jobs/confs/yaml/jobs/ovirt-host-deploy/ovirt-host-deploy_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-hosted-engine-ha/ovirt-hosted-engine-ha_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-hosted-engine-setup/ovirt-hosted-engine-setup_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-image-uploader/ovirt-image-uploader_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-iso-uploader/ovirt-iso-uploader_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-log-collector/ovirt-log-collector_create-rpms.yaml M jobs/confs/yaml/jobs/ovirt-reports/ovirt-reports_create-rpms.yaml M jobs/confs/yaml/jobs/python-windows/python-windows.yaml M jobs/confs/yaml/jobs/qemu/qemu_create-rpms.yaml M jobs/confs/yaml/jobs/repoman/repoman_build-artifacts.yaml M jobs/confs/yaml/jobs/slf4j/slf4j.yaml M jobs/confs/yaml/jobs/unboundid-ldapsdk/unboundid-ldapsdk_create-rpms.yaml M jobs/confs/yaml/jobs/vdsm-jsonrpc-java/vdsm-jsonrpc-java_create-rpms.yaml M jobs/confs/yaml/jobs/vdsm/vdsm_create-rpms.yaml M jobs/confs/yaml/jobs/vdsm/vdsm_create_libgfapi-rpms.yaml M jobs/confs/yaml/jobs/vdsm/vdsm_install-rpm-sanity.yaml M jobs/confs/yaml/scms/jenkins.yaml M jobs/confs/yaml/triggers/gerrit.yaml M jobs/confs/yaml/triggers/ovirt-node.yaml M jobs/confs/yaml/triggers/standard.yaml 34 files changed, 223 insertions(+), 273 deletions(-) Approvals: David Caro: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/39185 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e5b6c346653ad0e65b819e70225020db31711ca Gerrit-PatchSet: 6 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Support deactivate of Cinder Storage Domain
by mlipchuk＠redhat.com 25 Mar '15

25 Mar '15

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Support deactivate of Cinder Storage Domain ...................................................................... core: Support deactivate of Cinder Storage Domain Adding support for deactivate a Cinder Storage Domain Change-Id: Ie9e3764331b6296549698842e1d5b388d1a6b346 Bug-Url: https://bugzilla.redhat.com/1185826 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/DeactivateStorageDomainCommand.java 1 file changed, 11 insertions(+), 0 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/39169 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie9e3764331b6296549698842e1d5b388d1a6b346 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Adding deactivate method for Cinder Helper
by mlipchuk＠redhat.com 25 Mar '15

25 Mar '15

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Adding deactivate method for Cinder Helper ...................................................................... core: Adding deactivate method for Cinder Helper Adding deactivate method to the CINDER helper by seting the Storage Domain status to maintenance. Change-Id: I57c8a7400f40de8876ffb289127e8babe48bee87 Bug-Url: https://bugzilla.redhat.com/1185826 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/CINDERStorageHelper.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/39166 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I57c8a7400f40de8876ffb289127e8babe48bee87 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0