Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-09-07
02:08:16 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Add live-migration-force-after-timeout-stein spec https://review.openstack.org/600608
02:18:53 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Spec to add per instance timeout for LM - Stein https://review.openstack.org/600613
03:01:26 openstackgerrit Merged openstack/nova stable/rocky: Fix a failure to format config sample https://review.openstack.org/600336
03:09:08 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Detach and attach boot volumes - Stein https://review.openstack.org/600628
03:30:30 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Resource retrieving: add changes-before filter https://review.openstack.org/591976
05:38:46 openstackgerrit Merged openstack/nova master: Fix TypeError in nova-manage cell_v2 list_cells https://review.openstack.org/599861
05:46:59 openstackgerrit fupingxie proposed openstack/nova master: Add an example to add more pci devices in nova.conf https://review.openstack.org/592243
06:26:36 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Detach and attach boot volumes - Stein https://review.openstack.org/600628
06:28:31 openstackgerrit fupingxie proposed openstack/nova master: Delete allocations for instances that have been moved to another node https://review.openstack.org/582899
06:43:58 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Add live-migration-force-after-timeout-stein spec https://review.openstack.org/600608
06:44:13 openstackgerrit fupingxie proposed openstack/nova master: Delete allocations for instances that have been moved to another node https://review.openstack.org/582899
06:53:20 openstackgerrit fupingxie proposed openstack/nova master: Delete allocations for instances that is not in this node https://review.openstack.org/582899
06:55:06 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Spec to add per instance timeout for LM - Stein https://review.openstack.org/600613
07:03:54 openstackgerrit Radoslav Gerganov proposed openstack/nova master: Do not dump all instances in the scheduler https://review.openstack.org/598611
07:04:48 openstackgerrit Radoslav Gerganov proposed openstack/nova master: Do not dump all instances in the scheduler https://review.openstack.org/598611
07:08:44 artom Actually, sahid, if you're awake already, you might know this. Can NUMA resources be claimed without using a MoveClaim context?
07:08:51 openstackgerrit Chen proposed openstack/nova master: doc: update info for hypervisors https://review.openstack.org/599554
07:10:29 sahid artom: not sure what you mean? i thnk we have two ways to claim resources but you can implement a new one if necessary
07:11:20 artom sahid, so, for cold migration for instance, there's code in the conductor that essentially says, "with rt.cold_migration_claim: <do cold migration>"
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.

Earlier   Later