Dan Kenigsberg has submitted this change and it was merged. Change subject: jsonrpc: shorten executor thread system names ......................................................................
jsonrpc: shorten executor thread system names We shorten the basename of the jsonrpc executor, because the system has a hard limit of 15 ASCII character per thread name. The new name is less expressive, but leaves room for the worker ID, which is an overall usability improvement. Before, it looked like jsonrpc.Executo now it looks like jsonrpc/0 Change-Id: Ibefc7d2c60d7b5d23739b129088f8e7f80514540 Signed-off-by: Francesco Romani <from...@redhat.com> Reviewed-on: https://gerrit.ovirt.org/61978 Reviewed-by: Nir Soffer <nsof...@redhat.com> Reviewed-by: Irit Goihman <igoih...@redhat.com> Continuous-Integration: Jenkins CI Reviewed-by: Piotr Kliczewski <piotr.kliczew...@gmail.com> --- M lib/vdsm/rpc/bindingjsonrpc.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Looks good to me, approved Nir Soffer: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Irit Goihman: Looks good to me, but someone else must approve Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/61978 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibefc7d2c60d7b5d23739b129088f8e7f80514540 Gerrit-PatchSet: 10 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Irit Goihman <igoih...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org