| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-07-05 | |||
| 18:28:49 | dansmith | heh, okay | |
| 18:29:48 | fungi | i've about worn out all other tools at my disposal, so sarcasm is what's left at this point | |
| 18:30:27 | dansmith | yeah, you don't have to apologize to me for that.. I'm beyond exhausted.. I dunno what's after that before death, maybe "acceptance" ? | |
| 18:51:31 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Fix device type when booting from ISO image https://review.opendev.org/c/openstack/nova/+/909611 | |
| 19:33:15 | opendevreview | Dan Smith proposed openstack/nova master: Add iso file format inspector https://review.opendev.org/c/openstack/nova/+/923533 | |
| 19:54:58 | opendevreview | Merged openstack/nova stable/2023.2: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923287 | |
| 19:55:04 | opendevreview | Merged openstack/nova stable/2023.1: Check images with format_inspector for safety https://review.opendev.org/c/openstack/nova/+/923289 | |
| 20:11:30 | fungi | 923290 hit a package install error, and that knocked out 923291 as a child commit. i'll get them both back into play but they're the only changes left from the ossa | |
| 20:13:42 | fungi | waiting to promote them to the front since there's a stack of 5 succeeding changes that are due to merge in the next 10 minutes | |
| 20:14:08 | fungi | but once those land i'll shuffle them up | |
| 20:27:34 | fungi | and done | |
| 22:31:56 | opendevreview | Merged openstack/nova stable/2023.1: Additional qemu safety checking on base images https://review.opendev.org/c/openstack/nova/+/923290 | |
| 22:32:02 | opendevreview | Merged openstack/nova stable/2023.1: Fix vmdk_allowed_types checking https://review.opendev.org/c/openstack/nova/+/923291 | |
| 22:32:05 | fungi | yay!!! | |
| 22:44:43 | dansmith | fungi: thanks for all your work, I didn't appreciate how much that would involve you and your clicky finger | |
| 22:45:14 | fungi | no worries, it was thoroughly calloused already | |
| 22:46:02 | fungi | (no actual clicking was done, unless you count the clickity-clack of my mechanical keyboard) | |
| 22:46:29 | fungi | like all real software, zuul has a proper cli | |
| #openstack-nova - 2024-07-06 | |||
| 01:28:00 | JayF | hey, thanks to you both! and congratulations on getting the last of it landed | |
| #openstack-nova - 2024-07-07 | |||
| 00:02:40 | opendevreview | Merged openstack/nova master: scheduler: fix _get_sharing_providers to support unlimited aggr https://review.opendev.org/c/openstack/nova/+/921665 | |
| 01:00:47 | opendevreview | melanie witt proposed openstack/nova master: Consolidate vTPM and ephemeral encryption secret creation https://review.opendev.org/c/openstack/nova/+/912094 | |
| 01:00:47 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Add optional 'description' kwarg to create_secret() https://review.opendev.org/c/openstack/nova/+/919986 | |
| 01:00:48 | opendevreview | melanie witt proposed openstack/nova master: Follow up for encryption secret create consolidation https://review.opendev.org/c/openstack/nova/+/921586 | |
| 01:00:48 | opendevreview | melanie witt proposed openstack/nova master: Remove use of (hw:|hw_)ephemeral_encryption_format https://review.opendev.org/c/openstack/nova/+/921343 | |
| 01:00:50 | opendevreview | melanie witt proposed openstack/nova master: Add EncryptDetails object for ephemeral encryption https://review.opendev.org/c/openstack/nova/+/921344 | |
| 01:00:50 | opendevreview | melanie witt proposed openstack/nova master: Add database migration to ALTER encryption_options https://review.opendev.org/c/openstack/nova/+/921587 | |
| 01:01:38 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 01:01:41 | opendevreview | melanie witt proposed openstack/nova master: Validate key manager create access in API https://review.opendev.org/c/openstack/nova/+/919989 | |
| 01:01:44 | opendevreview | melanie witt proposed openstack/nova master: Clean up unused ephemeral encryption libvirt secrets https://review.opendev.org/c/openstack/nova/+/919990 | |
| 01:01:45 | opendevreview | melanie witt proposed openstack/nova master: libvirt: "Auto heal" ephemeral encryption secrets on guest launch https://review.opendev.org/c/openstack/nova/+/919991 | |
| 01:02:21 | opendevreview | melanie witt proposed openstack/nova master: Support (resize|cold migration) with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870933 | |
| 01:02:47 | opendevreview | melanie witt proposed openstack/nova master: Support live migration with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/905512 | |
| 01:03:26 | opendevreview | melanie witt proposed openstack/nova master: Support rebuild with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870939 | |
| 01:03:27 | opendevreview | melanie witt proposed openstack/nova master: Add Glance standardized encryption image properties https://review.opendev.org/c/openstack/nova/+/921345 | |
| 01:03:27 | opendevreview | melanie witt proposed openstack/nova master: Support encrypted source images for qcow2 https://review.opendev.org/c/openstack/nova/+/919992 | |
| 01:04:05 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 01:04:05 | opendevreview | melanie witt proposed openstack/nova master: Add backing_encryption_secret_uuid to BlockDeviceMapping https://review.opendev.org/c/openstack/nova/+/907960 | |
| 01:04:06 | opendevreview | melanie witt proposed openstack/nova master: Support encrypted backing files for qcow2 https://review.opendev.org/c/openstack/nova/+/907961 | |
| #openstack-nova - 2024-07-08 | |||
| 01:07:19 | opendevreview | Merged openstack/nova master: Server Rescue leads to Server ERROR state if original image is deleted https://review.opendev.org/c/openstack/nova/+/872385 | |
| 04:07:35 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870937 | |
| 04:07:35 | opendevreview | melanie witt proposed openstack/nova master: Support cross cell resize with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/909595 | |
| 04:07:36 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for raw with LUKS https://review.opendev.org/c/openstack/nova/+/884313 | |
| 04:07:36 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for rbd with LUKS https://review.opendev.org/c/openstack/nova/+/889912 | |
| 04:26:26 | spatel | any idea what is this error in nova_libvirt | |
| 04:26:29 | spatel | 2024-07-08 04:25:59.489+0000: 2691: error : qemuDomainObjBeginJobInternal:968 : Timed out during operation: cannot acquire state change lock (held by monitor=remoteDispatchDomainCreateWithFlags) | |
| 04:26:51 | spatel | This is happening after ceph failure. Now ceph is health but nova doesn't like to talk | |
| 07:18:50 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870937 | |
| 07:18:50 | opendevreview | melanie witt proposed openstack/nova master: Support cross cell resize with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/909595 | |
| 07:18:51 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for raw with LUKS https://review.opendev.org/c/openstack/nova/+/884313 | |
| 07:18:51 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for rbd with LUKS https://review.opendev.org/c/openstack/nova/+/889912 | |
| 10:19:48 | sean-k-mooney | gibi: regarding the time out how would you feel if i only tested one block size for iso | |
| 10:20:27 | sean-k-mooney | either 64 * units.Ki, or 1 * units.Mi | |
| 10:20:37 | sean-k-mooney | both would get the entiry header section in one read | |
| 10:21:05 | sean-k-mooney | 512 is not as bad as 17 but its proably why it takes so long | |
| 10:22:38 | sean-k-mooney | im going to try that locally and see how much of a speed up that gives | |
| 10:24:18 | sean-k-mooney | we can see it timed out in chunk = wrapper.read(block_size) so its definetly in that read loop and making progress it just takes too long | |
| 10:30:46 | sean-k-mooney | 5.5-5.9 second with the optimization vs 18.9-19 seconds without | |
| 10:30:56 | sean-k-mooney | i think that is a reasonable trade off | |
| 10:32:45 | sean-k-mooney | none of the logic im adding for the iso inspector cares about unaliged reads, that is just generic and shared with the other parsers so im personally happy with not testing it for these formats. | |
| 10:33:07 | sean-k-mooney | im going to commit that locally and adress the other nits but if you have concerns with that let me know | |
| 11:41:44 | gibi | sean-k-mooney: I'm fine skipping the unaligned read test for iso | |
| 11:43:58 | sean-k-mooney | ack im just adressing the other nits ill have new version up in the next 30 mins or so | |
| 12:03:42 | opendevreview | sean mooney proposed openstack/nova master: Reproduce iso regression with deep format inspection https://review.opendev.org/c/openstack/nova/+/923507 | |
| 12:03:42 | opendevreview | sean mooney proposed openstack/nova master: Add iso file format inspector https://review.opendev.org/c/openstack/nova/+/923533 | |
| 12:04:23 | sean-k-mooney | gibi: dansmith: ^ when your about those are functiontally the saem just with the nits addressed | |
| 12:04:49 | sean-k-mooney | im going to stack with those now and confirm if this correctly sovles the rebuild regression | |
| 12:05:34 | sean-k-mooney | i.e. boot form qcow usin a KNOPPIX or similar rescue iso to repair a vm | |
| 12:06:04 | sean-k-mooney | s/usin/then using/ | |
| 12:10:25 | gibi | sean-k-mooney: those two patches looks good to me. | |
| 12:11:07 | sean-k-mooney | hopefully that resolves the timeouts but we will see. the fact that it was only timing out in the coverage job ment it was borderline before | |
| 12:11:50 | sean-k-mooney | espicaly give the spread between tox py11 at 11 mins to py312 at 22 | |
| 12:13:19 | sean-k-mooney | where that job runs obviouly has a non trivial impact on the job time | |
| 12:24:14 | sean-k-mooney | gibi: ill comment on the patch but i was able t boot form an cirros qcow and then rescue with an alpine iso | |
| 12:24:24 | sean-k-mooney | unrescue reverted back to cirros as expected | |
| 12:41:11 | gibi | sounds good | |
| 12:41:25 | andrewbonney | Hi. Any reviews on https://review.opendev.org/c/openstack/nova/+/919961 would be appreciated when people have time. Thanks | |
| 13:08:33 | opendevreview | Robert Hoffmann proposed openstack/nova master: Fix: clean up volume attachments https://review.opendev.org/c/openstack/nova/+/923646 | |
| 13:37:14 | spatel | Any idea on nova_libvirt error : qemuDomainObjBeginJobInternal:968 : Timed out during operation: cannot acquire state change lock (held by monitor=init-instance-00000d99) | |
| 14:43:57 | elodilles | sean-k-mooney dansmith : should we wait for any other patch to merge, or is it OK if I generate now stable release patches for nova? (so that we release the fixes for bug/2059809) | |
| 14:44:23 | dansmith | elodilles: I think fungi would say yes, release | |
| 14:44:44 | elodilles | ACK, then I'm preparing the patch | |
| 14:44:48 | fungi | yeah, the sooner the better | |
| 14:45:38 | dansmith | elodilles, fungi: bauzas is out this week, will we need an ack? | |
| 14:46:41 | fungi | the release team is able to override/bypass that requirement, if they're willing to | |
| 14:46:52 | dansmith | cool | |
| 14:58:09 | elodilles | dansmith: https://review.opendev.org/c/openstack/releases/+/923656 | |
| 14:58:39 | elodilles | Uggla auniyal : ping, as you are also release liaisons ^^^ o:) | |
| 15:00:04 | elodilles | release jobs (validation and list-changes) logs should be available in ~10-20 mins | |
| 15:00:36 | dansmith | elodilles: hmm, are those the right hashes for each release? | |
| 15:00:50 | dansmith | seems like A and C are reversed, unless I'm missing something | |
| 15:01:11 | dansmith | wait, maybe not | |
| 15:01:33 | dansmith | okay yeah, mis-copy-paste I think, nevermind | |
| 16:14:10 | mnaser | hi nova friends. https://review.opendev.org/c/openstack/nova/+/923298/2 -- can someone look at this unmaintained/zed stack, it looks good to me so far and would be nice to get going into the gate | |
| 16:14:15 | mnaser | (CVE stuff) | |
| 16:16:54 | dansmith | mnaser: note that nova-core doesn't even have +2 on that branch because it's unmaintained | |
| 16:18:23 | dansmith | looks like it has the only tweak we made after the first push, so I'd assume that if it works for you it's good | |
| 16:20:12 | mnaser | yeah I got that in | |
| 16:20:23 | mnaser | well, elodilles did :) | |
| 16:25:14 | JayF | mnaser: feel free to add me as reviewer for any security backports in unmaintained/ that need additional review. | |
| 16:25:33 | JayF | mnaser: I have the one patch you've indicated opened in my window, will have a look | |