| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-05-27 | |||
| 15:57:53 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Run encrypted volume tests in nova-live-migration https://review.opendev.org/c/openstack/nova/+/843621 | |
| 18:54:19 | opendevreview | Ghanshyam proposed openstack/os-vif master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/os-vif/+/840020 | |
| 19:07:18 | artom | sean-k-mooney, so, I can't reproduce it on master, and I think that's because https://review.opendev.org/c/openstack/nova/+/804230/3/nova/compute/manager.py fixed it | |
| 19:07:26 | artom | And I even reviewed it, too | |
| 19:07:43 | artom | You may be done for the day, feel free to ignore this | |
| 19:08:01 | artom | I'm using (potentially absent) you as a rubber duck (again) | |
| 19:09:42 | artom | Lemme try backporting that to stable/wallaby instead of my WIP thing | |
| 19:19:25 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: fake: Ensure need_legacy_block_device_info returns False https://review.opendev.org/c/openstack/nova/+/843678 | |
| 19:19:26 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843679 | |
| 19:19:27 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843680 | |
| 19:19:42 | sean-k-mooney | artom: o/ | |
| 19:19:51 | sean-k-mooney | artom: im not around for long | |
| 19:20:04 | sean-k-mooney | but ill look quickly | |
| 19:20:36 | sean-k-mooney | artom: ah yes cool | |
| 19:21:10 | sean-k-mooney | and there is a functional repoducer https://review.opendev.org/c/openstack/nova/+/804229/3 | |
| 19:21:45 | sean-k-mooney | artom: ya so i think that is a better approch vs stable only | |
| 19:22:15 | artom | For sure | |
| 19:22:18 | sean-k-mooney | artom: nice find | |
| 19:22:48 | artom | :) I tried on master with iscsi + cryptsetup, which should have hit the same problem | |
| 19:22:50 | artom | It didn't | |
| 19:22:56 | artom | So I went git log diving | |
| 19:23:08 | artom | Luckily, Lee's commit message was well written and was easy to find. | |
| 19:23:15 | sean-k-mooney | so thats in xena so ya wallby to trait i guess | |
| 19:23:35 | sean-k-mooney | *to train | |
| 19:23:43 | sean-k-mooney | for the backports | |
| 19:24:29 | artom | We're going to burn out the SRT team (well, our 1 Nova dude Amit) in a matter of days | |
| 19:24:42 | artom | Like, he'll need therapy | |
| 19:24:44 | artom | I almost do | |
| 19:25:15 | sean-k-mooney | it hopefuly wont be too bad of a backpor but the fact we ned py27 suport for trian is a pain | |
| 19:25:49 | sean-k-mooney | even wallaby seam kind of old to me honestly | |
| 19:26:02 | artom | The code itself is mostly OK | |
| 19:26:13 | sean-k-mooney | anyway im going ot finish up for the weekend but ill check it on monday | |
| 19:26:24 | artom | But the price to pay for all the func tests improvements since train is backports being a PITA | |
| 19:26:53 | sean-k-mooney | ya | |
| 19:59:25 | opendevreview | melanie witt proposed openstack/nova master: Update [scheduler] limit_tenants_to_placement_aggregate config help https://review.opendev.org/c/openstack/nova/+/843684 | |
| 20:48:33 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: fake: Ensure need_legacy_block_device_info returns False https://review.opendev.org/c/openstack/nova/+/843678 | |
| 20:48:34 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843679 | |
| 20:48:35 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843680 | |
| 21:09:10 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843679 | |
| 21:09:11 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843680 | |
| #openstack-nova - 2022-05-28 | |||
| 03:25:25 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843679 | |
| 03:25:26 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843680 | |
| 03:28:44 | opendevreview | melanie witt proposed openstack/nova master: WIP Test volume extend actions in the nova-next job https://review.opendev.org/c/openstack/nova/+/843700 | |
| 04:23:30 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843679 | |
| 04:23:31 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843680 | |
| 04:59:44 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843680 | |
| 04:59:45 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843702 | |
| 06:39:17 | opendevreview | melanie witt proposed openstack/nova master: WIP Test volume extend actions in the nova-next job https://review.opendev.org/c/openstack/nova/+/843700 | |
| #openstack-nova - 2022-05-29 | |||
| 00:14:15 | opendevreview | Ghanshyam proposed openstack/nova stable/victoria: DNM: Testing https://review.opendev.org/c/openstack/tempest/+/843182 https://review.opendev.org/c/openstack/nova/+/843188 | |
| 00:43:55 | opendevreview | melanie witt proposed openstack/nova master: Test attached volume extend actions in the nova-next job https://review.opendev.org/c/openstack/nova/+/843700 | |
| #openstack-nova - 2022-05-30 | |||
| 08:09:44 | bauzas | good morning Nova | |
| 08:13:00 | gibi | bauzas: o/ | |
| 08:29:09 | Uggla | bauzas, lucky guy. ;) | |
| 08:59:37 | jhartkopf | Hey, I am currently working on the client-side implementation for an approved spec for Zed. Is it correct that the CLI parser changes should be implemented in python-openstackclient while the API changes are implemented in python-novaclient? Is openstacksdk involved in this at all? | |
| 09:01:31 | Uggla | jhartkopf, yes for python-openstackclient and python-novaclient. | |
| 09:03:30 | Uggla | I think openstacksdk is not involved. But it is good to update it if required, for people who needs it. | |
| 09:10:23 | jhartkopf | Uggla: Alright, so I will focus on openstackclient and novaclient first, thanks! | |
| 09:13:22 | Uggla | jhartkopf, you can have a look at https://review.opendev.org/c/openstack/python-novaclient/+/831651 and https://review.opendev.org/c/openstack/python-openstackclient/+/831902, this is what I'm currently doing. Maybe not the best example, but it gives an idea. | |
| 09:21:05 | opendevreview | Rico Lin proposed openstack/nova-specs master: Add vIOMMU device support for libvirt driver https://review.opendev.org/c/openstack/nova-specs/+/840310 | |
| 09:22:00 | ricolin | ^^^ stephenfin: sean-k-mooney just update viommu spec, please kindly review again. Thanks ;) | |
| 09:40:28 | jhartkopf | Uggla: Yeah that looks similar to what I had in mind, thank you :) | |
| 09:41:51 | sean-k-mooney | ricolin: you have one error that should be fixed regarding the default for the viommu value but im +2 otherwise | |
| 10:15:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename [pci]passthrough_whitelist to device_spec https://review.opendev.org/c/openstack/nova/+/843834 | |
| 10:15:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP: skeleton for asserting PCI resources in Placement https://review.opendev.org/c/openstack/nova/+/843291 | |
| 10:57:08 | opendevreview | Rico Lin proposed openstack/nova-specs master: Add vIOMMU device support for libvirt driver https://review.opendev.org/c/openstack/nova-specs/+/840310 | |
| 10:57:37 | ricolin | sean-k-mooney: right, it makes no sense to default to enable with performance overhead | |
| 10:58:10 | ricolin | sean-k-mooney: stephenfin spec updated, thanks for the quick reviews! | |
| 10:59:16 | sean-k-mooney | +2 | |
| 10:59:31 | ricolin | sean-k-mooney: thanks:) | |
| 10:59:32 | sean-k-mooney | stephenfin: ill leve the workflow to you | |
| 11:56:23 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix typos in help messages https://review.opendev.org/c/openstack/nova/+/843843 | |
| 11:58:47 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix typos in help messages https://review.opendev.org/c/openstack/nova/+/843843 | |
| 12:10:24 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fix typos https://review.opendev.org/c/openstack/nova/+/843127 | |
| 13:28:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename exception.PciConfigInvalidWhitelist to PciConfigInvalidSpec https://review.opendev.org/c/openstack/nova/+/843861 | |
| 13:28:39 | opendevreview | Balazs Gibizer proposed openstack/nova master: WIP: skeleton for asserting PCI resources in Placement https://review.opendev.org/c/openstack/nova/+/843291 | |
| 13:28:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename whitelist in tests https://review.opendev.org/c/openstack/nova/+/843862 | |
| 13:40:46 | opendevreview | Stephen Finucane proposed openstack/nova master: docs: Correct path used for config on Hyper-V https://review.opendev.org/c/openstack/nova/+/833248 | |
| 14:06:00 | opendevreview | Merged openstack/nova-specs master: Add vIOMMU device support for libvirt driver https://review.opendev.org/c/openstack/nova-specs/+/840310 | |
| 16:06:27 | bauzas | elodilles: arf, I wanted to update the nova-stable-maint member list as I just wrote but I saw that the ownership of this Gerrit group is made of stable-maint team members, whose I'm not | |
| 16:07:09 | bauzas | elodilles: you should be able to update the member list as you're from the release team which is subset of stable-maint | |
| 16:07:34 | bauzas | elodilles: could you maybe also change ownership and put nova-stable-maint as owner of itself ? | |
| 16:07:59 | bauzas | https://review.opendev.org/admin/groups/540,members and https://review.opendev.org/admin/groups/540 for context | |
| 16:28:43 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | |
| 16:28:56 | opendevreview | ribaudr proposed openstack/python-novaclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-novaclient/+/831651 | |
| 16:54:15 | opendevreview | Merged openstack/nova master: Fix duplicates keys https://review.opendev.org/c/openstack/nova/+/843605 | |
| 16:54:24 | opendevreview | Merged openstack/nova master: Add missing condition https://review.opendev.org/c/openstack/nova/+/843565 | |
| 16:54:32 | opendevreview | Merged openstack/nova master: Accept both 1 and Y as AMD SEV KVM kernel param value https://review.opendev.org/c/openstack/nova/+/843254 | |
| 16:54:40 | opendevreview | Merged openstack/nova master: Fix race condition in _get_pci_passthrough_devices https://review.opendev.org/c/openstack/nova/+/840993 | |
| 16:54:57 | opendevreview | Merged openstack/nova master: docs: Correct path used for config on Hyper-V https://review.opendev.org/c/openstack/nova/+/833248 | |
| 17:06:11 | opendevreview | Balazs Gibizer proposed openstack/nova stable/yoga: Accept both 1 and Y as AMD SEV KVM kernel param value https://review.opendev.org/c/openstack/nova/+/843819 | |
| 17:08:05 | opendevreview | Balazs Gibizer proposed openstack/nova stable/xena: Accept both 1 and Y as AMD SEV KVM kernel param value https://review.opendev.org/c/openstack/nova/+/843938 | |
| 17:08:56 | opendevreview | Balazs Gibizer proposed openstack/nova stable/wallaby: Accept both 1 and Y as AMD SEV KVM kernel param value https://review.opendev.org/c/openstack/nova/+/843939 | |
| 17:24:09 | opendevreview | Rajesh Tailor proposed openstack/nova stable/yoga: Add missing condition https://review.opendev.org/c/openstack/nova/+/843820 | |
| 17:25:02 | opendevreview | Rajesh Tailor proposed openstack/nova stable/xena: Add missing condition https://review.opendev.org/c/openstack/nova/+/843821 | |
| 18:15:48 | elodilles | bauzas: i've added gibi sean-k-mooney and stephenfin to nova-stable-maint group | |
| 18:16:37 | elodilles | bauzas: the ownership can only be edited in the project-config repository if i'm not mistaken | |
| 18:17:22 | artom | gibi, "Failed to detach device <snip> Libvirt did not report any error but the device is still in the config." that's the volume detach issue, yeah? | |
| 18:18:45 | gibi | artom: you should see | |
| 18:18:46 | sean-k-mooney | elodilles: in general anyone in a group can add members | |
| 18:18:46 | gibi | Run out of retry while detaching device .* with device alias .* from instance .* from the .* domain config. Device is still attached to the guest. | |