| opendevreview | Seyeong Kim proposed openstack/nova master: libvirt: Support boot_index for multiple block devices https://review.opendev.org/c/openstack/nova/+/963665 | 00:47 |
|---|---|---|
| opendevreview | Seyeong Kim proposed openstack/nova master: libvirt: Support boot_index for multiple block devices https://review.opendev.org/c/openstack/nova/+/963665 | 01:34 |
| *** mhen_ is now known as mhen | 02:30 | |
| nicolairuckel | For my patch (https://review.opendev.org/c/openstack/nova/+/959682), I tried to make cold migration work. Sean suggested using `libvirt_utils.copy_image` to copy the NVRAM file. Unfortunately, it still fails because of missing read permissions. When I set those manually the file will be present after the migration with the same permissions but my test UEFI variable is gone. Does anyone have an idea what I | 09:17 |
| nicolairuckel | could be missing here? This is what I tried: https://paste.opendev.org/show/bWX0QudxanRHdz8wpZs9/ | 09:17 |
| stephenfin | gibi: sean-k-mooney: Not urgent, but this series does the same pyproject.toml migration and wsgi_script removal for placement that we recently did for nova, if you want to take a look https://review.opendev.org/c/openstack/placement/+/919582 I think ye reviewed the nova ones | 11:31 |
| sean-k-mooney | stephenfin: those all look good to me | 11:51 |
| stephenfin | thanks | 11:55 |
| gibi | stephenfin: I have request in https://review.opendev.org/c/openstack/placement/+/967602 I'm +2+A on the rest | 12:14 |
| nicolairuckel | sean-k-mooney: Would you have time to look at my issue with copy_image? | 12:14 |
| sean-k-mooney | nicolairuckel: so to workaroudn the perrmeion issue we will need to invoke the copy with privsep (basically sudo cp vs cp) | 12:32 |
| opendevreview | Stephen Finucane proposed openstack/placement master: pre-commit: Bump versions https://review.opendev.org/c/openstack/placement/+/967602 | 13:11 |
| stephenfin | gibi: Good spot on https://review.opendev.org/c/openstack/placement/+/967602. Resolve now with comment left explaining where I was coming from (I did actually think about it at the time and still got it wrong 😅) | 13:12 |
| stephenfin | *Resolved | 13:12 |
| nicolairuckel | I'll look into that, thanks | 13:36 |
| opendevreview | Merged openstack/placement master: setup: Remove pbr's wsgi_scripts https://review.opendev.org/c/openstack/placement/+/919582 | 14:02 |
| opendevreview | Merged openstack/placement master: Correct default oslo.conf file https://review.opendev.org/c/openstack/placement/+/967600 | 14:02 |
| opendevreview | Merged openstack/placement master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/placement/+/967601 | 14:02 |
| Uggla | dansmith, yesterday I promised Luzy to ping you about the review for the image encryption bp & glance spec. | 15:02 |
| gibi | stephenfin: thanks +Ad it | 15:08 |
| dansmith | Uggla: thanks, but I get an email every time it's updated, it was just revised again yesterday and it's in my list to re-review.. but I have lots of things in the queue | 15:10 |
| Uggla | dansmith, it is what I explained yesterday, that you were busy with escalations etc... I don't know the timelines for Glance, maybe there is still time before spec freeze too. | 15:13 |
| opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Use built-in firmware auto-selection for UEFI firmware https://review.opendev.org/c/openstack/nova-specs/+/966584 | 15:20 |
| opendevreview | Merged openstack/nova stable/2024.2: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967569 | 16:11 |
| opendevreview | Merged openstack/placement master: pre-commit: Bump versions https://review.opendev.org/c/openstack/placement/+/967602 | 16:14 |
| opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Use built-in firmware auto-selection for UEFI firmware https://review.opendev.org/c/openstack/nova-specs/+/966584 | 16:18 |
| opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Use built-in firmware auto-selection for UEFI firmware https://review.opendev.org/c/openstack/nova-specs/+/966584 | 16:21 |
| opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Use built-in firmware auto-selection for UEFI firmware https://review.opendev.org/c/openstack/nova-specs/+/966584 | 16:32 |
| opendevreview | Merged openstack/nova stable/2024.2: nova-conductor puts instance in error state https://review.opendev.org/c/openstack/nova/+/961775 | 17:22 |
| nelljerram | FYI all backports for the "libvirt-tap-managed-no" topic have been merged now. Thanks so much to elodilles for nudging these along. masahito you may like to abandon https://review.opendev.org/c/openstack/nova/+/940135 now. If there are any related loose ends that I should work on, please do let me know. (I'm already thinking about how to plumb in a third-party Calico CI...) | 18:42 |
| nicolairuckel | Does migrate_disk_and_power_off not get called for cold migration? I thought that would be a good place to copy the NVRAM. | 20:43 |
| opendevreview | Ghanshyam proposed openstack/nova master: [Trivial]: Remove unused RBAC default alias https://review.opendev.org/c/openstack/nova/+/968410 | 21:28 |
| gmaan | Uggla: ^^ adding some notes and removing the common alias which should not be used. | 21:29 |
| opendevreview | Sean Mooney proposed openstack/nova-specs master: add spec for resouce notifications https://review.opendev.org/c/openstack/nova-specs/+/967712 | 22:48 |
| *** haleyb is now known as haleyb|out | 22:51 | |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!