Yair Zaslavsky has uploaded a new change for review. Change subject: aaa: adding correlation Id arguments at IrsBroker and VdsBroker This is the first patch towards removing corelation id from ThreadLocal. ......................................................................
aaa: adding correlation Id arguments at IrsBroker and VdsBroker This is the first patch towards removing corelation id from ThreadLocal. The following changes were introduced: 1. Adding corelationId to existing interaces and wrappers 2. Adding XXXClient intefaces - to let the vds and irs commands work with a similar interface as before. 3. Creating a dynamic proxy that conforms to the XXXClient interface and recives the corelation id + the correspoding broker object during creation. the invocatio nof the proxy finds the proper method at the broker, and adds the corelation id as argument. 4. Changes at XmlRpcUtils to extract the http header from the parameters and not from thread local and to send to vdsm one less parameter (i.e - without the correlation id). Change-Id: I828ad7a6e6ba64ed59bd935f500a9ed008244d6f Topic: AAA Signed-off-by: Yair Zaslavsky <yzasl...@redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ThreadLocalParamsContainer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IIrsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsBrokerCommand.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsClient.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsServerConnector.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/irsbroker/IrsServerWrapper.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcIIrsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/jsonrpc/JsonRpcVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/IVdsServer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerCommand.java A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsClient.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerConnector.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsServerWrapper.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/xmlrpc/XmlRpcUtils.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SetupNetworksVDSCommandTest.java 15 files changed, 1,433 insertions(+), 805 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/66/29366/1 -- To view, visit http://gerrit.ovirt.org/29366 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I828ad7a6e6ba64ed59bd935f500a9ed008244d6f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
