April 2014 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.1]: don't die when trying to convert None to float
by msivak＠redhat.com 28 Apr '14

28 Apr '14

Martin Sivák has submitted this change and it was merged. Change subject: don't die when trying to convert None to float ...................................................................... don't die when trying to convert None to float Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1091360 Change-Id: I1c0e5906a2708d546dcb5c8e1d68e07294edaa69 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/states.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Sivák: Looks good to me, approved Jiří Moskovčák: Verified -- To view, visit http://gerrit.ovirt.org/27071 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1c0e5906a2708d546dcb5c8e1d68e07294edaa69 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.1 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Fix issue with GetStorageDomainByNameQuery
by emesika＠redhat.com 28 Apr '14

28 Apr '14

Eli Mesika has submitted this change and it was merged. Change subject: core: Fix issue with GetStorageDomainByNameQuery ...................................................................... core: Fix issue with GetStorageDomainByNameQuery Commit 88756e3897ba467eb50055a7dda0c1afb11295d0 introduced function GetStorageDomainByNameQuery which faults at vm export. This patch fixes this issue. Change-Id: Ifae06739c658a1f602421081590790b11bba5806 Bug-Url: https://bugzilla.redhat.com/1087028 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/StorageDomainStaticDAODbFacadeImpl.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eli Mesika: Looks good to me, approved Allon Mureinik: Looks good to me, but someone else must approve Liran Zelkha: Verified -- To view, visit http://gerrit.ovirt.org/27088 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifae06739c658a1f602421081590790b11bba5806 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[ovirt-3.3]: packaging: setup: Filter db password in log
by ydary＠redhat.com 28 Apr '14

28 Apr '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: Filter db password in log ...................................................................... packaging: setup: Filter db password in log Change-Id: Ia795bf73f9f473225e73e90d3bbd0b03d92509ff Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/common_utils.py 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27049 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia795bf73f9f473225e73e90d3bbd0b03d92509ff Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: ovirt-3.3 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Forget HTTP session after invalidate
by juan.hernandez＠redhat.com 28 Apr '14

28 Apr '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Forget HTTP session after invalidate ...................................................................... restapi: Forget HTTP session after invalidate Currently when the RESTAPI receives a request that contains a valid session identifier, but that doesn't require persistent authentication we invalidate the session. This is correct, but we also try to use that invalidated session later, which generates exceptions. To avoid that this patch makes sure that when the session is invalidated the variable that references it is also set to null. Change-Id: I88dcd37f80eb1fa85aa0441c3cbdfed18e84d5f3 Bug-Url: https://bugzilla.redhat.com/1091371 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/common/jaxrs/src/main/java/org/ovirt/engine/api/common/security/auth/Challenger.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/27072 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I88dcd37f80eb1fa85aa0441c3cbdfed18e84d5f3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(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 samples-uiplugins[master]: gluster-nagios-dashboard : Add Trends Tab
by vszocs＠redhat.com 28 Apr '14

28 Apr '14

Vojtech Szocs has submitted this change and it was merged. Change subject: gluster-nagios-dashboard : Add Trends Tab ...................................................................... gluster-nagios-dashboard : Add Trends Tab Change-Id: Id7c00636e380da154f7f457066fe89475d04c9b4 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M gluster-nagios-dashboard/README M gluster-nagios-dashboard/gluster-nagios-dashboard.json A gluster-nagios-dashboard/src/images/print.png A gluster-nagios-dashboard/src/images/refresh.png A gluster-nagios-dashboard/src/js/common.js M gluster-nagios-dashboard/src/js/dashboard-init.js D gluster-nagios-dashboard/src/js/init.js A gluster-nagios-dashboard/src/js/trends.js A gluster-nagios-dashboard/src/lib/ui-bootstrap-tpls-0.10.0.js M gluster-nagios-dashboard/src/start.html A gluster-nagios-dashboard/src/trendsTab.html M gluster-nagios-dashboard/test/spec/dashboard-init-spec.js A gluster-nagios-dashboard/test/spec/trends-spec.js 13 files changed, 4,525 insertions(+), 63 deletions(-) Approvals: Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25471 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id7c00636e380da154f7f457066fe89475d04c9b4 Gerrit-PatchSet: 33 Gerrit-Project: samples-uiplugins Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.4]: packaging: setup: pep8 1.5.4 fixes
by oourfali＠redhat.com 28 Apr '14

28 Apr '14

Oved Ourfali has submitted this change and it was merged. Change subject: packaging: setup: pep8 1.5.4 fixes ...................................................................... packaging: setup: pep8 1.5.4 fixes Change-Id: I171dddd68243fa7895d25bae654e6bb723e1840e Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/pythonlib/ovirt_engine/service.py M packaging/setup/ovirt_engine_setup/domains.py M packaging/setup/ovirt_engine_setup/util.py M packaging/setup/plugins/ovirt-engine-common/base/core/uninstall.py M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine/config/ca.py M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine/files/simple.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/all-in-one/cpu.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/all-in-one/sshd.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/all-in-one/vdsm.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/iso_domain.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/system/nfs.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/system/sysctl.py 12 files changed, 31 insertions(+), 32 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27149 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I171dddd68243fa7895d25bae654e6bb723e1840e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: packaging: doc: extract AuditLogMessages.properties as doc f...
by oourfali＠redhat.com 28 Apr '14

28 Apr '14

Oved Ourfali has submitted this change and it was merged. Change subject: packaging: doc: extract AuditLogMessages.properties as doc for notifier ...................................................................... packaging: doc: extract AuditLogMessages.properties as doc for notifier Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1084138 Change-Id: Ifb6d56511011e76aebd3b0dc558d8c2f9c94d2d1 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile M ovirt-engine.spec.in M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.conf.in 3 files changed, 11 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, approved mooli tayer: Verified -- To view, visit http://gerrit.ovirt.org/27145 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb6d56511011e76aebd3b0dc558d8c2f9c94d2d1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: spiceProxyOverrideEnabled label overlapped in spanish
by tjelinek＠redhat.com 28 Apr '14

28 Apr '14

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: spiceProxyOverrideEnabled label overlapped in spanish ...................................................................... webadmin: spiceProxyOverrideEnabled label overlapped in spanish Fixed by setting the label big enough so the translated text fits in Change-Id: I1e02bd278e79d2571eb99b7dab70bab1c775bc81 Bug-Url: https://bugzilla.redhat.com/1088868 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified Vojtech Szocs: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26884 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e02bd278e79d2571eb99b7dab70bab1c775bc81 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(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 samples-uiplugins[master]: Gluster Nagios-dashboard plugin.
by vszocs＠redhat.com 28 Apr '14

28 Apr '14

Vojtech Szocs has submitted this change and it was merged. Change subject: Gluster Nagios-dashboard plugin. ...................................................................... Gluster Nagios-dashboard plugin. This plugin provides Nagios monitoring for gluster in oVirt. Dashboard will display the recent alerts in the oVirt and provides a summay for each cluster available in the system. Refere README for details on installation. Change-Id: I122f1b8640d72dbebc0c04b6fa42c98a212dca92 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- A gluster-nagios-dashboard/.gitignore A gluster-nagios-dashboard/Gruntfile.js A gluster-nagios-dashboard/README A gluster-nagios-dashboard/gluster-nagios-dashboard.json A gluster-nagios-dashboard/karma.conf.js A gluster-nagios-dashboard/package.json A gluster-nagios-dashboard/src/css/dashboard.css A gluster-nagios-dashboard/src/dashboard.html A gluster-nagios-dashboard/src/images/down.png A gluster-nagios-dashboard/src/images/up.png A gluster-nagios-dashboard/src/js/dashboard-init.js A gluster-nagios-dashboard/src/js/dashboard.js A gluster-nagios-dashboard/src/js/init.js A gluster-nagios-dashboard/src/js/services.js A gluster-nagios-dashboard/src/lib/angular/js/angular-animate.js A gluster-nagios-dashboard/src/lib/angular/js/angular-animate.min.js A gluster-nagios-dashboard/src/lib/angular/js/angular-animate.min.js.map A gluster-nagios-dashboard/src/lib/angular/js/angular-cookies.js A gluster-nagios-dashboard/src/lib/angular/js/angular-cookies.min.js A gluster-nagios-dashboard/src/lib/angular/js/angular-cookies.min.js.map A gluster-nagios-dashboard/src/lib/angular/js/angular-csp.css A gluster-nagios-dashboard/src/lib/angular/js/angular-loader.js A gluster-nagios-dashboard/src/lib/angular/js/angular-loader.min.js A gluster-nagios-dashboard/src/lib/angular/js/angular-loader.min.js.map A gluster-nagios-dashboard/src/lib/angular/js/angular-mocks.js A gluster-nagios-dashboard/src/lib/angular/js/angular-resource.js A gluster-nagios-dashboard/src/lib/angular/js/angular-resource.min.js A gluster-nagios-dashboard/src/lib/angular/js/angular-resource.min.js.map A gluster-nagios-dashboard/src/lib/angular/js/angular-route.js A gluster-nagios-dashboard/src/lib/angular/js/angular-route.min.js A gluster-nagios-dashboard/src/lib/angular/js/angular-route.min.js.map A gluster-nagios-dashboard/src/lib/angular/js/angular-sanitize.js A gluster-nagios-dashboard/src/lib/angular/js/angular-sanitize.min.js A gluster-nagios-dashboard/src/lib/angular/js/angular-sanitize.min.js.map A gluster-nagios-dashboard/src/lib/angular/js/angular-scenario.js A gluster-nagios-dashboard/src/lib/angular/js/angular-touch.js A gluster-nagios-dashboard/src/lib/angular/js/angular-touch.min.js A gluster-nagios-dashboard/src/lib/angular/js/angular-touch.min.js.map A gluster-nagios-dashboard/src/lib/angular/js/angular.js A gluster-nagios-dashboard/src/lib/angular/js/angular.min.js A gluster-nagios-dashboard/src/lib/angular/js/angular.min.js.map A gluster-nagios-dashboard/src/lib/angular/js/errors.json A gluster-nagios-dashboard/src/lib/angular/version.json A gluster-nagios-dashboard/src/lib/angular/version.txt A gluster-nagios-dashboard/src/lib/bootstrap/css/bootstrap-theme.css A gluster-nagios-dashboard/src/lib/bootstrap/css/bootstrap-theme.css.map A gluster-nagios-dashboard/src/lib/bootstrap/css/bootstrap-theme.min.css A gluster-nagios-dashboard/src/lib/bootstrap/css/bootstrap.css A gluster-nagios-dashboard/src/lib/bootstrap/css/bootstrap.css.map A gluster-nagios-dashboard/src/lib/bootstrap/css/bootstrap.min.css A gluster-nagios-dashboard/src/lib/bootstrap/fonts/glyphicons-halflings-regular.eot A gluster-nagios-dashboard/src/lib/bootstrap/fonts/glyphicons-halflings-regular.svg A gluster-nagios-dashboard/src/lib/bootstrap/fonts/glyphicons-halflings-regular.ttf A gluster-nagios-dashboard/src/lib/bootstrap/fonts/glyphicons-halflings-regular.woff A gluster-nagios-dashboard/src/lib/bootstrap/js/bootstrap.js A gluster-nagios-dashboard/src/lib/bootstrap/js/bootstrap.min.js A gluster-nagios-dashboard/src/lib/bootstrap/version.txt A gluster-nagios-dashboard/src/lib/ui-bootstrap/js/ui-bootstrap-tpls.js A gluster-nagios-dashboard/src/lib/ui-bootstrap/js/ui-bootstrap-tpls.min.js A gluster-nagios-dashboard/src/lib/ui-bootstrap/js/ui-bootstrap.js A gluster-nagios-dashboard/src/lib/ui-bootstrap/js/ui-bootstrap.min.js A gluster-nagios-dashboard/src/lib/ui-bootstrap/version.txt A gluster-nagios-dashboard/src/start.html A gluster-nagios-dashboard/test/lib/angular/js/angular-mocks.js A gluster-nagios-dashboard/test/spec/PlayerSpec.js A gluster-nagios-dashboard/test/spec/SpecHelper.js A gluster-nagios-dashboard/test/spec/dashboard-init-spec.js A gluster-nagios-dashboard/test/spec/dashboard-spec.js A gluster-nagios-dashboard/test/spec/services-spec.js A gluster-nagios-dashboard/test/spec/test-util.js 70 files changed, 79,553 insertions(+), 0 deletions(-) Approvals: Kanagaraj M: Verified Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25064 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I122f1b8640d72dbebc0c04b6fa42c98a212dca92 Gerrit-PatchSet: 22 Gerrit-Project: samples-uiplugins Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin : Pass cluster-id to ui-plugin when Volume is selec...
by vszocs＠redhat.com 28 Apr '14

28 Apr '14

Vojtech Szocs has submitted this change and it was merged. Change subject: webadmin : Pass cluster-id to ui-plugin when Volume is selected from System Tree ...................................................................... webadmin : Pass cluster-id to ui-plugin when Volume is selected from System Tree This patch passes cluster-id to ui-plugin when Volume is selected from System Tree. Change-Id: Ie12faea6a7ae0133be938b68bf1f35e2e658fed9 Signed-off-by: Anmol Babu <anbabu(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/plugin/entity/EntityObject.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: anmolbabu: Verified Kanagaraj M: Looks good to me, but someone else must approve Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26103 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie12faea6a7ae0133be938b68bf1f35e2e658fed9 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0