March 2014 - Engine-commits - oVirt List Archives

Change in ovirt-log-collector[master]: sos: added ovirt_engine_reports sos plugin
by sbonazzo＠redhat.com 10 Mar '14

10 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: sos: added ovirt_engine_reports sos plugin ...................................................................... sos: added ovirt_engine_reports sos plugin Bug-Url: https://bugzilla.redhat.com/1074468 Change-Id: Id76ce5a1cd835fa160057a7d9d71aae82f1669f5 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/__main__.py M src/sos/plugins/Makefile.am A src/sos/plugins/ovirt_engine_reports.py 3 files changed, 34 insertions(+), 0 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved Kiril Nesenko: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/25519 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id76ce5a1cd835fa160057a7d9d71aae82f1669f5 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Keith Robertson <kroberts(a)redhat.com> Gerrit-Reviewer: Kiril Nesenko <knesenko(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tomáš Došek <tdosek(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: engine: fix the allowed values for even guest distribution p...
by iheim＠redhat.com 10 Mar '14

10 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: engine: fix the allowed values for even guest distribution parameters ...................................................................... engine: fix the allowed values for even guest distribution parameters Change-Id: I4aae0f714b3b405a276e9a509ed8c2ec2cdcde2e Bug-Url: https://bugzilla.redhat.com/1069129 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- A packaging/dbscripts/upgrade/03_04_0680__fix_even_guest_distribution_parameters.sql 1 file changed, 7 insertions(+), 0 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25015 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4aae0f714b3b405a276e9a509ed8c2ec2cdcde2e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Kobi Ianko <kobi(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: core: resouces of VMs in launch state shouldn't be committed
by iheim＠redhat.com 10 Mar '14

10 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: core: resouces of VMs in launch state shouldn't be committed ...................................................................... core: resouces of VMs in launch state shouldn't be committed Currently we count pending in scheduling process, and committed memory is calculated in server side according to VMs attached to the host (including VMs in wait for launch state- host in VM entity is update when CreateVm command returns from VDSM). Pending memory is being released in powering-up state, so in fact we count resources twice: since the time VM is in wait for launch till it's powering up. Memory commitment in VDSM occurs after a vm was requested, so using this data will eliminate VM throttling that failed to reach powering up state. Change-Id: Ie2db20c213ac326d620fad578bb5c1c78ff0b3bc Bug-Url: https://bugzilla.redhat.com/1052024 Signed-off-by: Gilad Chaplik <gchaplik(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Gilad Chaplik: Verified Arik Hadas: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25461 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie2db20c213ac326d620fad578bb5c1c78ff0b3bc Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: webadmin: Add room for long vNIC names
by iheim＠redhat.com 10 Mar '14

10 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: webadmin: Add room for long vNIC names ...................................................................... webadmin: Add room for long vNIC names Increased the width of the label field, and redesigned the widgets around it to conform to the new width. Change-Id: I6f7d73632440d88e76c544a1aee754f47bdf8f21 Bug-Url: https://bugzilla.redhat.com/1029973 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> (cherry picked from commit 44792b0406857e456122212a840a304ac16ce6e0) --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/profile/ProfileInstanceTypeEditor.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/profile/ProfileInstanceTypeEditor.ui.xml 2 files changed, 23 insertions(+), 1 deletion(-) Approvals: Alona Kaplan: Looks good to me, approved Lior Vernia: Verified -- To view, visit http://gerrit.ovirt.org/25432 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6f7d73632440d88e76c544a1aee754f47bdf8f21 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: webadmin: Remove confusing coloring in VM interfaces widget
by iheim＠redhat.com 10 Mar '14

10 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: webadmin: Remove confusing coloring in VM interfaces widget ...................................................................... webadmin: Remove confusing coloring in VM interfaces widget The gray color led people to believe the widget was disabled, so changed it to black. Change-Id: I2ffcf14f55b6b14ee62adde63987a2a3bae3cd6c Bug-Url: https://bugzilla.redhat.com/1064431 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> (cherry picked from commit f3971c1782e4d443a94a19bd645765397021ccfb) --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/profile/ProfilesInstanceTypeEditor.ui.xml 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Alona Kaplan: Looks good to me, approved Lior Vernia: Verified -- To view, visit http://gerrit.ovirt.org/25431 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2ffcf14f55b6b14ee62adde63987a2a3bae3cd6c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: webadmin: Enable vertical scrolling in bond dialog
by iheim＠redhat.com 10 Mar '14

10 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: webadmin: Enable vertical scrolling in bond dialog ...................................................................... webadmin: Enable vertical scrolling in bond dialog This is required since the addition of labels, of which many could exist. Change-Id: I54ae2350eeb1dc23b222915f99df8f15882c67ae Bug-Url: https://bugzilla.redhat.com/1063852 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> (cherry picked from commit 49d7f0ea5d61f3981a4d8559f360999f381efd52) --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostBondPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/NicLabelEditor.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/NicLabelWidget.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/SetupNetworksBondPopupView.java 4 files changed, 5 insertions(+), 5 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved Lior Vernia: Verified -- To view, visit http://gerrit.ovirt.org/25423 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I54ae2350eeb1dc23b222915f99df8f15882c67ae Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: webadmin: Added check to prevent no-op case
by iheim＠redhat.com 10 Mar '14

10 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: webadmin: Added check to prevent no-op case ...................................................................... webadmin: Added check to prevent no-op case Adding a NIC to the bond already containing it wasn't marked as a no-op so it was displayed as a legitimate operation. Change-Id: Iebff9f9e3b5cd9973cfbebe307ae75a2e5d96a67 Bug-Url: https://bugzilla.redhat.com/1045656 Bug-Url: https://bugzilla.redhat.com/1045658 Signed-off-by: Lior Vernia <lvernia(a)redhat.com> (cherry picked from commit f1508e460cc3f238b52b3bc21a2c3cfdd68338b1) --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/NetworkOperationFactory.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved Lior Vernia: Verified -- To view, visit http://gerrit.ovirt.org/25419 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iebff9f9e3b5cd9973cfbebe307ae75a2e5d96a67 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: engine: allow values higher than 9 minutes for CpuOverCommit...
by iheim＠redhat.com 10 Mar '14

10 Mar '14

Itamar Heim has submitted this change and it was merged. Change subject: engine: allow values higher than 9 minutes for CpuOverCommitDuration ...................................................................... engine: allow values higher than 9 minutes for CpuOverCommitDuration Change-Id: I1b8b4f7b4f8ef76dc9774dc1620e6bb76aaf2f65 Bug-Url: https://bugzilla.redhat.com/1073776 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- A packaging/dbscripts/upgrade/03_03_1040_allow_values_higher_9_for_cpuovercommit.sql 1 file changed, 14 insertions(+), 0 deletions(-) Approvals: Jiří Moskovčák: Verified Gilad Chaplik: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25520 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b8b4f7b4f8ef76dc9774dc1620e6bb76aaf2f65 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Kobi Ianko <kobi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: reports: updated localization files
by ydary＠redhat.com 10 Mar '14

10 Mar '14

Yaniv Dary has submitted this change and it was merged. Change subject: reports: updated localization files ...................................................................... reports: updated localization files Change-Id: I85a101ad7ae157d9cae8555ee31948266fbd1c1c Related-To: https://bugzilla.redhat.com/1017183 Signed-off-by: Yaniv Dary <ydary(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/localization/ovirt_reports_bundle_de.properties.data M packaging/ovirt-reports/resources/reports_resources/localization/ovirt_reports_bundle_es.properties.data M packaging/ovirt-reports/resources/reports_resources/localization/ovirt_reports_bundle_fr.properties.data M packaging/ovirt-reports/resources/reports_resources/localization/ovirt_reports_bundle_ja.properties.data M packaging/ovirt-reports/resources/reports_resources/localization/ovirt_reports_bundle_pt_BR.properties.data M packaging/ovirt-reports/resources/reports_resources/localization/ovirt_reports_bundle_zh_CN.properties.data 6 files changed, 2,132 insertions(+), 2,132 deletions(-) Approvals: Yaniv Dary: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25561 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I85a101ad7ae157d9cae8555ee31948266fbd1c1c 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[ovirt-engine-3.4.0]: tools: fix example in configuration file.
by sbonazzo＠redhat.com 10 Mar '14

10 Mar '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: tools: fix example in configuration file. ...................................................................... tools: fix example in configuration file. The SNMP_MANAGERS example contains space and no quotes. Users following this example will lose all but first snmp managers. Change-Id: I30de620597eba51a0955d0d6c5cd1c4232c1ab44 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1074431 Signed-off-by: Mooli Tayer <mtayer(a)redhat.com> --- M packaging/services/ovirt-engine-notifier/ovirt-engine-notifier.conf.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/25500 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I30de620597eba51a0955d0d6c5cd1c4232c1ab44 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4.0 Gerrit-Owner: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com>

1 0