| opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754 | 12:01 |
|---|---|---|
| *** haleyb|out is now known as haleyb | 13:12 | |
| *** LarsErik1 is now known as LarsErikP | 13:28 | |
| 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:52 |
| 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. | 14:54 |
| dansmith | ack will look in a bit | 15:03 |
| 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 |
| 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:05 |
| sean-k-mooney | thats proably capatured in the commit message but thats one of the thigns to read tomorrow | 15:06 |
| gibi | yepp it is independent from the series | 15:35 |
| opendevreview | Merged openstack/nova master: libvirt: Enable autodeflate and freePageReporting for memballoon https://review.opendev.org/c/openstack/nova/+/952306 | 15:54 |
| 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:09 |
| gibi | bauzas: so if you are still on the eventlet series then this can be an easy addition | 16:10 |
| 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:10 |
| dansmith | gibi: bauzas melwitt sean-k-mooney gmaan any concerns? ^ | 16:11 |
| dansmith | that group used to have more "power in gerrit" than just BP administrivia, but it looks like not anymore | 16:11 |
| gibi | dansmith: no concerns. I trust Uggla | 16:13 |
| gmaan | ++ from me | 16:14 |
| Uggla | gmaan, gibi thx | 16:14 |
| opendevreview | sean mooney proposed openstack/nova master: [WIP] Add ResourceProviderWeigher for capacity-aware scheduling https://review.opendev.org/c/openstack/nova/+/953131 | 16:44 |
| 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:56 |
| 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:57 |
| 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 |
| sean-k-mooney | it allows configuring the bug teams and adding folks to those and all the normal branch(serises) config | 16:58 |
| dansmith | right, but.. those are PTL activities right? | 16:59 |
| sean-k-mooney | technically yes if we dont have a relesae liasion but i have no issue with ptls having that acess | 16:59 |
| sean-k-mooney | they are the person all that work defaults too if its not delegated | 16:59 |
| sean-k-mooney | so they should be able ot do it even when it is | 17:00 |
| 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:00 |
| dansmith | anyway, looks like we have over 50% "yes" so I'll go ahead and do it | 17:01 |
| 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:01 |
| dansmith | ack | 17:02 |
| sean-k-mooney | but yep i agree keep it small to the folkd doing the work | 17:02 |
| opendevreview | Ghanshyam proposed openstack/nova master: Add new policies for live migrate APIs https://review.opendev.org/c/openstack/nova/+/953063 | 19:04 |
| 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 |
| opendevreview | Michael Still proposed openstack/nova master: Implement sound model extra spec for libvirt. https://review.opendev.org/c/openstack/nova/+/940770 | 20: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 | 20: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 |
| opendevreview | Ghanshyam proposed openstack/nova master: Add new policies for live migrate APIs https://review.opendev.org/c/openstack/nova/+/953063 | 21:30 |
| opendevreview | Stephen Finucane proposed openstack/python-novaclient master: Replace keystoneclient with openstacksdk https://review.opendev.org/c/openstack/python-novaclient/+/952986 | 21: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 |
| opendevreview | Ghanshyam proposed openstack/nova master: Add project manager role in Nova API policy rule https://review.opendev.org/c/openstack/nova/+/953158 | 23:22 |
| opendevreview | Ghanshyam proposed openstack/nova master: Add project manager role in Nova API policy rule https://review.opendev.org/c/openstack/nova/+/941347 | 23:25 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!