| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-05-30 | |||
| 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: WIP: skeleton for asserting PCI resources in Placement https://review.opendev.org/c/openstack/nova/+/843291 | |
| 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: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: WIP: skeleton for asserting PCI resources in Placement https://review.opendev.org/c/openstack/nova/+/843291 | |
| 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: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 | gibi | Run out of retry while detaching device .* with device alias .* from instance .* from the .* domain config. Device is still attached to the guest. | |
| 18:18:46 | sean-k-mooney | elodilles: in general anyone in a group can add members | |
| 18:18:56 | gibi | when the retry loop gives up | |
| 18:19:05 | artom | gibi, yeah, that sounds right, keeps retrying and eventually stops/raises. | |
| 18:19:18 | gibi | if you see that then it is the detach issue | |
| 18:19:28 | elodilles | sean-k-mooney: i thought the same but it seems bauzas had no rights | |
| 18:19:48 | sean-k-mooney | neitehr do i https://review.opendev.org/admin/groups/21ce6c287ea33809980b2dec53915b07830cdb11,members but i think the config of this group is differnt then the defaul | |
| 18:19:51 | gibi | elodilles: thanks now I can go and ruin the stable branches too :D | |
| 18:20:19 | elodilles | sean-k-mooney: probably because in the past only stable-maint group audited everyone before adding them to a <project>-stable-maint group | |
| 18:20:24 | sean-k-mooney | elodilles: i can add and remove member to nova-core | |
| 18:20:42 | sean-k-mooney | so there is defintly a delta in the config | |
| 18:20:49 | elodilles | sean-k-mooney: but anyway, now it is said to be managed by the teams themselves | |
| 18:21:25 | sean-k-mooney | elodilles: ya the audit log setction is there for me in nova-core | |
| 18:21:27 | sean-k-mooney | https://review.opendev.org/admin/groups/54f6a1ec13b7453596635e8708f1b60bfd281ebd,audit-log | |
| 18:21:49 | sean-k-mooney | but not in nova-stable-maint | |
| 18:21:51 | elodilles | gibi: dooooooooon't :-o be careful :) and read the policy (or ask) if something is not clear :D | |
| 18:21:55 | sean-k-mooney | so that might be the differnce | |
| 18:22:42 | gibi | elodilles: sure ;) | |
| 18:23:08 | sean-k-mooney | elodilles: im sure we will still lean hevaly on your guidance in these matters | |
| 18:23:54 | sean-k-mooney | although as i siad downstream a few minutes ago, a doc review i was doing used all of my free brain power so im goign to call it a day | |
| 18:23:55 | sean-k-mooney | o/ | |
| 18:24:11 | elodilles | sean-k-mooney: o:) thanks :) i'll try to help :) | |
| 18:24:27 | elodilles | sean-k-mooney: have a nice evening! o/ | |
| 18:25:50 | elodilles | bauzas: i've also removed Matt and Lee as you wanted | |
| 18:33:51 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: fake: Ensure need_legacy_block_device_info returns False https://review.opendev.org/c/openstack/nova/+/843947 | |
| 18:33:52 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843948 | |
| 18:33:53 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843949 | |
| 18:36:57 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: fake: Ensure need_legacy_block_device_info returns False https://review.opendev.org/c/openstack/nova/+/843947 | |
| 18:36:58 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843948 | |
| 18:36:59 | opendevreview | Artom Lifshitz proposed openstack/nova stable/victoria: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843949 | |
| 18:38:23 | opendevreview | Artom Lifshitz proposed openstack/nova stable/ussuri: fake: Ensure need_legacy_block_device_info returns False https://review.opendev.org/c/openstack/nova/+/843950 | |
| 18:38:24 | opendevreview | Artom Lifshitz proposed openstack/nova stable/ussuri: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843951 | |
| 18:38:25 | opendevreview | Artom Lifshitz proposed openstack/nova stable/ussuri: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843952 | |
| 20:25:39 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: fake: Ensure need_legacy_block_device_info returns False https://review.opendev.org/c/openstack/nova/+/843958 | |
| 20:25:40 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: Add a regression test for bug 1939545 https://review.opendev.org/c/openstack/nova/+/843959 | |
| 20:25:41 | opendevreview | Artom Lifshitz proposed openstack/nova stable/train: compute: Ensure updates to bdms during pre_live_migration are saved https://review.opendev.org/c/openstack/nova/+/843960 | |
| 20:49:40 | mloza1 | hello, how can I tell if a VM created in to a specific AZ? | |
| 21:24:30 | opendevreview | Merged openstack/os-vif master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/os-vif/+/791970 | |
| #openstack-nova - 2022-05-31 | |||
| 07:37:30 | gibi | bauzas: o/ There is a good chance that I have to be away from the keyboard during the team meeting today | |
| 08:07:07 | gibi | stephenfin, sean-k-mooney[m]: your view would me much appreciated here https://review.opendev.org/c/openstack/nova/+/843443/1#message-ac73126bbaa46cbbccdccb15c4c6d28358a328fc | |
| 08:16:26 | bauzas | elodilles: thanks for the nova-stable-maint additions ;) | |
| 08:16:59 | bauzas | gibi: ack, no worries | |
| 08:28:16 | elodilles | bauzas: np :) | |
| 09:48:50 | sean-k-mooney2 | gibi: ah the autopep8 change. i intentionally did not put it in test-reqirements since others experessed that they wanted ti to be optional and not require autopep8 | |
| 09:49:11 | sean-k-mooney | am | |
| 09:49:18 | sean-k-mooney | so we could do as you suggest | |
| 09:49:23 | sean-k-mooney | and move it there and pin | |
| 09:49:30 | gibi | I affraid that we start listing deps in different palces | |
| 09:49:31 | gibi | places | |
| 09:49:37 | sean-k-mooney | or we could add it to global requirements and pin in upper-constraits | |
| 09:49:58 | sean-k-mooney | gibi: now that devestack does not install test-requiremetns | |
| 09:50:00 | gibi | I don't know if we track test only requirements in global | |
| 09:50:10 | sean-k-mooney | its less of an issue with me to do that | |
| 09:50:19 | sean-k-mooney | gibi: we have flake8 i think | |
| 09:50:43 | sean-k-mooney | and pytest | |
| 09:50:57 | gibi | noe flake is not in global but pytest is in global | |
| 09:50:59 | sean-k-mooney | gibi: https://github.com/openstack/requirements/blob/master/global-requirements.txt#L396= | |