| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-23 | |||
| 18:10:25 | mriedem | ok | |
| 18:10:31 | mriedem | you know about the ara report right? | |
| 18:10:39 | mriedem | to see what runs before and after you with what values | |
| 18:13:52 | sean-k-mooney | artom: once i have it up an running i can give you remote acess to poke around if you want. i have a static ip so i can easily port forwad you to i the hosts. ubuntu is install now so i just need to add a couple of kernel arges to make sriov work then stack with your code | |
| 18:14:07 | artom | sean-k-mooney, ack, that works | |
| 18:15:20 | artom | sean-k-mooney, I'm thinking of this bit in the resource tracker: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L283-L295 | |
| 18:15:35 | artom | For NUMA LM, we call that during check_live_migration_at_destination | |
| 18:16:47 | efried | mriedem: no, I knew it was there, but never really understood what it meant. | |
| 18:18:55 | mriedem | btw, speaking of that nova.image.download module https://logs.opendev.org/84/676684/1/check/neutron-grenade-multinode/b0a76ac/logs/screen-n-api.txt#_Aug_15_13_25_04_744293 | |
| 18:18:59 | mriedem | i'm not sure why that's happening | |
| 18:19:05 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L294-L295 might be a proablem | |
| 18:19:31 | sean-k-mooney | since we have already claimed them on the dest it could claim them twice | |
| 18:19:50 | mriedem | oh gdi of course https://review.opendev.org/#/c/676684/ | |
| 18:19:51 | mriedem | hehh | |
| 18:19:56 | sean-k-mooney | i think currenlty we only use that for cold migration right | |
| 18:21:30 | artom | sean-k-mooney, *currently* | |
| 18:21:45 | artom | But NUMA LM is based on doing a MoveClaim | |
| 18:22:09 | sean-k-mooney | right so we would have to modify this or it wont work with sriov live migration | |
| 18:22:18 | sean-k-mooney | or rather might not | |
| 18:23:18 | sean-k-mooney | will you be passing in somthing to indicate its a live migration | |
| 18:23:20 | artom | Or just ignore pci_requests entirely if it's a live migration? | |
| 18:23:31 | sean-k-mooney | that is what i was going to suggest | |
| 18:23:33 | artom | Yeah, the migration will hava a type set | |
| 18:23:34 | openstackgerrit | Andreas Jaeger proposed openstack/nova master: PDF documentation build https://review.opendev.org/676730 | |
| 18:23:54 | sean-k-mooney | its nice that it is claiming alias based pci devices | |
| 18:24:02 | sean-k-mooney | we dont currently support that in the sriov migration | |
| 18:24:13 | sean-k-mooney | but that also means we woudl not update the xml | |
| 18:24:43 | artom | Yeah, we can't claim resource we have no intention of using | |
| 18:24:44 | sean-k-mooney | so what we need to to is 1 check that there is not alias based pci device and two not call the cpi claim bit if its a live migration | |
| 18:25:06 | artom | Why is 2 alone not enough? | |
| 18:25:30 | sean-k-mooney | it is but the other is a safty mesure | |
| 18:25:48 | sean-k-mooney | e.g. we shoudl not attemt to migrate if you have alias based pci passthough | |
| 18:25:54 | sean-k-mooney | its not supported at all | |
| 18:26:08 | sean-k-mooney | we might already check that somewhere | |
| 18:26:16 | sean-k-mooney | if we dont we should in the conductor | |
| 18:26:35 | artom | Seems like that should be separate from NUMA LM tho | |
| 18:26:44 | sean-k-mooney | ya it is. | |
| 18:27:10 | artom | NUMA LM just cares about not stepping on SRIOV LM's toes | |
| 18:27:15 | sean-k-mooney | so for numa the if(migration.type!=live) shoudl be enough | |
| 18:28:12 | artom | We have functional tests for SRIOV LM? | |
| 18:28:20 | artom | I wonder if we could try both in a func test | |
| 18:28:41 | sean-k-mooney | we dont | |
| 18:29:06 | artom | Func tests for neutron SRIOV at all? | |
| 18:29:27 | sean-k-mooney | nope | |
| 18:29:38 | sean-k-mooney | we have for flavor based pci passthough | |
| 18:29:43 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/tests/functional/libvirt/test_pci_sriov_servers.py | |
| 18:29:52 | sean-k-mooney | but no fucntional test for neutorn sriov | |
| 18:30:03 | artom | So manual testing it is | |
| 18:30:07 | sean-k-mooney | so we would have to extend the neutron mock | |
| 18:30:12 | artom | Yeah | |
| 18:30:15 | sean-k-mooney | but we could add them if we did | |
| 18:30:57 | sean-k-mooney | i dont think we have any livemigration functional tests that use the libvirt fake driver | |
| 18:31:09 | artom | We didn't | |
| 18:31:16 | artom | NUMA LM's going to be the first | |
| 18:31:37 | artom | You meant fakelibvirt, right? | |
| 18:31:42 | sean-k-mooney | so your going to implement the multiple port bidnign workflow in the neutron fixtures | |
| 18:31:46 | sean-k-mooney | yes | |
| 18:31:47 | artom | Because we have a couple that use fake virt | |
| 18:31:49 | sean-k-mooney | fakelibvirt | |
| 18:32:03 | artom | sean-k-mooney, Ha. Ha. Ha. | |
| 18:32:26 | sean-k-mooney | artom: you will need that form live migration testing i think | |
| 18:32:37 | sean-k-mooney | you would for sriov lm | |
| 18:32:50 | artom | sean-k-mooney, heh, one thing at a time :) | |
| 18:32:55 | sean-k-mooney | for numa you could fall back to the old codepath | |
| 18:33:06 | artom | If we decide neutron sriov func test is something we want, I don't mind doing it | |
| 18:33:19 | artom | But for NUMA LM it's irrelevant | |
| 18:36:14 | sean-k-mooney | it would be nice to have functional test for this at some point but i think we shoudl wait till U to do that | |
| 18:36:28 | sean-k-mooney | at least that we we can do some firstpart testing beyond unit tests | |
| 18:37:24 | mriedem | sean-k-mooney: https://review.opendev.org/#/c/678098/2 needs to mention the reason for the difference in the backport (in the commit message) | |
| 18:37:27 | mriedem | otherwise it looks OK | |
| 18:37:47 | sean-k-mooney | ok ill respin it now | |
| 18:38:51 | sean-k-mooney | it was this https://github.com/openstack/os-vif/commit/ee124d2e98c2660ced7d71c1bd10525bf095a939 | |
| 18:38:58 | mriedem | Id8b71172fb06b435cf169a7e55c11233f22fa65b | |
| 18:38:58 | mriedem | yeah i just ofund that | |
| 18:44:02 | openstackgerrit | sean mooney proposed openstack/os-vif stable/stein: only disable mac ageing for ovs hybrid plug https://review.opendev.org/678098 | |
| 18:44:10 | sean-k-mooney | something like ^ | |
| 18:44:27 | sean-k-mooney | the other option we have would be to backport Id8b71172fb06b435cf169a7e55c11233f22fa65b | |
| 18:49:01 | mriedem | no we don't want to backport that big refactor | |
| 18:49:02 | mriedem | +2 on the other | |
| 18:49:20 | mriedem | melwitt: maybe you want to hit https://review.opendev.org/#/c/678098/ since lee is probably done | |
| 18:58:54 | mriedem | melwitt: any changes of getting https://review.opendev.org/#/c/507486/ and/or https://review.opendev.org/#/c/640898/ updated soonish before i lose context on those? | |
| 18:58:58 | mriedem | *chance | |
| 19:04:45 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Introduce live_migration_claim() https://review.opendev.org/635669 | |
| 19:04:46 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: LM: add support for augmenting migrate_data with info from claims https://review.opendev.org/634828 | |
| 19:04:46 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: New objects for NUMA live migration https://review.opendev.org/634827 | |
| 19:04:47 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA live migration support https://review.opendev.org/634606 | |
| 19:04:47 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: LM: add support for updating NUMA-related XML on the source https://review.opendev.org/635229 | |
| 19:04:48 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Functional test for NUMA live migration https://review.opendev.org/672595 | |
| 19:04:48 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Deprecate CONF.workarounds.enable_numa_live_migration https://review.opendev.org/640021 | |
| 19:16:39 | sean-k-mooney | artom: the last patch ^ is the one i should deploy yes? | |
| 19:16:51 | artom | sean-k-mooney, hol'up, things are in motion | |
| 19:16:57 | sean-k-mooney | have you added the sriov if? | |
| 19:16:58 | sean-k-mooney | ok | |
| 19:17:02 | artom | But in general, yeah | |
| 19:17:10 | artom | I did add the if | |
| 19:17:26 | sean-k-mooney | im not quite ready yet to run it but its close | |
| 19:17:50 | sean-k-mooney | i normlly run a vanila devstack install first then switch to the ting i want to test | |
| 19:18:01 | openstackgerrit | Eric Fried proposed openstack/nova master: Allow strict_proxies for sdk Connection https://review.opendev.org/676926 | |
| 19:19:00 | efried | I think I oughtta rebase the ironic series on that ^ and switch it on. | |
| 19:24:05 | openstackgerrit | Eric Fried proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899 | |