Dima Kuznetsov has posted comments on this change. Change subject: supervdsm: Decorators for supervdsm proxy calls. ......................................................................
Patch Set 2: The patches above this one demonstrate one such use. The case I'm trying to make is that supervdsm server contains a lot of unrelated to each other code, and I think it is better to have the logic dealing with a specific area in a specific module (that deals with this area). So I propose a decorator that upon invocation checks if it should be dispatched through supervdsm, and on the other end, supervdsm callback should call this function directly. To take it a bit further, we can pass not just the function name but also a module name and automatically import it and eliminate the need for specific callback in the supervdsm server code. -- To view, visit https://gerrit.ovirt.org/44598 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ibea2b07b91a7faffda16c71c6b5831f96d116c15 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dima Kuznetsov <dkuzn...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Dima Kuznetsov <dkuzn...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches