- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: foreman integration - showing foreman hosts in new host dial...
by oourfali＠redhat.com 27 May '13

27 May '13

Oved Ourfali has submitted this change and it was merged. Change subject: foreman integration - showing foreman hosts in new host dialog ...................................................................... foreman integration - showing foreman hosts in new host dialog This patch adds external host provider interface, with an implementation for Foreman, to allow viewing foreman hosts from the "add new host" dialog. Limitations (will be addressed later): 1. Will only work with non-SSL foreman servers Change-Id: I30ea180e477a8f0714c4dc97ec55f29be515723a Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M backend/manager/modules/bll/pom.xml A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetHostListFromExternalProviderQuery.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/HostProviderProxy.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/ForemanHost.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/ForemanHostProviderProxy.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/ForemanHostWrapper.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/ProviderProxyFactory.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetHostListFromExternalProviderQueryTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/ProviderType.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetHostListFromExternalProviderParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterGuideModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/EditHostModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/NewHostModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/popup/host/HostPopupPresenterWidget.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostPopupView.ui.xml 22 files changed, 1,056 insertions(+), 129 deletions(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14582 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I30ea180e477a8f0714c4dc97ec55f29be515723a Gerrit-PatchSet: 16 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi : Cookies are being set even when not using persiste...
by mpastern＠redhat.com 27 May '13

27 May '13

Michael Pasternak has submitted this change and it was merged. Change subject: restapi : Cookies are being set even when not using persistent authentication ...................................................................... restapi : Cookies are being set even when not using persistent authentication API sets jsession id when not using persistent authentication. This patch creates a session only when persistent authentication header is passed in the request. Change-Id: Ieb004fefbd9a276efacb2ce6fa8b7a48a9516c6e Bug-Url: https://bugzilla.redhat.com/962730 Signed-off-by: Ravi Nori <rnori(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, 1 insertion(+), 1 deletion(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14891 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb004fefbd9a276efacb2ce6fa8b7a48a9516c6e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin, root: Prevent JSESSIONID cookie for root path
by ecohen＠redhat.com 27 May '13

27 May '13

Einav Cohen has submitted this change and it was merged. Change subject: webadmin,root: Prevent JSESSIONID cookie for root path ...................................................................... webadmin,root: Prevent JSESSIONID cookie for root path Engine "root" web application containing various utility servlets sets JSESSIONID cookie for / (root path) upon following occasions: a. when requesting document file for a missing language, DocsServlet ensures HttpSession -> JSESSIONID cookie in order to read/store "langPageShown" attribute: "Show missing language page only for the first time" b. when processing any JSP page, since each JSP gets HttpSession -> JSESSIONID cookie created eagerly: - ovirt-engine.jsp (splash page) - no_lang.jsp (missing language page) However, setting cookie X for path=/ essentially shadows any other cookie(s) with same name (X) set for path=/foo Consider following example: 1. User visits WebAdmin: - new cookie JSESSIONID for path=/webadmin - WebAdmin JavaScript reads JSESSIONID cookie, correct value is returned 2. User visits "root" web application: - new cookie JSESSIONID for path=/ - WebAdmin JavaScript reads JSESSIONID cookie, wrong value is returned because there are two JSESSIONID cookies and path=/ takes precedence This patch prevents "root" web application from using JSESSIONID cookie (see occasions a. and b. above), which prevents cookie shadowing for WebAdmin. In future, we should consider using different "session" cookie names for different web applications in order to avoid such problems. Change-Id: I3b4c95f0a716bf3cc05d102a1026b3c6aee5879c Bug-Url: https://bugzilla.redhat.com/966525 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M backend/manager/modules/root/src/main/java/org/ovirt/engine/core/DocsServlet.java M backend/manager/modules/root/src/main/webapp/WEB-INF/help/no_lang.jsp M backend/manager/modules/root/src/main/webapp/WEB-INF/ovirt-engine.jsp 3 files changed, 26 insertions(+), 5 deletions(-) Approvals: Einav Cohen: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15057 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3b4c95f0a716bf3cc05d102a1026b3c6aee5879c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(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>

1 0

Change in ovirt-host-deploy[master]: core: split out hardware check as it is shared with ovirt-en...
by Alon Bar-Lev 26 May '13

26 May '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: core: split out hardware check as it is shared with ovirt-engine-setup ...................................................................... core: split out hardware check as it is shared with ovirt-engine-setup Change-Id: I0e9aa168b6f30e2bc2d4dbc3439562e9402dca59 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M po/POTFILES.in M src/ovirt_host_deploy/Makefile.am A src/ovirt_host_deploy/hardware.py M src/plugins/ovirt-host-deploy/vdsm/hardware.py 4 files changed, 222 insertions(+), 154 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15073 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0e9aa168b6f30e2bc2d4dbc3439562e9402dca59 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>

1 0

Change in ovirt-engine[master]: Remove VDS super.clone method
by shavivi＠redhat.com 26 May '13

26 May '13

Shahar Havivi has submitted this change and it was merged. Change subject: Remove VDS super.clone method ...................................................................... Remove VDS super.clone method Not supported by GWT (the actual type that is not supported is the exception: java.lang.CloneNotSupportedException) Change-Id: I3673da05f1cc9c1de834cda4f9ef5865a058d6c8 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java 1 file changed, 0 insertions(+), 5 deletions(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3673da05f1cc9c1de834cda4f9ef5865a058d6c8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0

Change in ovirt-reports[master]: reports: fixed br9 query syntax error (#967276)
by ydary＠redhat.com 26 May '13

26 May '13

Yaniv Dary has submitted this change and it was merged. Change subject: reports: fixed br9 query syntax error (#967276) ...................................................................... reports: fixed br9 query syntax error (#967276) Bug-Url: https://bugzilla.redhat.com/967276 Change-Id: Iae4e8519323637e00c9dc348bedac8214197136d Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M reports/repository_files/resources/Reports/Inventory/VM_Inventory_files/full details.jrxml.data 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15070 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae4e8519323637e00c9dc348bedac8214197136d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: align path field in local storage dialog
by tnisan＠redhat.com 26 May '13

26 May '13

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: align path field in local storage dialog ...................................................................... webadmin: align path field in local storage dialog Align path field in local storage popup dialog so it will be aligned with rest of the fields in dialog. Change-Id: Iaf55a833fb0bb0084ec2548a90b094ef40daf4b9 Signed-off-by: Alissa Bonas <abonas(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/LocalStorageView.ui.xml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Tal Nisan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14882 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaf55a833fb0bb0084ec2548a90b094ef40daf4b9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: fix failure of cleaning database
by oschreib＠redhat.com 26 May '13

26 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: fix failure of cleaning database ...................................................................... packaging: setup: fix failure of cleaning database Change-Id: I698eead168e25019f57feb0b9493bc87aa38ce33 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-remove/db/clear.py 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15063 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I698eead168e25019f57feb0b9493bc87aa38ce33 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>

1 0

Change in ovirt-engine[master]: packaging: setup: separate pki config and keys groups
by oschreib＠redhat.com 26 May '13

26 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: separate pki config and keys groups ...................................................................... packaging: setup: separate pki config and keys groups using separate pki config and keys for uninstall groups Change-Id: Ied09af0cb1ace07101629273746ed3eb820a78ee Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/config/ca.py M packaging/setup/plugins/ovirt-engine-setup/pki/ca.py 2 files changed, 13 insertions(+), 14 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15014 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ied09af0cb1ace07101629273746ed3eb820a78ee Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(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[master]: packaging: setup: fixed uninstall UX
by oschreib＠redhat.com 26 May '13

26 May '13

Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: setup: fixed uninstall UX ...................................................................... packaging: setup: fixed uninstall UX - avoid to create empty section in uninstall file - avoid to ask more than one time about uninstalling a group Change-Id: I95dddc2205a7e16dc4f4848fb6fdeab7b1e2ec2b Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-remove/files/simple.py M packaging/setup/plugins/ovirt-engine-setup/core/uninstall.py 2 files changed, 65 insertions(+), 31 deletions(-) Approvals: Ofer Schreiber: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15016 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I95dddc2205a7e16dc4f4848fb6fdeab7b1e2ec2b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(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