| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-10-03 | |||
| 08:40:03 | ralonsoh | (sorry) | |
| 08:40:22 | ralonsoh | regarding WSGIServer. We have a similar implementation in Neutron (UnixDomainWSGIServer) | |
| 08:40:52 | ralonsoh | and is implemented with eventlet | |
| 08:41:15 | ralonsoh | I'm wondering if you have an alternative or if you have already started migrating it | |
| 09:02:00 | sean-k-mooney | ralonsoh: so we had 2 converstain on related topic. the first related to the nova-api consol script where we were considering entirly removing it and only provide wthe wsgi script allowing people to run it under uwsgi,apache2 mod_wsgi or gunicorn the alternitive was to to use one of python built in web servers | |
| 09:02:02 | sean-k-mooney | (https://docs.python.org/3/library/wsgiref.html#module-wsgiref.simple_server or https://docs.python.org/3/library/http.server.html#http.server.ThreadingHTTPServer) or use and external one like ( gunicorn or flask) | |
| 09:02:50 | sean-k-mooney | ralonsoh: the second converastaion was the wsgi framework to use instead fo eventlet and i was suggeign movign to falsk since keystone already did | |
| 09:03:36 | sean-k-mooney | there are others like fastapi ectra but flask is already used in openstack so why add another depency if we can avoid it | |
| 09:06:32 | sean-k-mooney | ralonsoh: i know neutron is in a slightly diffent place because you did not have a wsgi script and that was not the default way to run the api | |
| 09:07:10 | sean-k-mooney | ralonsoh: for nova the nova-api console script/binary has not been the recommended way to run it for 6+ years | |
| 09:07:45 | sean-k-mooney | in production and even in devstack the freestandign wsgi script is how that was executed so we were not using the eventlet web server | |
| 09:07:58 | opendevreview | Pierre Riteau proposed openstack/nova master: Fix missing backtick in configuration option help https://review.opendev.org/c/openstack/nova/+/931253 | |
| 09:15:27 | bauzas | ioni: just saw your message, could you try an older version of openstackclient by installing it in a venv ? | |
| 09:16:17 | bauzas | OSC changed a lot between 6.x and 7.x as it now uses the SDK but we may have another regression :( | |
| 10:02:35 | ralonsoh | sean-k-mooney, ok, I see. The WSGIServer is used in WSGIService and only for the API | |
| 10:03:49 | ralonsoh | we had a different problem: the UnixDomainWSGIServer is used on the agents to provide a wsgi server for the metadata | |
| 10:07:59 | sean-k-mooney | ralonsoh: i think you can just swap the socket passed to the http server with a unix domain socket instead of tcp | |
| 10:08:28 | ioni | @bauzas, ive downgraded python-openstackclient to 6.6.0, still the same issue | |
| 10:08:38 | ioni | should i downgrade sdk as well? | |
| 10:08:41 | sean-k-mooney | i have not tired that but i dont think the stdlib webserver actully depend on the transprot protocol | |
| 10:08:43 | ioni | to what version? | |
| 10:09:29 | ralonsoh | sean-k-mooney, UnixDomainWSGIServer is using a unix socket | |
| 10:09:39 | bauzas | ioni: well, could you then use DEBUG level with OSC to see the call and the return ? | |
| 10:10:26 | ioni | @bauzas, sure thing, can you give me instructions ? | |
| 10:14:24 | sean-k-mooney | ralonsoh: right but what i mean is i think they way you use https://docs.python.org/3/library/http.server.html#http.server.SimpleHTTPRequestHandler for example is you create a socket server use that to serve the web server | |
| 10:19:30 | ioni | bauzas, whats interesting is that in --debug, i can see details regarding servers on that hypervisor but it doesn't print them | |
| 10:20:17 | bauzas | ioni: so this is a OSC bug | |
| 10:20:26 | sean-k-mooney | ralonsoh: currently https://docs.python.org/3/library/socketserver.html only provide tcp and udp concreate implamtion but you coudl inherit form https://docs.python.org/3/library/socketserver.html#socketserver.BaseServer and create a unix variaint and use the https://docs.python.org/3/library/socketserver.html#socketserver.ThreadingMixIn if needed | |
| 10:20:48 | bauzas | ioni: can you please provide a paste ? | |
| 10:20:53 | ioni | sure | |
| 10:21:02 | sean-k-mooney | ralonsoh: there may be better wasy i.e. if flask has a unix server or one of our other exsitng deps does | |
| 10:21:31 | bauzas | (with the DEBUG level, but if you want, just modify the compute nodes) | |
| 10:22:26 | ioni | @bauzas, https://paste.xinu.at/ZFZ8ic/ | |
| 10:22:33 | ioni | @bauzas, is this what you need? | |
| 10:23:02 | bauzas | ioni: perfect | |
| 10:23:17 | ioni | @bauzas, i will delete the paste when you are done with it | |
| 10:23:19 | bauzas | stephenfin: apparently we have a OSC bug (even with 6.x) ^ | |
| 10:23:40 | ioni | that's with 7.1.2 but with 6.6.0 is the same behavior | |
| 10:27:02 | bauzas | ioni: very quickly before I go off, I can see that Nova returns all the instances | |
| 10:27:45 | bauzas | REQ: curl -g -i -X GET "https://xxx.cloud:8774/v2.1/servers/detail?flavor=None&image=None&name=None&project_id=None&reservation_id=None&status=None&user_id=None&deleted=False&host=cj201&ip=None&ip6=None&changes-since=None&changes-before=None&all_tenants=True" -H "Accept: application/json" -H "OpenStack-API-Version: compute 2.95" -H "User-Agent: openstacksdk/4.0.0 | |
| 10:27:47 | bauzas | keystoneauth1/5.8.0 python-requests/2.32.3 CPython/3.12.7" -H "X-Auth-Token: {SHA256}7be3037be38c6e0589e4a584f9d26b254f9fd49a435336b9f510b2fbfa64fa65" -H "X-OpenStack-Nova-API-Version: 2.95" | |
| 10:27:49 | bauzas | https://xxx.cloud:8774 "GET /v2.1/servers/detail?deleted=False&host=cj201&all_tenants=True HTTP/11" 200 171022 | |
| 10:28:22 | bauzas | but then OSC returns none of them | |
| 10:28:35 | ioni | yes | |
| 10:29:30 | ioni | osc bug, i have an old venv with older versions of pip packages and it works | |
| 10:29:40 | ioni | but not sure in which module of python is the bug | |
| 10:29:57 | bauzas | gtk | |
| 10:41:21 | stephenfin | ioni: bauzas: https://review.opendev.org/c/openstack/openstacksdk/+/929742 https://review.opendev.org/c/openstack/releases/+/930934 | |
| 10:42:28 | ioni | stephenfin, i' | |
| 10:42:33 | ioni | stephenfin, i'll wait for 4.0.1 | |
| 10:43:18 | ioni | stephenfin, but i don't understand who migrate affects a server list --host | |
| 10:43:50 | stephenfin | ah, wait, I saw host and read no further... | |
| 10:44:32 | ioni | my issue is that server list --all-projects --host cj201 for example doesnt return anything anymore with latest versions | |
| 10:44:45 | ioni | with --debug i see stuff there | |
| 10:47:32 | stephenfin | Ah, that's this https://review.opendev.org/c/openstack/python-openstackclient/+/926193 | |
| 10:47:36 | stephenfin | I thought we'd backported that... | |
| 10:48:07 | ioni | let me test that | |
| 10:48:10 | ioni | is just one line | |
| 10:49:45 | ioni | stephenfin, that was it | |
| 10:49:47 | ioni | thanks | |
| 10:50:15 | stephenfin | No worries. The backport is proposed now and will be out in 7.1.3 probably early next week https://review.opendev.org/c/openstack/python-openstackclient/+/931261 | |
| 10:50:52 | ioni | stephenfin, nice. i was worried that maybe 2023.1 doesn't work with latest openstackclient | |
| 10:52:42 | stephenfin | Nah, we just migrated all of the remaining novaclient usage in OSC to SDK in 2024.2 and our test coverage was not as good as we had thought :( There will likely be more small things uncovered but nothing that isn't an easy fix | |
| 10:58:42 | ioni | stephenfin, thank you | |
| 11:08:32 | opendevreview | Merged openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call https://review.opendev.org/c/openstack/nova/+/910627 | |
| 13:56:44 | opendevreview | Cyril Roelandt proposed openstack/python-novaclient master: [DNM] Replace glanceclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/931276 | |
| 15:07:24 | melwitt | ioni: looks like I have openstacksdk 3.3.0 so that's why I didn't see what you were seeing. glad yall were able to find what was causing the issue | |
| 15:25:53 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop usage of pkg_resources https://review.opendev.org/c/openstack/nova/+/931312 | |
| 20:00:23 | opendevreview | Cyril Roelandt proposed openstack/python-novaclient master: [DNM] Replace glanceclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/931276 | |
| 21:47:31 | opendevreview | Cyril Roelandt proposed openstack/python-novaclient master: [DNM] Replace glanceclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/931276 | |
| #openstack-nova - 2024-10-04 | |||
| 01:18:06 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop usage of pkg_resources https://review.opendev.org/c/openstack/nova/+/931312 | |
| 04:49:25 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/929719 | |
| 09:04:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Refactor obj_make_compatible to reduce complexity https://review.opendev.org/c/openstack/nova/+/928590 | |
| 09:04:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: [ovo]Add igb value to hw_vif_model image property https://review.opendev.org/c/openstack/nova/+/928456 | |
| 09:04:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: [doc]Developer doc about PCI and SRIOV testing https://review.opendev.org/c/openstack/nova/+/928834 | |
| 09:04:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: [libvirt]Support hw_vif_model = igb https://review.opendev.org/c/openstack/nova/+/931372 | |
| 09:08:16 | opendevreview | Balazs Gibizer proposed openstack/nova master: [libvirt]Support hw_vif_model = igb https://review.opendev.org/c/openstack/nova/+/928584 | |
| 09:08:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: [doc]Developer doc about PCI and SRIOV testing https://review.opendev.org/c/openstack/nova/+/928834 | |
| 09:43:27 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Add new option to enforce multipath volume connections https://review.opendev.org/c/openstack/nova/+/845660 | |
| 10:15:20 | opendevreview | Merged openstack/placement master: Add placement.wsgi.api module https://review.opendev.org/c/openstack/placement/+/919569 | |
| 10:47:02 | opendevreview | Amit Uniyal proposed openstack/nova master: Libvirt: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926521 | |
| 11:43:52 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Remove Python 3.8 support https://review.opendev.org/c/openstack/python-novaclient/+/931477 | |
| 11:47:36 | opendevreview | Balazs Gibizer proposed openstack/nova master: [doc]Fix the device_spec config doc about placement https://review.opendev.org/c/openstack/nova/+/931479 | |
| 11:48:13 | gibi | bauzas: sean-k-mooney: ^^ trivial doc fix | |
| 12:00:27 | opendevreview | Takashi Natsume proposed openstack/os-vif master: Remove Python 3.8 support https://review.opendev.org/c/openstack/os-vif/+/931480 | |
| 12:02:24 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Remove Python 3.8 support https://review.opendev.org/c/openstack/python-novaclient/+/931477 | |
| 12:02:54 | opendevreview | Takashi Natsume proposed openstack/os-vif master: Remove Python 3.8 support https://review.opendev.org/c/openstack/os-vif/+/931480 | |
| #openstack-nova - 2024-10-05 | |||
| 20:19:35 | opendevreview | Amit Uniyal proposed openstack/nova master: Libvirt: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926521 | |
| 20:19:35 | opendevreview | Amit Uniyal proposed openstack/nova master: VMware: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926522 | |
| 20:19:36 | opendevreview | Amit Uniyal proposed openstack/nova master: zvm: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926523 | |
| 20:19:36 | opendevreview | Amit Uniyal proposed openstack/nova master: Ironic: updates resource provider trait list https://review.opendev.org/c/openstack/nova/+/926524 | |
| #openstack-nova - 2024-10-06 | |||
| 05:50:00 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for 2025.1 Epoxy https://review.opendev.org/c/openstack/nova/+/929918 | |
| 11:25:42 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on netifaces https://review.opendev.org/c/openstack/nova/+/931582 | |
| 13:03:37 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on netifaces https://review.opendev.org/c/openstack/nova/+/931582 | |
| 13:27:37 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on netifaces https://review.opendev.org/c/openstack/nova/+/931582 | |
| 16:13:51 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on netifaces https://review.opendev.org/c/openstack/nova/+/931582 | |
| #openstack-nova - 2024-10-07 | |||
| 06:48:33 | opendevreview | Merged openstack/nova master: [doc]Fix the device_spec config doc about placement https://review.opendev.org/c/openstack/nova/+/931479 | |
| 09:55:07 | auniyal | HI sean-k-mooney, review reminder, can you please have another look at https://review.opendev.org/c/openstack/nova/+/926521 and https://review.opendev.org/c/openstack/nova/+/929858 | |
| 10:30:18 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.2: [doc]Fix the device_spec config doc about placement https://review.opendev.org/c/openstack/nova/+/931635 | |
| 10:30:58 | opendevreview | Balazs Gibizer proposed openstack/nova stable/2024.1: [doc]Fix the device_spec config doc about placement https://review.opendev.org/c/openstack/nova/+/931636 | |