- Engine-commits - oVirt List Archives

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

Change in ovirt-node-plugin-vdsm[ovirt-3.6]: Revert "Disable engine registration if hosted engine is conf...
by dougsland＠redhat.com 25 Jan '16

25 Jan '16

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: Revert "Disable engine registration if hosted engine is configured" ...................................................................... Revert "Disable engine registration if hosted engine is configured" This reverts commit b18463d1992ac0c130c1b4d448e2eb9f8b7627c2. Change-Id: I2eba6f8d632f6ffe7f264447612acda9c4e3f829 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1301486 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Reviewed-on: https://gerrit.ovirt.org/52675 Continuous-Integration: Jenkins CI Reviewed-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> Reviewed-by: Ryan Barry <rbarry(a)redhat.com> Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com> Continuous-Integration: Fabian Deutsch <fabiand(a)redhat.com> Tested-by: Fabian Deutsch <fabiand(a)redhat.com> (cherry picked from commit 673ad1aabda99907d0d612e290098f8393011cef) Reviewed-on: https://gerrit.ovirt.org/52709 Continuous-Integration: Douglas Schilling Landgraf <dougsland(a)redhat.com> Tested-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M src/engine_page.py 1 file changed, 0 insertions(+), 26 deletions(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/52709 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2eba6f8d632f6ffe7f264447612acda9c4e3f829 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-node-plugin-vdsm[master]: Revert "Disable engine registration if hosted engine is conf...
by fabiand＠redhat.com 25 Jan '16

25 Jan '16

Fabian Deutsch has submitted this change and it was merged. Change subject: Revert "Disable engine registration if hosted engine is configured" ...................................................................... Revert "Disable engine registration if hosted engine is configured" This reverts commit b18463d1992ac0c130c1b4d448e2eb9f8b7627c2. Change-Id: I2eba6f8d632f6ffe7f264447612acda9c4e3f829 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1301486 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Reviewed-on: https://gerrit.ovirt.org/52675 Continuous-Integration: Jenkins CI Reviewed-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> Reviewed-by: Ryan Barry <rbarry(a)redhat.com> Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com> Continuous-Integration: Fabian Deutsch <fabiand(a)redhat.com> Tested-by: Fabian Deutsch <fabiand(a)redhat.com> --- M src/engine_page.py 1 file changed, 0 insertions(+), 26 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved; Passed CI tests Douglas Schilling Landgraf: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Ryan Barry: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/52675 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2eba6f8d632f6ffe7f264447612acda9c4e3f829 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in mom[master]: Release mom-0.5.2
by msivak＠redhat.com 25 Jan '16

25 Jan '16

Martin Sivák has submitted this change and it was merged. Change subject: Release mom-0.5.2 ...................................................................... Release mom-0.5.2 Change-Id: Idf9747a8f94b3b1ca8966a9ea8e7b9fadf24b363 Signed-off-by: Martin Sivak <msivak(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/52695 Continuous-Integration: Jenkins CI --- M configure.ac 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Martin Sivák: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/52695 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf9747a8f94b3b1ca8966a9ea8e7b9fadf24b363 Gerrit-PatchSet: 2 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0