| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-01-19 | |||
| 18:35:58 | sean-k-mooney | gmann: yes but im not sure we want it too | |
| 18:36:14 | sean-k-mooney | if we did we would need to test multipel storage backend for example | |
| 18:36:15 | gmann | overall idea is have a common template which can cover all the related tests (which might be in different jobs due to requried configuration) | |
| 18:36:26 | sean-k-mooney | so we would need at least 2 jobs running full tempest | |
| 18:36:55 | gmann | 'full tempest' cannot be covered by two jobs. full name is confusing there always :) | |
| 18:37:33 | sean-k-mooney | no i meen we need two jobs running all tempest test with different config | |
| 18:38:17 | sean-k-mooney | for exmple nova libvirt backed has 3 ways to pass sotrage to a guest so if you wante test all 3 with live migraiton you need basially 2 jobs | |
| 18:38:43 | sean-k-mooney | one with lvm/iscis to test cinder voluems and block stoarge and one with ceph to test qemu driect connect | |
| 18:38:53 | sean-k-mooney | *qemu network stoage | |
| 18:39:16 | sean-k-mooney | we also have thigs liek multi cell | |
| 18:39:32 | sean-k-mooney | so if you want to test every thign in gate you have to run most of check again | |
| 18:40:32 | sean-k-mooney | personally i think the way we defien the jobs in the project repo today is better then trying to put service speicic repos in tempest | |
| 18:40:52 | gmann | sure, I mean all tempest tests at least with any one configuration. | |
| 18:41:30 | gmann | we have tests also in Tempest for 6 services so same way jobs. I feel that is easy to maintain and control also | |
| 18:42:18 | gmann | previously infra used to do that, maintain integrated template. and with migration to opendev it is more QA doing those jobs | |
| 18:43:31 | sean-k-mooney | right but i am not sure its adding much value currently at least for nova everythin in the template is also in our project config | |
| 18:43:59 | sean-k-mooney | if we removed - integrated-gate-compute template it would not change the runnign jobs at all | |
| 18:45:03 | sean-k-mooney | we overrite the jobs in repo anyway https://github.com/openstack/nova/blob/master/.zuul.yaml#L451-L471 | |
| 18:45:11 | sean-k-mooney | https://github.com/openstack/nova/blob/master/.zuul.yaml#L497-L500 | |
| 18:45:29 | sean-k-mooney | so we can pass our own irrelevant-files list | |
| 18:59:10 | openstackgerrit | Merged openstack/nova stable/train: Update pci stat pools based on PCI device changes https://review.opendev.org/c/openstack/nova/+/761725 | |
| 19:55:23 | openstackgerrit | sean mooney proposed openstack/nova-specs master: libvirt: support vdpa kernel networking https://review.opendev.org/c/openstack/nova-specs/+/764999 | |
| 19:59:25 | sean-k-mooney | artom: hopefully that adds what you wanted ^ | |
| 20:00:10 | sean-k-mooney | gibi: stephenfin assuming artom is happy can ye rereivew tommorow | |
| 21:26:43 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: compute: Lock by instance.uuid lock during swap_volume https://review.opendev.org/c/openstack/nova/+/758730 | |
| 21:27:18 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/stein: libvirt: Skip encryption metadata lookups if secret already exists on host https://review.opendev.org/c/openstack/nova/+/765772 | |
| 21:27:36 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: Use absolute path during qemu img rebase https://review.opendev.org/c/openstack/nova/+/757098 | |
| 22:42:15 | openstackgerrit | Ghanshyam proposed openstack/nova master: DNM: Testing system scope in tempest https://review.opendev.org/c/openstack/nova/+/740124 | |
| #openstack-nova - 2021-01-20 | |||
| 07:07:01 | 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 | |
| 08:49:45 | 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:46 | 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:46 | 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:47 | 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:47 | openstackgerrit | Brin Zhang proposed openstack/nova master: Replace tenants* with projects* of policies https://review.opendev.org/c/openstack/nova/+/765315 | |
| 09:54:35 | 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 | |
| 10:09:22 | stephenfin | brinzhang: ack | |
| 10:10:18 | stephenfin | lyarwood: Does the 'nova server-migration-list' command only list *ongoing* migrations or does it list all migrations in the past? | |
| 10:10:21 | stephenfin | (I don't have a multi-node deployment or I'd test this myself) | |
| 10:10:48 | stephenfin | I tried to migrate a server on single node deployment, which justifiably failed, but it doesn't appear in that list | |
| 10:16:10 | brinzhang | stephenfin: thanks | |
| 10:16:50 | lyarwood | stephenfin: I think it's all | |
| 10:17:00 | lyarwood | stephenfin: I don't have an env to check | |
| 10:20:36 | stephenfin | Okay, I'll say all for the OSC docs so | |
| 10:38:24 | 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:32 | lyarwood | gibi: ^ thanks for the review | |
| 10:39:21 | 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:29 | stephenfin | sure thing | |
| 10:39:33 | lyarwood | thanks | |
| 10:39:40 | stephenfin | looking at artom's spec atm, that's next on the queue | |
| 10:41:02 | lyarwood | ack thanks, I'll also try to look at specs this morning | |
| 11:09:22 | 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:38:09 | lyarwood | hemanth_n: looking | |
| 11:40:48 | openstackgerrit | Merged openstack/nova-specs master: `socket` PCI NUMA-affinity Policy https://review.opendev.org/c/openstack/nova-specs/+/765551 | |
| 11:40:53 | openstackgerrit | Merged openstack/nova-specs master: remove unicode from code https://review.opendev.org/c/openstack/nova-specs/+/769260 | |
| 11:49:17 | 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:51:47 | 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:52:18 | sean-k-mooney | its not really an optimisation in that you cant limit the placment allcoation set if you dont have it | |
| 11:52:33 | sean-k-mooney | if we were to go the provider.yaml route | |
| 11:52:38 | sean-k-mooney | then it would be a custom trait | |
| 11:52:48 | sean-k-mooney | whcih we can do but i would prefer to do that in code | |
| 11:52:57 | sean-k-mooney | i can drop it but it fells like a hack to not have it | |
| 11:53:04 | stephenfin | True. Same issue with PCI devices, NUMA topologies etc. though | |
| 11:53:28 | sean-k-mooney | not entirely the same thign although i waint traits for numa support | |
| 11:53:30 | 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:33 | sean-k-mooney | i just have not needed them | |
| 11:53:56 | sean-k-mooney | well traits are for capablities | |
| 11:54:30 | sean-k-mooney | lets see what gibi thinks and ill go with whatever the two of ye decied unless other weigh in | |
| 11:54:37 | stephenfin | Yeah, it's a very fuzzy line, that's for sure | |
| 11:54:41 | stephenfin | Makes sense | |
| 11:55:22 | stephenfin | as for your PCI devices in placement question, RE: what to do with VFs when a PF is consumed | |
| 11:55:28 | stephenfin | I'd just kill that "feature" | |
| 11:55:59 | stephenfin | make it necessary to state whether you want to allow passthrough of the PF or the VFs, and only report one | |
| 11:56:01 | 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:12 | sean-k-mooney | but its very similar to how isolate used to work | |
| 11:56:20 | sean-k-mooney | its the same problem form a plamcent point of view | |
| 11:56:25 | stephenfin | Yup | |
| 11:56:29 | sean-k-mooney | so killing it would be valid but contentious | |
| 11:56:38 | sean-k-mooney | which is why i dont think we will agree on that by tomorowy | |
| 11:56:43 | stephenfin | I agree | |
| 11:56:48 | stephenfin | Plus, you already have two specs | |
| 11:56:57 | stephenfin | More than enough for the three months we have left :) | |
| 11:57:22 | stephenfin | Anywho | |
| 12:02:37 | 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:03:57 | sean-k-mooney | stephenfin: we dont have 3 months left | |
| 12:04:52 | sean-k-mooney | stephenfin: m3 is march 11th so half that we have 6 weeks left but i agree | |
| 12:05:13 | sean-k-mooney | its 3 months till the offical release | |
| 12:05:24 | sean-k-mooney | april 15th | |
| 12:22:20 | openstackgerrit | John Garbutt proposed openstack/nova master: Fix PCI passthrough race on reschedule (refresh) https://review.opendev.org/c/openstack/nova/+/710848 | |
| 13:09:14 | sean-k-mooney | johnthetubaguy: since your about QQ do you know if anyoen is working on unified limits this cycle | |
| 13:10:31 | 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:32:35 | openstackgerrit | Merged openstack/nova master: libvirt: Mock get_arch during some stable rescue unit tests https://review.opendev.org/c/openstack/nova/+/769916 | |
| 13:41:24 | 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:42:33 | gibi | sean-k-mooney: which trait specifically you dicussed above with stephenfin above? | |
| 13:45:11 | sean-k-mooney | gibi: the vdpa one | |
| 13:46:10 | sean-k-mooney | gibi: ill get you the link to the comment in the spec | |
| 13:46:35 | sean-k-mooney | https://review.opendev.org/c/openstack/nova-specs/+/764999/4/specs/wallaby/approved/libvirt-vdpa-support.rst#79 | |
| 13:47:41 | 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:48:11 | sean-k-mooney | gibi: unfortuetlly the PF/VF thing is the same problem we had with hyperthread and PCPUs | |
| 13:48:39 | sean-k-mooney | fundementally we cant express requirments to placment that require different resouced dependding on the selected host | |
| 13:49:23 | 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 | |