September 2014 - Engine-commits - oVirt List Archives

Change in ovirt-reports[ovirt-engine-reports-3.5]: reports: do not display input control in ce dashboards
by ydary＠redhat.com 23 Sep '14

23 Sep '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: do not display input control in ce dashboards ...................................................................... reports: do not display input control in ce dashboards Change-Id: I2b633bddfb4f4ca61aa70309ecaa4cb4437501f8 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> (cherry picked from commit 6b356af96bc2cfec79da29b2b8e26fc29b9ecced) --- M packaging/ovirt-reports/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard.xml M packaging/ovirt-reports/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard.xml M packaging/ovirt-reports/resources/reports_resources/webadmin_dashboards/ce/system_dashboard.xml 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33270 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b633bddfb4f4ca61aa70309ecaa4cb4437501f8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.5 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-reports[ovirt-engine-reports-3.5]: packaging: setup: fix link to dashboard
by ydary＠redhat.com 23 Sep '14

23 Sep '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: fix link to dashboard ...................................................................... packaging: setup: fix link to dashboard Change-Id: I498c2918005e0ba0f6a7d2e188c7d55104a84f8d Bug-Url: https://bugzilla.redhat.com/1144079 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit cb481081a78f843d1c174b19ab667ee3de07502c) --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/engine.py 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33269 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I498c2918005e0ba0f6a7d2e188c7d55104a84f8d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.5 Gerrit-Owner: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-reports[master]: reports: do not display input control in ce dashboards
by ydary＠redhat.com 23 Sep '14

23 Sep '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: do not display input control in ce dashboards ...................................................................... reports: do not display input control in ce dashboards Change-Id: I2b633bddfb4f4ca61aa70309ecaa4cb4437501f8 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/webadmin_dashboards/ce/cluster_dashboard.xml M packaging/ovirt-reports/resources/reports_resources/webadmin_dashboards/ce/datacenter_dashboard.xml M packaging/ovirt-reports/resources/reports_resources/webadmin_dashboards/ce/system_dashboard.xml 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33268 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b633bddfb4f4ca61aa70309ecaa4cb4437501f8 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> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.5]: packaging: setup: fix older pep8 compatibility
by oourfali＠redhat.com 23 Sep '14

23 Sep '14

Oved Ourfali has submitted this change and it was merged. Change subject: packaging: setup: fix older pep8 compatibility ...................................................................... packaging: setup: fix older pep8 compatibility regardless, this is better... Change-Id: Ibd6b2cf7c1c9f5427a6cce6ceb8df2873bc43888 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> (cherry picked from commit 19a2ed147aff0cde12f2ad144617e7c7240ab0da) --- M packaging/setup/ovirt_engine_setup/remote_engine.py 1 file changed, 7 insertions(+), 7 deletions(-) Approvals: Yedidyah Bar David: Verified; Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33267 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibd6b2cf7c1c9f5427a6cce6ceb8df2873bc43888 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: fix older pep8 compatibility
by emesika＠redhat.com 23 Sep '14

23 Sep '14

Eli Mesika has submitted this change and it was merged. Change subject: packaging: setup: fix older pep8 compatibility ...................................................................... packaging: setup: fix older pep8 compatibility regardless, this is better... Change-Id: Ibd6b2cf7c1c9f5427a6cce6ceb8df2873bc43888 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/remote_engine.py 1 file changed, 7 insertions(+), 7 deletions(-) Approvals: Simone Tiraboschi: Looks good to me, but someone else must approve Eli Mesika: Verified; Looks good to me, approved oVirt Jenkins CI Server: Looks good to me, but someone else must approve Yedidyah Bar David: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/33255 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibd6b2cf7c1c9f5427a6cce6ceb8df2873bc43888 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(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-reports[master]: packaging: setup: fix link to dashboard
by ydary＠redhat.com 23 Sep '14

23 Sep '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: fix link to dashboard ...................................................................... packaging: setup: fix link to dashboard Change-Id: I498c2918005e0ba0f6a7d2e188c7d55104a84f8d Bug-Url: https://bugzilla.redhat.com/1144079 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/engine.py 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Yaniv Dary: Looks good to me, approved Yedidyah Bar David: Verified -- To view, visit http://gerrit.ovirt.org/33266 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I498c2918005e0ba0f6a7d2e188c7d55104a84f8d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: clear numa data that causes parsing error
by gchaplik＠redhat.com 23 Sep '14

23 Sep '14

Gilad Chaplik has submitted this change and it was merged. Change subject: core: clear numa data that causes parsing error ...................................................................... core: clear numa data that causes parsing error In change 875214c4, numa node distance field is parsed and persist differently, so in order to avoid parsing exception this fix clears the data, and it will be filled in next iteration of getVdsCaps. Change-Id: I38bf5bb45199b9151d8a88667fe8b2de4b614209 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- A packaging/dbscripts/upgrade/03_06_0400_clear_numa_node_distance.sql 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33245 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I38bf5bb45199b9151d8a88667fe8b2de4b614209 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-host-deploy[master]: node: support plugin features instead by version
by Alon Bar-Lev 23 Sep '14

23 Sep '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: node: support plugin features instead by version ...................................................................... node: support plugin features instead by version Change-Id: I60647e0e51670ea58b3313e5e7b06e8ecf63dffe Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/ovirt_host_deploy/constants.py M src/plugins/ovirt-host-deploy/kdump/packages.py M src/plugins/ovirt-host-deploy/node/detect.py 3 files changed, 24 insertions(+), 27 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33069 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I60647e0e51670ea58b3313e5e7b06e8ecf63dffe Gerrit-PatchSet: 3 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> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Don't write empty description/comment to OVF
by juan.hernandez＠redhat.com 23 Sep '14

23 Sep '14

Juan Hernandez has submitted this change and it was merged. Change subject: core: Don't write empty description/comment to OVF ...................................................................... core: Don't write empty description/comment to OVF Currently when we generate OVF files we write the description and comment properties of the VM even if they are null. As a result when a VM without a description or comment is re-imported it has an empty string a value (instead of null) for these properties. This patch changes the OVF writer so that it won't write the description or comment properties if they are null. Change-Id: Iffd0f7010bc32402c45882dc6087bf02a55a14d9 Bug-Url: https://bugzilla.redhat.com/1145466 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfWriter.java 1 file changed, 10 insertions(+), 6 deletions(-) Approvals: Juan Hernandez: Verified Arik Hadas: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33247 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iffd0f7010bc32402c45882dc6087bf02a55a14d9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: core: vmID parameter not passed during jsonrpc call
by oourfali＠redhat.com 23 Sep '14

23 Sep '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: vmID parameter not passed during jsonrpc call ...................................................................... core: vmID parameter not passed during jsonrpc call There was missing parameter when running updateVmPolicy verb. Bug-Url: https://bugzilla.redhat.com/1142851 Change-Id: I7dea865c9053f3960e2f5eff608d29be388e4080 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Piotr Kliczewski: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33257 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7dea865c9053f3960e2f5eff608d29be388e4080 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0