September 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6]: packaging: setup: don't re-add dropped VdcOptions
by piotr.kliczewski＠gmail.com 18 Sep '15

18 Sep '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: setup: don't re-add dropped VdcOptions ...................................................................... packaging: setup: don't re-add dropped VdcOptions https://gerrit.ovirt.org/34693 dropped deprecated and unused VdcOptions from db. Avoid to re-add them during setup. Bug-Url: https://bugzilla.redhat.com/1254961 Change-Id: I87e4364f80985471937a99eaeba0b31c26b75a16 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 73811cb2f154b5ff475db7867441a7fd9531873a) --- M packaging/setup/ovirt_engine_setup/engine/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/__init__.py D packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/macrange.py 3 files changed, 0 insertions(+), 90 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved; Passed CI tests Martin Mucha: Verified -- To view, visit https://gerrit.ovirt.org/46349 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87e4364f80985471937a99eaeba0b31c26b75a16 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: automation: dropped el6 dependencies
by sbonazzo＠redhat.com 18 Sep '15

18 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: automation: dropped el6 dependencies ...................................................................... automation: dropped el6 dependencies Change-Id: Ied14916412adc1c3a5820db069f3a599ed2b16d7 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- D automation/build-artifacts.packages.el6 1 file changed, 0 insertions(+), 8 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46351 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ied14916412adc1c3a5820db069f3a599ed2b16d7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master 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]: core: Remove permanent generation options
by sbonazzo＠redhat.com 18 Sep '15

18 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: core: Remove permanent generation options ...................................................................... core: Remove permanent generation options The permanent generation has been removed in Java 8, thus the "PermSize" and "MaxPermSize" options don't have any meaning. Change-Id: Ic41c25e15d438d40a93f8db04c2d44a5b35c92ef Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M README.developer M automation/build-artifacts.sh M automation/check-patch.sh M frontend/webadmin/modules/pom.xml M packaging/services/ovirt-engine/ovirt-engine.conf.in M packaging/services/ovirt-engine/ovirt-engine.py 7 files changed, 3 insertions(+), 15 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46284 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic41c25e15d438d40a93f8db04c2d44a5b35c92ef Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: setup: don't re-add dropped VdcOptions
by sbonazzo＠redhat.com 18 Sep '15

18 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: don't re-add dropped VdcOptions ...................................................................... packaging: setup: don't re-add dropped VdcOptions https://gerrit.ovirt.org/34693 dropped deprecated and unused VdcOptions from db. Avoid to re-add them during setup. Bug-Url: https://bugzilla.redhat.com/1254961 Change-Id: I87e4364f80985471937a99eaeba0b31c26b75a16 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/engine/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/__init__.py D packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/macrange.py 3 files changed, 0 insertions(+), 90 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Eli Mesika: Looks good to me, but someone else must approve Martin Mucha: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/45743 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87e4364f80985471937a99eaeba0b31c26b75a16 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(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-engine[master]: webadmin: don't set watchdog action to null
by tjelinek＠redhat.com 18 Sep '15

18 Sep '15

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: don't set watchdog action to null ...................................................................... webadmin: don't set watchdog action to null It is not correct to set it to null - it renders as an empty value which is not correct - better just to disable the field. Change-Id: I2792c7b4b50d402fa6cb9622f5d3174b2653853b Bug-Url: https://bugzilla.redhat.com/1261015 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java 1 file changed, 0 insertions(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Jakub Niedermertl: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46348 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2792c7b4b50d402fa6cb9622f5d3174b2653853b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Require Java 8
by sbonazzo＠redhat.com 18 Sep '15

18 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: core: Require Java 8 ...................................................................... core: Require Java 8 This patch changes the build instructions and the RPM .spec so that they requires Java 8 both for building and running. This is needed to be able to introduce new components that will require Java 8. Change-Id: Ieb3f2a49e42a826bba8178559f2d802217dfbc74 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M README.developer M ovirt-engine.spec.in 2 files changed, 3 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46271 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb3f2a49e42a826bba8178559f2d802217dfbc74 Gerrit-PatchSet: 2 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-release[master]: ovirt-release-master: move to 4.0 development
by sbonazzo＠redhat.com 18 Sep '15

18 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-release-master: move to 4.0 development ...................................................................... ovirt-release-master: move to 4.0 development - Bumped version to 002 for 4.0 release cycle - Dropped EL 6 support Change-Id: I6fb1bf652e4c933b41c2beb884928b71bc37e104 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-release-master/Makefile.am M ovirt-release-master/configure.ac D ovirt-release-master/ovirt-el6-deps.repo.in M ovirt-release-master/ovirt-release-master.spec.in 4 files changed, 8 insertions(+), 56 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests David Caro: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46340 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6fb1bf652e4c933b41c2beb884928b71bc37e104 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(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: VM-like dialogs cleanup
by piotr.kliczewski＠gmail.com 18 Sep '15

18 Sep '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: VM-like dialogs cleanup ...................................................................... webadmin: VM-like dialogs cleanup Aligning widgets of VM, Templete and VM Pool dialog, tabs General, System, Initial Run, Console. Bug-Url: https://bugzilla.redhat.com/1231159 Change-Id: I51db16d7d89fa9eeca2ea0778192c8d9b42d4430 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/EntityModelWidgetWithInfo.ui.xml A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/GenericWidgetWithLabel.java A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/GenericWidgetWithLabel.ui.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/dialog/tab/DialogTabPanel.ui.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/generic/EntityModelDetachableWidget.ui.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/generic/EntityModelDetachableWidgetWithLabel.ui.xml 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 8 files changed, 143 insertions(+), 33 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified Jenkins CI: Passed CI tests Michal Skrivanek: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46320 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I51db16d7d89fa9eeca2ea0778192c8d9b42d4430 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: ovirt-engine: drop el6 on master
by sbonazzo＠redhat.com 18 Sep '15

18 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine: drop el6 on master ...................................................................... ovirt-engine: drop el6 on master On 4.0 engine won't be supported on el6. Change-Id: I77b4eb20b57e9b89bb66a45e69b836bf8b71a68a Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml M jobs/confs/projects/ovirt-image-uploader/ovirt-image-uploader_standard.yaml M jobs/confs/projects/ovirt-iso-uploader/ovirt-iso-uploader_create-rpms.yaml M jobs/confs/projects/ovirt-log-collector/ovirt-log-collector_create-rpms.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-master.yaml M jobs/confs/yaml/builders/copy-create-job-artifact.yaml 6 files changed, 52 insertions(+), 28 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests David Caro: Looks good to me -- To view, visit https://gerrit.ovirt.org/45170 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I77b4eb20b57e9b89bb66a45e69b836bf8b71a68a Gerrit-PatchSet: 9 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(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: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Introduce AsserJ for test assertions
by sbonazzo＠redhat.com 18 Sep '15

18 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: engine: Introduce AsserJ for test assertions ...................................................................... engine: Introduce AsserJ for test assertions AssertJ is a very useful assertion library which has the extra advantages of full autocomplete support in your favourite IDE with fluent and very readable assertions. Examples: assertThat(list).isNotNull().isEmpty(); assertThat(list).hasSize(15).contains(1,2).doesNotContain(5) assertThat(listA).hasSameSizeAs(listb) assertThat(objectA).isEqualTo(objectB) assertThat(objectA).isSameAs(objectB) ... It is also easy to create domain specific assertions which can be used the same way. See http://joel-costigliola.github.io/assertj/ for further details. Change-Id: I750249be3e966b721be6a6a56a1d3a281c312179 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesEnforcerTest.java M ovirt-engine.spec.in M pom.xml 3 files changed, 21 insertions(+), 16 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Roman Mohr: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45980 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I750249be3e966b721be6a6a56a1d3a281c312179 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0