September 2015 - Engine-commits - oVirt List Archives

Change in ovirt-log-collector[ovirt-log-collector-3.6]: build: ovirt-log-collector-3.6.0
by sbonazzo＠redhat.com 22 Sep '15

22 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-log-collector-3.6.0 ...................................................................... build: ovirt-log-collector-3.6.0 Change-Id: I89256499b0af352834442f4c7d8487334ef1e129 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-log-collector.spec.in 2 files changed, 7 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46500 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I89256499b0af352834442f4c7d8487334ef1e129 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: ovirt-log-collector-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: fixing coverity issue #1320611
by masayag＠redhat.com 22 Sep '15

22 Sep '15

Moti Asayag has submitted this change and it was merged. Change subject: engine: fixing coverity issue #1320611 ...................................................................... engine: fixing coverity issue #1320611 Trying to remove from List<HostDevice> a String value. Change-Id: I2c97de2b93c129f0d9bdc0d8d06822da25d11d82 Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/host/NetworkDeviceHelperImplTest.java 1 file changed, 15 insertions(+), 4 deletions(-) Approvals: Alona Kaplan: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/46499 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2c97de2b93c129f0d9bdc0d8d06822da25d11d82 Gerrit-PatchSet: 2 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: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-image-uploader[ovirt-image-uploader-3.6]: build: ovirt-image-uploader-3.6.0
by sbonazzo＠redhat.com 22 Sep '15

22 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-image-uploader-3.6.0 ...................................................................... build: ovirt-image-uploader-3.6.0 Change-Id: Icfa73121afb1886c3dd59e9c607776199e4793ad Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-image-uploader.spec.in 2 files changed, 7 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46498 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icfa73121afb1886c3dd59e9c607776199e4793ad Gerrit-PatchSet: 1 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: ovirt-image-uploader-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-iso-uploader[ovirt-iso-uploader-3.6]: build: ovirt-iso-uploader-3.6.0
by sbonazzo＠redhat.com 22 Sep '15

22 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-iso-uploader-3.6.0 ...................................................................... build: ovirt-iso-uploader-3.6.0 Change-Id: I36ab7c5cc6eb9317f02bb2b9aeb57edc9284c10a Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-iso-uploader.spec.in 2 files changed, 6 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46497 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I36ab7c5cc6eb9317f02bb2b9aeb57edc9284c10a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: ovirt-iso-uploader-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-iso-uploader[ovirt-iso-uploader-3.6]: pep8: fixing errors
by sbonazzo＠redhat.com 22 Sep '15

22 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: pep8: fixing errors ...................................................................... pep8: fixing errors Change-Id: Ie102be0a3e5d7078b17ef86db970ba484564b543 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/__main__.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46496 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie102be0a3e5d7078b17ef86db970ba484564b543 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: ovirt-iso-uploader-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-image-uploader[master]: ovirt-image-uploader: fix ovirt-engine api url
by sbonazzo＠redhat.com 22 Sep '15

22 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-image-uploader: fix ovirt-engine api url ...................................................................... ovirt-image-uploader: fix ovirt-engine api url Change-Id: I9c37c104c49e3d7ba03eeb8c9597d91e219d4b2d Bug-Url: https://bugzilla.redhat.com/1264424 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M src/__main__.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Rafael Martins: Verified -- To view, visit https://gerrit.ovirt.org/46374 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9c37c104c49e3d7ba03eeb8c9597d91e219d4b2d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-image-uploader Gerrit-Branch: master Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(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

1 0

Change in ovirt-iso-uploader[master]: ovirt-iso-uploader: fix ovirt-engine api url
by sbonazzo＠redhat.com 22 Sep '15

22 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-iso-uploader: fix ovirt-engine api url ...................................................................... ovirt-iso-uploader: fix ovirt-engine api url Change-Id: I9e3daea3a9603f5a3763be6b6a809e01da3c62f8 Bug-Url: https://bugzilla.redhat.com/1264542 Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M src/__main__.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Rafael Martins: Verified -- To view, visit https://gerrit.ovirt.org/46376 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e3daea3a9603f5a3763be6b6a809e01da3c62f8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-iso-uploader Gerrit-Branch: master Gerrit-Owner: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: vmconsole: helper: exit with non zero if not http success
by sbonazzo＠redhat.com 22 Sep '15

22 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: vmconsole: helper: exit with non zero if not http success ...................................................................... vmconsole: helper: exit with non zero if not http success Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1223671 Change-Id: I1e3be061695afcae389a92b8e953743577438331 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/libexec/ovirt-vmconsole-proxy-helper/ovirt-vmconsole-list.py 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Francesco Romani: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46480 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e3be061695afcae389a92b8e953743577438331 Gerrit-PatchSet: 2 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: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin:remove customProperties from nic
by piotr.kliczewski＠gmail.com 22 Sep '15

22 Sep '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin:remove customProperties from nic ...................................................................... webadmin:remove customProperties from nic Change-Id: Icc367e3d11f2731237d2b52049d39f33364f1e84 Bug-Url: https://bugzilla.redhat.com/1246060 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostSetupNetworksModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/LogicalNetworkModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/NetworkOperation.java M frontend/webadmin/modules/uicommonweb/src/test/java/org/ovirt/engine/ui/uicommonweb/models/hosts/network/ExecuteNetworkCommandInNetworkOperationTest.java 4 files changed, 43 insertions(+), 25 deletions(-) Approvals: Martin Mucha: Verified Alona Kaplan: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46490 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icc367e3d11f2731237d2b52049d39f33364f1e84 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: redundant scrollbar in SetupNetworks dialog
by piotr.kliczewski＠gmail.com 22 Sep '15

22 Sep '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: redundant scrollbar in SetupNetworks dialog ...................................................................... webadmin: redundant scrollbar in SetupNetworks dialog The setup networks dialog was slightly too small and a redundant vertical scroll bar was displayed. The dialog has been made slightly higher to accomodate all its contents without the scroll bar. Change-Id: I3af6cdcdccd59ba45292d13ffd16cb6d820987ce Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1140526 Signed-off-by: Marcin Mirecki <mmirecki(a)redhat.com> (cherry picked from commit 564e2b012447142fa155707f9fb36f102fcb8dbd) --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostSetupNetworksPopupView.ui.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Passed CI tests Marcin Mirecki: Verified; Looks good to me, but someone else must approve Alona Kaplan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45905 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3af6cdcdccd59ba45292d13ffd16cb6d820987ce Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0