| *** mfo is now known as Guest717 | 05:40 | |
| *** mfo_ is now known as mfo | 05:40 | |
| *** bhagyashris is now known as bhagyashris|rover | 05:44 | |
| bauzas | good morning Nova | 08:09 |
|---|---|---|
| * bauzas is back from long weekend | 08:09 | |
| gibi | bauzas: o/ | 08:13 |
| Uggla | bauzas, lucky guy. ;) | 08:29 |
| 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? | 08:59 |
| Uggla | jhartkopf, yes for python-openstackclient and python-novaclient. | 09:01 |
| Uggla | I think openstacksdk is not involved. But it is good to update it if required, for people who needs it. | 09:03 |
| jhartkopf | Uggla: Alright, so I will focus on openstackclient and novaclient first, thanks! | 09:10 |
| 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:13 |
| 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:21 |
| ricolin | ^^^ stephenfin: sean-k-mooney just update viommu spec, please kindly review again. Thanks ;) | 09:22 |
| jhartkopf | Uggla: Yeah that looks similar to what I had in mind, thank you :) | 09:40 |
| sean-k-mooney | ricolin: you have one error that should be fixed regarding the default for the viommu value but im +2 otherwise | 09:41 |
| 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 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rename [pci]passthrough_whitelist to device_spec https://review.opendev.org/c/openstack/nova/+/843834 | 10:15 |
| 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 |
| ricolin | sean-k-mooney: right, it makes no sense to default to enable with performance overhead | 10:57 |
| ricolin | sean-k-mooney: stephenfin spec updated, thanks for the quick reviews! | 10:58 |
| sean-k-mooney | +2 | 10:59 |
| ricolin | sean-k-mooney: thanks:) | 10:59 |
| sean-k-mooney | stephenfin: ill leve the workflow to you | 10:59 |
| opendevreview | Rajesh Tailor proposed openstack/nova master: Fix typos in help messages https://review.opendev.org/c/openstack/nova/+/843843 | 11:56 |
| opendevreview | Rajesh Tailor proposed openstack/nova master: Fix typos in help messages https://review.opendev.org/c/openstack/nova/+/843843 | 11:58 |
| opendevreview | Rajesh Tailor proposed openstack/nova master: Fix typos https://review.opendev.org/c/openstack/nova/+/843127 | 12:10 |
| 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 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rename exception.PciConfigInvalidWhitelist to PciConfigInvalidSpec https://review.opendev.org/c/openstack/nova/+/843861 | 13:28 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Rename whitelist in tests https://review.opendev.org/c/openstack/nova/+/843862 | 13:28 |
| opendevreview | Stephen Finucane proposed openstack/nova master: docs: Correct path used for config on Hyper-V https://review.opendev.org/c/openstack/nova/+/833248 | 13:40 |
| opendevreview | Merged openstack/nova-specs master: Add vIOMMU device support for libvirt driver https://review.opendev.org/c/openstack/nova-specs/+/840310 | 14:06 |
| 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:06 |
| 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 |
| bauzas | elodilles: could you maybe also change ownership and put nova-stable-maint as owner of itself ? | 16:07 |
| bauzas | https://review.opendev.org/admin/groups/540,members and https://review.opendev.org/admin/groups/540 for context | 16:07 |
| opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | 16:28 |
| 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:28 |
| opendevreview | Merged openstack/nova master: Fix duplicates keys https://review.opendev.org/c/openstack/nova/+/843605 | 16:54 |
| opendevreview | Merged openstack/nova master: Add missing condition https://review.opendev.org/c/openstack/nova/+/843565 | 16:54 |
| 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 |
| opendevreview | Merged openstack/nova master: Fix race condition in _get_pci_passthrough_devices https://review.opendev.org/c/openstack/nova/+/840993 | 16:54 |
| opendevreview | Merged openstack/nova master: docs: Correct path used for config on Hyper-V https://review.opendev.org/c/openstack/nova/+/833248 | 16:54 |
| 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:06 |
| 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 |
| 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:08 |
| opendevreview | Rajesh Tailor proposed openstack/nova stable/yoga: Add missing condition https://review.opendev.org/c/openstack/nova/+/843820 | 17:24 |
| opendevreview | Rajesh Tailor proposed openstack/nova stable/xena: Add missing condition https://review.opendev.org/c/openstack/nova/+/843821 | 17:25 |
| elodilles | bauzas: i've added gibi sean-k-mooney and stephenfin to nova-stable-maint group | 18:15 |
| elodilles | bauzas: the ownership can only be edited in the project-config repository if i'm not mistaken | 18:16 |
| 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:17 |
| gibi | artom: you should see | 18:18 |
| 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 |
| sean-k-mooney | elodilles: in general anyone in a group can add members | 18:18 |
| gibi | when the retry loop gives up | 18:18 |
| artom | gibi, yeah, that sounds right, keeps retrying and eventually stops/raises. | 18:19 |
| gibi | if you see that then it is the detach issue | 18:19 |
| elodilles | sean-k-mooney: i thought the same but it seems bauzas had no rights | 18:19 |
| 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 |
| gibi | elodilles: thanks now I can go and ruin the stable branches too :D | 18: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 |
| sean-k-mooney | elodilles: i can add and remove member to nova-core | 18:20 |
| sean-k-mooney | so there is defintly a delta in the config | 18:20 |
| elodilles | sean-k-mooney: but anyway, now it is said to be managed by the teams themselves | 18:20 |
| sean-k-mooney | elodilles: ya the audit log setction is there for me in nova-core | 18:21 |
| sean-k-mooney | https://review.opendev.org/admin/groups/54f6a1ec13b7453596635e8708f1b60bfd281ebd,audit-log | 18:21 |
| sean-k-mooney | but not in nova-stable-maint | 18:21 |
| elodilles | gibi: dooooooooon't :-o be careful :) and read the policy (or ask) if something is not clear :D | 18:21 |
| sean-k-mooney | so that might be the differnce | 18:21 |
| gibi | elodilles: sure ;) | 18:22 |
| sean-k-mooney | elodilles: im sure we will still lean hevaly on your guidance in these matters | 18:23 |
| 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 |
| sean-k-mooney | o/ | 18:23 |
| elodilles | sean-k-mooney: o:) thanks :) i'll try to help :) | 18:24 |
| elodilles | sean-k-mooney: have a nice evening! o/ | 18:24 |
| elodilles | bauzas: i've also removed Matt and Lee as you wanted | 18:25 |
| 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 |
| 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 |
| 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:33 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 |
| 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 | 18:38 |
| *** iurygregory_ is now known as iurygregory | 19:37 | |
| 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 |
| 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 |
| 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:25 |
| mloza1 | hello, how can I tell if a VM created in to a specific AZ? | 20:49 |
| opendevreview | Merged openstack/os-vif master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/os-vif/+/791970 | 21:24 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!