October 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: migrate the LoggedUtils into slf4j
by yzaslavs＠redhat.com 21 Oct '14

21 Oct '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: migrate the LoggedUtils into slf4j ...................................................................... core: migrate the LoggedUtils into slf4j Keeping some backward compatibility within VdcCommandBase, IrsBrokerCommand. Bug-Url: https://bugzilla.redhat.com/1109871 Change-Id: I3e17eeb0670c456a39f354e766a771e69d160002 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/VdcCommandBase.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/log/LoggedUtils.java M backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/log/LoggedUtilsTest.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java 4 files changed, 117 insertions(+), 109 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34295 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3e17eeb0670c456a39f354e766a771e69d160002 Gerrit-PatchSet: 5 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: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: services: drop ovirt logger
by yzaslavs＠redhat.com 21 Oct '14

21 Oct '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: services: drop ovirt logger ...................................................................... services: drop ovirt logger Bug-Url: https://bugzilla.redhat.com/1109871 Change-Id: I8cb2bdf2bc0894b22e3ef320cc79fe193f31d8d7 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/services/src/main/java/org/ovirt/engine/core/services/GetSessionUser.java M backend/manager/modules/services/src/main/java/org/ovirt/engine/core/services/HealthStatus.java M backend/manager/modules/services/src/main/java/org/ovirt/engine/core/services/RegisterServlet.java 3 files changed, 45 insertions(+), 52 deletions(-) Approvals: Alon Bar-Lev: Verified Martin Peřina: Looks good to me, but someone else must approve Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34296 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8cb2bdf2bc0894b22e3ef320cc79fe193f31d8d7 Gerrit-PatchSet: 1 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: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: fixing development environment
by sbonazzo＠redhat.com 21 Oct '14

21 Oct '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fixing development environment ...................................................................... packaging: setup: fixing development environment Fixing imports and external tool usage for notroot developing environment. Change-Id: Ifa388eeec683ec2ec4f7e3b3949e296f2078180e Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1109326 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/system/nfs.py 1 file changed, 11 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified oVirt Jenkins CI Server: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/34257 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifa388eeec683ec2ec4f7e3b3949e296f2078180e Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: fixing development environment on remote e...
by sbonazzo＠redhat.com 21 Oct '14

21 Oct '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fixing development environment on remote engine ...................................................................... packaging: setup: fixing development environment on remote engine Fixing imports for notroot developing environment. Change-Id: I9b7e75dbf8ffe7a8c251232648685b0d6dbf592b Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-common/base/remote_engine/remote_engine_root_ssh.py 1 file changed, 11 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/34259 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b7e75dbf8ffe7a8c251232648685b0d6dbf592b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-cli[cli_3.5]: packaging: spec: fix fedora build requirements
by juan.hernandez＠redhat.com 21 Oct '14

21 Oct '14

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: spec: fix fedora build requirements ...................................................................... packaging: spec: fix fedora build requirements In Fedora < 21 and EL >= 6 python-setuptools-devel was provided by python-setuptools. In Fedora 21 such package is not provided anymore. Also fixed URL for Source. Change-Id: I2b1ff672bb6dd4adaf3695bb1ab87d0f2f0f8d64 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 3a2d3c478a9f9746259aac47e7286daa0f16e831) --- M ovirt-engine-cli.spec.in 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified Juan Hernandez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34307 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b1ff672bb6dd4adaf3695bb1ab87d0f2f0f8d64 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> 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[ovirt-engine-3.5]: packaging: setup: Use common code for remote wsp pki
by tnisan＠redhat.com 21 Oct '14

21 Oct '14

Tal Nisan has submitted this change and it was merged. Change subject: packaging: setup: Use common code for remote wsp pki ...................................................................... packaging: setup: Use common code for remote wsp pki Code to handle PKI stuff with remote engine has been moved to a common place to be shared between different plugins. Using it. Change-Id: Id72711fb1577d5ee97454a36650456c2675206be Bug-Url: https://bugzilla.redhat.com/1125188 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/websocket_proxy/constants.py M packaging/setup/plugins/ovirt-engine-setup/websocket_proxy/config.py M packaging/setup/plugins/ovirt-engine-setup/websocket_proxy/pki.py 3 files changed, 200 insertions(+), 282 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified -- To view, visit http://gerrit.ovirt.org/33941 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id72711fb1577d5ee97454a36650456c2675206be Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Added extra gluster repos for vdsm
by dcaroest＠redhat.com 21 Oct '14

21 Oct '14

David Caro has submitted this change and it was merged. Change subject: Added extra gluster repos for vdsm ...................................................................... Added extra gluster repos for vdsm Added to el6 and el7 as it started failing with the rpms from the centos updates repo Change-Id: Ieb624afd8b313113f62d6d8a2287af109e7ac11a Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/yaml/jobs/vdsm/vdsm_create-rpms.yaml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: David Caro: Verified; Looks good to me, approved Dan Kenigsberg: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/34298 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb624afd8b313113f62d6d8a2287af109e7ac11a Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Ohad Basan <obasan(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-cli[master]: packaging: spec: fix fedora build requirements
by juan.hernandez＠redhat.com 21 Oct '14

21 Oct '14

Juan Hernandez has submitted this change and it was merged. Change subject: packaging: spec: fix fedora build requirements ...................................................................... packaging: spec: fix fedora build requirements In Fedora < 21 and EL >= 6 python-setuptools-devel was provided by python-setuptools. In Fedora 21 such package is not provided anymore. Also fixed URL for Source. Change-Id: I2b1ff672bb6dd4adaf3695bb1ab87d0f2f0f8d64 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M ovirt-engine-cli.spec.in 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified Juan Hernandez: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/34301 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2b1ff672bb6dd4adaf3695bb1ab87d0f2f0f8d64 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> 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]: utils: suppress compiler warnings (serial)
by yzaslavs＠redhat.com 21 Oct '14

21 Oct '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: utils: suppress compiler warnings (serial) ...................................................................... utils: suppress compiler warnings (serial) Change-Id: I2cf34e4b41e77a5bdc99235a2411f213aebd5ddd Signed-off-by: Tim Speetjens <tim.speetjens(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/collections/DefaultValueMap.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/serialization/json/JsonVmPayloadMixIn.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ssl/AuthSSLInitializationException.java 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/33742 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2cf34e4b41e77a5bdc99235a2411f213aebd5ddd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tim Speetjens <tim.speetjens(a)gmail.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: AuditLogDirectorTest 'revisit'.
by yzaslavs＠redhat.com 21 Oct '14

21 Oct '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: core: AuditLogDirectorTest 'revisit'. ...................................................................... core: AuditLogDirectorTest 'revisit'. code was commented out 2 years ago --> this is dead code. -removing unused code -removing TOTO Change-Id: Ibf2d2358aa6e22ce10795c6f22b3874c43e814ab Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dal/dbbroker/auditloghandling/AuditLogDirectorTest.java 1 file changed, 1 insertion(+), 62 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/29590 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibf2d2358aa6e22ce10795c6f22b3874c43e814ab Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> 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: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0