April 2013 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: make validateFQDN case insensitive
by juan.hernandez＠redhat.com 04 Apr '13

04 Apr '13

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: make validateFQDN case insensitive ...................................................................... packaging: make validateFQDN case insensitive If the specified FQDN to rhevm-setup is a different case to that returned by the DNS PTR record, then validateFQDN reports that the reverse-resolved name doesn't match the supplied name and fails the validation and hence rhevm-setup fails. This patch makes validateFQDN case insensitive so it doesn't matter if the supplied FQDN differs in case to that retured by the DNS PTR record, because DNS itself is case insensitive. Change-Id: I7619971956c1eba8b6d3f2a345b8fc85a778384e Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=928165 Signed-off-by: Mark Huth <mhuth(a)redhat.com> --- M packaging/fedora/setup/engine_validators.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13539 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7619971956c1eba8b6d3f2a345b8fc85a778384e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mark Huth <mhuth(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Mark Huth <mhuth(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]: pki: cleanup: Found reliance on default encoding
by yzaslavs＠redhat.com 04 Apr '13

04 Apr '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: pki: cleanup: Found reliance on default encoding ...................................................................... pki: cleanup: Found reliance on default encoding Change-Id: Id561cc7e04301ef92000f66301a7cb81713cdde1 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/crypt/EncryptionUtils.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/hostinstall/OpenSslCAWrapper.java 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13512 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id561cc7e04301ef92000f66301a7cb81713cdde1 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: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: remove hide popup feature.
by ecohen＠redhat.com 03 Apr '13

03 Apr '13

Einav Cohen has submitted this change and it was merged. Change subject: webadmin: remove hide popup feature. ...................................................................... webadmin: remove hide popup feature. - Removed feature to hide error popups when logging out as they are no longer needed Change-Id: I2fc7ee921159aafc7e9c85cd2093b7882fcfe3b4 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/system/ErrorPopupManager.java 1 file changed, 3 insertions(+), 12 deletions(-) Approvals: Einav Cohen: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12023 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2fc7ee921159aafc7e9c85cd2093b7882fcfe3b4 Gerrit-PatchSet: 5 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: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: on handing over a VM mark it as MigratingTo
by rgolan＠redhat.com 03 Apr '13

03 Apr '13

Roy Golan has submitted this change and it was merged. Change subject: core: on handing over a VM mark it as MigratingTo ...................................................................... core: on handing over a VM mark it as MigratingTo This patch aims to reflect the VM handover more clear so instead of moving a VM to Unknown it will move to MigratingTo (unless the host is *already* NonResponging) so tracking a VM status transition wouldn't puzzle users (at least would be less confusing and more script friendly as the script won't discover a VM is suddenly Uknown) Non responding treatment has been changed to collect also VMs that are migrating to the host (i.e status MigratingTo and run_on_vds == nonRespongindVds) Change-Id: I0cc072b445ef2616e209fd9dfb30f65abc0ad3da Signed-off-by: Roy Golan <rgolan(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/909932 --- M backend/manager/dbscripts/vms_sp.sql M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAODbFacadeImpl.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 5 files changed, 55 insertions(+), 10 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13560 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0cc072b445ef2616e209fd9dfb30f65abc0ad3da Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>

1 0

Change in ovirt-engine-sdk[sdk_3.2]: sdk: user should not see admin permissions the DC #924357
by mpastern＠redhat.com 03 Apr '13

03 Apr '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: user should not see admin permissions the DC #924357 ...................................................................... sdk: user should not see admin permissions the DC #924357 https://bugzilla.redhat.com/show_bug.cgi?id=924357 Change-Id: I85819a56aecbd5c4289b63af7fa3b93da91a8991 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtsdk/infrastructure/proxy.py 1 file changed, 9 insertions(+), 5 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/13563 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I85819a56aecbd5c4289b63af7fa3b93da91a8991 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.2 Gerrit-Owner: Michael Pasternak <mpastern(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-sdk[master]: sdk: user should not see admin permissions the DC #924357
by mpastern＠redhat.com 03 Apr '13

03 Apr '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: user should not see admin permissions the DC #924357 ...................................................................... sdk: user should not see admin permissions the DC #924357 Change-Id: I85819a56aecbd5c4289b63af7fa3b93da91a8991 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtsdk/infrastructure/proxy.py 1 file changed, 9 insertions(+), 5 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/13562 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I85819a56aecbd5c4289b63af7fa3b93da91a8991 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(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]: history: fixed storage domain status collection (#922499)
by yzaslavs＠redhat.com 03 Apr '13

03 Apr '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: history: fixed storage domain status collection (#922499) ...................................................................... history: fixed storage domain status collection (#922499) Had mistake in cross database status function, this patch corrects this. Bug-Url: https://bugzilla.redhat.com/922499 Change-Id: Iba80c84946d1566208f8ac1ac133f842696dd7e6 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M backend/manager/dbscripts/create_dwh_views.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13358 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iba80c84946d1566208f8ac1ac133f842696dd7e6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: history: fixed user usage data collection (#926952)
by yzaslavs＠redhat.com 03 Apr '13

03 Apr '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: history: fixed user usage data collection (#926952) ...................................................................... history: fixed user usage data collection (#926952) was collecting the os system user, changed to collect the spice session user. Bug-Url: https://bugzilla.redhat.com/926952 Change-Id: I50d4521ef733add0c5439d48c3dbc31bd5ad4e5f Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M backend/manager/dbscripts/create_dwh_views.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13357 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I50d4521ef733add0c5439d48c3dbc31bd5ad4e5f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin, userportal: introducing German (de) - part 1
by ecohen＠redhat.com 03 Apr '13

03 Apr '13

Einav Cohen has submitted this change and it was merged. Change subject: webadmin, userportal: introducing German (de) - part 1 ...................................................................... webadmin, userportal: introducing German (de) - part 1 With this patch, the German (de) locale will be available for display in the GUI. [German properties files will be inroduced in a separate patch] Change-Id: Id84038dcbf056d734e8262b1818b93e1dc099277 Signed-off-by: Einav Cohen <ecohen(a)redhat.com> --- M frontend/webadmin/modules/pom.xml M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/UserPortal.gwt.xml M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/WebAdmin.gwt.xml 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Einav Cohen: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12115 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id84038dcbf056d734e8262b1818b93e1dc099277 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Einav Cohen <ecohen(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]: userportal, webadmin: introducing German (de) - part 2
by ecohen＠redhat.com 03 Apr '13

03 Apr '13

Einav Cohen has submitted this change and it was merged. Change subject: userportal, webadmin: introducing German (de) - part 2 ...................................................................... userportal, webadmin: introducing German (de) - part 2 Pulled German (de) files from the "master" version in the "oVirt" project in Zanata. Change-Id: I47dadb536e7790371bd8907adfbde9cd72cb1896 Signed-off-by: Einav Cohen <ecohen(a)redhat.com> --- A frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants_de.properties A frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationMessages_de.properties A frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Constants_de.properties A frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/LocalizedEnums_de.properties A frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Messages_de.properties A frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/SpiceRedKeys_de.properties A frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_de.properties A frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors_de.properties A frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors_de.properties A frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationConstants_de.properties A frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationMessages_de.properties A frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors_de.properties A frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors_de.properties A frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/VdsmErrors_de.properties A frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants_de.properties A frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationMessages_de.properties 16 files changed, 5,311 insertions(+), 0 deletions(-) Approvals: Einav Cohen: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12116 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I47dadb536e7790371bd8907adfbde9cd72cb1896 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com>

1 0