| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-06-20 | |||
| 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: shell: Hard fail if v2.0 URL detected https://review.opendev.org/c/openstack/python-novaclient/+/952985 | |
| 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: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:41 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Replace keystoneclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/952986 | |
| 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: Replace keystoneclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/952986 | |
| 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: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: Use pre-commit for lint jobs https://review.opendev.org/c/openstack/python-novaclient/+/952987 | |
| 15:40:47 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Replace keystoneclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/952986 | |
| 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 | |
| 16:57:36 | dansmith | sean-k-mooney: it doesn't appear in project-config at all, so no gerrit perms, AFAIK.. what launchpad access does it provide aside that you're thinking of? | |
| 16:58:15 | dansmith | I mean, it allows for the management of the nova project's milestones and such, which is kinda the point here, but is there more? | |
| 16:58:43 | sean-k-mooney | it allows configuring the bug teams and adding folks to those and all the normal branch(serises) config | |
| 16:59:01 | dansmith | right, but.. those are PTL activities right? | |
| 16:59:23 | sean-k-mooney | technically yes if we dont have a relesae liasion but i have no issue with ptls having that acess | |
| 16:59:54 | sean-k-mooney | they are the person all that work defaults too if its not delegated | |
| 17:00:01 | sean-k-mooney | so they should be able ot do it even when it is | |
| 17:00:29 | dansmith | okay, I don't think it should be opened any wider than necessary (i.e. PTL and long term cores), but PTL seems important for managing all that paperwork | |
| 17:01:03 | dansmith | anyway, looks like we have over 50% "yes" so I'll go ahead and do it | |
| 17:01:33 | sean-k-mooney | the only reaosn i brought it up is i asked for access before to manage that when i was the release liason for nova and i was not added at the time | |
| 17:02:26 | dansmith | ack | |
| 17:02:27 | sean-k-mooney | but yep i agree keep it small to the folkd doing the work | |
| 19:04:56 | opendevreview | Ghanshyam proposed openstack/nova master: Add new policies for live migrate APIs https://review.opendev.org/c/openstack/nova/+/953063 | |
| 20:01:00 | opendevreview | Michael Still proposed openstack/nova master: Implement sound model extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/940770 | |
| 20:01:00 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for sound model. https://review.opendev.org/c/openstack/nova/+/926126 | |
| 20:01:01 | opendevreview | Michael Still proposed openstack/nova master: Implement USB controller extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/950643 | |
| 20:01:01 | opendevreview | Michael Still proposed openstack/nova master: libvirt: Add objects and notifications for USB controller model. https://review.opendev.org/c/openstack/nova/+/927354 | |
| 21:30:35 | opendevreview | Ghanshyam proposed openstack/nova master: Add new policies for live migrate APIs https://review.opendev.org/c/openstack/nova/+/953063 | |
| 21:46:46 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Use pre-commit for lint jobs https://review.opendev.org/c/openstack/python-novaclient/+/952987 | |
| 21:46:46 | opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Replace keystoneclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/952986 | |
| 23:22:53 | opendevreview | Ghanshyam proposed openstack/nova master: Add project manager role in Nova API policy rule https://review.opendev.org/c/openstack/nova/+/953158 | |
| 23:25:29 | opendevreview | Ghanshyam proposed openstack/nova master: Add project manager role in Nova API policy rule https://review.opendev.org/c/openstack/nova/+/941347 | |