January 2013 - Engine-commits - oVirt List Archives

Change in ovirt-host-deploy[master]: build: add missing .gitignores to dist
by Alon Bar-Lev 15 Jan '13

15 Jan '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: build: add missing .gitignores to dist ...................................................................... build: add missing .gitignores to dist Change-Id: Ie7e72995a00ee6a68e7ed631fa498bf0463b35ae Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile.am M src/plugins/ovirt-host-deploy/vdsmhooks/Makefile.am 2 files changed, 6 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11050 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie7e72995a00ee6a68e7ed631fa498bf0463b35ae 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 otopi[master]: build: add missing .gitignores to dist
by Alon Bar-Lev 15 Jan '13

15 Jan '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: build: add missing .gitignores to dist ...................................................................... build: add missing .gitignores to dist Change-Id: Icc3f8c1f90193a252be6af045c58a2b552a06826 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile.am 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11049 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc3f8c1f90193a252be6af045c58a2b552a06826 Gerrit-PatchSet: 1 Gerrit-Project: otopi 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-sdk-java[master]: sdk: replace mvn-release plugin with local
by mpastern＠redhat.com 15 Jan '13

15 Jan '13

Michael Pasternak has submitted this change and it was merged. Change subject: sdk: replace mvn-release plugin with local ...................................................................... sdk: replace mvn-release plugin with local Change-Id: I4ee2aa4edb306eed465eb14451a3837e39351800 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M ovirt-engine-sdk-java/pom.xml 1 file changed, 50 insertions(+), 66 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11048 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4ee2aa4edb306eed465eb14451a3837e39351800 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: improve behavior of bookmark list (2/2)
by gchaplik＠redhat.com 15 Jan '13

15 Jan '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: improve behavior of bookmark list (2/2) ...................................................................... webadmin: improve behavior of bookmark list (2/2) This patch series fixes some problems with the behavior of the bookmark list. This patch fixes re-selection of already-selected bookmarks, which was broken by gwt issue 6310 (which causes setSelectedItem to not behave as expected when a SingleSelectionModel's keyboardSelectionPolicy is BOUND_TO_SELECTION). To work around the problem, we use a different keyboardSelectionPolicy that isn't affected by the bug, and handle the keypress events manually. Change-Id: I25a9e721d8b7ccd982708eadf579e9eb229dedd5 Bug-Url: https://bugzilla.redhat.com/880499 Signed-off-by: Greg Padgett <gpadgett(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/bookmark/BookmarkList.java 1 file changed, 36 insertions(+), 2 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10819 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I25a9e721d8b7ccd982708eadf579e9eb229dedd5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>

1 0

Change in ovirt-engine[master]: build: use release java packages of otopi and ovirt-host-dep...
by yzaslavs＠redhat.com 15 Jan '13

15 Jan '13

Yair Zaslavsky has submitted this change and it was merged. Change subject: build: use release java packages of otopi and ovirt-host-deploy ...................................................................... build: use release java packages of otopi and ovirt-host-deploy Change-Id: I5a06b72097ba52d7a69a3904d297f506ee069f18 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M pom.xml 1 file changed, 2 insertions(+), 10 deletions(-) Approvals: Alon Bar-Lev: Verified Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11046 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a06b72097ba52d7a69a3904d297f506ee069f18 Gerrit-PatchSet: 1 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-host-deploy[master]: packaging: java: post 1.0.0 release
by Alon Bar-Lev 15 Jan '13

15 Jan '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: java: post 1.0.0 release ...................................................................... packaging: java: post 1.0.0 release Change-Id: Ia5efc2477dfb26906192ed5573861b50c4321de7 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M configure.ac 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11044 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia5efc2477dfb26906192ed5573861b50c4321de7 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 otopi[master]: packaging: java: post 1.0.0 release
by Alon Bar-Lev 15 Jan '13

15 Jan '13

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: java: post 1.0.0 release ...................................................................... packaging: java: post 1.0.0 release Change-Id: I571391ed61ccaf57d83b68d2d47456630927e2a1 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M configure.ac 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11043 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I571391ed61ccaf57d83b68d2d47456630927e2a1 Gerrit-PatchSet: 1 Gerrit-Project: otopi 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]: webadmin: Add cluster policy to edit cluster
by gchaplik＠redhat.com 15 Jan '13

15 Jan '13

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Add cluster policy to edit cluster ...................................................................... webadmin: Add cluster policy to edit cluster Added the ability to set the cluster policy in the create/edit cluster popup view The original ClusterPolicyPopupView was dropped and the link from the original button was changed to the ClusterPopupView Change-Id: I195d9ec1c7901a31e6e656221964d82bc90c9695 Signed-off-by: Ofri Masad <omasad(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterGeneralModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterPolicyModel.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/gin/PresenterModule.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/gin/uicommon/ClusterModule.java D frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/popup/cluster/ClusterPolicyPopupPresenterWidget.java D frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPolicyPopupView.java D frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPolicyPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/SubTabClusterGeneralView.java 13 files changed, 400 insertions(+), 490 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10573 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I195d9ec1c7901a31e6e656221964d82bc90c9695 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: ofri masad <omasad(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: ofri masad <omasad(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: remove expensive use of ObjectDescriptor.toString
by rgolan＠redhat.com 15 Jan '13

15 Jan '13

Roy Golan has submitted this change and it was merged. Change subject: core: remove expensive use of ObjectDescriptor.toString ...................................................................... core: remove expensive use of ObjectDescriptor.toString Change-Id: Id0bb51579a9af1f76248cb1152e29ad7cb82d8a0 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- D backend/manager/modules/compat/src/test/java/org/ovirt/engine/core/compat/backendcompat/PropertyInfoTest.java D backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ObjectDescriptor.java D backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/ObjectDescriptorTest.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/StatusReturnForXmlRpc.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/StatusForXmlRpc.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/StatusOnlyReturnForXmlRpc.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml 7 files changed, 13 insertions(+), 160 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10356 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0bb51579a9af1f76248cb1152e29ad7cb82d8a0 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Alissa Bonas <abonas(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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[master]: engine: SPICE should be in upper case [TEXT]
by rgolan＠redhat.com 15 Jan '13

15 Jan '13

Roy Golan has submitted this change and it was merged. Change subject: engine: SPICE should be in upper case [TEXT] ...................................................................... engine: SPICE should be in upper case [TEXT] String substitution has been done with ssed, an enhanced version of sed that support lookbehind regexp find . -regex ".*\.properties" | xargs sed -R -e 's/(?ism)(?<=[=])spice/SPICE/g' -i sed -R -e 's/(?ism)(?<=[= ])spice/SPICE/g' -i frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java Change-Id: I23918285d24a88836493eaa2776886d43ebd80c0 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=876900 Signed-off-by: Roy Golan <rgolan(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants_es.properties M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants_fr.properties M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants_pt_BR.properties M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/ui/common/CommonApplicationConstants_zh_CN.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Constants_es.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Constants_fr.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Constants_pt_BR.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Constants_zh_CN.properties M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors_es.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors_fr.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors_zh_CN.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationConstants_es.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationConstants_fr.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationConstants_pt_BR.properties M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/userportal/ApplicationConstants_zh_CN.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors_es.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors_fr.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/ConsoleErrors_zh_CN.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants_es.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants_fr.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants_pt_BR.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants_zh_CN.properties 26 files changed, 32 insertions(+), 32 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10791 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23918285d24a88836493eaa2776886d43ebd80c0 Gerrit-PatchSet: 4 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> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>

1 0