March 2016 - Engine-commits - oVirt List Archives

Change in ovirt-host-deploy[master]: vdsm: packages: drop legacy configuration
by sbonazzo＠redhat.com 24 Mar '16

24 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: vdsm: packages: drop legacy configuration ...................................................................... vdsm: packages: drop legacy configuration `vdsm-tool configure` was introduced in ovirt-3.4. We no longer support hosts older than that, so we can drop this stale code. Change-Id: I8646cd55b03226ee49f6be97da69af28a3c11099 Signed-off-by: Dan Kenigsberg <danken(a)redhat.com> --- M src/plugins/ovirt-host-deploy/vdsm/packages.py 1 file changed, 10 insertions(+), 27 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55040 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8646cd55b03226ee49f6be97da69af28a3c11099 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in otopi[master]: pep8: E701 multiple statements on one line (colon)
by didi＠redhat.com 24 Mar '16

24 Mar '16

Yedidyah Bar David has submitted this change and it was merged. Change subject: pep8: E701 multiple statements on one line (colon) ...................................................................... pep8: E701 multiple statements on one line (colon) Change-Id: I5aebd4c9243ea56da09abf5866f0e5def4195e58 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M src/plugins/otopi/debug/force_fail/force_fail.py M src/plugins/otopi/debug/wait_on_error/wait_on_error.py 2 files changed, 4 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/55187 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5aebd4c9243ea56da09abf5866f0e5def4195e58 Gerrit-PatchSet: 1 Gerrit-Project: otopi 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fix of audit log message in UpdateVmCommand
by ahadas＠redhat.com 24 Mar '16

24 Mar '16

Arik Hadas has submitted this change and it was merged. Change subject: core: Fix of audit log message in UpdateVmCommand ...................................................................... core: Fix of audit log message in UpdateVmCommand Fix of copy-paste mistake: AuditLogType.FAILED_HOT_SET_NUMBER_OF_CPUS -> AuditLogType.FAILED_HOT_SET_MEMORY. Change-Id: I16a3a7dcf02f977ab626257d19e39f86785b0cc6 Bug-Url: https://bugzilla.redhat.com/1311899 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jakub Niedermertl: Verified Arik Hadas: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/55028 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I16a3a7dcf02f977ab626257d19e39f86785b0cc6 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: engine: Allow Html5 as one of the default spice modes
by tnisan＠redhat.com 24 Mar '16

24 Mar '16

Tal Nisan has submitted this change and it was merged. Change subject: engine: Allow Html5 as one of the default spice modes ...................................................................... engine: Allow Html5 as one of the default spice modes Html5 is added among allowed default spice modes for engine-config. Change-Id: I7f6439cd67f48a3eb38703dc1414805cdecbe9c2 Bug-Url: https://bugzilla.redhat.com/1311408 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M packaging/etc/engine-config/engine-config.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Marek Libra: Verified -- To view, visit https://gerrit.ovirt.org/55130 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7f6439cd67f48a3eb38703dc1414805cdecbe9c2 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(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

Change in ovirt-release[ovirt-3.6]: build: post ovirt-release36-007
by sbonazzo＠redhat.com 24 Mar '16

24 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-release36-007 ...................................................................... build: post ovirt-release36-007 Change-Id: I1042a36802e70ae5d47e8c30f2b9dba34af67173 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M automation/check-patch.sh M configure.ac M ovirt-release36.spec.in 4 files changed, 10 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55178 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1042a36802e70ae5d47e8c30f2b9dba34af67173 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-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-release[ovirt-3.6]: build: ovirt-release36-007
by sbonazzo＠redhat.com 24 Mar '16

24 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-release36-007 ...................................................................... build: ovirt-release36-007 Change-Id: I3c8cad6089ee369c8386b4c68e75f67e6bd509f8 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M automation/check-patch.sh M configure.ac M ovirt-release36.spec.in 4 files changed, 6 insertions(+), 9 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55177 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3c8cad6089ee369c8386b4c68e75f67e6bd509f8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-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-release[ovirt-3.6]: master: fix Virt SIG URL for EL7 Server
by sbonazzo＠redhat.com 24 Mar '16

24 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: master: fix Virt SIG URL for EL7 Server ...................................................................... master: fix Virt SIG URL for EL7 Server Fix the URL for RHEL 7.x Server variant. Change-Id: I36e3de45aaa5cceee1ef10560cd73f4bab2af50f Bug-Url: https://bugzilla.redhat.com/1294083 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-el7-deps.repo.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55176 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I36e3de45aaa5cceee1ef10560cd73f4bab2af50f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-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]: build: animal-sniffer in compile time
by amureini＠redhat.com 23 Mar '16

23 Mar '16

Allon Mureinik has submitted this change and it was merged. Change subject: build: animal-sniffer in compile time ...................................................................... build: animal-sniffer in compile time animal-sniffer is an important check that prevents future breakages in GWT. As it's no longer run in CI, and as it takes a negligible time to run this patch makes sure it will be run as part of the build, and is governed by a property (like, e.g., checkstyle, which is much more expensive) Change-Id: I678cf246a446f2146de93b5f96a35ec0091234fc Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/common/pom.xml M backend/manager/modules/compat/pom.xml M backend/manager/modules/extensions-api-root/pom.xml M backend/manager/modules/searchbackend/pom.xml M build-tools-root/ovirt-findbugs-filters/pom.xml M frontend/webadmin/modules/pom.xml M pom.xml 7 files changed, 44 insertions(+), 98 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Allon Mureinik: Verified; Passed CI tests Vojtech Szocs: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/54997 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I678cf246a446f2146de93b5f96a35ec0091234fc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[master]: packaging: updated AUTHORS
by sbonazzo＠redhat.com 23 Mar '16

23 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: updated AUTHORS ...................................................................... packaging: updated AUTHORS Change-Id: Ic76fb07be2dd377c76078e728d949ecf6c95df30 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M AUTHORS 1 file changed, 25 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/55149 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic76fb07be2dd377c76078e728d949ecf6c95df30 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-host-deploy 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[master]: packaging: gettext: use 0.18.2 or latest
by sbonazzo＠redhat.com 23 Mar '16

23 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: gettext: use 0.18.2 or latest ...................................................................... packaging: gettext: use 0.18.2 or latest Use gettext 0.18.2 or newest: gettext 0.19.6 introduces a new macro, allowing to use latest gettext provided it's newer than requires See: https://lists.gnu.org/archive/html/bug-gettext/2015-08/msg00026.html - Solved: autoreconf: running: automake --add-missing --copy --force-missing configure.ac:39: warning: The 'AM_PROG_MKDIR_P' macro is deprecated, and its use is discouraged. configure.ac:39: You should use the Autoconf-provided 'AC_PROG_MKDIR_P' macro instead, configure.ac:39: and use '$(MKDIR_P)' instead of '$(mkdir_p)'in your Makefile.am files. - ignore autogenerated intl directory - Drop requirement on GCC for having gettext support, we're using Python. Change-Id: I32a81cc3e3892aa0e7096d0694ebc2346e465aa4 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M .gitignore M configure.ac M ovirt-host-deploy.spec.in M po/Makevars M po/POTFILES.in 5 files changed, 50 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/55148 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I32a81cc3e3892aa0e7096d0694ebc2346e465aa4 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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: gerrit-hooks <automation(a)ovirt.org>

1 0