- Engine-commits - oVirt List Archives

Change in ovirt-reports[master]: cleanup: dos2unix *.xml
by ydary＠redhat.com 22 Aug '12

22 Aug '12

Yaniv Dary has submitted this change and it was merged. Change subject: cleanup: dos2unix *.xml ...................................................................... cleanup: dos2unix *.xml Change-Id: I2ddfe672406632a3376948fb8a0072549fb680d7 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M reports/jars/ChartsCustomizers/pom.xml M reports/jars/ReportsLineBarChartTheme/pom.xml M reports/jars/ReportsPieChartTheme/pom.xml M reports/jars/WebadminLineBarChartTheme/pom.xml 4 files changed, 152 insertions(+), 152 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7405 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2ddfe672406632a3376948fb8a0072549fb680d7 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-cli[master]: cli: shell does not exit when using /exit/ command in script
by mpastern＠redhat.com 22 Aug '12

22 Aug '12

Michael Pasternak has submitted this change and it was merged. Change subject: cli: shell does not exit when using /exit/ command in script ...................................................................... cli: shell does not exit when using /exit/ command in script Change-Id: I38e41185d2f3016536f6791bb2d8bdf0f28be0d9 Signed-off-by: Michael Pasternak <mpastern(a)redhat.com> --- M src/ovirtcli/shell/engineshell.py 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7397 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I38e41185d2f3016536f6791bb2d8bdf0f28be0d9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>

1 0

Change in ovirt-dwh[master]: packaging: fixed setup failure on missing file (#850330)
by ydary＠redhat.com 22 Aug '12

22 Aug '12

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: fixed setup failure on missing file (#850330) ...................................................................... packaging: fixed setup failure on missing file (#850330) getHostParams was broken since a config file was removed. Reimplemented getHostParams to fetch this data from another source. https://bugzilla.redhat.com/850330 Change-Id: Ic3166c233dfd29a715b0d5ed80e4ff59d23b40ae Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M packaging/common_utils.py M packaging/ovirt-engine-dwh-setup.py 2 files changed, 38 insertions(+), 33 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7392 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic3166c233dfd29a715b0d5ed80e4ff59d23b40ae Gerrit-PatchSet: 6 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-dwh[master]: build: support files with spaces in tarball
by ydary＠redhat.com 22 Aug '12

22 Aug '12

Yaniv Dary has submitted this change and it was merged. Change subject: build: support files with spaces in tarball ...................................................................... build: support files with spaces in tarball Change-Id: I9910e972906d388c5791158776beef1f81977f95 Signed-off-by: Alon Bar-Lev <alon.barlev(a)gmail.com> --- M Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified Juan Hernandez: Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7404 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9910e972906d388c5791158776beef1f81977f95 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alexey Chub <achub(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: UI: change Make-Template to more readable text (#848104)
by shavivi＠redhat.com 22 Aug '12

22 Aug '12

Shahar Havivi has submitted this change and it was merged. Change subject: UI: change Make-Template to more readable text (#848104) ...................................................................... UI: change Make-Template to more readable text (#848104) https://bugzilla.redhat.com/show_bug.cgi?id=848104 Change-Id: I9055763d66b09da175810ad3ba2ace0ff8316264 Signed-off-by: Shahar Havivi <shavivi(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmMakeTemplatePopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmMakeTemplatePopupWidget.ui.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewTemplateVmModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java 6 files changed, 31 insertions(+), 37 deletions(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7216 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9055763d66b09da175810ad3ba2ace0ff8316264 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: add ENGINE_FQDN variable to service configuration
by juan.hernandez＠redhat.com 21 Aug '12

21 Aug '12

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: add ENGINE_FQDN variable to service configuration ...................................................................... packaging: add ENGINE_FQDN variable to service configuration The engine FQDN is required for dependencies such as dwhj and reports to successfully find engine. Change-Id: Icc2a12280813bc7462ba62bf9e9a7b5358811147 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/fedora/engine-service.sysconfig M packaging/fedora/setup/common_utils.py M packaging/fedora/setup/engine-setup.py 3 files changed, 10 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7384 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc2a12280813bc7462ba62bf9e9a7b5358811147 Gerrit-PatchSet: 3 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: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-dwh[master]: packaging: changed built jar version to hard coded values
by ydary＠redhat.com 21 Aug '12

21 Aug '12

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: changed built jar version to hard coded values ...................................................................... packaging: changed built jar version to hard coded values Change-Id: Iabd56405617f9e67a1819e463ba344bbf8d50c1b Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M data-warehouse/history_etl/etl_sources/advancedPersistentLookupLib/pom.xml M data-warehouse/history_etl/etl_sources/historyETLProcedure/pom.xml M data-warehouse/history_etl/etl_sources/talendRoutines/pom.xml M pom.xml 4 files changed, 5 insertions(+), 7 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7379 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iabd56405617f9e67a1819e463ba344bbf8d50c1b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-dwh[master]: packaging: use place holders for version/release
by ydary＠redhat.com 21 Aug '12

21 Aug '12

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: use place holders for version/release ...................................................................... packaging: use place holders for version/release Currently Makefile substitutes complete lines within spec in order to inject version information. The hard coded version information within the spec is misleading. Modify process to use substitute variables to ease maintenance. Change-Id: I265574b32285c9596414d293a1e5b465cc223215 Signed-off-by: Alon Bar-Lev <alon.barlev(a)gmail.com> --- M Makefile M packaging/ovirt-engine-dwh.spec.in 2 files changed, 4 insertions(+), 4 deletions(-) Approvals: Alon Bar-Lev: Verified Juan Hernandez: Looks good to me, but someone else must approve Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7371 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I265574b32285c9596414d293a1e5b465cc223215 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: fixing NPE in Reports (#849623)
by alkaplan＠redhat.com 21 Aug '12

21 Aug '12

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: fixing NPE in Reports (#849623) ...................................................................... webadmin: fixing NPE in Reports (#849623) https://bugzilla.redhat.com/849623 There was a change in behavior, when fetching Reports.xml, in the past, the jboss used to return null, and now it returns something, so added a try/catch clause to protect the call. Change-Id: I4e9f9e47d258b9def47a050bb7eeaafcbdd2d342 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/ReportInit.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/ReportParser.java 2 files changed, 20 insertions(+), 13 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7344 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4e9f9e47d258b9def47a050bb7eeaafcbdd2d342 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: Webadmin: Tree pane is not refreshed (#845813)
by shavivi＠redhat.com 21 Aug '12

21 Aug '12

Shahar Havivi has submitted this change and it was merged. Change subject: Webadmin: Tree pane is not refreshed (#845813) ...................................................................... Webadmin: Tree pane is not refreshed (#845813) https://bugzilla.redhat.com/show_bug.cgi?id=845813 Signed-off-by: Shahar Havivi<shavivi(a)redhat.com> Change-Id: I6b1a315ea4f7c8bfedc5b13b591c2eb8c90ee5a2 --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/tree/SystemTree.java 1 file changed, 14 insertions(+), 0 deletions(-) Approvals: Shahar Havivi: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7374 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b1a315ea4f7c8bfedc5b13b591c2eb8c90ee5a2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com>

1 0