| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-09-07 | |||
| 07:11:35 | artom | And IIUC this claim context is how resources are claimed on the destination node | |
| 07:12:01 | artom | Is there a less opaque way of doing, for example by calling something like rt.claim_these_resources_on_the_compute_host? | |
| 07:12:28 | artom | I'm essentially trying to avoid having to understand claim contexts :D | |
| 07:15:00 | sahid | artom: well... i did not have yet finish my coffe but i would say, you will have to reuse moveclaim that to avoid rewriting everything, | |
| 07:15:24 | artom | sahid, aha, thanks! That's... what I was afraid of :( | |
| 07:16:45 | sahid | artom there is nothing so complicted in moveclaim, basically we just create a migration object context and update resources on destination host based on the instance that we are migrating | |
| 07:17:22 | artom | sahid, heh, you keep saying that, and I just can never bring myself to go through that code and understand what's going on and what else it affects | |
| 07:17:40 | artom | sahid, maybe a bjns when you have time so you can bring me up to speed | |
| 07:18:00 | sahid | what is blocking you? | |
| 07:18:19 | artom | Besides my own stupidity and laziness? ;) | |
| 07:18:38 | sahid | ahahah :) | |
| 07:19:35 | artom | So, the context is, I'm updating the live migration spec for Stein, and I was pinning my hopes on placement to claim resources on the dest | |
| 07:20:31 | artom | But! Placement is only qualitative - ie, you can say I'm claiming 2 NUMA nodes, but you can't say I'm claiming NUMA node 0 and 3, and pinning CPUs 1-4 and 4-8 | |
| 07:20:52 | artom | So we'll need claims on the compute node regardless, even in the distance future when placement starts doing NUMA for us | |
| 07:21:33 | artom | Because for NUMA we need to know *which* specific resources are being claimed, not just how many | |
| 07:21:49 | artom | The mellanox guys want to do live migration with SRIOV, and they have the same problem | |
| 07:22:59 | artom | I'll continue updating the spec, and them, if you're willing, come back with a more concrete list of questions than just "HALP!!11" | |
| 07:23:37 | sahid | so basically i would say first you will have to update hardware.fit_instance_to_host() to work with placement | |
| 07:24:55 | sahid | well actually i have some doubt you can achieve that work with placement right now | |
| 07:25:13 | artom | Well, the conversion of hardware/resource_tracker to use placement will come later, for now there isn't even a spec for Stein for NUMA RPs in placement (though Sylvain did have one proposed for Rocky) | |
| 07:25:29 | artom | sahid, yeah, it's for later, the pieces aren't in placement yet | |
| 07:25:40 | sahid | ok so i would say forget about placement, we will be able to update our code at time | |
| 07:25:42 | artom | I need to bite the bullet and work with the claim context | |
| 07:26:02 | sahid | yes you should make this working for claim using resource_tracker | |
| 07:26:30 | artom | Which I've been desperately trying to avoid, but I don't think there's a way around it :( | |
| 07:36:52 | openstackgerrit | Chen proposed openstack/nova master: doc: update info for hypervisors https://review.openstack.org/599554 | |
| 08:03:51 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Update launch-from-volume doc to latest version. https://review.openstack.org/572241 | |
| 09:07:27 | openstackgerrit | Silvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts https://review.openstack.org/522245 | |
| 09:07:52 | openstackgerrit | Chen proposed openstack/nova master: Use six.string_types to improve python2/3 compatibility https://review.openstack.org/600677 | |
| 09:42:39 | wuchongyao | ||
| 10:17:22 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: DNM: Use claim context during live migration https://review.openstack.org/576222 | |
| 10:35:13 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge security groups extension response into server view builder https://review.openstack.org/585475 | |
| 10:35:24 | gmann | mriedem there was issue on making 'security_groups' field optional. fixed it now. ^^. i will fix api-ref for later where 'security_groups' filed is not mentioned optional in server response. | |
| 11:15:53 | openstackgerrit | Merged openstack/nova master: Update volume-attachment API url in policies https://review.openstack.org/596726 | |
| 11:16:09 | openstackgerrit | Merged openstack/nova master: Docs: update link for remote debugging https://review.openstack.org/591316 | |
| 11:16:18 | openstackgerrit | Merged openstack/nova master: Add encrypted volume support to feature matrix docs https://review.openstack.org/570255 | |
| 11:16:24 | openstackgerrit | Merged openstack/nova master: fup: Fix import order and test nit https://review.openstack.org/600474 | |
| 12:36:33 | openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova-specs master: Modelling passthrough devices for report to placement https://review.openstack.org/591037 | |
| 12:54:50 | openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Do not dump all instances in the scheduler https://review.openstack.org/598611 | |
| 13:15:20 | openstackgerrit | Merged openstack/nova master: Configure placement DB context manager for nova-manage/status https://review.openstack.org/599744 | |
| 13:34:15 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge extended_status extension response into server view builder https://review.openstack.org/592092 | |
| 14:44:21 | openstackgerrit | Artom Lifshitz proposed openstack/nova-specs master: WIP: Re-propose numa-aware-live-migration spec https://review.openstack.org/599587 | |
| 15:25:19 | mriedem | lyarwood: dansmith: can i get your eyes on the rocky backport of the fix for the 3rd party ci failures https://review.openstack.org/#/c/600464/ | |
| 16:10:52 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Resource retrieving: add changes-before filter https://review.openstack.org/591976 | |
| 16:15:42 | mriedem | johnthetubaguy: you should review this spec https://review.openstack.org/#/c/600608/ and https://review.openstack.org/600613 - they were osic specs previously approved in pike so you'd probably have the best context on those | |
| 16:15:49 | mriedem | per-instance live migration timeout | |
| 16:16:05 | mriedem | cfriesen: you too for starlingx has some similar functionality ^ | |
| 17:10:13 | cfriesen | mriedem_bbq: will take a look | |
| 17:11:39 | cfriesen | mriedem_bbq: oh man, I thought that got sorted out a couple releases back. guess not. | |
| 17:22:29 | dansmith | mriedem_bbq: did you want me to put my money in place of my mouth on this? https://review.openstack.org/#/c/600085/ | |
| 17:54:10 | mriedem | dansmith: i can update it, unless you're looking to poke on it | |
| 17:54:16 | mriedem | was doing spec reviews before the ptg | |
| 17:54:20 | mriedem | trying to anyway | |
| 17:59:12 | dansmith | I'm happy for you to do it, I just wasn't sure if you were expecting me to | |
| 18:11:01 | openstackgerrit | Merged openstack/nova-specs master: Resource retrieving: add changes-before filter https://review.openstack.org/591976 | |
| 18:18:04 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Add live-migration-force-after-timeout-stein spec https://review.openstack.org/600608 | |
| 18:18:23 | mriedem | johnthetubaguy: Kevin_Zheng: dansmith: ^ cleaned up and modernized with context since it's been awhile since ocata GA | |
| 18:27:01 | dansmith | mriedem: sorry, I was busy shoveling poo on leakypipes' pet-loving transfer ownership spec | |
| 18:27:28 | dansmith | let it be known henceforth that leakypipes loves pets and will be up for helping with future BFV hardening | |
| 18:27:30 | mriedem | that guy, he does love pets | |
| 18:27:32 | mriedem | and bfv i've heard | |
| 18:27:37 | mriedem | haha | |
| 18:30:22 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix formatting in changes-since guide https://review.openstack.org/600150 | |
| 18:31:24 | leakypipes | dansmith: I don't see how wanting to allow nova to participate in the multi-service task of transferring ownership of an instance is similar to BFV. | |
| 18:32:02 | dansmith | leakypipes: I was referring to the point of this being purely for pets | |
| 18:32:14 | dansmith | BFV has uses outside the world of pets even | |
| 18:32:18 | dansmith | but not this transfer thing | |
| 18:32:21 | dansmith | that's pets pets pets | |
| 18:32:42 | dansmith | PETS! | |
| 18:33:08 | leakypipes | agreed. | |
| 18:33:32 | leakypipes | pets exist, though. | |
| 18:33:34 | dansmith | leakypipes: so before I will approve your spec, I will require some sort of public admission that you love pets, and that BFV has a place in this world | |
| 18:33:38 | leakypipes | we've got 150 thousand of them. | |
| 18:34:04 | dansmith | oh man, I can't wait to quote you the next time we're talking about something pet-based :D | |
| 18:34:20 | dansmith | 2018, this happened: [11:33:32] <leakypipes>pets exist, though. | |
| 18:34:24 | leakypipes | dansmith: I can publicly admit my desire to facilitate pet management. I shall never submit to your demand that BFV is a good idea, though. | |
| 18:34:41 | leakypipes | :P | |
| 18:35:16 | dansmith | that is not what I said | |
| 18:35:21 | leakypipes | dansmith: if there were a different service that handled pets instead of ephemeral stuffs, I'd be proposing this spec at that project. But.... | |
| 18:35:21 | dansmith | "has a place in this world" | |
| 18:35:39 | leakypipes | oh, has a place in the world... ok, I might consider acquiescing to that. | |
| 18:35:46 | dansmith | you'll have to. | |
| 18:35:53 | leakypipes | bend a knee, huh? | |
| 18:36:15 | dansmith | anyway, we can figure that all out once you scrape all the poo off it | |
| 18:36:26 | dansmith | might need a picture on twitter sort of ting. | |
| 18:36:31 | dansmith | mriedem and I can get a t-shirt made | |
| 18:36:45 | leakypipes | I am indeed good at designing t-shirts, as you know. | |
| 18:37:55 | dansmith | heh | |
| 18:50:30 | mriedem | i still have my shelve t-shirt design sitting around | |
| 18:50:44 | mriedem | nova: shelving at the speed of business | |
| 18:51:03 | mriedem | https://www.customink.com/designs/novashelve/scj0-00b6-zdmw/hotlink?pc=HL-142344&utm_campaign=hotlink_201601&utm_source=hotlink&utm_medium=email&utm_content=designback | |
| 18:51:21 | mriedem | i'm just too f'ing cheap to get one made | |
| 18:53:03 | mriedem | fwiw, our product people want the owner transfer thing as well. i sent them the spec and asked to get any feedback, yay or nay, but of course i haven't heard back. | |
| 18:53:09 | mriedem | which is the hua-wei | |
| 18:53:20 | mriedem | see what i did there? | |
| 18:53:37 | mriedem | Kevin_Zheng: ^ maybe you should kick some people locally to check jay's instance owner transfer spec | |
| 18:53:43 | mriedem | since it sounds like dansmith loves it | |
| 18:53:47 | dansmith | I know you want it and other people do too | |
| 18:54:16 | dansmith | I'm just not really looking forward to the churn | |
| 18:54:19 | mriedem | this spec is starting to sound like a dirty little secret | |