| *** tosky has quit IRC | 00:02 | |
| *** ociuhandu has quit IRC | 00:03 | |
| *** iurygregory has quit IRC | 00:06 | |
| *** artom has quit IRC | 00:16 | |
| *** artom has joined #openstack-nova | 00:16 | |
| *** xarlos has joined #openstack-nova | 00:18 | |
| *** songwenping__ has joined #openstack-nova | 00:18 | |
| *** rcernin_ has joined #openstack-nova | 00:19 | |
| *** swp20 has quit IRC | 00:19 | |
| *** macz_ has quit IRC | 00:19 | |
| *** rcernin has quit IRC | 00:20 | |
| *** amodi has quit IRC | 00:26 | |
| *** LinPeiWen84 has joined #openstack-nova | 00:42 | |
| *** iurygregory has joined #openstack-nova | 00:43 | |
| *** artom has quit IRC | 00:45 | |
| *** songwenping__ has quit IRC | 00:48 | |
| *** sapd1 has quit IRC | 00:48 | |
| *** tbachman has quit IRC | 00:59 | |
| *** tbachman has joined #openstack-nova | 01:00 | |
| *** jamesden_ has joined #openstack-nova | 01:00 | |
| *** abhishekk has quit IRC | 01:02 | |
| *** LinPeiWen84 has quit IRC | 01:02 | |
| *** zenkuro has quit IRC | 01:03 | |
| *** jamesdenton has quit IRC | 01:03 | |
| *** zenkuro has joined #openstack-nova | 01:03 | |
| *** abhishekk has joined #openstack-nova | 01:04 | |
| *** LinPeiWen20 has joined #openstack-nova | 01:04 | |
| *** mlavalle has quit IRC | 01:05 | |
| *** hamalq has quit IRC | 01:26 | |
| *** martinkennelly has quit IRC | 01:39 | |
| *** sapd1 has joined #openstack-nova | 01:46 | |
| *** spatel has joined #openstack-nova | 01:51 | |
| *** sapd1 has quit IRC | 01:59 | |
| *** jangutter has joined #openstack-nova | 02:11 | |
| *** jangutter has quit IRC | 02:12 | |
| *** jangutter has joined #openstack-nova | 02:13 | |
| *** jangutter_ has quit IRC | 02:13 | |
| *** rouk has joined #openstack-nova | 02:36 | |
| *** zzzeek has quit IRC | 03:04 | |
| *** zzzeek has joined #openstack-nova | 03:05 | |
| *** zenkuro has quit IRC | 03:09 | |
| *** hemanth_n has joined #openstack-nova | 03:12 | |
| *** rcernin_ has quit IRC | 03:23 | |
| *** mkrai has joined #openstack-nova | 03:36 | |
| *** jmlowe has quit IRC | 03:39 | |
| *** rcernin_ has joined #openstack-nova | 04:01 | |
| *** psachin has joined #openstack-nova | 04:11 | |
| *** ratailor has joined #openstack-nova | 04:33 | |
| *** mkrai has quit IRC | 04:34 | |
| *** mkrai_ has joined #openstack-nova | 04:34 | |
| *** sapd1 has joined #openstack-nova | 04:34 | |
| *** sapd1 has quit IRC | 04:42 | |
| *** LinPeiWen20 has quit IRC | 05:06 | |
| *** hemanth_n has quit IRC | 05:07 | |
| *** hemanth_n has joined #openstack-nova | 05:07 | |
| *** LinPeiWen15 has joined #openstack-nova | 05:13 | |
| *** spatel has quit IRC | 05:32 | |
| *** iurygregory has quit IRC | 05:33 | |
| *** vishalmanchanda has joined #openstack-nova | 05:52 | |
| *** gyee has quit IRC | 05:59 | |
| *** bbowen has quit IRC | 06:31 | |
| *** bbowen has joined #openstack-nova | 06:31 | |
| *** nightmare_unreal has joined #openstack-nova | 06:45 | |
| *** slaweq has joined #openstack-nova | 07:04 | |
| *** openstackgerrit has joined #openstack-nova | 07:07 | |
| openstackgerrit | Hemanth N proposed openstack/nova stable/stein: Update pci stat pools based on PCI device changes https://review.opendev.org/c/openstack/nova/+/761727 | 07:07 |
|---|---|---|
| *** mkrai_ has quit IRC | 07:07 | |
| *** avolkov has joined #openstack-nova | 07:14 | |
| *** luksky has joined #openstack-nova | 07:23 | |
| *** rcernin_ has quit IRC | 07:28 | |
| *** slaweq has quit IRC | 07:30 | |
| *** rcernin_ has joined #openstack-nova | 07:32 | |
| *** xek has joined #openstack-nova | 07:35 | |
| *** sapd1 has joined #openstack-nova | 07:35 | |
| *** sapd1 has quit IRC | 07:59 | |
| *** slaweq has joined #openstack-nova | 08:00 | |
| *** mkrai_ has joined #openstack-nova | 08:00 | |
| *** mkrai_ has quit IRC | 08:04 | |
| *** tesseract has joined #openstack-nova | 08:09 | |
| *** andrewbonney has joined #openstack-nova | 08:09 | |
| *** rpittau|afk is now known as rpittau | 08:17 | |
| *** sapd1 has joined #openstack-nova | 08:26 | |
| *** tesseract has quit IRC | 08:31 | |
| *** tesseract has joined #openstack-nova | 08:31 | |
| *** mkrai has joined #openstack-nova | 08:32 | |
| *** rcernin_ has quit IRC | 08:35 | |
| *** mkrai_ has joined #openstack-nova | 08:40 | |
| *** abhishekk has quit IRC | 08:40 | |
| *** mkrai has quit IRC | 08:41 | |
| *** hemanth_n has quit IRC | 08:41 | |
| *** psachin has quit IRC | 08:41 | |
| *** mgariepy has quit IRC | 08:41 | |
| *** abhishekk has joined #openstack-nova | 08:41 | |
| *** mgariepy has joined #openstack-nova | 08:41 | |
| *** hemanth_n has joined #openstack-nova | 08:41 | |
| *** tosky has joined #openstack-nova | 08:47 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List SG API https://review.opendev.org/c/openstack/nova/+/766726 | 08:49 |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from Flavor Access APIs https://review.opendev.org/c/openstack/nova/+/767704 | 08:49 |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Replaces tenant_id with project_id from List/Show usage APIs https://review.opendev.org/c/openstack/nova/+/768509 | 08:49 |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Replace tenants* with projects* of policies https://review.opendev.org/c/openstack/nova/+/765315 | 08:49 |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Replace os-simple-tenant-usage with os-simple-project-usage https://review.opendev.org/c/openstack/nova/+/768852 | 08:49 |
| *** sapd1 has quit IRC | 08:50 | |
| *** spatel has joined #openstack-nova | 09:09 | |
| *** luksky has quit IRC | 09:12 | |
| *** luksky has joined #openstack-nova | 09:13 | |
| *** spatel has quit IRC | 09:14 | |
| *** DinaBelova has quit IRC | 09:17 | |
| *** tristanC has quit IRC | 09:18 | |
| *** DinaBelova has joined #openstack-nova | 09:19 | |
| *** tristanC has joined #openstack-nova | 09:20 | |
| *** songwenping__ has joined #openstack-nova | 09:31 | |
| *** brinzhang_ has quit IRC | 09:34 | |
| *** songwenping_ has quit IRC | 09:35 | |
| *** songwenping__ has quit IRC | 09:47 | |
| *** songwenping__ has joined #openstack-nova | 09:48 | |
| *** derekh has joined #openstack-nova | 09:49 | |
| *** brinzhang has joined #openstack-nova | 09:52 | |
| brinzhang | stephenfin, gibi: there is a small change for the host.nodename, pls review, it's updated the sean-k-monney's comments in the shelve/unshelve patch https://review.opendev.org/c/openstack/nova/+/762499 | 09:54 |
| *** martinkennelly has joined #openstack-nova | 10:07 | |
| *** ociuhandu has joined #openstack-nova | 10:07 | |
| stephenfin | brinzhang: ack | 10:09 |
| stephenfin | lyarwood: Does the 'nova server-migration-list' command only list *ongoing* migrations or does it list all migrations in the past? | 10:10 |
| stephenfin | (I don't have a multi-node deployment or I'd test this myself) | 10:10 |
| stephenfin | I tried to migrate a server on single node deployment, which justifiably failed, but it doesn't appear in that list | 10:10 |
| *** ociuhandu has quit IRC | 10:12 | |
| brinzhang | stephenfin: thanks | 10:16 |
| lyarwood | stephenfin: I think it's all | 10:16 |
| lyarwood | stephenfin: I don't have an env to check | 10:17 |
| *** mkrai_ has quit IRC | 10:18 | |
| *** rpittau is now known as rpittau|bbl | 10:20 | |
| stephenfin | Okay, I'll say all for the OSC docs so | 10:20 |
| *** ociuhandu has joined #openstack-nova | 10:26 | |
| *** martinkennelly has quit IRC | 10:30 | |
| *** martinkennelly has joined #openstack-nova | 10:30 | |
| *** LinPeiWen15 has quit IRC | 10:31 | |
| *** priteau has joined #openstack-nova | 10:32 | |
| *** rcernin_ has joined #openstack-nova | 10:34 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: libvirt: Add support for flavor and image defined ephemeral encryption https://review.opendev.org/c/openstack/nova-specs/+/771474 | 10:38 |
| lyarwood | gibi: ^ thanks for the review | 10:38 |
| lyarwood | stephenfin / bauzas ; can I ask for reviews on that spec today please? Appreciate it's late etc, I should be able to respin quickly today outside of ~13:30-15:00 GMT | 10:39 |
| stephenfin | sure thing | 10:39 |
| lyarwood | thanks | 10:39 |
| stephenfin | looking at artom's spec atm, that's next on the queue | 10:39 |
| lyarwood | ack thanks, I'll also try to look at specs this morning | 10:41 |
| *** rcernin_ has quit IRC | 10:41 | |
| *** dtantsur|afk is now known as dtantsur | 10:44 | |
| *** mkrai_ has joined #openstack-nova | 10:49 | |
| *** ociuhandu has quit IRC | 10:54 | |
| *** ociuhandu_ has joined #openstack-nova | 10:54 | |
| *** ociuhandu_ has quit IRC | 10:54 | |
| *** ociuhandu has joined #openstack-nova | 10:54 | |
| hemanth_n | lyarwood stephenfin elod: sorry to bother you again, can you review stein backported patch https://review.opendev.org/c/openstack/nova/+/761727 when you get some time | 11:09 |
| *** ociuhandu has quit IRC | 11:14 | |
| *** ociuhandu_ has joined #openstack-nova | 11:14 | |
| *** rpittau|bbl is now known as rpittau | 11:24 | |
| *** iurygregory has joined #openstack-nova | 11:27 | |
| *** macz_ has joined #openstack-nova | 11:34 | |
| lyarwood | hemanth_n: looking | 11:38 |
| *** macz_ has quit IRC | 11:39 | |
| openstackgerrit | Merged openstack/nova-specs master: `socket` PCI NUMA-affinity Policy https://review.opendev.org/c/openstack/nova-specs/+/765551 | 11:40 |
| openstackgerrit | Merged openstack/nova-specs master: remove unicode from code https://review.opendev.org/c/openstack/nova-specs/+/769260 | 11:40 |
| *** ociuhandu_ has quit IRC | 11:42 | |
| *** rcernin_ has joined #openstack-nova | 11:42 | |
| *** LinPeiWen52 has joined #openstack-nova | 11:42 | |
| *** ratailor has quit IRC | 11:44 | |
| sean-k-mooney | stephenfin: this is why the trait is needed https://review.opendev.org/c/openstack/nova-specs/+/764999/4/specs/wallaby/approved/libvirt-vdpa-support.rst#79 | 11:49 |
| stephenfin | sean-k-mooney: so it's an optimization. Would it make sense to avoid this pre-optimization and suggest that people use provider.yaml if they hit the issue? | 11:51 |
| sean-k-mooney | its not really an optimisation in that you cant limit the placment allcoation set if you dont have it | 11:52 |
| sean-k-mooney | if we were to go the provider.yaml route | 11:52 |
| sean-k-mooney | then it would be a custom trait | 11:52 |
| sean-k-mooney | whcih we can do but i would prefer to do that in code | 11:52 |
| sean-k-mooney | i can drop it but it fells like a hack to not have it | 11:52 |
| stephenfin | True. Same issue with PCI devices, NUMA topologies etc. though | 11:53 |
| sean-k-mooney | not entirely the same thign although i waint traits for numa support | 11:53 |
| stephenfin | Yeah, I'd agree with this normally but it does feel like we're using a trait as a stand in for proper resource reporting | 11:53 |
| sean-k-mooney | i just have not needed them | 11:53 |
| sean-k-mooney | well traits are for capablities | 11:53 |
| sean-k-mooney | lets see what gibi thinks and ill go with whatever the two of ye decied unless other weigh in | 11:54 |
| stephenfin | Yeah, it's a very fuzzy line, that's for sure | 11:54 |
| stephenfin | Makes sense | 11:54 |
| stephenfin | as for your PCI devices in placement question, RE: what to do with VFs when a PF is consumed | 11:55 |
| stephenfin | I'd just kill that "feature" | 11:55 |
| stephenfin | make it necessary to state whether you want to allow passthrough of the PF or the VFs, and only report one | 11:55 |
| sean-k-mooney | well it has been a feature for a very long time making it so you dont have to pre allocated devices | 11:56 |
| sean-k-mooney | but its very similar to how isolate used to work | 11:56 |
| sean-k-mooney | its the same problem form a plamcent point of view | 11:56 |
| stephenfin | Yup | 11:56 |
| sean-k-mooney | so killing it would be valid but contentious | 11:56 |
| sean-k-mooney | which is why i dont think we will agree on that by tomorowy | 11:56 |
| stephenfin | I agree | 11:56 |
| stephenfin | Plus, you already have two specs | 11:56 |
| stephenfin | More than enough for the three months we have left :) | 11:56 |
| stephenfin | Anywho | 11:57 |
| * stephenfin goes for lunch | 11:57 | |
| *** bbowen has quit IRC | 11:57 | |
| *** bbowen has joined #openstack-nova | 11:59 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: libvirt: Add support for flavor and image defined ephemeral encryption https://review.opendev.org/c/openstack/nova-specs/+/771474 | 12:02 |
| *** raildo has joined #openstack-nova | 12:03 | |
| sean-k-mooney | stephenfin: we dont have 3 months left | 12:03 |
| sean-k-mooney | stephenfin: m3 is march 11th so half that we have 6 weeks left but i agree | 12:04 |
| sean-k-mooney | its 3 months till the offical release | 12:05 |
| sean-k-mooney | april 15th | 12:05 |
| *** zzzeek has quit IRC | 12:18 | |
| *** zzzeek has joined #openstack-nova | 12:21 | |
| openstackgerrit | John Garbutt proposed openstack/nova master: Fix PCI passthrough race on reschedule (refresh) https://review.opendev.org/c/openstack/nova/+/710848 | 12:22 |
| *** mkrai_ has quit IRC | 12:38 | |
| *** ociuhandu has joined #openstack-nova | 12:42 | |
| *** ociuhandu has quit IRC | 12:44 | |
| *** ociuhandu has joined #openstack-nova | 12:45 | |
| *** ociuhandu has quit IRC | 12:54 | |
| *** ociuhandu has joined #openstack-nova | 12:56 | |
| *** rcernin_ has quit IRC | 12:58 | |
| *** ociuhandu has quit IRC | 13:00 | |
| *** rcernin_ has joined #openstack-nova | 13:08 | |
| sean-k-mooney | johnthetubaguy: since your about QQ do you know if anyoen is working on unified limits this cycle | 13:09 |
| sean-k-mooney | johnthetubaguy: i dont see a spec approved just wondering if that is on holed indefinetly or are people still makeing progress? melwitt you might also know? | 13:10 |
| *** spatel has joined #openstack-nova | 13:10 | |
| *** spatel has quit IRC | 13:15 | |
| *** ociuhandu has joined #openstack-nova | 13:19 | |
| *** artom has joined #openstack-nova | 13:22 | |
| *** ociuhandu has quit IRC | 13:26 | |
| *** sapd1 has joined #openstack-nova | 13:27 | |
| *** dasp_ has joined #openstack-nova | 13:31 | |
| *** whoami-rajat___ has joined #openstack-nova | 13:31 | |
| openstackgerrit | Merged openstack/nova master: libvirt: Mock get_arch during some stable rescue unit tests https://review.opendev.org/c/openstack/nova/+/769916 | 13:32 |
| *** songwenping__ has quit IRC | 13:33 | |
| *** dasp has quit IRC | 13:34 | |
| *** brinzhang has quit IRC | 13:37 | |
| *** zenkuro has joined #openstack-nova | 13:38 | |
| *** amodi has joined #openstack-nova | 13:38 | |
| *** ociuhandu has joined #openstack-nova | 13:38 | |
| gibi | stephenfin: we have customers that are relying the feature that allows consuming the whole PF or consuming just VFs from it dynamically. So while I like the idea to uncomplicate things I have to agree with sean-k-mooney that this feature is alreasy sold and supported out there | 13:41 |
| gibi | sean-k-mooney: which trait specifically you dicussed above with stephenfin above? | 13:42 |
| * gibi had a doctor's appointment that made IRC hard to follow | 13:43 | |
| *** ociuhandu has quit IRC | 13:45 | |
| sean-k-mooney | gibi: the vdpa one | 13:45 |
| sean-k-mooney | gibi: ill get you the link to the comment in the spec | 13:46 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova-specs/+/764999/4/specs/wallaby/approved/libvirt-vdpa-support.rst#79 | 13:46 |
| *** ociuhandu has joined #openstack-nova | 13:47 | |
| sean-k-mooney | gibi: i can drop the trait and rely on the pci passthough filter but i wante to provide a slightly better schdulign expericne before looking at tracking in plamcnet next cycle when i have more time/brain power | 13:47 |
| sean-k-mooney | gibi: unfortuetlly the PF/VF thing is the same problem we had with hyperthread and PCPUs | 13:48 |
| sean-k-mooney | fundementally we cant express requirments to placment that require different resouced dependding on the selected host | 13:48 |
| sean-k-mooney | so if we want to track pci devices in plamcnet in the long term we either have to try and update the allcoation candiate after we claim it or we have to remove the fature | 13:49 |
| *** brinzhang has joined #openstack-nova | 13:49 | |
| sean-k-mooney | the same way we removed the ablity for the isolate policy to claim both hypertreads on host with SMT enabled | 13:49 |
| *** swp20 has joined #openstack-nova | 13:50 | |
| sean-k-mooney | anywya pci device in placment is hopefully out of scope for vdpa this cycle | 13:50 |
| *** brinzhang has quit IRC | 13:51 | |
| *** swp20 has quit IRC | 13:51 | |
| *** ociuhandu has quit IRC | 13:51 | |
| sean-k-mooney | gibi: so the options we have are no trait and just rely on pci passhtough filter, standard trait added by prefilter, custom trati added by prefilter, and custom trait added by provider.yaml and manually in the flavor/image | 13:51 |
| *** brinzhang has joined #openstack-nova | 13:51 | |
| *** derekh has quit IRC | 13:51 | |
| *** ociuhandu has joined #openstack-nova | 13:54 | |
| * gibi thinks | 13:56 | |
| gibi | I agree that currenty no support in placement to make some VF resource unavailable if the parent PF is consumed. Current it is done by the PCI tracker and I guess we can keep this behavior in the compute node even after PCI is modelled in placement. E.g. the pci claim that happens on the compute can simply reserve the VFs in placement if the PF is claimed | 13:59 |
| gibi | it has the same racy behavior as of today | 14:00 |
| gibi | but not worse | 14:00 |
| sean-k-mooney | yep if we do the claim in the conductor imieatly after the allcoation calim | 14:00 |
| sean-k-mooney | we can make that better | 14:01 |
| gibi | yes, that can be done as well | 14:01 |
| sean-k-mooney | espcially if said claim also make sure the resrvation in placment is updated first | 14:01 |
| *** spatel has joined #openstack-nova | 14:01 | |
| gibi | we can basically add a placement post-filter that gets the selected allocation candidate that is being sent back to placement and adds the extra reservation to it | 14:02 |
| sean-k-mooney | ya that an option but not in the next 6 weeks right | 14:02 |
| gibi | yes, it is after W | 14:02 |
| * sean-k-mooney this is why i intentionally declared pci device in placment out of scope | 14:02 | |
| sean-k-mooney | ok | 14:02 |
| gibi | agree | 14:02 |
| sean-k-mooney | so the real question is what to do for W | 14:03 |
| gibi | yes | 14:03 |
| sean-k-mooney | standard tait that wont be need after devices are in placment | 14:03 |
| sean-k-mooney | custom trait | 14:03 |
| sean-k-mooney | or no trait | 14:03 |
| gibi | the custom trait solution would still require the pre-filter to be implemented, isn't it? | 14:03 |
| sean-k-mooney | i think so | 14:04 |
| sean-k-mooney | provider.yaml coudl report it | 14:04 |
| gibi | then the custom trait is bad option in my eyes | 14:04 |
| sean-k-mooney | but requesting would need a flavor/image with it | 14:04 |
| sean-k-mooney | and using a specal falvor/imge for vdpa seams wrong in my eyes | 14:05 |
| gibi | agree ^ | 14:05 |
| sean-k-mooney | there is no way to add custom traits to a neutron port right? | 14:05 |
| gibi | vdpa is a port specific thing | 14:05 |
| sean-k-mooney | yep exactly | 14:05 |
| gibi | sean-k-mooney: not from the user side | 14:05 |
| gibi | neutron could add it from code | 14:05 |
| sean-k-mooney | ya that is what i tought | 14:05 |
| gibi | but that also mehh | 14:05 |
| sean-k-mooney | via the resouce requests right | 14:06 |
| gibi | right | 14:06 |
| sean-k-mooney | but we are not using them for this | 14:06 |
| gibi | if there is qos then the vnic type is added as custom triat | 14:06 |
| sean-k-mooney | so unfortunetly not useful | 14:06 |
| sean-k-mooney | am dont you mean physnet | 14:06 |
| sean-k-mooney | not vnic type | 14:06 |
| gibi | hm, | 14:06 |
| *** nweinber has joined #openstack-nova | 14:06 | |
| sean-k-mooney | anyway it wont work without neutron code chagnes is the imporant point | 14:07 |
| sean-k-mooney | so really its standard trait with prefilter or no trait. | 14:07 |
| gibi | both | 14:07 |
| gibi | both the physnet and the vnic_type is a custom trait in qos | 14:07 |
| sean-k-mooney | oh that is what you ment :) | 14:08 |
| sean-k-mooney | ok good to know | 14:08 |
| gibi | anyhow | 14:08 |
| gibi | I agree it is either standard trait, or nothing | 14:08 |
| sean-k-mooney | stephenfin: ^ | 14:08 |
| sean-k-mooney | do either of ye have a preference then | 14:09 |
| sean-k-mooney | i feel like ye are leaning towoard nothing? | 14:09 |
| sean-k-mooney | and just using the pci passthough filter? | 14:09 |
| gibi | personally if having no trait and a possibility of a bugreport due to the limit in allocation candidates that motivates us to do the PCI in placement then | 14:09 |
| gibi | yes, I would go with nothing | 14:09 |
| gibi | and focus on PCI in placement in X | 14:09 |
| gibi | (I have my personal reason to look into PCI in placement soon) | 14:10 |
| stephenfin | yeah, I vote nothing | 14:10 |
| sean-k-mooney | ya personally i like the possibel cross project goal of decalring X a maintaince cycle | 14:10 |
| gibi | the current PCIFilter based qos + pci solution has limitations until PCI in placement is a thing | 14:10 |
| sean-k-mooney | but i am happy to push internally that we use that to pay donw our pci debth | 14:10 |
| gibi | sean-k-mooney: lets have X a maintenance cylce where we discuss the PCI in placement from spec perspective :D | 14:11 |
| sean-k-mooney | hehe well i would like to spend some time fixing some of thepci related move operations but ya | 14:11 |
| gibi | and yes, technical dept removal is a good form of maintenance | 14:11 |
| sean-k-mooney | ok ill update the spec to remove the trait and prefilter | 14:12 |
| sean-k-mooney | and abandon the patches i submited for those | 14:12 |
| gibi | sean-k-mooney: thanks | 14:12 |
| sean-k-mooney | gibi: stephenfin was that the only outstanding question ye had | 14:12 |
| sean-k-mooney | is so ill respin the spec now | 14:12 |
| stephenfin | yup, for me anyway | 14:12 |
| gibi | sean-k-mooney: nothing else from me on that sepc | 14:13 |
| gibi | spec | 14:13 |
| sean-k-mooney | ok thanks both of you. | 14:13 |
| *** rcernin_ has quit IRC | 14:13 | |
| sean-k-mooney | gibi: your main concen with the port numa policies is the precidence right | 14:14 |
| sean-k-mooney | ill poke other to review in our internal team meeting but should i add it to the adgenda for tomorrows | 14:14 |
| sean-k-mooney | or maybe a ML post and ask for feedback there? | 14:15 |
| *** vishalmanchanda has quit IRC | 14:15 | |
| gibi | sean-k-mooney: anything that puts more eyes on the spec from the perspective | 14:15 |
| gibi | I think tomorrows agenda is a good idea | 14:15 |
| gibi | if no objection there then I will upgrade my vote to +2 | 14:16 |
| *** sapd1 has quit IRC | 14:16 | |
| sean-k-mooney | thanks ok ill send a short mail to the list and say it to the redhat folk internally too and we can decied tomorrow | 14:16 |
| *** derekh has joined #openstack-nova | 14:17 | |
| nightmare_unreal | is there a reason why placement takes 1.0 api as default ? why it does not take latest like in nova | 14:19 |
| *** hemanth_n has quit IRC | 14:20 | |
| *** jangutter_ has joined #openstack-nova | 14:20 | |
| sean-k-mooney | latest is a legacy thing for the nova client | 14:21 |
| sean-k-mooney | latest also might not always do what you think it wont nessiarly do negociation | 14:21 |
| *** jangutter has quit IRC | 14:24 | |
| *** irclogbot_3 has quit IRC | 14:27 | |
| nightmare_unreal | is there a way to specify/config that ? | 14:28 |
| *** sapd1 has joined #openstack-nova | 14:29 | |
| sean-k-mooney | so osc when you use latest just uses that latest version constnat in the nova_client lib | 14:30 |
| sean-k-mooney | it does not negociatation | 14:30 |
| sean-k-mooney | nova client if you dont specify a microverison negociates | 14:30 |
| sean-k-mooney | if you explcitly use latest i dont no if it blindly uses it like osc or does negociation | 14:31 |
| sean-k-mooney | at the api level i think its an alias for the latest microverion in nova | 14:31 |
| *** irclogbot_3 has joined #openstack-nova | 14:31 | |
| sean-k-mooney | but that is not a stardard part of microverions | 14:31 |
| sean-k-mooney | if you use latest the behavior will change between cloud or after upgrades so generally you sould not use it unless you are manually using clients to do something | 14:32 |
| sean-k-mooney | e.g. if this is used in code somewhere you shoudl not use latest | 14:32 |
| sean-k-mooney | you should use the minium microverion for the feature you are using | 14:33 |
| sean-k-mooney | that is what nova and other service do internally when calling each other | 14:33 |
| nightmare_unreal | I see | 14:36 |
| nightmare_unreal | thanks for explanation | 14:36 |
| sean-k-mooney | i assume you were using the openstack client when you said placment was defaulting to 1.0 | 14:37 |
| *** zoharm has joined #openstack-nova | 14:37 | |
| sean-k-mooney | osc does not do microverion negocation to provide a stable striptable unifrom inteface. some plugins like ironic disagree with that philosocy and implemented negoication themselves but that was inteionally not done osc itself | 14:39 |
| sean-k-mooney | the placment team did like the osc phioscophy so intentionlly did not implment microverion negociation in there placment plugin | 14:40 |
| *** rcernin_ has joined #openstack-nova | 14:40 | |
| *** rcernin_ has quit IRC | 14:45 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: libvirt: Add support for flavor and image defined ephemeral encryption https://review.opendev.org/c/openstack/nova-specs/+/771474 | 14:45 |
| gibi | hm, does somebody has an idea how can it be that a server in ERROR state does not have a fault field? http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22testtools.matchers._impl.MismatchError%3A%20'fault'%20not%20in%20%7B%5C%22 | 14:47 |
| sean-k-mooney | um is it in cell0? | 14:49 |
| sean-k-mooney | im not sure if the code path that lands you there would result in that but no | 14:50 |
| sean-k-mooney | not of the top of my head | 14:50 |
| sean-k-mooney | allthough you are looking at the unit/func tests? | 14:50 |
| sean-k-mooney | so we might just not have mocked it right | 14:50 |
| sean-k-mooney | in real code it maight always be set but in our unit test it might not due to incorrect mocking maybe? | 14:51 |
| gibi | it is tempest | 14:52 |
| sean-k-mooney | fyi ill be afk for a while but ill update the spec when i get back | 14:52 |
| gibi | sean-k-mooney: ack | 14:52 |
| sean-k-mooney | ah ok ya i guess using testtool makes sense there | 14:52 |
| *** vishalmanchanda has joined #openstack-nova | 14:52 | |
| sean-k-mooney | oh needed to exted time frame so its the min bandwith tests | 14:53 |
| sean-k-mooney | weird | 14:53 |
| gibi | I see in the api log that fault is lazy loaded | 14:53 |
| gibi | for this request | 14:53 |
| gibi | so somehow the fault is missing in the db | 14:54 |
| *** rcernin_ has joined #openstack-nova | 14:58 | |
| *** whoami-rajat___ is now known as whoami-rajat__ | 15:02 | |
| *** rcernin_ has quit IRC | 15:03 | |
| *** amodi has quit IRC | 15:06 | |
| *** rcernin_ has joined #openstack-nova | 15:07 | |
| *** rcernin_ has quit IRC | 15:12 | |
| lyarwood | https://review.opendev.org/c/openstack/nova/+/754702/ - there's a first, subnode deployment failed but we still tried to run multinode tests | 15:12 |
| *** slaweq has quit IRC | 15:21 | |
| *** slaweq has joined #openstack-nova | 15:23 | |
| *** sapd1 has quit IRC | 15:31 | |
| kashyap | Unrelated ... TIL, Red Hat announced gratis RHEL (you can now run upto 16 production instances): https://www.redhat.com/en/blog/new-year-new-red-hat-enterprise-linux-programs-easier-ways-access-rhel | 15:44 |
| kashyap | lyarwood: --^ Wonder if that allows one to start testing w/ proper RHEL in the upstream CI | 15:44 |
| lyarwood | kashyap: I wouldn't expect it would no | 15:45 |
| lyarwood | kashyap: and in any case testing RHEL in master CI seems a little pointless if we have centOS streams | 15:46 |
| kashyap | lyarwood: Ah, right; if we're testing w/ CentOS | 15:46 |
| kashyap | ... Streams, it's indeed not worth it | 15:46 |
| kashyap | I brought it up because some upstreams (like QEMU) will start using it now | 15:47 |
| lyarwood | I'd like to introduce *some* centOS streams coverage into the Nova gate if at all possible but that's about it | 15:48 |
| kashyap | lyarwood: Right; any non-trivial enterprise distro testing is upto relevant downstreams (as it's part of their value prop). | 15:51 |
| kashyap | I, for one, would be happy if a Fedora job that is sustained w/o bit-rot. I don't care about CentOS or Streams, personally. | 15:52 |
| lyarwood | we have that in the experimental queue | 15:52 |
| lyarwood | but centOS is a supported distro unlike Fedora | 15:53 |
| lyarwood | so I'd rather have it in the check and gate queues | 15:53 |
| kashyap | lyarwood: Yeah; I know you did some work on that; and I use it occasionally. To which, I thank you! | 15:54 |
| lyarwood | a few of us normally try to support the new fedora releases | 15:54 |
| kashyap | You're saying CentOS as "supported" as in, LTS, I take it | 15:54 |
| kashyap | lyarwood: Right; I did it in the past, and didn't do it after that. | 15:55 |
| kashyap | I also know ianw++ does some tireless maint work | 15:55 |
| sean-k-mooney | lyarwood: fedora is techniclaly not in the PTI | 15:55 |
| sean-k-mooney | if your talking about devstack then its only partly supported | 15:55 |
| sean-k-mooney | https://github.com/openstack/devstack/blob/master/doc/source/overview.rst#base-os | 15:56 |
| kashyap | lyarwood: BTW, upstream libvirt tests CentOS7, 8, Streams, and a crap-ton of other distros: https://gitlab.com/libvirt/libvirt/-/pipelines/244260219 | 15:58 |
| kashyap | It's at least good that they (including QEMU) test on RHEL; to catch regressions that can sneak in through rebases. | 15:58 |
| sean-k-mooney | kashyap: by the way the previous developer licencing explcitly disallowed rhel | 15:58 |
| kashyap | (And upper layers benefit from that.) | 15:58 |
| lyarwood | sean-k-mooney: huh I was going on the runtimes for each release | 15:59 |
| kashyap | sean-k-mooney: Yep; that's fixed now. The developer subscription is now far more useful. | 15:59 |
| sean-k-mooney | lyarwood: fedora is not in the support runtime for any release | 15:59 |
| sean-k-mooney | right now centos stream is not either technically | 15:59 |
| sean-k-mooney | although i think that will change in the y release | 16:00 |
| sean-k-mooney | i know rdo are moving to stream now | 16:00 |
| sean-k-mooney | but technially that is not a supproted runtime | 16:00 |
| lyarwood | yup I'd still like to move ahead with some centOS coverage of some form this cycle if possible | 16:00 |
| sean-k-mooney | yes we could do that though we proably shoudl not use stream | 16:01 |
| sean-k-mooney | at least until the offical runtime list is either ammened or next cycle | 16:02 |
| lyarwood | yeah I'll just use whatever is in the base job from devstack/tempest (/me forgets which) | 16:02 |
| sean-k-mooney | lyarwood: well both are avaiable | 16:02 |
| sean-k-mooney | and i fixed the only stream issue earlier in the cycle | 16:02 |
| sean-k-mooney | so we can use both if needed | 16:03 |
| sean-k-mooney | its just up to use to choose the correct nodeset | 16:03 |
| sean-k-mooney | kashyap: the limit on 16 instance basically means we cant use the "No-cost RHEL for small production workloads" clause | 16:03 |
| kashyap | sean-k-mooney: What do you mean can't use? It is for CI/developer purposes | 16:04 |
| kashyap | That's their intentional limit | 16:04 |
| kashyap | sean-k-mooney: Many upstreams benefit from that | 16:04 |
| kashyap | It is not supposed to solve every use case of every project. But for vast majority; this is a good move. | 16:04 |
| sean-k-mooney | that document does not explcitly state you arfree to use it for CI | 16:04 |
| *** macz_ has joined #openstack-nova | 16:05 | |
| sean-k-mooney | it describe 2 catagories "No-cost RHEL for customer development teams" upstream ci is not a customer | 16:06 |
| sean-k-mooney | the other one is "No-cost RHEL for small production workloads" | 16:07 |
| sean-k-mooney | which is adderssed by "Individual Developer subscription for RHEL can be used in production for up to 16 systems." | 16:12 |
| *** ociuhandu has quit IRC | 16:25 | |
| *** mlavalle has joined #openstack-nova | 16:26 | |
| *** ociuhandu has joined #openstack-nova | 16:36 | |
| *** gyee has joined #openstack-nova | 16:50 | |
| kashyap | The annoucement clearly spells out "community projects maintaining large CI/Build systems". | 16:54 |
| dansmith | gibi: lyarwood: If you're around, could we get this landed? https://review.opendev.org/c/openstack/nova/+/771234 | 17:02 |
| dansmith | we're currently copying glance-internal properties into snapshots, which we need to stop so glance can stop letting api users set internals :P | 17:02 |
| *** nightmare_unreal has quit IRC | 17:03 | |
| openstackgerrit | Merged openstack/nova master: tox: Stop linting as part of docs target https://review.opendev.org/c/openstack/nova/+/735176 | 17:04 |
| openstackgerrit | Merged openstack/nova master: doc: require openstack client change for every new API microversion https://review.opendev.org/c/openstack/nova/+/717727 | 17:04 |
| *** rcernin_ has joined #openstack-nova | 17:08 | |
| lyarwood | dansmith: sorry was on a call, looking | 17:12 |
| *** rcernin_ has quit IRC | 17:12 | |
| *** artom has quit IRC | 17:15 | |
| gibi | dansmith: looks good to me | 17:16 |
| dansmith | gibi: thanks | 17:17 |
| lyarwood | dansmith: did you want to backport this? | 17:17 |
| dansmith | lyarwood: we don't really need to, as I don't think the glance part will get backported | 17:17 |
| lyarwood | dansmith: if you do could you add a bug otherwise this LGTM | 17:17 |
| lyarwood | dansmith: an ack then I'll +W | 17:17 |
| lyarwood | ah* | 17:17 |
| dansmith | thanks | 17:17 |
| dansmith | if we backported the enforcement in glance, we'd start breaking nova if you yum updated glance before, so I don't think it's a good idea | 17:18 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Replace blind retry with libvirt event waiting in detach https://review.opendev.org/c/openstack/nova/+/770246 | 17:19 |
| *** dtantsur is now known as dtantsur|afk | 17:23 | |
| *** rpittau is now known as rpittau|afk | 17:26 | |
| openstackgerrit | Merged openstack/nova master: libvirt: Only add a USB controller if it's necessary https://review.opendev.org/c/openstack/nova/+/756549 | 17:30 |
| *** tesseract has quit IRC | 17:33 | |
| *** ociuhandu_ has joined #openstack-nova | 17:38 | |
| *** ociuhandu has quit IRC | 17:42 | |
| *** ociuhandu_ has quit IRC | 17:42 | |
| *** artom has joined #openstack-nova | 17:47 | |
| *** artom has quit IRC | 17:47 | |
| *** artom has joined #openstack-nova | 17:47 | |
| openstackgerrit | Merged openstack/nova master: libvirt: Remove support for '[libvirt] use_usb_tablet' https://review.opendev.org/c/openstack/nova/+/756550 | 17:53 |
| openstackgerrit | Merged openstack/nova master: Make PCI claim NUMA aware during live migration https://review.opendev.org/c/openstack/nova/+/748453 | 17:54 |
| openstackgerrit | Merged openstack/nova master: api-ref: Add notes about volume attach and detach being async https://review.opendev.org/c/openstack/nova/+/768104 | 17:55 |
| openstackgerrit | Merged openstack/nova master: Docs: correct cpu_thread_policy explanation https://review.opendev.org/c/openstack/nova/+/765172 | 17:55 |
| *** ralonsoh has quit IRC | 17:58 | |
| *** derekh has quit IRC | 18:01 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: libvirt: start tracking NUMACell.socket for hosts https://review.opendev.org/c/openstack/nova/+/766816 | 18:21 |
| stephenfin | lyarwood: This can wait til after M2, but you should probably look at https://review.opendev.org/c/openstack/python-openstackclient/+/771699 at some point | 18:23 |
| lyarwood | stephenfin: ack | 18:23 |
| stephenfin | Suggestions for how to make that less horrific for users are welcome, though I think that's just part of the complexity of BDM v2, unfortunately :-( | 18:23 |
| lyarwood | stephenfin: speaking of M2 would you mind hitting https://review.opendev.org/c/openstack/nova-specs/+/771474 again | 18:24 |
| stephenfin | sure | 18:24 |
| *** vishalmanchanda has quit IRC | 18:40 | |
| openstackgerrit | Artom Lifshitz proposed openstack/os-traits master: WIP: Add COMPUTE_SOCKET_PCI_NUMA_AFFINITY trait https://review.opendev.org/c/openstack/os-traits/+/771705 | 18:55 |
| artom | Oh we get notified in both channels eh | 18:55 |
| sean-k-mooney | all the placment and os traits related stuff is now here | 19:01 |
| artom | So is -placement ded? | 19:02 |
| sean-k-mooney | kind of | 19:02 |
| artom | zombie? | 19:02 |
| sean-k-mooney | i mean i think most of us are still there too | 19:02 |
| sean-k-mooney | but gibi is ptl of both currently and three really isnt much of a placment core team | 19:02 |
| sean-k-mooney | at least that is active | 19:02 |
| *** rcernin_ has joined #openstack-nova | 19:09 | |
| *** andrewbonney has quit IRC | 19:09 | |
| *** dcapone2004 has joined #openstack-nova | 19:13 | |
| *** rcernin_ has quit IRC | 19:14 | |
| *** rcernin_ has joined #openstack-nova | 19:25 | |
| *** rcernin_ has quit IRC | 19:30 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Add reference docs for internal block device structures https://review.opendev.org/c/openstack/nova/+/771711 | 19:32 |
| lyarwood | stephenfin: ^ for the morning, spent some time this afternoon cleaning an old email from mdbooth up and turning it into a reference doc | 19:32 |
| *** zoharm has quit IRC | 19:40 | |
| *** rcernin_ has joined #openstack-nova | 20:12 | |
| openstackgerrit | sean mooney proposed openstack/nova-specs master: libvirt: support vdpa kernel networking https://review.opendev.org/c/openstack/nova-specs/+/764999 | 20:21 |
| dansmith | lyarwood: there's another failure in the multistore job I'm seeing evidence of, which actually might be caused by us snapshotting an image which includes glance internals, which prevents us from doing multistore things, thinking they're already done | 20:22 |
| dansmith | lyarwood: which hopefully that do-not-inherit patch will fix | 20:22 |
| dansmith | hadn't seen this one failure until just now | 20:22 |
| *** rcernin_ has quit IRC | 20:23 | |
| *** avolkov has quit IRC | 20:24 | |
| lyarwood | dansmith: ack cool, hopefully that's the failure I saw before the break and your change should resolve it. I've not seen any more recently tbh, most of our issues at the moment are with the nova-live-migration job, subnode devstack failures and libvirtd. | 20:34 |
| * lyarwood drops for the night \o | 20:34 | |
| *** dave-mccowan has joined #openstack-nova | 20:40 | |
| *** tosky has quit IRC | 20:41 | |
| *** raildo has quit IRC | 20:42 | |
| *** raildo_ has joined #openstack-nova | 20:42 | |
| *** tosky has joined #openstack-nova | 20:42 | |
| *** mgariepy has quit IRC | 20:42 | |
| *** spatel has quit IRC | 20:43 | |
| *** mgariepy has joined #openstack-nova | 20:43 | |
| *** rcernin_ has joined #openstack-nova | 20:48 | |
| dansmith | lyarwood: most of them were the tempest-induced OOM that I fixed earlier I think, but.. yeah | 20:59 |
| *** dcapone2004 has quit IRC | 21:11 | |
| *** sean-k-mooney has quit IRC | 21:16 | |
| *** rcernin_ has quit IRC | 21:49 | |
| *** rcernin has joined #openstack-nova | 21:50 | |
| *** whoami-rajat__ has quit IRC | 21:51 | |
| *** jamesden_ has quit IRC | 21:51 | |
| *** jamesdenton has joined #openstack-nova | 21:51 | |
| *** nweinber has quit IRC | 22:05 | |
| *** xek has quit IRC | 22:13 | |
| *** hamalq has joined #openstack-nova | 22:25 | |
| *** slaweq has quit IRC | 22:32 | |
| *** raildo_ has quit IRC | 23:09 | |
| *** lbragstad has quit IRC | 23:13 | |
| *** lbragstad_ has joined #openstack-nova | 23:13 | |
| *** brinzhang has quit IRC | 23:20 | |
| *** brinzhang has joined #openstack-nova | 23:29 | |
| *** luksky has quit IRC | 23:33 | |
| *** tosky has quit IRC | 23:44 | |
| *** hoonetorg has quit IRC | 23:48 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!