Friday, 2023-07-28

opendevreviewmelanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2  https://review.opendev.org/c/openstack/nova/+/87093204:21
opendevreviewmelanie witt proposed openstack/nova master: Support resize with ephemeral encryption for qcow2  https://review.opendev.org/c/openstack/nova/+/87093304:21
opendevreviewmelanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property  https://review.opendev.org/c/openstack/nova/+/87093504:21
opendevreviewmelanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase  https://review.opendev.org/c/openstack/nova/+/87093604:21
opendevreviewmelanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption for qcow2  https://review.opendev.org/c/openstack/nova/+/87093704:21
opendevreviewmelanie witt proposed openstack/nova master: Support rebuild and unshelve with ephemeral encryption  https://review.opendev.org/c/openstack/nova/+/87093904:21
opendevreviewmelanie witt proposed openstack/nova master: WIP Support rescue with ephemeral encryption  https://review.opendev.org/c/openstack/nova/+/87367504:21
opendevreviewmelanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS  https://review.opendev.org/c/openstack/nova/+/77227304:21
opendevreviewmelanie witt proposed openstack/nova master: WIP raw imagebackend support  https://review.opendev.org/c/openstack/nova/+/88431304:21
opendevreviewmelanie witt proposed openstack/nova master: WIP rbd imagebackend support  https://review.opendev.org/c/openstack/nova/+/88991204:21
opendevreviewmelanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw, rbd  https://review.opendev.org/c/openstack/nova/+/86241604:30
opendevreviewmelanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw, rbd  https://review.opendev.org/c/openstack/nova/+/86241604:37
Ugglabauzas, I'm gonna push the series using the SDK in a couple of minutes, just finishing it is working well in devstack. Then I will introduce the manila lock api calls.09:23
bauzas++09:23
Ugglathis is on par with the previous code except the sdk for the moment.09:24
Ugglamaybe more than a couple of minutes, I'm running functional tests and devstack in //. My cpu is 100% and the machine is slow as hell.09:27
opendevreviewribaudr proposed openstack/nova master: Allow config to support virtiofs (driver)  https://review.opendev.org/c/openstack/nova/+/88652209:53
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (db)  https://review.opendev.org/c/openstack/nova/+/83119309:53
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (objects)  https://review.opendev.org/c/openstack/nova/+/83940109:53
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction)  https://review.opendev.org/c/openstack/nova/+/83119409:53
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part)  https://review.opendev.org/c/openstack/nova/+/83309009:53
opendevreviewribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process  https://review.opendev.org/c/openstack/nova/+/88007509:53
opendevreviewribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion  https://review.opendev.org/c/openstack/nova/+/88147209:53
opendevreviewribaudr proposed openstack/nova master: Add metadata for shares  https://review.opendev.org/c/openstack/nova/+/85050009:53
opendevreviewribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part)  https://review.opendev.org/c/openstack/nova/+/85482309:53
opendevreviewribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part)  https://review.opendev.org/c/openstack/nova/+/85482409:53
opendevreviewribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part)  https://review.opendev.org/c/openstack/nova/+/86028409:53
opendevreviewribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part)  https://review.opendev.org/c/openstack/nova/+/86028509:53
opendevreviewribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares  https://review.opendev.org/c/openstack/nova/+/86028609:53
opendevreviewribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part)  https://review.opendev.org/c/openstack/nova/+/86028709:53
opendevreviewribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute manager part)  https://review.opendev.org/c/openstack/nova/+/86028809:53
opendevreviewribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol  https://review.opendev.org/c/openstack/nova/+/88386209:53
opendevreviewribaudr proposed openstack/nova master: Check shares support (compute manager)  https://review.opendev.org/c/openstack/nova/+/88575109:53
opendevreviewribaudr proposed openstack/nova master: Check shares support (only API exception)  https://review.opendev.org/c/openstack/nova/+/88575209:53
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API)  https://review.opendev.org/c/openstack/nova/+/83683009:53
opendevreviewribaudr proposed openstack/nova master: Check shares support (API)  https://review.opendev.org/c/openstack/nova/+/85049909:53
opendevreviewribaudr proposed openstack/nova master: Add helper methods to attach/detach shares  https://review.opendev.org/c/openstack/nova/+/88575309:53
opendevreviewribaudr proposed openstack/nova master: Add instance.share_attach notification  https://review.opendev.org/c/openstack/nova/+/85050109:53
opendevreviewribaudr proposed openstack/nova master: Add instance.share_detach notification  https://review.opendev.org/c/openstack/nova/+/85102809:53
opendevreviewribaudr proposed openstack/nova master: Add shares to InstancePayload  https://review.opendev.org/c/openstack/nova/+/85102909:53
opendevreviewribaudr proposed openstack/nova master: Add instance.share_attach_error notification  https://review.opendev.org/c/openstack/nova/+/86028209:53
opendevreviewribaudr proposed openstack/nova master: Add instance.share_detach_error notification  https://review.opendev.org/c/openstack/nova/+/86028309:53
opendevreviewribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working.  https://review.opendev.org/c/openstack/nova/+/85208609:53
opendevreviewribaudr proposed openstack/nova master: Add virt/libvirt error test cases  https://review.opendev.org/c/openstack/nova/+/85208709:53
opendevreviewribaudr proposed openstack/nova master: Docs about Manila shares API usage  https://review.opendev.org/c/openstack/nova/+/87164209:53
opendevreviewAmit Uniyal proposed openstack/nova master: Add function to get all attachments in Cinder.API module  https://review.opendev.org/c/openstack/nova/+/88145711:20
opendevreviewAmit Uniyal proposed openstack/nova master: Delete dangling bdms  https://review.opendev.org/c/openstack/nova/+/88228411:20
opendevreviewAmit Uniyal proposed openstack/nova master: Reproducer for dangling bdms  https://review.opendev.org/c/openstack/nova/+/88994711:20
opendevreviewAmit Uniyal proposed openstack/nova master: Add function to get all attachments in Cinder.API module  https://review.opendev.org/c/openstack/nova/+/88145711:24
opendevreviewAmit Uniyal proposed openstack/nova master: Reproducer for dangling bdms  https://review.opendev.org/c/openstack/nova/+/88994711:24
opendevreviewAmit Uniyal proposed openstack/nova master: Delete dangling bdms  https://review.opendev.org/c/openstack/nova/+/88228411:24
opendevreviewAmit Uniyal proposed openstack/nova master: Add function to get all attachments in Cinder.API module  https://review.opendev.org/c/openstack/nova/+/88145711:54
opendevreviewAmit Uniyal proposed openstack/nova master: Reproducer for dangling bdms  https://review.opendev.org/c/openstack/nova/+/88994711:54
opendevreviewAmit Uniyal proposed openstack/nova master: Delete dangling bdms  https://review.opendev.org/c/openstack/nova/+/88228411:54
opendevreviewDanila Balagansky proposed openstack/nova master: Show free vCPUs in HostState __repr__  https://review.opendev.org/c/openstack/nova/+/88640015:41
*** gibi is now known as gibi_pto16:15
sean-k-mooneydansmith: it can proably wait till monday but did we decied a direction for fixing the nova-lvm job17:26
sean-k-mooneyhave we temporally skipped it17:27
dansmithyeah gmann skipped it IIRC17:27
sean-k-mooneywell the test. ok i just didnt want to leave it blocked over the weekend17:27
sean-k-mooneyhopefully some things can merge17:27
dansmithhopefully, although it wasn't blocking any of my stuff and I'm having a hard enough time as it is :/17:28
dansmithzuul is currently taunting me with three patches in the gate queue all currently passing everything17:28
dansmithit likes to wait until the very end to fail one job while I'm not looking17:28
dansmithusually just one in the bottom most patch so the rest have to go through a recheck which takes about 20 tries to make it back to gate17:28
sean-k-mooneyack well the reason i ask other then the fact you were here yestereday when we were debugging it was to know if it was blocking your stuff 17:29
dansmithyeah, it's not, luckily :_17:29
sean-k-mooneytl;dr i was hoping as im sure you are that those could merge today/over the weekend 17:30
dansmithso not caring is a little selfish but whatever, I've been slamming my head too much lately17:30
dansmithyeah, I'd like that17:30
dansmithif that were to happen I'd tear up the current job application I'm working on17:30
dansmithI was just writing "I'm very qualified to flip burgers because my current job involves lots of futile repetitive behavior so I'm used to it"17:31
sean-k-mooney:)17:31
gmannyes it is in the gate17:32
dansmithgmann: is it?17:32
gmanndansmith: ah  mean approved but in check pipeline still, i just rechecked17:33
dansmithah okay17:33
gmannat leaast port sec grp test fix is merged, hopefully we will get all in-progress fixes during this weekend17:34
dansmithorly, I missed that.. that's definitely good17:37
dansmithI just noticed that we're running a non-voting expensive job in the gate queue17:47
dansmithnova-live-migration-ceph17:47
dansmithbetter fix that17:47
dansmither, hmm, it's not marked as such in .zuul.conf17:48
gmannyeah, i see it voting17:54
dansmithzuul status display says non-voting next to it17:55
dansmithhttps://zuul.opendev.org/t/openstack/status#87990517:55
gmanndansmith: ah, that is due to this? https://github.com/openstack/devstack-plugin-ceph/blob/master/.zuul.yaml#L12317:56
gmannparent job17:57
dansmithah, voting on the job def and not the queue? but why is the regular job voting?17:57
gmanndansmith: regular job ?17:57
dansmiththe regular ceph job17:58
dansmithmaybe it doesn't inherit from multinode?17:58
gmannyeah, multinode17:58
dansmithack17:58
dansmithI think we could take that out of the gate queue probably17:58
gmannyeah we should do that until that is voting17:59
dansmithack17:59
gmannand if no plan to fix in nearby may be move to periodic ?17:59
gmannrunning known failing non voting in check pipeline also costly 17:59
opendevreviewDan Smith proposed openstack/nova master: Remove n-v ceph live migration job from gate  https://review.opendev.org/c/openstack/nova/+/88999218:00
dansmithit doesn't fail that I know of18:00
dansmitheven if it's spotty, it's good to have it in check, but I think it's more just because it has been unstable in the past, and is not yet converted to cephadm, etc18:01
dansmithI think it's also reasonable to just not gate on it anyway, even if it's voting in check18:01
dansmithgiven that we require clean rechecks, I think we could probably trim down our gate jobs a bit anyway and reduce the number of fails we have there18:02
dansmiththe chances of threading the needle between check and gate for something that works on regular lm and not with ceph, that changed between check and gate is pretty dang unlikely, IMHO18:02
gmannack18:05
dansmithmelwitt: I can haz? https://review.opendev.org/c/openstack/nova/+/88999218:19
melwittsure why not18:20
dansmiththanks18:22
* dansmith holds his breath18:33
dansmithdamn18:34
dansmithI thought we were going to get three patches to merge18:34
dansmithbut no.. one failure in one job at the last minute, of course18:34
opendevreviewMerged openstack/nova master: Online migrate missing Instance.compute_id fields  https://review.opendev.org/c/openstack/nova/+/87990518:37
dansmithwhatever, I'll take it18:37

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!