- Engine-commits - oVirt List Archives

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

Change in ovirt-engine[master]: engine: Add header to each response.
by awels＠redhat.com 16 Apr '14

16 Apr '14

Alexander Wels has submitted this change and it was merged. Change subject: engine: Add header to each response. ...................................................................... engine: Add header to each response. - This patch adds headers to each response with the option of defining extra headers in the web.xml. There are 3 headers added by default. - X-FRAME-OPTIONS: SAMEORIGIN (To stop click jacking) - X-CONTENT-TYPE-OPTIONS: NOSNIFF (To stop mime type attacks) - X-XSS-PROTECTION: 1; MODE=BLOCK (To enable IE XSS filter) Change-Id: I4e9c26dddd0d0300d6b3d078ae763b78600f25d1 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M backend/manager/modules/docs/src/main/webapp/WEB-INF/web.xml M backend/manager/modules/root/src/main/webapp/WEB-INF/web.xml A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/servlet/HeaderFilter.java A backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/servlet/HeaderFilterTest.java M backend/manager/modules/welcome/src/main/webapp/WEB-INF/web.xml M frontend/webadmin/modules/frontend/src/main/resources/META-INF/web-fragment.xml M frontend/webadmin/modules/userportal-gwtp/src/main/webapp/WEB-INF/web.xml M frontend/webadmin/modules/webadmin/src/main/webapp/WEB-INF/web.xml 8 files changed, 258 insertions(+), 0 deletions(-) Approvals: Alexander Wels: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25937 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4e9c26dddd0d0300d6b3d078ae763b78600f25d1 Gerrit-PatchSet: 3 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: Alon Bar-Lev <alonbl(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]: restapi: Only one method to set mapping locator
by juan.hernandez＠redhat.com 16 Apr '14

16 Apr '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Only one method to set mapping locator ...................................................................... restapi: Only one method to set mapping locator The method that sets the mapping loator has been recently moved to the backend base resource class, so it is no longer necessary to have it in other resources and to explicitly check the type of the resource when calling it. Change-Id: I28595e36feb41bf2915e2fb0b28f78d7b3b65f74 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/BackendApplication.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResource.java 2 files changed, 1 insertion(+), 14 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26428 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I28595e36feb41bf2915e2fb0b28f78d7b3b65f74 Gerrit-PatchSet: 2 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0