| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-20 | |||
| 09:34:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix neutron client dict grabbing https://review.opendev.org/c/openstack/nova/+/952966 | |
| 09:39:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix neutron client dict grabbing https://review.opendev.org/c/openstack/nova/+/952966 | |
| 10:36:30 | sean-k-mooney | gibi: so i wondered about doing something like that as well but i expected the behaivor to be the same __getattr__ is more convetional so that nice | |
| 10:36:51 | sean-k-mooney | have you tested this in devstack? | |
| 10:37:56 | sean-k-mooney | ill quickly rebase my test patch but it was faillng on a weired mysl isse so its proably goign to fail for unrelated reasons | |
| 10:41:47 | gibi | sean-k-mooney: yepp I tested it in devstack locally | |
| 10:42:03 | opendevreview | sean mooney proposed openstack/nova master: [DNM] testing py3.13 eventlet bug workaround https://review.opendev.org/c/openstack/nova/+/951749 | |
| 10:42:37 | gibi | the cpython bug is triggered by having two refs on the internal __dict__ so __getattr__ removes that | |
| 10:42:39 | sean-k-mooney | gibi: ok i was tryign to test it with ^ but for some reason i was getting mysql issues iwth the iniall root passward config | |
| 10:43:02 | sean-k-mooney | gibi: i see how did you find that | |
| 10:43:47 | sean-k-mooney | did you find the cpython bug through searching the web | |
| 10:44:10 | gibi | it wasn't me. It was jkulik who managed to create the small reproduction and then engaged with the python community | |
| 10:44:42 | gibi | the discussion happened on eventlet-removal channel as first it was thought that this is yet another eventlet induced bug | |
| 10:45:29 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only apply "soft" additionalProperties validation to requests https://review.opendev.org/c/openstack/nova/+/952936 | |
| 10:45:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volumes APIs https://review.opendev.org/c/openstack/nova/+/952348 | |
| 10:45:30 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349 | |
| 10:45:31 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volume attachments APIs https://review.opendev.org/c/openstack/nova/+/952350 | |
| 10:45:31 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use valid UUIDs for cinder resources https://review.opendev.org/c/openstack/nova/+/952971 | |
| 10:45:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for floating IP APIs https://review.opendev.org/c/openstack/nova/+/952972 | |
| 10:45:32 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for security group APIs https://review.opendev.org/c/openstack/nova/+/952973 | |
| 10:45:49 | sean-k-mooney | right i spent 4 or 5 hours at the weekend tyring to create a repoducer for this | |
| 10:47:12 | sean-k-mooney | i even through a bunch of AIs at it including gemini deep reasearch https://docs.google.com/document/d/10U6wnSO2vUOZjjHJQvd0Mfei2Kep-uho1LxJIPC_0VQ/edit?usp=sharing | |
| 11:04:54 | gibi | that is a long doc :) | |
| 11:56:37 | elodilles | hi o/ i know it's Friday, but may i get a core review to this trivial gate fix (python-novaclient)? o:) (docs job is broken in Zuul without this) https://review.opendev.org/c/openstack/python-novaclient/+/947690 | |
| 12:04:56 | elodilles | got a +W, thanks! o:) | |
| 12:28:27 | sean-k-mooney | elodilles: the patch is fine even if we have existing tech debt there | |
| 12:28:36 | sean-k-mooney | we ar enot properly tagging things in novaclient in general | |
| 12:28:49 | sean-k-mooney | i.e. using doc and pdf-docs | |
| 12:28:59 | sean-k-mooney | even in nova we coudl do a better job of that | |
| 12:30:04 | sean-k-mooney | we have a bunch of things that are not tagged or using the wrogn tag in novas bindep https://review.opendev.org/c/openstack/nova/+/951749/9/bindep.txt | |
| 12:43:33 | elodilles | sean-k-mooney: true. to be honest, since we just extending the bindep here and python-novaclient is not that project that gets tons of patches nowadays regularly i was just happy that the gate is working with the fix o:) but you are right, probably it would be better to just tag libpcre with 'doc' tag to only get it installed in docs job | |
| 12:43:58 | sean-k-mooney | yep an i +2'd it because the chagne is fine | |
| 12:44:13 | sean-k-mooney | i was jsut pointing that out as a nit mroe then anything else to acknowlage the tech debt | |
| 12:44:32 | elodilles | +1 | |
| 12:45:01 | sean-k-mooney | the one that annoys me most in nova is that we dont tag mysql and postgress with test | |
| 12:45:20 | sean-k-mooney | i dont nessiarly alwasy want to install multipe database on my laptop | |
| 12:45:22 | sean-k-mooney | or vm | |
| 12:45:51 | elodilles | ah, true. :/ | |
| 12:46:27 | sean-k-mooney | when i want to run the opertunistic test locally i normaly use a devstack env | |
| 13:15:40 | bauzas | is it me or review.opendev.org is b0rken ? | |
| 13:17:22 | elodilles | looks fine to me | |
| 13:19:37 | priteau | bauzas: I have issues too, I can SSH to it but I can SSH to it manually, I can pull, but I can't submit a patch with git-review. | |
| 13:19:52 | bauzas | hmmm | |
| 13:19:54 | priteau | It only started very recently. Are you with Orange too? | |
| 13:20:19 | bauzas | nope, I use the Free ISP | |
| 13:20:31 | priteau | So probably not a peering issue | |
| 13:20:48 | priteau | elodilles: have you tried to submit a patch recently? | |
| 13:21:25 | bauzas | https://downforeveryoneorjustme.com/review.opendev.org | |
| 13:23:07 | elodilles | priteau: no, this time i just checked review.o.o in browser | |
| 13:28:14 | bauzas | gibi: really sorry, wanted to review your eventlet series :( | |
| 13:28:50 | opendevreview | Merged openstack/python-novaclient master: Add libpcre3-dev in bindep.txt for pcre.h https://review.opendev.org/c/openstack/python-novaclient/+/947690 | |
| 13:33:44 | opendevreview | Elod Illes proposed openstack/python-novaclient master: DNM: gate health test https://review.opendev.org/c/openstack/python-novaclient/+/952928 | |
| 13:35:30 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop explicit setting of iommu for AMD SEV support https://review.opendev.org/c/openstack/nova/+/909635 | |
| 13:35:40 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: tests: Remove support for running against keystone v2 https://review.opendev.org/c/openstack/python-novaclient/+/952984 | |
| 13:35:40 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: shell: Hard fail if v2.0 URL detected https://review.opendev.org/c/openstack/python-novaclient/+/952985 | |
| 13:35:41 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Replace keystoneclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/952986 | |
| 13:35:41 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Use pre-commit for lint jobs https://review.opendev.org/c/openstack/python-novaclient/+/952987 | |
| 13:35:57 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for security group APIs https://review.opendev.org/c/openstack/nova/+/952973 | |
| 13:57:44 | opendevreview | Stephen Finucane proposed openstack/nova-specs master: Add flavor-search-by-name spec https://review.opendev.org/c/openstack/nova-specs/+/940440 | |
| 14:39:42 | opendevreview | Merged openstack/python-novaclient master: Replace cinderclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/931961 | |
| 14:39:44 | opendevreview | Merged openstack/python-novaclient master: Replace neutronclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/932057 | |
| 15:38:45 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: tests: Remove support for running against keystone v2 https://review.opendev.org/c/openstack/python-novaclient/+/952984 | |
| 15:38:46 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: shell: Hard fail if v2.0 URL detected https://review.opendev.org/c/openstack/python-novaclient/+/952985 | |
| 15:38:46 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Replace keystoneclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/952986 | |
| 15:38:47 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Use pre-commit for lint jobs https://review.opendev.org/c/openstack/python-novaclient/+/952987 | |
| 15:40:46 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: tests: Stop "testing" against Identity v2 API https://review.opendev.org/c/openstack/python-novaclient/+/952984 | |
| 15:40:47 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Replace keystoneclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/952986 | |
| 15:40:47 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Use pre-commit for lint jobs https://review.opendev.org/c/openstack/python-novaclient/+/952987 | |
| 16:00:25 | zigo | gibi: I'm not sure fixing the consequence of the Python 3.13 bug is the way. Hopefully, the interpreter itself can be fixed. | |
| 16:04:10 | zigo | I'll grab and use that patch anyways, as a temp workaround. | |
| 16:21:49 | zigo | nova_31.0.0-4_source.changes uploaded. Hopefully, it will make its way to Trixie ! | |
| 16:54:39 | sean-k-mooney | zigo: well its a workaround for now | |
| 16:54:46 | sean-k-mooney | the interpreter shoudl be fixed | |
| 16:55:01 | sean-k-mooney | but if we can get it to a point where we can start running devstack ectra | |
| 16:55:22 | sean-k-mooney | that will help build some confidnce that there arent other issue hiding behind this | |
| 17:35:04 | opendevreview | Merged openstack/nova-specs master: Support for tracking traits removed from provider.yaml https://review.opendev.org/c/openstack/nova-specs/+/937587 | |
| 19:13:21 | opendevreview | Masahito Muroi proposed openstack/nova master: Add network_id query for list server API and list server detail API https://review.opendev.org/c/openstack/nova/+/953011 | |
| #openstack-nova - 2025-06-21 | |||
| 12:02:18 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove unused validator for uri element https://review.opendev.org/c/openstack/nova/+/951084 | |
| 12:02:47 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop unused method https://review.opendev.org/c/openstack/nova/+/927680 | |
| #openstack-nova - 2025-06-22 | |||
| 19:19:57 | opendevreview | Ghanshyam proposed openstack/nova master: Add new policies for live migrate APIs https://review.opendev.org/c/openstack/nova/+/953063 | |
| 19:22:12 | opendevreview | Ghanshyam proposed openstack/nova master: Add new policies for live migrate APIs https://review.opendev.org/c/openstack/nova/+/953063 | |
| #openstack-nova - 2025-06-23 | |||
| 12:01:42 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754 | |
| 14:52:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: Note on RPC error decorators around build_and_run_instance https://review.opendev.org/c/openstack/nova/+/953121 | |
| 14:54:00 | gibi | sean-k-mooney, dansmith ^^ this is the build_and_run_instance / SpawnIsSynchronous fix. It turned out that we don't have a real bug in the production code just some dead code and a very complicated layered error handling mechanism. | |
| 15:03:53 | dansmith | ack will look in a bit | |
| 15:05:16 | sean-k-mooney | gibi: nice. im trying to focus on well 2 things today but i would like to avoid making it 3 :) im goign to try and dedicate tomorow as a code review day specificly for the eventet series and stephenfin's open api series. feel free to ping me to review your eventlet serise if you dont see me doing it by the afternoon | |
| 15:05:45 | sean-k-mooney | that patch is standalone form that serise right so we can merge it as a general bug fix without any depency correct? | |
| 15:06:43 | sean-k-mooney | thats proably capatured in the commit message but thats one of the thigns to read tomorrow | |
| 15:35:57 | gibi | yepp it is independent from the series | |
| 15:54:33 | opendevreview | Merged openstack/nova master: libvirt: Enable autodeflate and freePageReporting for memballoon https://review.opendev.org/c/openstack/nova/+/952306 | |
| 16:09:51 | gibi | bauzas: https://review.opendev.org/c/openstack/nova/+/953121 is not part of the eventlet series but either we merge this and then I rebase the series, or I will replicated the test changes in the series as well to make things passing on the spawn_on patch in the series | |
| 16:10:12 | gibi | bauzas: so if you are still on the eventlet series then this can be an easy addition | |
| 16:10:50 | Uggla | dansmith, can you please add me to the Nova Drivers" team on launchpad. So I could deal with BP and some kind of admin tasks. | |
| 16:11:11 | dansmith | gibi: bauzas melwitt sean-k-mooney gmaan any concerns? ^ | |
| 16:11:42 | dansmith | that group used to have more "power in gerrit" than just BP administrivia, but it looks like not anymore | |
| 16:13:49 | gibi | dansmith: no concerns. I trust Uggla | |
| 16:14:03 | gmaan | ++ from me | |
| 16:14:16 | Uggla | gmaan, gibi thx | |
| 16:44:34 | opendevreview | sean mooney proposed openstack/nova master: [WIP] Add ResourceProviderWeigher for capacity-aware scheduling https://review.opendev.org/c/openstack/nova/+/953131 | |
| 16:56:35 | sean-k-mooney | dansmith: not specificly. we have dicussed if we wanted to expand that in the past. it provides quite a lot of access ot teh launchpad config but im ok with includeing Uggla | |