Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UUID warning on ControllerPublishVolume #3

Closed
Akrog opened this issue Jun 26, 2018 · 0 comments
Closed

UUID warning on ControllerPublishVolume #3

Akrog opened this issue Jun 26, 2018 · 0 comments
Labels
bug Something isn't working

Comments

@Akrog
Copy link
Member

Akrog commented Jun 26, 2018

When we do a ControllerPublishVolume we get the following warning:

/home/geguileo/code/reuse-cinder-drivers/cinderlib-csi/.venv/lib/python2.7/site-packages/oslo_versionedobjects/fields.py:368: FutureWarning: u'think.localdomain' is an invalid UUID. Using UUIDFields with invalid UUIDs is no longer supported, and will be removed in a future release. Please update your code to input valid UUIDs or accept ValueErrors for invalid UUIDs. See https://docs.openstack.org/oslo.versionedobjects/latest/reference/fields.html#oslo_versionedobjects.fields.UUIDField for further details
  FutureWarning)

We shouldn't be getting any warnings unrelated to the CSI plugin.

@Akrog Akrog added the bug Something isn't working label Jun 26, 2018
@Akrog Akrog closed this as completed in cbfbf79 Jun 29, 2018
Akrog added a commit to Akrog/ember-csi that referenced this issue Mar 8, 2019
All concurrent requests are sharing the same request ID, so when we
receive a new request it overwrites the request ID of the previous
thread and they all start reporting the same ID.

Timeline:

- Request embercsi#1 comes in --> Start logging as embercsi#1
- Request embercsi#2 comes in --> Start logging as embercsi#2
- Request embercsi#1 logs as request embercsi#2
- Request embercsi#2 logs as request embercsi#2
- Request embercsi#3 comes in --> Start logging as embercsi#3
- Request embercsi#1 logs as request embercsi#3
- Request embercsi#2 logs as request embercsi#3

This is caused by not monkey patching the threading library soon enough
and the thread storage is actually using the storage of the native
thread instead of locations for each greenthread.

There is only 1 case where the code will fail with this change, and that
is if the `hostname` is not in /etc/hosts as the greenthread DNS library
does not check /etc/hostname, only /etc/hosts.

The solution is to either add the hostname to /etc/hosts or set
environmental variable `EVENTLET_NO_GREENDNS=yes`.  But we don't
recommend the environmental variable, since this may also affect our
drivers if they are using hostnames in their configurations.
Akrog added a commit to Akrog/ember-csi that referenced this issue Mar 12, 2019
All concurrent requests are sharing the same request ID, so when we
receive a new request it overwrites the request ID of the previous
thread and they all start reporting the same ID.

Timeline:

- Request embercsi#1 comes in --> Start logging as embercsi#1
- Request embercsi#2 comes in --> Start logging as embercsi#2
- Request embercsi#1 logs as request embercsi#2
- Request embercsi#2 logs as request embercsi#2
- Request embercsi#3 comes in --> Start logging as embercsi#3
- Request embercsi#1 logs as request embercsi#3
- Request embercsi#2 logs as request embercsi#3

This is caused by not monkey patching the threading library soon enough
and the thread storage is actually using the storage of the native
thread instead of locations for each greenthread.

There is only 1 case where the code will fail with this change, and that
is if the `hostname` is not in /etc/hosts as the greenthread DNS library
does not check /etc/hostname, only /etc/hosts.

The solution is to either add the hostname to /etc/hosts or set
environmental variable `EVENTLET_NO_GREENDNS=yes`.  But we don't
recommend the environmental variable, since this may also affect our
drivers if they are using hostnames in their configurations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant