| *** JohnnyW52 is now known as JohnnyW5 | 08:34 | |
| opendevreview | efineshi proposed openstack/nova master: Not destroy the vm as it migrate back before evacuate clean https://review.opendev.org/c/openstack/nova/+/887564 | 09:42 |
|---|---|---|
| opendevreview | efineshi proposed openstack/nova master: Not destroy the vm as it migrate back before evacuate clean https://review.opendev.org/c/openstack/nova/+/887564 | 09:44 |
| opendevreview | dzogchen proposed openstack/nova master: Adding a wait before attempting to update a resource https://review.opendev.org/c/openstack/nova/+/888452 | 10:04 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Added context manager for instance lock https://review.opendev.org/c/openstack/nova/+/873648 | 11:24 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | 11:24 |
| *** ralonsoh is now known as ralonsoh_lunch | 11:50 | |
| *** ralonsoh_lunch is now known as ralonsoh | 12:51 | |
| dansmith | bauzas: most of the other online migrations use count instead of max_count, but some of them are right. So I'm going to put a trivial patch on top of this to move them all (including mine) to max_count | 13:34 |
| bauzas | dansmith: no worries, I just told +1 but this is a close +2 | 13:35 |
| bauzas | just in case you need to provide a new revision | 13:35 |
| dansmith | bauzas: the way I have it matches the other one in the file just above it, so I think the patch is okay as-is for consistency, and I will push another on top to fix these both and all the others | 13:37 |
| dansmith | running tests now | 13:37 |
| dansmith | also, I think we probably need to think about removing some of these online migrations at some point.. we probably don't need to be scanning for situations that were introduced in pike | 13:39 |
| opendevreview | Dan Smith proposed openstack/nova master: Fix up inconsistent max_count argument in o-d-ms https://review.opendev.org/c/openstack/nova/+/888470 | 14:07 |
| dansmith | bauzas: ^ | 14:07 |
| * bauzas looks | 14:08 | |
| dansmith | hopefully that earns me a +2 on the one below :) | 14:10 |
| dansmith | sean-k-mooney[m]: if you're looking for some easy nova work, this series could use some +Ws (or review comments): https://review.opendev.org/q/topic:bp/compute-object-ids+status:open | 15:28 |
| opendevreview | Amit Uniyal proposed openstack/nova master: Disconnecting volume from the compute host https://review.opendev.org/c/openstack/nova/+/877446 | 15:38 |
| opendevreview | yatin proposed openstack/nova master: Add config option to configure TB cache size https://review.opendev.org/c/openstack/nova/+/868419 | 16:23 |
| opendevreview | melanie witt proposed openstack/nova stable/ussuri: Reproducer for bug 2003991 unshelving offloaded instance https://review.opendev.org/c/openstack/nova/+/888205 | 22:25 |
| opendevreview | melanie witt proposed openstack/nova stable/ussuri: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888206 | 22:25 |
| opendevreview | melanie witt proposed openstack/nova stable/train: Reproducer for bug 2003991 unshelving offloaded instance https://review.opendev.org/c/openstack/nova/+/888508 | 22:40 |
| opendevreview | melanie witt proposed openstack/nova stable/train: Enforce quota usage from placement when unshelving https://review.opendev.org/c/openstack/nova/+/888509 | 22:40 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!