November 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.5]: jsonrpc: version bump
by piotr.kliczewski＠gmail.com 21 Nov '14

21 Nov '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: jsonrpc: version bump ...................................................................... jsonrpc: version bump New version of jsonrpc with jackson parsing flags. Change-Id: I2a3fdf2835c49c3555796e9d4f26b3cc2aec96bc Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> Bug-Url: https://bugzilla.redhat.com/1155146 --- M ovirt-engine.spec.in M pom.xml 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Piotr Kliczewski: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35418 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2a3fdf2835c49c3555796e9d4f26b3cc2aec96bc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: userportal, webadmin: Multi view table columns inconsistent
by awels＠redhat.com 21 Nov '14

21 Nov '14

Alexander Wels has submitted this change and it was merged. Change subject: userportal,webadmin: Multi view table columns inconsistent ...................................................................... userportal,webadmin: Multi view table columns inconsistent - since a463914769e87b5fb55999bcd25d56167b8e3d16 columns available in all view would not get removed and would always end up at the front of the grid even if other columns that can disappear were in front of them. This patch fixes that problem. Change-Id: I143894af611f8364da943433e81378690adcb0e8 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1157215 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/resize/ColumnResizeCellTable.java 1 file changed, 21 insertions(+), 37 deletions(-) Approvals: Alexander Wels: Verified Daniel Erez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35146 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I143894af611f8364da943433e81378690adcb0e8 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: webadmin: refresh button wrapping
by awels＠redhat.com 21 Nov '14

21 Nov '14

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: refresh button wrapping ...................................................................... webadmin: refresh button wrapping - Fixed refresh button wrapping when there is not enough room to fit all the buttons. Change-Id: I1fc13aea7950ab3f4965131dcc2b32e4ea837192 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1163929 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M packaging/branding/ovirt.brand/ovirt-patternfly-compat.css 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alexander Wels: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35171 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1fc13aea7950ab3f4965131dcc2b32e4ea837192 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(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 ovirt-engine[master]: webadmin: Fix exception on VM status column
by awels＠redhat.com 21 Nov '14

21 Nov '14

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Fix exception on VM status column ...................................................................... webadmin: Fix exception on VM status column - When hovering over or clicking on the status column in the VM main tab the console would display an exception. This patch solves that. This is the right status column Change-Id: I1d5cfaa291fe26b1a7439a8a71e8e04d8dc539b0 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/column/StatusCompositeCellWithElementId.java 1 file changed, 13 insertions(+), 2 deletions(-) Approvals: Alexander Wels: Verified Greg Sheremeta: Looks good to me, but someone else must approve Vojtech Szocs: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35180 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1d5cfaa291fe26b1a7439a8a71e8e04d8dc539b0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(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 ovirt-host-deploy[master]: gluster: start glusterpmd service after reboot.
by Alon Bar-Lev 21 Nov '14

21 Nov '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: gluster: start glusterpmd service after reboot. ...................................................................... gluster: start glusterpmd service after reboot. Added start of glusterpmd service after reboot. Change-Id: I6b4d631b23250b513b42ff0509e8d230b1cb3d9b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1138108 Signed-off-by: ndarshan <dnarayan(a)redhat.com> --- M ChangeLog M src/plugins/ovirt-host-deploy/gluster/glusterpmd.py 2 files changed, 2 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, approved Darshan N: Verified -- To view, visit http://gerrit.ovirt.org/35415 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b4d631b23250b513b42ff0509e8d230b1cb3d9b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Darshan N <dnarayan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: mla: deny access to specific ID for users in user level API
by piotr.kliczewski＠gmail.com 21 Nov '14

21 Nov '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: mla: deny access to specific ID for users in user level API ...................................................................... mla: deny access to specific ID for users in user level API User who don't have manipulate_permission action group should not see users in system, even he shouldn't access them directly by their ID's. Change-Id: I90ec94fd0194680548e159f5d9bc010f5c233b91 Bug-Url: https://bugzilla.redhat.com/1160443 Signed-off-by: Ondra Machacek <machacek.ondra(a)gmail.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/GetDbUserByUserIdQuery.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DbUserDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DbUserDAODbFacadeImpl.java M packaging/dbscripts/user_sp.sql 4 files changed, 21 insertions(+), 4 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Ondřej Macháček: Verified Eli Mesika: Looks good to me, but someone else must approve Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/35301 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I90ec94fd0194680548e159f5d9bc010f5c233b91 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Ondřej Macháček <machacek.ondra(a)gmail.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: add custom properties to templates
by juan.hernandez＠redhat.com 21 Nov '14

21 Nov '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: add custom properties to templates ...................................................................... restapi: add custom properties to templates currently custom properties are vm only feature, so adding also to templates. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1011735 Change-Id: I95261d17138a8a19d52b16f7f84322c051d7bf17 Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmBaseMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 3 files changed, 12 insertions(+), 9 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Omer Frenkel: Verified -- To view, visit http://gerrit.ovirt.org/35118 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I95261d17138a8a19d52b16f7f84322c051d7bf17 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: frontend: disable edit of vm-pool custom properties
by ofrenkel＠redhat.com 21 Nov '14

21 Nov '14

Omer Frenkel has submitted this change and it was merged. Change subject: frontend: disable edit of vm-pool custom properties ...................................................................... frontend: disable edit of vm-pool custom properties Change-Id: I291cb3ec893780ba1fceefb37c5a678699398e1a Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ExistingPoolModelBehavior.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Omer Frenkel: Verified -- To view, visit http://gerrit.ovirt.org/35393 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I291cb3ec893780ba1fceefb37c5a678699398e1a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in jenkins[master]: jobs: add mom build
by dcaroest＠redhat.com 20 Nov '14

20 Nov '14

David Caro has submitted this change and it was merged. Change subject: jobs: add mom build ...................................................................... jobs: add mom build Change-Id: I72a466a119083fcce55d45ce04c4ebae820b4df3 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A jobs/confs/yaml/jobs/mom/mom_create-rpms.yaml A jobs/confs/yaml/scms/mom.yaml 2 files changed, 42 insertions(+), 0 deletions(-) Approvals: Adam Litke: Looks good to me, but someone else must approve David Caro: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33400 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I72a466a119083fcce55d45ce04c4ebae820b4df3 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(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: oVirt Jenkins CI Server

1 0

Change in samples-uiplugins[master]: gluster-nagios-monitoring: remove gluster-nagios-monitoring ...
by vszocs＠redhat.com 20 Nov '14

20 Nov '14

Vojtech Szocs has submitted this change and it was merged. Change subject: gluster-nagios-monitoring: remove gluster-nagios-monitoring plugin ...................................................................... gluster-nagios-monitoring: remove gluster-nagios-monitoring plugin removing gluster-nagios-monitoring ui plugin is as it is moved to an individual repo at ssh://gerrit.ovirt.org:29418/gluster-nagios-monitoring.git Change-Id: I0ba048385f060d9df4a280fce05a77bde322a132 Signed-off-by: Ramesh Nachimuthu <rnachimu(a)redhat.com> --- D gluster-nagios-monitoring/.gitignore D gluster-nagios-monitoring/Gruntfile.js D gluster-nagios-monitoring/Makefile D gluster-nagios-monitoring/README D gluster-nagios-monitoring/gluster-nagios-monitoring.json D gluster-nagios-monitoring/gluster-nagios-monitoring.spec.in D gluster-nagios-monitoring/karma.conf.js D gluster-nagios-monitoring/package.json D gluster-nagios-monitoring/src/css/dashboard.css D gluster-nagios-monitoring/src/dashboard.html D gluster-nagios-monitoring/src/images/down.png D gluster-nagios-monitoring/src/images/print.png D gluster-nagios-monitoring/src/images/refresh.png D gluster-nagios-monitoring/src/images/up.png D gluster-nagios-monitoring/src/js/common.js D gluster-nagios-monitoring/src/js/dashboard-init.js D gluster-nagios-monitoring/src/js/dashboard.js D gluster-nagios-monitoring/src/js/services.js D gluster-nagios-monitoring/src/js/trends.js D gluster-nagios-monitoring/src/lib/angular/js/angular-animate.js D gluster-nagios-monitoring/src/lib/angular/js/angular-animate.min.js D gluster-nagios-monitoring/src/lib/angular/js/angular-animate.min.js.map D gluster-nagios-monitoring/src/lib/angular/js/angular-cookies.js D gluster-nagios-monitoring/src/lib/angular/js/angular-cookies.min.js D gluster-nagios-monitoring/src/lib/angular/js/angular-cookies.min.js.map D gluster-nagios-monitoring/src/lib/angular/js/angular-csp.css D gluster-nagios-monitoring/src/lib/angular/js/angular-loader.js D gluster-nagios-monitoring/src/lib/angular/js/angular-loader.min.js D gluster-nagios-monitoring/src/lib/angular/js/angular-loader.min.js.map D gluster-nagios-monitoring/src/lib/angular/js/angular-mocks.js D gluster-nagios-monitoring/src/lib/angular/js/angular-resource.js D gluster-nagios-monitoring/src/lib/angular/js/angular-resource.min.js D gluster-nagios-monitoring/src/lib/angular/js/angular-resource.min.js.map D gluster-nagios-monitoring/src/lib/angular/js/angular-route.js D gluster-nagios-monitoring/src/lib/angular/js/angular-route.min.js D gluster-nagios-monitoring/src/lib/angular/js/angular-route.min.js.map D gluster-nagios-monitoring/src/lib/angular/js/angular-sanitize.js D gluster-nagios-monitoring/src/lib/angular/js/angular-sanitize.min.js D gluster-nagios-monitoring/src/lib/angular/js/angular-sanitize.min.js.map D gluster-nagios-monitoring/src/lib/angular/js/angular-scenario.js D gluster-nagios-monitoring/src/lib/angular/js/angular-touch.js D gluster-nagios-monitoring/src/lib/angular/js/angular-touch.min.js D gluster-nagios-monitoring/src/lib/angular/js/angular-touch.min.js.map D gluster-nagios-monitoring/src/lib/angular/js/angular.js D gluster-nagios-monitoring/src/lib/angular/js/angular.min.js D gluster-nagios-monitoring/src/lib/angular/js/angular.min.js.map D gluster-nagios-monitoring/src/lib/angular/js/errors.json D gluster-nagios-monitoring/src/lib/angular/version.json D gluster-nagios-monitoring/src/lib/angular/version.txt D gluster-nagios-monitoring/src/lib/bootstrap/css/bootstrap-theme.css D gluster-nagios-monitoring/src/lib/bootstrap/css/bootstrap-theme.css.map D gluster-nagios-monitoring/src/lib/bootstrap/css/bootstrap-theme.min.css D gluster-nagios-monitoring/src/lib/bootstrap/css/bootstrap.css D gluster-nagios-monitoring/src/lib/bootstrap/css/bootstrap.css.map D gluster-nagios-monitoring/src/lib/bootstrap/css/bootstrap.min.css D gluster-nagios-monitoring/src/lib/bootstrap/fonts/glyphicons-halflings-regular.eot D gluster-nagios-monitoring/src/lib/bootstrap/fonts/glyphicons-halflings-regular.svg D gluster-nagios-monitoring/src/lib/bootstrap/fonts/glyphicons-halflings-regular.ttf D gluster-nagios-monitoring/src/lib/bootstrap/fonts/glyphicons-halflings-regular.woff D gluster-nagios-monitoring/src/lib/bootstrap/js/bootstrap.js D gluster-nagios-monitoring/src/lib/bootstrap/js/bootstrap.min.js D gluster-nagios-monitoring/src/lib/bootstrap/version.txt D gluster-nagios-monitoring/src/lib/ui-bootstrap-tpls-0.10.0.js D gluster-nagios-monitoring/src/lib/ui-bootstrap/js/ui-bootstrap-tpls.js D gluster-nagios-monitoring/src/lib/ui-bootstrap/js/ui-bootstrap-tpls.min.js D gluster-nagios-monitoring/src/lib/ui-bootstrap/js/ui-bootstrap.js D gluster-nagios-monitoring/src/lib/ui-bootstrap/js/ui-bootstrap.min.js D gluster-nagios-monitoring/src/lib/ui-bootstrap/version.txt D gluster-nagios-monitoring/src/start.html D gluster-nagios-monitoring/src/trendsTab.html D gluster-nagios-monitoring/test/lib/angular/js/angular-mocks.js D gluster-nagios-monitoring/test/spec/PlayerSpec.js D gluster-nagios-monitoring/test/spec/SpecHelper.js D gluster-nagios-monitoring/test/spec/dashboard-init-spec.js D gluster-nagios-monitoring/test/spec/dashboard-spec.js D gluster-nagios-monitoring/test/spec/services-spec.js D gluster-nagios-monitoring/test/spec/test-util.js D gluster-nagios-monitoring/test/spec/trends-spec.js D gluster-nagios-monitoring/version.mak 79 files changed, 0 insertions(+), 84,289 deletions(-) Approvals: Kanagaraj M: Looks good to me, but someone else must approve Vojtech Szocs: Looks good to me, approved Ramesh N: Verified -- To view, visit http://gerrit.ovirt.org/35101 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0ba048385f060d9df4a280fce05a77bde322a132 Gerrit-PatchSet: 1 Gerrit-Project: samples-uiplugins Gerrit-Branch: master Gerrit-Owner: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Ramesh N <rnachimu(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: anmolbabu <anbabu(a)redhat.com>

1 0