April 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: engine: fixed checkstyle plugin problem
by gchaplik＠redhat.com 18 Apr '14

18 Apr '14

Gilad Chaplik has submitted this change and it was merged. Change subject: engine: fixed checkstyle plugin problem ...................................................................... engine: fixed checkstyle plugin problem checkstyle plugin on windows path will report no-argument constructor problem on aaa module Change-Id: Iacb017c2ad0209e74c7f55cbfba2f3220dda711c Signed-off-by: Jason Liao <chuan.liao(a)hp.com> --- M build-tools-root/checkstyles/src/main/resources/checkstyle-suppressions.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26875 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iacb017c2ad0209e74c7f55cbfba2f3220dda711c Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jason Liao <chuan.liao(a)hp.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(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]: aaa: Changing MaxLDAPQueryPartsNumber to long
by yzaslavs＠redhat.com 18 Apr '14

18 Apr '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: aaa: Changing MaxLDAPQueryPartsNumber to long ...................................................................... aaa: Changing MaxLDAPQueryPartsNumber to long Changed MaxLDAPQueryPartsNumber to be long to conform to extensions-api. In addition added support to Config for Long Change-Id: I5d001fdb816b4ca7f690ac2578159a75c9f3d058 topic: AAA Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/generic/DBConfigUtils.java 2 files changed, 3 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26888 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5d001fdb816b4ca7f690ac2578159a75c9f3d058 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-node-plugin-vdsm[node-3.0]: spec: remove uneeded sed's
by dougsland＠redhat.com 17 Apr '14

17 Apr '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: spec: remove uneeded sed's ...................................................................... spec: remove uneeded sed's We have 3 sed command in spec file that are deprecated: 1. > # semanage is not present in the image and virt_use_nfs is on (see rhbz#642209) > # remove it from vdsmd startup script to avoid error > sed -i 's#/usr/sbin/semanage#/bin/true#' /etc/rc.d/init.d/vdsmd We don't have semanage command anymore in /etc/rc.d/init.d/vdsmd and ovirt-node these days provide semanage command. 2. > # libvirtd upstart job is already configured on rhevh > sed -i 's/ && start_libvirtd$//' /etc/rc.d/init.d/vdsmd There is no start_libvirtd anymore in /etc/rc.d/init.d/vdsmd 3. > # chkconfig results (symlinks) cannnot be peristed > sed -i 's#/sbin/chkconfig \$srv off##' /etc/rc.d/init.d/vdsmd No more /sbin/chkconfig $srv off in /etc/rc.d/init.d/vdsmd Change-Id: I98ecf410204f081b5cea730a87b2aca8786e3a87 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1088218 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-node-plugin-vdsm.spec.in 1 file changed, 0 insertions(+), 10 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26899 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I98ecf410204f081b5cea730a87b2aca8786e3a87 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: node-3.0 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: Joey Boggs <jboggs(a)redhat.com> Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in otopi[master]: core: typo in comment
by Alon Bar-Lev 17 Apr '14

17 Apr '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: typo in comment ...................................................................... core: typo in comment Change-Id: I2668dd18a1f689d8a1f20647137d63f159dc7837 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M src/otopi/plugin.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26287 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2668dd18a1f689d8a1f20647137d63f159dc7837 Gerrit-PatchSet: 2 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: spec: remove uneeded sed's
by dougsland＠redhat.com 17 Apr '14

17 Apr '14

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: spec: remove uneeded sed's ...................................................................... spec: remove uneeded sed's We have 3 sed command in spec file that are deprecated: 1. > # semanage is not present in the image and virt_use_nfs is on (see rhbz#642209) > # remove it from vdsmd startup script to avoid error > sed -i 's#/usr/sbin/semanage#/bin/true#' /etc/rc.d/init.d/vdsmd We don't have semanage command anymore in /etc/rc.d/init.d/vdsmd and ovirt-node these days provide semanage command. 2. > # libvirtd upstart job is already configured on rhevh > sed -i 's/ && start_libvirtd$//' /etc/rc.d/init.d/vdsmd There is no start_libvirtd anymore in /etc/rc.d/init.d/vdsmd 3. > # chkconfig results (symlinks) cannnot be peristed > sed -i 's#/sbin/chkconfig \$srv off##' /etc/rc.d/init.d/vdsmd No more /sbin/chkconfig $srv off in /etc/rc.d/init.d/vdsmd Change-Id: I98ecf410204f081b5cea730a87b2aca8786e3a87 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1088218 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M ovirt-node-plugin-vdsm.spec.in 1 file changed, 0 insertions(+), 10 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26865 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I98ecf410204f081b5cea730a87b2aca8786e3a87 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)fedoraproject.org> Gerrit-Reviewer: Joey Boggs <jboggs(a)redhat.com> Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in samples-uiplugins[master]: AngularJS Demo Plugin
by vszocs＠redhat.com 17 Apr '14

17 Apr '14

Vojtech Szocs has submitted this change and it was merged. Change subject: AngularJS Demo Plugin ...................................................................... AngularJS Demo Plugin This plugin does the following - Add Test main tab - when selected, Test main tab sends message to the plugin - plugin validates the message and invokes function on Test main tab - Test main tab's function updates AngularJS scope which in turn updates tab UI This plugin uses AngularJS Javascript framework * AngularJS, Model-View-Controller framework for JavaScript http://angularjs.org/ Change-Id: Ibd6965c7317378d310682e5efb3eb974ec04cd00 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M .gitignore A angular-demo-plugin/README A angular-demo-plugin/src/js/common.js A angular-demo-plugin/src/js/plugin.js A angular-demo-plugin/src/js/tab.js A angular-demo-plugin/src/lib/angular/js/angular-animate.js A angular-demo-plugin/src/lib/angular/js/angular-animate.min.js A angular-demo-plugin/src/lib/angular/js/angular-animate.min.js.map A angular-demo-plugin/src/lib/angular/js/angular-cookies.js A angular-demo-plugin/src/lib/angular/js/angular-cookies.min.js A angular-demo-plugin/src/lib/angular/js/angular-cookies.min.js.map A angular-demo-plugin/src/lib/angular/js/angular-loader.js A angular-demo-plugin/src/lib/angular/js/angular-loader.min.js A angular-demo-plugin/src/lib/angular/js/angular-loader.min.js.map A angular-demo-plugin/src/lib/angular/js/angular-resource.js A angular-demo-plugin/src/lib/angular/js/angular-resource.min.js A angular-demo-plugin/src/lib/angular/js/angular-resource.min.js.map A angular-demo-plugin/src/lib/angular/js/angular-route.js A angular-demo-plugin/src/lib/angular/js/angular-route.min.js A angular-demo-plugin/src/lib/angular/js/angular-route.min.js.map A angular-demo-plugin/src/lib/angular/js/angular-sanitize.js A angular-demo-plugin/src/lib/angular/js/angular-sanitize.min.js A angular-demo-plugin/src/lib/angular/js/angular-sanitize.min.js.map A angular-demo-plugin/src/lib/angular/js/angular-touch.js A angular-demo-plugin/src/lib/angular/js/angular-touch.min.js A angular-demo-plugin/src/lib/angular/js/angular-touch.min.js.map A angular-demo-plugin/src/lib/angular/js/angular.js A angular-demo-plugin/src/lib/angular/js/angular.min.js A angular-demo-plugin/src/lib/angular/js/angular.min.js.map A angular-demo-plugin/src/lib/angular/version.txt A angular-demo-plugin/src/plugin.html A angular-demo-plugin/src/tab.html A angular-demo-plugin/test.json 33 files changed, 25,778 insertions(+), 0 deletions(-) Approvals: Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26129 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibd6965c7317378d310682e5efb3eb974ec04cd00 Gerrit-PatchSet: 2 Gerrit-Project: samples-uiplugins Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in samples-uiplugins[master]: Moved .gitignore to root directory
by vszocs＠redhat.com 17 Apr '14

17 Apr '14

Vojtech Szocs has submitted this change and it was merged. Change subject: Moved .gitignore to root directory ...................................................................... Moved .gitignore to root directory Change-Id: I828e5305e5321ffbba858e3456629be7571f55db Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- A .gitignore D space-shooter-plugin/.gitignore 2 files changed, 5 insertions(+), 5 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Vojtech Szocs: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25522 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I828e5305e5321ffbba858e3456629be7571f55db Gerrit-PatchSet: 3 Gerrit-Project: samples-uiplugins Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[master]: frontend: Fix incorrect text wrap
by tjelinek＠redhat.com 17 Apr '14

17 Apr '14

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: Fix incorrect text wrap ...................................................................... frontend: Fix incorrect text wrap In Japanese, caption for remaping control-alt-del is broken into two lines. Change-Id: I0c62c286d2c85d43da5aec89df01f3d43e63171d Bug-Url: https://bugzilla.redhat.com/1080322 Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/ConsolePopupView.ui.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Looks good to me, approved Frank Kobzik: Verified -- To view, visit http://gerrit.ovirt.org/26851 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c62c286d2c85d43da5aec89df01f3d43e63171d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(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 ovirt-engine[master]: Fixing a typo
by sbonazzo＠redhat.com 17 Apr '14

17 Apr '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: Fixing a typo ...................................................................... Fixing a typo Auhtorization -> Authorization Change-Id: I111117a4d751059429bfffcb7876888c4d2637ea Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M backend/manager/modules/aaa/pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/26854 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I111117a4d751059429bfffcb7876888c4d2637ea Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-setup[master]: packaging: setup: bridge: support tagged VLANs and bond
by sbonazzo＠redhat.com 17 Apr '14

17 Apr '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: bridge: support tagged VLANs and bond ...................................................................... packaging: setup: bridge: support tagged VLANs and bond Added support for tagged VLANs Added support for bond interfaces Change-Id: I0abd18b3ae36265406afb80a1ee082d2d14d4a92 Bug-Url: https://bugzilla.redhat.com/1072027 Bug-Url: https://bugzilla.redhat.com/1076944 Bug-Url: https://bugzilla.redhat.com/1078206 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/network/bridge.py 1 file changed, 46 insertions(+), 15 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/25894 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0abd18b3ae36265406afb80a1ee082d2d14d4a92 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Leonid Natapov <leonidnatapov(a)gmail.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Meital Bourvine <mbourvin(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