- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: exttool: core: reduce log verbosity
by masayag＠redhat.com 16 Jul '15

16 Jul '15

Moti Asayag has submitted this change and it was merged. Change subject: exttool: core: reduce log verbosity ...................................................................... exttool: core: reduce log verbosity no need for milliseconds. the method name makes the output unreadable. Change-Id: Ib33b6586a361c80a4e698c69b3f0f870e5d2921b Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/conf/extensions-tool-logging.properties 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43699 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib33b6586a361c80a4e698c69b3f0f870e5d2921b 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: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: exttool: core: create Map for substitions variables
by masayag＠redhat.com 16 Jul '15

16 Jul '15

Moti Asayag has submitted this change and it was merged. Change subject: exttool: core: create Map for substitions variables ...................................................................... exttool: core: create Map for substitions variables Change-Id: Ib0457fa7581d355a82218047fddd2f8386fb7106 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/extensions-tool/src/main/java/org/ovirt/engine/exttool/core/ExtensionsToolExecutor.java M backend/manager/extensions-tool/src/main/java/org/ovirt/engine/exttool/core/ModuleService.java M backend/manager/extensions-tool/src/main/java/org/ovirt/engine/exttool/logger/LoggerServiceImpl.java 3 files changed, 25 insertions(+), 20 deletions(-) Approvals: Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43641 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib0457fa7581d355a82218047fddd2f8386fb7106 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: exttool: enable checkstyle and fix
by masayag＠redhat.com 16 Jul '15

16 Jul '15

Moti Asayag has submitted this change and it was merged. Change subject: exttool: enable checkstyle and fix ...................................................................... exttool: enable checkstyle and fix not sure why it was disabled, it was too quite... Change-Id: Ic29c808037a98cef1f1558d085117452b0c7c56d Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/extensions-tool/pom.xml M backend/manager/extensions-tool/src/main/java/org/ovirt/engine/exttool/core/ExtensionsToolExecutor.java M backend/manager/extensions-tool/src/main/java/org/ovirt/engine/exttool/core/ModuleService.java M backend/manager/extensions-tool/src/main/java/org/ovirt/engine/exttool/logger/LoggerServiceImpl.java 4 files changed, 12 insertions(+), 15 deletions(-) Approvals: Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43676 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic29c808037a98cef1f1558d085117452b0c7c56d Gerrit-PatchSet: 3 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: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: extapi: authn: logout should accept auth record
by masayag＠redhat.com 16 Jul '15

16 Jul '15

Moti Asayag has submitted this change and it was merged. Change subject: extapi: authn: logout should accept auth record ...................................................................... extapi: authn: logout should accept auth record this must have been an error in documentation, logout should accept an auth record, from which it can extract principal and other information such as session id. we can fix this now as there is no known usage of logout so far. Change-Id: I8616bebdcdb620e8d38c171e07af8c93edfd9b17 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/LogoutSessionCommand.java M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/aaa/Authn.java 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43691 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8616bebdcdb620e8d38c171e07af8c93edfd9b17 Gerrit-PatchSet: 3 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: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: handle failure in vms monitoring
by ofrenkel＠redhat.com 15 Jul '15

15 Jul '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: handle failure in vms monitoring ...................................................................... core: handle failure in vms monitoring currently if exception is thrown during vms monitoring (mostly during analyzing) the exception is not caught. since monitoring is always called from scheduler/event dispatcher, the only log is ~: [] Failed to invoke scheduled method perform: null which is useless so instead, changed VmsMonitoring.perform() to catch the exception and log it. Change-Id: I2c7c86f2b4b6d8041717b284c9dd07a84ada6969 Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VmsMonitoring.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Omer Frenkel: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43671 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2c7c86f2b4b6d8041717b284c9dd07a84ada6969 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: adding support for serial child commands execution
by laravot＠redhat.com 15 Jul '15

15 Jul '15

Liron Aravot has submitted this change and it was merged. Change subject: core: adding support for serial child commands execution ...................................................................... core: adding support for serial child commands execution This patch adds SerialChildCommandsExecutionCallback and SerialChildExecutingCommand. The SerialChildCommandsExecutionCallback is a callback that currently supports serial child execution until a failure is encountered or until the execution is finished. The use the callback the command needs to implement the SerialChildExecutingCommand interface. Change-Id: I729c97375525eaa2cf4274d4365765bcb10bd19e Signed-off-by: laravot(a)redhat.com <laravot(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SerialChildCommandsExecutionCallback.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SerialChildExecutingCommand.java 2 files changed, 73 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Liron Aravot: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/41699 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I729c97375525eaa2cf4274d4365765bcb10bd19e Gerrit-PatchSet: 16 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: adding ConcurrentChildCommandsExecutionCallback
by laravot＠redhat.com 15 Jul '15

15 Jul '15

Liron Aravot has submitted this change and it was merged. Change subject: core: adding ConcurrentChildCommandsExecutionCallback ...................................................................... core: adding ConcurrentChildCommandsExecutionCallback This patch adds the ConcurrentChildCommandsExecutionCallback callback. This callback is used to track the execution of number of child commands and call the endAction() method when the execution is done. Change-Id: I69d6d592a76e24466af32462ff9fc4f9c888b6f3 Signed-off-by: laravot(a)redhat.com <laravot(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ConcurrentChildCommandsExecutionCallback.java 1 file changed, 28 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Liron Aravot: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/41698 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I69d6d592a76e24466af32462ff9fc4f9c888b6f3 Gerrit-PatchSet: 16 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: support printing arrays in vdsm resposne
by ofrenkel＠redhat.com 15 Jul '15

15 Jul '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: support printing arrays in vdsm resposne ...................................................................... core: support printing arrays in vdsm resposne Change-Id: I88041fd06cd439af066294b988d5fc172c22b0d3 Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/xmlrpc/XmlRpcObjectDescriptor.java 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Omer Frenkel: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43612 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I88041fd06cd439af066294b988d5fc172c22b0d3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: adding ChildCommandsCallbackBase
by laravot＠redhat.com 15 Jul '15

15 Jul '15

Liron Aravot has submitted this change and it was merged. Change subject: core: adding ChildCommandsCallbackBase ...................................................................... core: adding ChildCommandsCallbackBase Change-Id: I1d5eb9ec92e20b1de5220e5b9101d93971f74487 Signed-off-by: laravot(a)redhat.com <laravot(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChildCommandsCallbackBase.java 1 file changed, 119 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Liron Aravot: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/42359 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1d5eb9ec92e20b1de5220e5b9101d93971f74487 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Get signed WebSocket proxy ticket over REST
by tjelinek＠redhat.com 15 Jul '15

15 Jul '15

Tomas Jelinek has submitted this change and it was merged. Change subject: restapi: Get signed WebSocket proxy ticket over REST ...................................................................... restapi: Get signed WebSocket proxy ticket over REST POST /vms/{vmid}/graphicsconsoles/{consoleid}/proxyticket <action/> - <action> <proxy_ticket> <value>ticket_content</value> </proxy_ticket> </action> Change-Id: I1701fee8cd36c714cd5ac0d28ef076c51531b9d6 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1181030 --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/VmGraphicsConsoleResource.java M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmGraphicsConsoleResource.java 4 files changed, 56 insertions(+), 0 deletions(-) Approvals: Jakub Niedermertl: Verified Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/42412 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1701fee8cd36c714cd5ac0d28ef076c51531b9d6 Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0