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

Reply via email to