| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | 01:45 |
|---|---|---|
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Extract share management from ComputeManager https://review.opendev.org/c/openstack/nova/+/999349 | 01:45 |
| *** Jeff is now known as Guest14559 | 01:53 | |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Extract share management from ComputeManager https://review.opendev.org/c/openstack/nova/+/999349 | 02:02 |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | 02:02 |
| *** Guest14559 is now known as JeffYang | 02:31 | |
| gibi | gmaan: sorry I was already done for the day at that point yesterday. | 07:35 |
| opendevreview | Roman Hros proposed openstack/nova master: Delete inactive port bindings during live migration https://review.opendev.org/c/openstack/nova/+/999374 | 08:03 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: check tempest for instability https://review.opendev.org/c/openstack/nova/+/999378 | 08:25 |
| opendevreview | Joan Gilabert proposed openstack/nova master: Report OWNER_NOVA trait on resource providers https://review.opendev.org/c/openstack/nova/+/994299 | 09:30 |
| opendevreview | Joan Gilabert proposed openstack/nova master: Add libvirt driver MDEV ARQ support for Cyborg https://review.opendev.org/c/openstack/nova/+/995840 | 09:30 |
| opendevreview | Roman Hros proposed openstack/nova master: Delete inactive port bindings during live migration https://review.opendev.org/c/openstack/nova/+/999374 | 09:34 |
| opendevreview | Roman Hros proposed openstack/nova master: Delete inactive port bindings during live migration https://review.opendev.org/c/openstack/nova/+/999374 | 10:20 |
| ralonsoh | sean-k-mooney, hi! will you have some time to talk about https://bugs.launchpad.net/nova/+bug/2161894? | 10:34 |
| ralonsoh | In particular about `create_tap`!=`ovs_create_tap` | 10:34 |
| sean-k-mooney | sure am are you free now ish, or at the top of the hour? | 10:38 |
| sean-k-mooney | i was going to go grab a coffee quickly | 10:38 |
| sean-k-mooney | but we can chat in say 5 mins | 10:38 |
| ralonsoh | perfect | 10:39 |
| sean-k-mooney | those https://meet.google.com/nbj-ernx-ymb work for you other are fee to join to we will be talking about https://bugs.launchpad.net/nova/+bug/2161894 a live migration instablity with ml2/ovs | 10:51 |
| opendevreview | Roman Hros proposed openstack/nova master: Delete inactive port bindings during live migration https://review.opendev.org/c/openstack/nova/+/999374 | 12:51 |
| gmaan | gibi: no issue, I commented on the change, I think doing option 3 (moving all thread/executors things in a separate file) will make it easy to maintain in long term | 17:21 |
| gmaan | anyways, will wait for your feedback before I start doing the modification | 17:22 |
| gmaan | dansmith: thanks for review graceful shutdown change, I replied please check, mainly for this comment if the approach i replied is ok https://review.opendev.org/c/openstack/nova/+/996299/comment/5c3a3d23_ff206cc3/ | 18:29 |
| gmaan | for revert_resize I think making dest revert_resize idempotent will work, but I will handle that in separate change with functional reproducer test | 18:30 |
| dansmith | gmaan: reading but just realizing I forgot to delete half a sentence in there :/ | 18:36 |
| dansmith | gmaan: replied | 18:46 |
| gmaan | dansmith: thanks, for task end. I end task for both success and failure case. do you think recording the status also can be useful for shutdown case? we will be returning the erorr to users in that case but in case that is missed as we are shutting down, having tasks failed can be useful for users to trigger them again? | 18:58 |
| dansmith | gmaan: you mean logging what tasks failed during shutdown right? | 18:58 |
| gmaan | yeah | 18:58 |
| dansmith | yeah, definitely seems good to log that stuff | 18:59 |
| gmaan | ok, will add that | 18:59 |
| dansmith | on the multiple end thing, I'm really just saying that if we expect one start and one end for each task, but we get a second end it might mean we raced or lost track of something and so it'd be good to flag it .. if it is expected for some reason then no need | 18:59 |
| gmaan | ah i see yeah missed to record start task is good point. | 19:02 |
| dansmith | yeah exactly | 19:04 |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Fix deny_share host locality for NFS https://review.opendev.org/c/openstack/nova/+/998752 | 20:51 |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Extract share management from ComputeManager https://review.opendev.org/c/openstack/nova/+/999349 | 21:44 |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | 21:44 |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Extract share management from ComputeManager https://review.opendev.org/c/openstack/nova/+/999349 | 22:01 |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | 22:01 |
| * gouthamr messes up rebases, hates patch trains | 22:05 | |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Task tracking mechanism for graceful shutdown https://review.opendev.org/c/openstack/nova/+/996299 | 22:12 |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Extract share management from ComputeManager https://review.opendev.org/c/openstack/nova/+/999349 | 22:21 |
| opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | 22:21 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: Call manager graceful_shutdown while RPC server waits https://review.opendev.org/c/openstack/nova/+/996583 | 22:46 |
| gmaan | dansmith: updated the changes. Its public holiday here on Monday so will be checking it back on tuesday | 22:46 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!