| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-03-07 | |||
| 20:29:16 | bauzas | it can work with "le café" | |
| 20:29:22 | bauzas | "le budget" | |
| 20:29:25 | artom | dansmith, ah, and your response was "there's the connection, by having the PCI filter work with this new trusted thing in pci requests" | |
| 20:29:30 | dansmith | yes | |
| 20:29:58 | bauzas | yeah the spec was unclear, I asked for details | |
| 20:30:02 | mriedem | so the api calls create_pci_requests_for_sriov_ports which gets the binding profile off the port and gets the pci request spec with the new trusted attribute | |
| 20:31:15 | mriedem | ok and then the pci filter magically picks a host based on the pci request spec for a trusted vf, tied via flavor extra spec? | |
| 20:31:33 | dansmith | not just that, it picks an _actual_ pci device I think | |
| 20:31:35 | dansmith | it's AWESOME | |
| 20:33:34 | mriedem | PciPassthroughFilter right? | |
| 20:34:04 | mriedem | man i'm confused | |
| 20:34:20 | mriedem | good thing the release note would have documented how all of this works for a deployer that wants to use it | |
| 20:34:30 | mriedem | artom: i guess? | |
| 20:34:48 | mriedem | the spec just says, "operators will have to use host aggregates" | |
| 20:35:19 | artom | I sort of assumed it would be in the same way that we have pinned CPUs now | |
| 20:35:31 | artom | Aggregates for trusted VFs and aggregated for normal VFs | |
| 20:35:46 | artom | So that any VF in a 'trusted' aggregate is, err, trusted | |
| 20:38:27 | dansmith | that would kindof suck | |
| 20:39:40 | mriedem | the PciPassthroughFilter is looking at HostState.pci_stats, but i don't know what goes into that thing saying "this host reported it supports trusted VFs" | |
| 20:40:09 | openstackgerrit | Merged openstack/nova master: Only attempt a rebuild claim for an evacuation to a new host https://review.openstack.org/546268 | |
| 20:42:13 | mriedem | i guess https://review.openstack.org/#/c/458514/13/releasenotes/notes/trusted-vfs-abee6dff7c9b6940.yaml ? | |
| 20:42:27 | mriedem | so the pci passthrough_whitelist has to report the devices that are trusted, | |
| 20:42:37 | mriedem | and the PciPassthroughFilter links those to the ports in the pci request spec? | |
| 20:43:25 | mriedem | and pci passthrough_whitelist is our poor man's placement inventory system yes? | |
| 20:43:26 | mriedem | jaypipes: ^? | |
| 20:43:47 | openstackgerrit | Dan Smith proposed openstack/nova master: Make nova-manage db purge take --all-cells https://review.openstack.org/550502 | |
| 20:44:52 | dansmith | mriedem: this ^ passed, so I converted the hook to run --all-cells on the purge | |
| 20:45:20 | dansmith | all the other patches have a clean swath of green from zuul too | |
| 20:45:22 | mriedem | +2 on the bottom change in the series | |
| 20:46:06 | dansmith | the other guy hasn't updated the archive patch yet, but if he has tomorrow I'll rebase on this and convert that part of the hook too | |
| 20:46:26 | mriedem | Kevin_Zheng ? | |
| 20:46:34 | dansmith | someone else was responding right? | |
| 20:46:39 | mriedem | oh idk | |
| 20:47:06 | dansmith | also downstream QE here is going to try to get some real world database snapshots and run archive/purge against them | |
| 20:48:51 | bauzas | jaypipes: mriedem: dansmith: as we discussed at the PTG, I'm about to provide a new spec for NUMA topology, should I use that one https://review.openstack.org/#/c/543062/1 or just a new one given we agreed on a different way | |
| 20:49:00 | tssurya | dansmith : Zhenyu Zheng is Kevin_Zheng :) | |
| 20:49:14 | dansmith | tssurya: oh, hah | |
| 20:49:57 | dansmith | I stopped pattern matching at "Z != K; break" | |
| 20:50:25 | tssurya | dansmith: :D | |
| 20:54:34 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Update the nova-manage db archive_deleted_rows description https://review.openstack.org/550600 | |
| 20:54:48 | artom | dansmith, actually, about that, if we grab that mitaka dump from the "original" bug a long time ago, and just run it through nova-manage db-sync...? | |
| 20:55:06 | dansmith | artom: it was newton I think | |
| 20:55:13 | dansmith | artom: we could, but it'd be nicer to have something a little fresher | |
| 20:55:17 | jaypipes | mriedem: yes, you are correct. (about the pci_passthrough_whitelist and PciPassthroughFilter cruft. | |
| 20:55:19 | dansmith | but if nothing else, that will work | |
| 20:55:40 | jaypipes | bauzas: definitely a new one. | |
| 20:55:56 | bauzas | jaypipes: roger. | |
| 20:58:18 | mriedem | jaypipes: ok cool, | |
| 20:58:28 | mriedem | it's about time, since juno, that i start to try to understand how that stuff works | |
| 20:58:39 | mriedem | i would really rather not have to though | |
| 20:59:06 | openstackgerrit | Merged openstack/nova stable/queens: Check for multiattach before removing connections https://review.openstack.org/550221 | |
| 21:03:27 | bauzas | just to make it clear : "le"/"la" | |
| 21:03:36 | bauzas | le woof, la bye | |
| 21:03:50 | bauzas | :p | |
| 21:09:39 | mriedem | maybe in le france | |
| 21:09:42 | mriedem | i'm in 'merica | |
| 21:18:39 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Rename the 'recreate' param in rebuild_instance to 'evacuate' https://review.openstack.org/508190 | |
| 21:22:45 | bauzas | oh man, getting mad with the imagebackend | |
| 21:30:28 | mriedem | dansmith: not sure if you want to clean up the nits in https://review.openstack.org/#/c/550182/ before i +W? | |
| 21:31:16 | mriedem | bauzas: https://i.chzbgr.com/full/6425329152/hA7A48AD5/ | |
| 21:32:03 | bauzas | meh | |
| 21:32:14 | bauzas | trying to understand spaghetti code | |
| 21:32:36 | mriedem | i think mdbooth said the libvirt imagebackend code was done | |
| 21:36:12 | bauzas | mriedem: man, it's your fault | |
| 21:36:39 | bauzas | mriedem: I need to understand why we preallocate LVM volumes | |
| 21:36:41 | bauzas | :p | |
| 21:37:00 | mriedem | is pdragbrady still around? | |
| 21:37:06 | mriedem | *pdraigbrady? | |
| 21:37:50 | mriedem | oh facebook i see | |
| 21:40:56 | dansmith | mriedem: sure, give me a sec | |
| 21:43:10 | openstackgerrit | Dan Smith proposed openstack/nova master: Add --purge helper flag to archive_deleted_rows https://review.openstack.org/550182 | |
| 21:43:11 | openstackgerrit | Dan Smith proposed openstack/nova master: Make nova-manage db purge take --all-cells https://review.openstack.org/550502 | |
| 21:43:17 | dansmith | mriedem: le boom ^ | |
| 21:43:28 | mriedem | le thanks | |
| 21:47:42 | mriedem | dansmith: assuming we should get this into a 17.0.1 as well https://review.openstack.org/#/c/550557/ if you want to fast approve | |
| 21:47:46 | mriedem | then i can request the release | |
| 21:48:39 | dansmith | mriedem: hmm, for the backport we should probably clamp to the 4.x latest right? | |
| 21:48:52 | mriedem | wasn't sure | |
| 21:48:57 | dansmith | it should be the minimum that release supports | |
| 21:49:10 | dansmith | so actually not 4.max | |
| 21:49:32 | dansmith | 4.18 or 'pike' | |
| 21:49:40 | mriedem | yeah was going to say 4.18 | |
| 21:49:59 | dansmith | although | |
| 21:50:11 | mriedem | then why didn't we need that on master? | |
| 21:50:12 | dansmith | if someone is starting up a new deployment of queens... | |
| 21:50:23 | dansmith | because 5.0 is the minimum we support on master | |
| 21:50:38 | dansmith | actually, you know, yeah, let's just leave it | |
| 21:50:44 | dansmith | it doesn't cache it, | |
| 21:50:50 | dansmith | so if it's really first startup, 5.0 is fine on queens | |
| 21:51:07 | dansmith | and if something else comes online that is older, it won't have cached it and will do the right thing | |
| 21:51:50 | dansmith | mriedem: I will say this is not critical on queens though, since queens will speak 4.11 if you ask it to, | |
| 21:51:55 | dansmith | so it won't break like master did | |
| 21:52:12 | mriedem | i wanted to ask odyssey4me if they hit this in OSA on queens or master | |
| 21:52:32 | mriedem | if we don't need this in queens, i'm fine to abandon | |
| 21:52:36 | dansmith | really shouldn't on queens, it's after we dropped 4.x support in master that pinning to 4.11 breaks | |
| 21:52:53 | dansmith | let's just hold off, ask odyssey4me, and release 17.0.1 without it | |
| 21:52:56 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Deprecate sparse LVs https://review.openstack.org/549771 | |
| 21:53:00 | dansmith | and if I'm missing something we can merge it later, okay? | |
| 21:53:11 | mriedem | abandoned | |
| 21:53:11 | mriedem | yeah | |