January 2016 - Engine-commits - oVirt List Archives

Change in ovirt-release[master]: node: Show the pretty name only
by sbonazzo＠redhat.com 26 Jan '16

26 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: node: Show the pretty name only ...................................................................... node: Show the pretty name only Change-Id: Ib33427bbb43bbcd2418d20c3502429808ca8bcb7 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release-master/host-node/cockpit/branding/branding.css 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/52721 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib33427bbb43bbcd2418d20c3502429808ca8bcb7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-3.6]: build: ovirt-live-3.6.2
by sbonazzo＠redhat.com 26 Jan '16

26 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-live-3.6.2 ...................................................................... build: ovirt-live-3.6.2 Change-Id: Ideac8c9a64cfd76455379320393e471a0b10bfd0 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/Makefile M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52720 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ideac8c9a64cfd76455379320393e471a0b10bfd0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: allow to balance migrate vm that run on specific host
by rgolan＠redhat.com 26 Jan '16

26 Jan '16

Roy Golan has submitted this change and it was merged. Change subject: core: allow to balance migrate vm that run on specific host ...................................................................... core: allow to balance migrate vm that run on specific host Change-Id: I1b34923ad7659d426eca51eb360c6ae0add01a54 Bug-Url: https://bugzilla.redhat.com/1258751 Signed-off-by: Artyom Lukianov <alukiano(a)redhat.com> Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/utils/FindVmAndDestinations.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/utils/FindVmAndDestinationsTest.java 2 files changed, 86 insertions(+), 3 deletions(-) Approvals: Martin Sivák: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved Artyom Lukianov: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/45559 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1b34923ad7659d426eca51eb360c6ae0add01a54 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Artyom Lukianov <alukiano(a)redhat.com> Gerrit-Reviewer: Artyom Lukianov <alukiano(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-3.6]: setup: removing --disable-factory option
by sbonazzo＠redhat.com 26 Jan '16

26 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: setup: removing --disable-factory option ...................................................................... setup: removing --disable-factory option --disable-factory is no longer supported on gnome-terminal 3.14, removing it. Change-Id: I910200ad2bd519c2794cce4879b2528a2f4073d4 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> (cherry picked from commit 21f7b22f1e7b7fb849ec25dd83f789e906717dcf) --- M centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/engine-setup-yad.sh M fedora/oVirtLiveFiles/engine-setup-yad.sh 2 files changed, 2 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52719 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I910200ad2bd519c2794cce4879b2528a2f4073d4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[master]: setup: removing --disable-factory option
by sbonazzo＠redhat.com 26 Jan '16

26 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: setup: removing --disable-factory option ...................................................................... setup: removing --disable-factory option --disable-factory is no longer supported on gnome-terminal 3.14, removing it. Change-Id: I910200ad2bd519c2794cce4879b2528a2f4073d4 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M centos-7/oVirtLiveFiles/root/home/liveuser/oVirtLiveFiles/engine-setup-yad.sh M fedora/oVirtLiveFiles/engine-setup-yad.sh 2 files changed, 2 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/52679 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I910200ad2bd519c2794cce4879b2528a2f4073d4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-reports[master]: reports: fixed typo in capacity planning report br48
by sradco＠redhat.com 26 Jan '16

26 Jan '16

Shirly Radco has submitted this change and it was merged. Change subject: reports: fixed typo in capacity planning report br48 ...................................................................... reports: fixed typo in capacity planning report br48 Change-Id: I657889066ae7befadecc265a6497da598f696960 Bug-Url: https://bugzilla.redhat.com/1300240 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M packaging/ovirt-reports/resources/reports_resources/localization/ovirt_reports_bundle_en_US.properties.data 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Shirly Radco: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/52689 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I657889066ae7befadecc265a6497da598f696960 Gerrit-PatchSet: 1 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: mark fetchIsoDomains method as synchronized
by derez＠redhat.com 26 Jan '16

26 Jan '16

Daniel Erez has submitted this change and it was merged. Change subject: core: mark fetchIsoDomains method as synchronized ...................................................................... core: mark fetchIsoDomains method as synchronized Marking fetchIsoDomains method as synchronized in IsoDomainListSyncronizer (instead of addRepoFileToProblematicList/resetProblematicList methods) to avoid an unguarded read of problematicRepoFileList on fetchIsoDomains method. Coverity CID: 1348304 Change-Id: I257a9bb6a5c85013a633c7a46a1fcd7dbdb1fef0 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/IsoDomainListSyncronizer.java 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Verified Maor Lipchuk: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/52523 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I257a9bb6a5c85013a633c7a46a1fcd7dbdb1fef0 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Removed ClusterListModel's unused methods
by amureini＠redhat.com 26 Jan '16

26 Jan '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: Removed ClusterListModel's unused methods ...................................................................... core: Removed ClusterListModel's unused methods Change-Id: If65a1cee8f242a8174f58bbca605a551f8952c27 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java 1 file changed, 0 insertions(+), 42 deletions(-) Approvals: Tal Nisan: Looks good to me, but someone else must approve Alexander Wels: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52250 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If65a1cee8f242a8174f58bbca605a551f8952c27 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(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: 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[master]: restapi: Fix order of assert arguments in host nics test
by juan.hernandez＠redhat.com 25 Jan '16

25 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Fix order of assert arguments in host nics test ...................................................................... restapi: Fix order of assert arguments in host nics test The "expected" and "actual" arguments are swapped in one of the tests in the host nics test class. This makes the results confusing when they fail. This patch fixes that. Change-Id: I3fd0929219cf308389cdcbbd87a3a2ca6224fc46 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResourceTest.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Martin Mucha: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/52685 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3fd0929219cf308389cdcbbd87a3a2ca6224fc46 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Generate JAXB classes with V3 prefix
by juan.hernandez＠redhat.com 25 Jan '16

25 Jan '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Generate JAXB classes with V3 prefix ...................................................................... restapi: Generate JAXB classes with V3 prefix This patch changes the build process so that the JAXB classes are generated twice: once without prefix and onother time with the V3 prefix in the name. These prefixed classes will be used by later patches to support transforming objects between versions of the API. Change-Id: I81af2480431790a4681f7192c4ac0230b40fd8cc Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/pom.xml A backend/manager/modules/restapi/interface/definition/src/main/resources/v3/api.xjb M backend/manager/modules/restapi/interface/definition/src/main/resources/v3/api.xsd 3 files changed, 80 insertions(+), 58 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51733 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I81af2480431790a4681f7192c4ac0230b40fd8cc Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0