August 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: setup: print termination message
by oschreib＠redhat.com 26 Aug '13

26 Aug '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: print termination message ...................................................................... packaging: setup: print termination message Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=999964 Change-Id: Ia2f27ba721f79606de38b83c04ce33095cf32b0b Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/core/misc.py 1 file changed, 21 insertions(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18440 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia2f27ba721f79606de38b83c04ce33095cf32b0b 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> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core: Adds task description for SshSoftFencingCommand
by yzaslavs＠redhat.com 26 Aug '13

26 Aug '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Adds task description for SshSoftFencingCommand ...................................................................... core: Adds task description for SshSoftFencingCommand Adds task description for SshSoftFencingCommand. Change-Id: Iaa947dde1d93c7e8c96e111b02a55a2a179d3f9b Bug-Url: https://bugzilla.redhat.com/988305 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Martin Peřina: Verified Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18494 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaa947dde1d93c7e8c96e111b02a55a2a179d3f9b 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]: engine: Propagate error message when querying for LDAP users
by emesika＠redhat.com 26 Aug '13

26 Aug '13

Eli Mesika has submitted this change and it was merged. Change subject: engine: Propagate error message when querying for LDAP users ...................................................................... engine: Propagate error message when querying for LDAP users A previous commit fixed this issue for users/groups(dcfd219). call setHandleFailure(true) on users as well so the handleQueryError code will be called on ldap provider errors. Change-Id: I9362e48a36be61fd75c386883b9ccaf28ed15a27 Signed-off-by: Mooli Tayer <mtayer(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/users/AdElementListModel.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18363 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9362e48a36be61fd75c386883b9ccaf28ed15a27 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: Asaf Shakarchi <asaf(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Support for gluster hooks added
by kmayilsa＠redhat.com 26 Aug '13

26 Aug '13

Kanagaraj M has submitted this change and it was merged. Change subject: restapi: Support for gluster hooks added ...................................................................... restapi: Support for gluster hooks added Gluster hooks are deployed during installation of gluster in the servers of a gluster cluster. The ovirt engine polls each of the servers in the cluster and stores information about the gluster hooks - primarily to check if the hook scripts are in sync across servers. This patch adds API support for managing gluster hooks. - api/cluster/{id}/glusterhooks lists all gluster hooks in the cluster along with conflict status - api/cluster/{id}/glusterhooks/{glusterhook:id}|rel=delete resolve missing hook conflict by removing hook script from all servers and engine database - api/cluster/{id}/glusterhooks/{id} show details of a gluster hook actions supported: - enable enables hook in all servers in the cluster - disable disables hook in all servers in the cluster - resolve action.resolutionType = COPY resolve content conflict by copying content from either engine or from one of the servers to all or servers where the hook is missing as applicable (based on parameters) action.resolutionType = ADD resolve missing hook conflict by copying hook stored in engine database to all servers where the hook is missing Change-Id: Ibf9c0299977448e0b50feeed6e38a015c60b86fd Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M backend/manager/modules/restapi/interface/common/jaxrs/src/main/java/org/ovirt/engine/api/common/util/LinkHelper.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/HookContentType.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/HookStage.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/HookStatus.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/ResolutionType.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/ClusterResource.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/gluster/GlusterHookResource.java A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/gluster/GlusterHooksResource.java M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendClusterResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendClustersResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterHookResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterHooksResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/utils/FeaturesHelper.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterHookResourceTest.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterHooksResourceTest.java A backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/GlusterHookMapper.java 19 files changed, 1,186 insertions(+), 3 deletions(-) Approvals: Kanagaraj M: Looks good to me, approved Sahina Bose: Verified -- To view, visit http://gerrit.ovirt.org/16512 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibf9c0299977448e0b50feeed6e38a015c60b86fd Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Fix the Network QoS unit translation
by masayag＠redhat.com 25 Aug '13

25 Aug '13

Moti Asayag has submitted this change and it was merged. Change subject: core: Fix the Network QoS unit translation ...................................................................... core: Fix the Network QoS unit translation engine is using Megabits as units for the Network QoS 'Average' and 'Peak' properties. vdsm is using Kilobytes for those properties. current translation is wrong (1:1024). it should be 1:128 Change-Id: Ib299bc401aa9863717a663666d19ba1ae4ef3aba Bug-Url: https://bugzilla.redhat.com/999799 Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VmInfoBuilder.java 1 file changed, 5 insertions(+), 4 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18398 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib299bc401aa9863717a663666d19ba1ae4ef3aba Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: ofri masad <omasad(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Livnat Peer <lpeer(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0

Change in otopi[master]: core: sort or randomize events
by Alon Bar-Lev 25 Aug '13

25 Aug '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: sort or randomize events ...................................................................... core: sort or randomize events Before this change, the order of events was semi-random, except for specific conditions (before, after, priority, stage). This change either sorts the list of events, which is the default behavior, randomizes this list, if directed to do so by the environment. Change-Id: I3a81095dc4f90e523028b3f5b30c6045829f8ca1 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/otopi/constants.py M src/otopi/context.py 2 files changed, 11 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18478 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a81095dc4f90e523028b3f5b30c6045829f8ca1 Gerrit-PatchSet: 4 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Added setVmName() to VmNetworkInterface mapper
by masayag＠redhat.com 25 Aug '13

25 Aug '13

Moti Asayag has submitted this change and it was merged. Change subject: engine: Added setVmName() to VmNetworkInterface mapper ...................................................................... engine: Added setVmName() to VmNetworkInterface mapper Got lost among the VNIC profile patches and caused a minor UI bug. Change-Id: I0f83a7f6aac644d7a59998ad88b1ef2aead73b5f Bug-Url: https://bugzilla.redhat.com/1000297 Bug-Url: https://bugzilla.redhat.com/1000306 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/network/VmNetworkInterfaceDaoDbFacadeImpl.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Moti Asayag: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18477 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0f83a7f6aac644d7a59998ad88b1ef2aead73b5f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: Fix width of migration mode selector
by gchaplik＠redhat.com 25 Aug '13

25 Aug '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Fix width of migration mode selector ...................................................................... webadmin: Fix width of migration mode selector Width of Migration support selector (in VM popup view) was extended to show the full text. Change-Id: Ie828baa759e32bde042f19b8b6b690ebfeff747f Bug-Url: https://bugzilla.redhat.com/992993 Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml 2 files changed, 13 insertions(+), 2 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18410 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie828baa759e32bde042f19b8b6b690ebfeff747f Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: ofri masad <omasad(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Eldan Hildesheim <info(a)eldanet.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: move class to another package
by dfediuck＠redhat.com 25 Aug '13

25 Aug '13

Doron Fediuck has submitted this change and it was merged. Change subject: restapi: move class to another package ...................................................................... restapi: move class to another package moved two classes to another package in another project Change-Id: I36f4bfe8506a6d78aff6a3f90e6f6197f2e62d30 Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/pom.xml R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/WatchdogAction.java R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/model/WatchdogModel.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendWatchdogsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/WatchdogValidator.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendWatchdogResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendWatchdogsResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/WatchdogMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/WatchdogMapperTest.java 10 files changed, 20 insertions(+), 12 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18293 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I36f4bfe8506a6d78aff6a3f90e6f6197f2e62d30 Gerrit-PatchSet: 2 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: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: fix add watchdog
by dfediuck＠redhat.com 25 Aug '13

25 Aug '13

Doron Fediuck has submitted this change and it was merged. Change subject: engine: fix add watchdog ...................................................................... engine: fix add watchdog Fixes the watchdog add in restapi. - added validation for adding and updating entities - removed WatchdogsResource as it was not needed Change-Id: I95362fda69b8ec7c02893ba30cdf19be7eee5eac Signed-off-by: Laszlo Hornyak <lhornyak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddWatchdogCommand.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/WatchdogsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendWatchdogResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendWatchdogsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/rsdl/RsdlBuilder.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/WatchdogAction.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/WatchdogMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/WatchdogModel.java 8 files changed, 29 insertions(+), 20 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/17434 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I95362fda69b8ec7c02893ba30cdf19be7eee5eac Gerrit-PatchSet: 7 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: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0