- Engine-commits - oVirt List Archives

Change in ovirt-reports[master]: build: after build reports for 3.6.1 respin
by sradco＠redhat.com 07 Dec '15

07 Dec '15

Shirly Radco has submitted this change and it was merged. Change subject: build: after build reports for 3.6.1 respin ...................................................................... build: after build reports for 3.6.1 respin Change-Id: I86d5f5fbc62ea35b2c22013ee2ea261ef427156b Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M version.mak 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/50003 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I86d5f5fbc62ea35b2c22013ee2ea261ef427156b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-reports[master]: build: build reports for 3.6.1 respin
by sradco＠redhat.com 07 Dec '15

07 Dec '15

Shirly Radco has submitted this change and it was merged. Change subject: build: build reports for 3.6.1 respin ...................................................................... build: build reports for 3.6.1 respin Change-Id: If201700125568a2bc969d5431371892380ed8166 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M ovirt-engine-reports.spec.in M version.mak 2 files changed, 7 insertions(+), 3 deletions(-) Approvals: Shirly Radco: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/50002 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If201700125568a2bc969d5431371892380ed8166 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: add missing space in message text
by oourfali＠redhat.com 07 Dec '15

07 Dec '15

Oved Ourfali has submitted this change and it was merged. Change subject: core: add missing space in message text ...................................................................... core: add missing space in message text Adding a missing space to the message part that is concatenated to the original message example : Power Management test failed for Host dell-r210ii-04.example.com.No reason was returned for this operation failure. See logs for further details. should be: Power Management test failed for Host dell-r210ii-04.example.com. No reason was returned for this operation failure. See logs for further details. Change-Id: If3ae23595df53cf2261efc2c4f1b2e207a0ecd3c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1288107 Signed-off-by: emesika <emesika(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirector.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eli Mesika: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/50019 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If3ae23595df53cf2261efc2c4f1b2e207a0ecd3c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6.1]: Frontend: ConsoleUserName & ClientIP update regression fix
by tnisan＠redhat.com 07 Dec '15

07 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: Frontend: ConsoleUserName & ClientIP update regression fix ...................................................................... Frontend: ConsoleUserName & ClientIP update regression fix Due to comparison of the same variable which never can be false, the values for the ConsoleUserName and ClientIP fields for console users was not updated. This patch fixes this regression which was introduced in the refactoring of the sessions tab into the guest info tab. Change-Id: Ibb13bd4e9e8a77ee606520133c681464ac119ba5 Bug-Url: https://bugzilla.redhat.com/1272978 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmGuestInfoModel.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Vinzenz Feenstra: Verified -- To view, visit https://gerrit.ovirt.org/50016 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibb13bd4e9e8a77ee606520133c681464ac119ba5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.1 Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6.1]: webadmin: Exception thrown when trying to move/copy a disk w...
by tnisan＠redhat.com 07 Dec '15

07 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Exception thrown when trying to move/copy a disk with no up SDs ...................................................................... webadmin: Exception thrown when trying to move/copy a disk with no up SDs When trying to move or copy a disk where no destination storage domain is available the dialog throws an exception Change-Id: Ief00d53f1c1a8cbadb5c286268028d6aac36e7cc Bug-Url: https://bugzilla.redhat.com/1286557 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/DisksAllocationModel.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/50026 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief00d53f1c1a8cbadb5c286268028d6aac36e7cc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.1 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: config: increase heartbeat interval
by tnisan＠redhat.com 07 Dec '15

07 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: config: increase heartbeat interval ...................................................................... config: increase heartbeat interval We see communication issues due to "heartbeat exceeded" exception. In order to mitigate those situation we increase heartbeat interval from 10 to 30 seconds. Change-Id: I68b9627ab49f607c746e841476e60aaf127d0ed1 Signed-off-by: pkliczewski <piotr.kliczewski(a)gmail.com> Bug-Url: https://bugzilla.redhat.com/1289123 --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 2 files changed, 5 insertions(+), 2 deletions(-) Approvals: Piotr Kliczewski: Verified Eli Mesika: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, but someone else must approve Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/49703 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I68b9627ab49f607c746e841476e60aaf127d0ed1 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Exception thrown when trying to move/copy a disk w...
by tnisan＠redhat.com 07 Dec '15

07 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Exception thrown when trying to move/copy a disk with no up SDs ...................................................................... webadmin: Exception thrown when trying to move/copy a disk with no up SDs When trying to move or copy a disk where no destination storage domain is available the dialog throws an exception Change-Id: Ief00d53f1c1a8cbadb5c286268028d6aac36e7cc Bug-Url: https://bugzilla.redhat.com/1286557 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/DisksAllocationModel.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Verified; Passed CI tests Daniel Erez: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/50025 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief00d53f1c1a8cbadb5c286268028d6aac36e7cc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Exception thrown when trying to move/copy a disk w...
by tnisan＠redhat.com 07 Dec '15

07 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Exception thrown when trying to move/copy a disk with no up SDs ...................................................................... webadmin: Exception thrown when trying to move/copy a disk with no up SDs When trying to move or copy a disk where no destination storage domain is available the dialog throws an exception Change-Id: Ief00d53f1c1a8cbadb5c286268028d6aac36e7cc Bug-Url: https://bugzilla.redhat.com/1286557 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/DisksAllocationModel.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Verified; Passed CI tests Daniel Erez: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/50022 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief00d53f1c1a8cbadb5c286268028d6aac36e7cc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: frontend: Remove AbstractVmPopupWidget.localize() method
by vszocs＠redhat.com 07 Dec '15

07 Dec '15

Vojtech Szocs has submitted this change and it was merged. Change subject: frontend: Remove AbstractVmPopupWidget.localize() method ...................................................................... frontend: Remove AbstractVmPopupWidget.localize() method Moved manual setting of label/text properties to ui.xml file, since method contained no additional logic to setting it declaratively in the ui.xml file. Change-Id: Ib0917cbb7eabe2a3bc80a168ad06136342cf09a8 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml 2 files changed, 93 insertions(+), 217 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Martin Betak: Verified -- To view, visit https://gerrit.ovirt.org/49112 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib0917cbb7eabe2a3bc80a168ad06136342cf09a8 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: there are no such things as ghost in MacRangeWidget
by tnisan＠redhat.com 07 Dec '15

07 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: there are no such things as ghost in MacRangeWidget ...................................................................... webadmin: there are no such things as ghost in MacRangeWidget 'isGhost' mechanism is faulty. Correct behavior is, acording to colleges and code notes, that there's at most one ghost and it has to be the last record. Currently implementation allows existence of multiple ghosts, which needs not to be trailing records. For simplicity, in this dialog we assume that there cannot exist ghosts, until ghost problem is solved generally Change-Id: I8091cedfd3cf51dfcb93bb7c7de45234088bfdd8 Bug-Url: https://bugzilla.redhat.com/1217940 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/macpool/MacRangeWidget.java 1 file changed, 1 insertion(+), 3 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Martin Mucha: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/49966 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8091cedfd3cf51dfcb93bb7c7de45234088bfdd8 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0