| *** ministry is now known as __ministry | 01:28 | |
| opendevreview | Wenping Song proposed openstack/osc-placement master: Remove usage of six https://review.opendev.org/c/openstack/osc-placement/+/815223 | 07:22 |
|---|---|---|
| opendevreview | Wenping Song proposed openstack/osc-placement master: Remove usage of six https://review.opendev.org/c/openstack/osc-placement/+/815223 | 07:26 |
| opendevreview | Wenping Song proposed openstack/nova master: Cleanup guest process when vm evacuate failed and then deleted https://review.opendev.org/c/openstack/nova/+/814746 | 08:08 |
| bauzas | good morning Nova | 08:23 |
| * bauzas starts writing some email about the PTG | 08:23 | |
| gibi | bauzas: o/ | 08:25 |
| *** noonedeadpunk_ is now known as noonedeadpunk | 10:36 | |
| artom_ | johnthetubaguy[m], TheJulia, so here's an idea (to follow up on the node reassignment hash ring changed thing) | 13:00 |
| artom_ | Move the whole thing to the conductor | 13:00 |
| artom_ | We would in effect be moving virt-driver-specific code into the conductor, but it sure would make everything else easier... | 13:02 |
| *** artom_ is now known as artom | 13:06 | |
| kashyap | clarkb: Very cool; this is merged - https://review.opendev.org/c/openstack/devstack/+/815020 :) | 13:51 |
| opendevreview | Erlon R. Cruz proposed openstack/nova master: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/815324 | 15:18 |
| opendevreview | Erlon R. Cruz proposed openstack/nova master: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/815324 | 15:18 |
| kashyap | clarkb: Responded on the list. Hope that answers your questions. (Even if it's not fully satisfactory) | 15:37 |
| clarkb | kashyap: it does, thanks | 15:37 |
| clarkb | kashyap: also https://bugzilla.redhat.com/show_bug.cgi?id=2010058 seems to be at least part of our fedora problems | 15:38 |
| opendevreview | Merged openstack/nova master: Reproducer unit test for bug 1934094 https://review.opendev.org/c/openstack/nova/+/812646 | 16:54 |
| *** ministry is now known as __ministry | 16:59 | |
| gibi | melwitt, sean-k-mooney[m]: I think https://review.opendev.org/c/openstack/nova/+/814036 is the way to go. My alternative trial to kill leaking eventlet failed in https://review.opendev.org/c/openstack/nova/+/815017 | 17:42 |
| TheJulia | artom: hmm, I don't know the nova conductor code path well enough to really know for sure. That feels like we're just shifting a point of failure, but at the same time the compute hosts are basically stateless and just fulfill a role... | 18:27 |
| artom | TheJulia, it might be workable if we build a decent abstraction around it... like, the conductor gets the list of running compute services, calls them and passes the list, the services+driver combo do the hash ring thing and return the result to the conductor, the conductor the saves that | 18:30 |
| artom | Really rough initial first idea | 18:30 |
| TheJulia | artom: I guess the question I would ahve is what spured the thought to think to move it | 18:33 |
| artom | TheJulia, well the current problem is that in your proposed patch we'd be modifying instance.host and potentially doing it while something else is accessing it. Except we can't use the lock that everything else uses because we have no guarantee that we're on the same compute host | 18:35 |
| artom | And the lock is host-local | 18:35 |
| TheJulia | Well, it is a single field update on a row, the db should lock it for the transaction unless sqlachemy's save is update all fields or insert/delete | 18:37 |
| TheJulia | and *if* something should happen where there is a split perception, it should reconcile | 18:37 |
| TheJulia | on the net run | 18:37 |
| TheJulia | s/net/next | 18:37 |
| artom | I... guess? In any case, there be dragons | 18:39 |
| TheJulia | there are dragons today :) | 18:39 |
| TheJulia | and they think we are tasty with ketchup | 18:39 |
| opendevreview | mitya-eremeev-2 proposed openstack/nova master: Close Glance image if downloading failed. https://review.opendev.org/c/openstack/nova/+/815347 | 19:10 |
| opendevreview | Gustavo Santos proposed openstack/nova master: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/815373 | 19:37 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!