Skip to content
This repository has been archived by the owner on Nov 9, 2020. It is now read-only.

Print the VM name for which request is being processed. #456

Merged
merged 1 commit into from Jun 10, 2016

Conversation

kerneltime
Copy link
Contributor

This change prints the name for the VM that issued the request being served.

Testing:
make sure /etc/vmware/vmdkops/log_config.json is deleted
make deploy-esx
run docker volume ls

[root@promc-2n-dhcp24:~] 06/10/16 21:56:26 2795426 [Ubuntu 64-bit] [INFO   ] executeRequest 'list' completed with ret=[{u'Attributes': {}, u'Name': 'MyVolume'}, {u'Attributes': {}, u'Name': 'rg'}, {u'Attributes': {}, u'Name': 'radio2016'}, {u'Attributes': {}, u'Name': 'MyVolume2'}]

This change prints the name for the VM that issued the request being served.

Testing:
make sure /etc/vmware/vmdkops/log_config.json is deleted
make deploy-esx
run docker volume ls
```
[root@promc-2n-dhcp24:~] 06/10/16 21:56:26 2795426 [Ubuntu 64-bit] [INFO   ] executeRequest 'list' completed with ret=[{u'Attributes': {}, u'Name': 'MyVolume'}, {u'Attributes': {}, u'Name': 'rg'}, {u'Attributes': {}, u'Name': 'radio2016'}, {u'Attributes': {}, u'Name': 'MyVolume2'}]
```
@kerneltime
Copy link
Contributor Author

When we do use multiple threads this change should still work.
From: https://docs.python.org/2/library/threading.html

name
A string used for identification purposes only. It has no semantics. Multiple threads may be given the same name. The initial name is set by the constructor.

@andrewjstone
Copy link
Contributor

LGTM

@pdhamdhere
Copy link
Contributor

Nice. LGTM. Thanks!

@kerneltime kerneltime merged commit 6d7f217 into master Jun 10, 2016
@kerneltime kerneltime deleted the saymyname.kerneltime branch June 10, 2016 22:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants