| *** elodille1 is now known as elodilles | 04:41 | |
| artom | dansmith, sean-k-mooney, I think https://review.opendev.org/c/openstack/nova/+/915139/3 (the cpu_shared_set live migration fix) is at a point where it should be an easy-ish +2/+A - can we get that going? I've got a couple of whitebox changed that need it, which will allow us to cross a few tests off our list | 14:17 |
|---|---|---|
| sean-k-mooney | ah ya i didnt get to reveiw that last week | 14:18 |
| sean-k-mooney | i have 12 mins before my next meeting so ill take a look | 14:18 |
| artom | Cheers :) | 14:21 |
| simondodsley | is there a known prgramatic method to get the unique ID of a hypervisor? I know it is in the Nova DB but not sure which construct I can use to get that out through a python call | 14:36 |
| sean-k-mooney | do you mean the compute node UUID | 14:41 |
| sean-k-mooney | that is aviable to admins via the hypervior api | 14:41 |
| dansmith | Uggla_: mind if I just quickly fix the truthiness nits in https://review.opendev.org/c/openstack/nova/+/915139/3/nova/virt/libvirt/migration.py and then I'll +2? | 14:46 |
| dansmith | and also the commit message nit | 14:51 |
| dansmith | (which is not really a nit, IMHO) | 14:52 |
| artom | dansmith, mind if I proxy for him and just tell you to do it? ;) | 14:53 |
| artom | (With appreciation for your review time) | 14:53 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: scheduler: AggregateMultitenancyIsolation to support unlimited tenant https://review.opendev.org/c/openstack/nova/+/896512 | 14:56 |
| opendevreview | Dan Smith proposed openstack/nova master: Fix: migration configuration with cpu_shared_set (object part) https://review.opendev.org/c/openstack/nova/+/903706 | 14:59 |
| opendevreview | Dan Smith proposed openstack/nova master: Fix: migration configuration with cpu_shared_set (libvirt part) https://review.opendev.org/c/openstack/nova/+/915139 | 14:59 |
| opendevreview | Dan Smith proposed openstack/nova master: Test live migration between hosts with differnet cpu_shared_sets https://review.opendev.org/c/openstack/nova/+/913744 | 14:59 |
| *** blarnath is now known as d34dh0r53 | 15:10 | |
| dansmith | tkajinam: did you want to opine on this before I flesh it out? https://review.opendev.org/c/openstack/nova/+/915911/1 | 15:31 |
| opendevreview | Max proposed openstack/nova master: feat: nova-manage db instace events cleanup https://review.opendev.org/c/openstack/nova/+/916927 | 15:32 |
| tkajinam | dansmith, I'll check the series of these patches and will leave any comments (or +1 if no) | 15:34 |
| *** tobias-urdin4 is now known as tobias-urdin | 15:57 | |
| opendevreview | Merged openstack/nova master: Fix: migration configuration with cpu_shared_set (object part) https://review.opendev.org/c/openstack/nova/+/903706 | 19:58 |
| opendevreview | Merged openstack/nova master: Fix: migration configuration with cpu_shared_set (libvirt part) https://review.opendev.org/c/openstack/nova/+/915139 | 20:37 |
| simondodsley | sean-k-mooney: sorry for the delay in responding. I need to get the compute UUID from within a cinder driver. Do you know where/how I could do that? | 21:07 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!