- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: tool: restore excidently removed 'private static final'
by oourfali＠redhat.com 26 Oct '14

26 Oct '14

Oved Ourfali has submitted this change and it was merged. Change subject: tool: restore excidently removed 'private static final' ...................................................................... tool: restore excidently removed 'private static final' I removed it by mistake[1]. [1] http://gerrit.ovirt.org/#/c/33441/1..2/backend/manager/tools/src/main/java/… Change-Id: Ic51771917e5e8ec22042f2ee6ebdc67af10e2f01 Signed-off-by: Mooli Tayer <mtayer(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/notifier/transport/snmp/Snmp.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33746 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic51771917e5e8ec22042f2ee6ebdc67af10e2f01 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: mooli tayer <mtayer(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: osinfo: Enable NIC hotplugging in ppc64
by ofrenkel＠redhat.com 26 Oct '14

26 Oct '14

Omer Frenkel has submitted this change and it was merged. Change subject: osinfo: Enable NIC hotplugging in ppc64 ...................................................................... osinfo: Enable NIC hotplugging in ppc64 This patch enables NIC hotplugging in ppc64 guests, since it is supported in PowerKVM. Change-Id: I755c599c2b93ee901b51572415b95841cb5263c1 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1154630 Signed-off-by: Vitor de Lima <vdelima(a)redhat.com> --- M packaging/conf/osinfo-defaults.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34327 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I755c599c2b93ee901b51572415b95841cb5263c1 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vitor de Lima <vdelima(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vdelima(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]: engine: blank template no need default vm_init
by tnisan＠redhat.com 26 Oct '14

26 Oct '14

Tal Nisan has submitted this change and it was merged. Change subject: engine: blank template no need default vm_init ...................................................................... engine: blank template no need default vm_init By having a default vm_init all inherit VMs are getting blank vm_init as well. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1155951 Change-Id: Ifa39cad168d523f1a902095b28cde91bf49f14db Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- A packaging/dbscripts/upgrade/03_05_1110_remove_vm_init_for_blank_template.sql 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Shahar Havivi: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34410 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifa39cad168d523f1a902095b28cde91bf49f14db Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[ovirt-engine-reports-3.5]: packaging: Added release notes
by sradco＠redhat.com 26 Oct '14

26 Oct '14

Shirly Radco has submitted this change and it was merged. Change subject: packaging: Added release notes ...................................................................... packaging: Added release notes Change-Id: Iea8c481aab96d0eb3e63af4f2818f9e80e830313 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M ovirt-engine-reports.spec.in 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34476 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iea8c481aab96d0eb3e63af4f2818f9e80e830313 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.5 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(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: require java detection
by sradco＠redhat.com 26 Oct '14

26 Oct '14

Shirly Radco has submitted this change and it was merged. Change subject: packaging: setup: require java detection ...................................................................... packaging: setup: require java detection Will be required after http://gerrit.ovirt.org/33410 is merged. Change-Id: I0c4f1868f6d3fd71bae52fce4f05395f808c18d4 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit af8d472a55d9ea5234e8c5ccca23c2e0a07023b1) --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-reports/config/jboss.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34472 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c4f1868f6d3fd71bae52fce4f05395f808c18d4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-reports Gerrit-Branch: ovirt-engine-reports-3.5 Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-host-deploy[master]: build: force java-1.5 source and target
by Alon Bar-Lev 26 Oct '14

26 Oct '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: build: force java-1.5 source and target ...................................................................... build: force java-1.5 source and target soon java-8 will be used to build, we should force our preferences. Change-Id: I74c4e1ed5b94e24ba70b046243332c59da506575 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/java/Makefile.am M src/java/pom.xml.in 2 files changed, 10 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34469 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74c4e1ed5b94e24ba70b046243332c59da506575 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> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in otopi[master]: build: force java-1.5 source and target
by Alon Bar-Lev 26 Oct '14

26 Oct '14

Alon Bar-Lev has submitted this change and it was merged. Change subject: build: force java-1.5 source and target ...................................................................... build: force java-1.5 source and target soon java-8 will be used to build, we should force our preferences. Change-Id: Ic5efc391c5142663eea99b98dd864326ec8586b8 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M src/java/Makefile.am M src/java/pom.xml.in 2 files changed, 10 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34468 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic5efc391c5142663eea99b98dd864326ec8586b8 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> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Fix host deploy logs path displayed in Events tab
by oourfali＠redhat.com 24 Oct '14

24 Oct '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: Fix host deploy logs path displayed in Events tab ...................................................................... core: Fix host deploy logs path displayed in Events tab Fixes host deploy logs path displayed in Events tab in case of unsuccessfull deployment of host. Change-Id: Ifcba8acd649bb97dc793a2f82062f9bc71a36720 Bug-Url: https://bugzilla.redhat.com/1155603 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Peřina: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34448 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifcba8acd649bb97dc793a2f82062f9bc71a36720 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: userportal, webadmin: Fix frontend compile fedora 21
by sbonazzo＠redhat.com 24 Oct '14

24 Oct '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: userportal,webadmin: Fix frontend compile fedora 21 ...................................................................... userportal,webadmin: Fix frontend compile fedora 21 - On java < 8 the missing class definition for GenericApiGWTServiceAsync was silently ignored and would not cause an issue, but with fedora 21 you can't downgrade to java < 8 so the missing class would cause the compile to fail. The class is generated but we never added the generated classes path to the source definition in the POM.xml Change-Id: Ib7fc9c9ecebf39dcae3a0b9c3923e2790dbb179a Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/frontend/pom.xml 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Alexander Wels: Verified -- To view, visit http://gerrit.ovirt.org/34402 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib7fc9c9ecebf39dcae3a0b9c3923e2790dbb179a Gerrit-PatchSet: 1 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: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in mom[master]: Correct typo in vcpuLimit name element
by alitke＠redhat.com 24 Oct '14

24 Oct '14

Adam Litke has submitted this change and it was merged. Change subject: Correct typo in vcpuLimit name element ...................................................................... Correct typo in vcpuLimit name element There is typo in name of XML element - vcpulimit is incorrect, because it is vcpuLimit. Change-Id: I13e88aff78fd4125c146d72e9faade0cb4a7b544 Signed-off-by: Martin Pavlasek <mpavlase(a)redhat.com> Reviewed-on: http://gerrit.ovirt.org/34212 Reviewed-by: Adam Litke <alitke(a)redhat.com> --- M mom/HypervisorInterfaces/libvirtInterface.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Adam Litke: Looks good to me, approved Martin Pavlásek: Verified -- To view, visit http://gerrit.ovirt.org/34212 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I13e88aff78fd4125c146d72e9faade0cb4a7b544 Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Martin Pavlásek <mpavlase(a)redhat.com> Gerrit-Reviewer: Adam Litke <alitke(a)redhat.com> Gerrit-Reviewer: Martin Pavlásek <mpavlase(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0