| *** dtantsur_ is now known as dtantsur | 01:32 | |
| opendevreview | Pierre Riteau proposed openstack/nova-specs master: Fix typo in release name https://review.opendev.org/c/openstack/nova-specs/+/930860 | 04:59 |
|---|---|---|
| opendevreview | Pierre Riteau proposed openstack/nova-specs master: Fix typos in release name https://review.opendev.org/c/openstack/nova-specs/+/930860 | 08:33 |
| opendevreview | Merged openstack/nova-specs master: Fix typos in release name https://review.opendev.org/c/openstack/nova-specs/+/930860 | 08:53 |
| *** ykarel_ is now known as ykarel | 09:01 | |
| opendevreview | Takashi Kajinami proposed openstack/placement master: Remove Python 3.8 support https://review.opendev.org/c/openstack/placement/+/925648 | 09:19 |
| *** __ministry is now known as Guest4990 | 10:44 | |
| *** ykarel_ is now known as ykarel | 12:26 | |
| *** kgube1 is now known as kgube | 12:30 | |
| opendevreview | Dan Smith proposed openstack/nova master: Fix image format error on vol-backed snapshot https://review.opendev.org/c/openstack/nova/+/930754 | 13:50 |
| *** haleyb|out is now known as haleyb | 13:58 | |
| *** ykarel_ is now known as ykarel | 14:02 | |
| opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing finish_time https://review.opendev.org/c/openstack/nova/+/928933 | 14:18 |
| sean-k-mooney1 | dansmith: regarding ^ i belive the reason we create glance images is mainly for metadata for thing like shelve but also so you can boot a new vm from it using the "boot form volume form image" flow without needed to use the "boot form volume form volumes snapshot" flow | 14:26 |
| dansmith | sean-k-mooney1: it doesn't look to me like there's enough info in the image to do that | 14:26 |
| *** sean-k-mooney1 is now known as sean-k-mooney | 14:26 | |
| sean-k-mooney | i have not looked at that closely in a while just i think that was the orignal reasoning (shelve) | 14:27 |
| sean-k-mooney | im more or less seculating about the secondary usecase | 14:27 |
| sean-k-mooney | s/seculating/guessing/ | 14:29 |
| sean-k-mooney | i belive the image just has a refence to the volume snapshot | 14:29 |
| dansmith | sean-k-mooney: it was not the original rationale as best I can tell, based on the history I provided.. it was literally to unify some workflow where we called a qemu command we don't even call anymore | 14:29 |
| dansmith | sean-k-mooney: it doesn't, as far as I can tell | 14:29 |
| dansmith | sean-k-mooney: regardless, the fix above is needed, the re-think is optional | 14:30 |
| sean-k-mooney | oh ok. then ya if its not for that im not sure why that code still exits | 14:30 |
| sean-k-mooney | ack, i just skimmed the commit message for what its worth | 14:30 |
| sean-k-mooney | i can try and take a look at it more closely later in the week | 14:30 |
| *** whoami-rajat_ is now known as whoami-rajat | 17:08 | |
| opendevreview | Merged openstack/nova stable/2023.1: Fix regression with live migration on shared storage https://review.opendev.org/c/openstack/nova/+/929969 | 17:38 |
| opendevreview | Merged openstack/nova master: Support os-brick specific lock_path https://review.opendev.org/c/openstack/nova/+/849328 | 18:51 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!