Thursday, 2018-06-21

*** tetsuro has joined #openstack-placement00:36
*** takashin has joined #openstack-placement00:41
*** tssurya has joined #openstack-placement00:42
*** tssurya has quit IRC00:47
openstackgerrithuanhongda proposed openstack/nova stable/ocata: [Stable Only] Remove soft-deleted instances from quota_usages  https://review.openstack.org/57065301:02
*** gjayavelu has quit IRC01:13
*** tssurya has joined #openstack-placement02:05
*** tssurya has quit IRC02:09
*** bhagyashris has quit IRC02:14
*** bhagyashris has joined #openstack-placement02:15
openstackgerritChen proposed openstack/nova master: Specify default update_resources_interval  https://review.openstack.org/57686402:21
openstackgerritDinesh Bhor proposed openstack/nova master: PCPU: Define numa dedicated CPU resource class  https://review.openstack.org/56177002:24
openstackgerritDinesh Bhor proposed openstack/nova master: PCPU: Add respective conf options  https://review.openstack.org/56177102:24
openstackgerritDinesh Bhor proposed openstack/nova master: NUMACell, InstanceNUMACell: Adopt 'PCPU' changes  https://review.openstack.org/57602102:24
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add instance.lock notification  https://review.openstack.org/57670202:27
rpifanhi02:29
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add instance.lock notification  https://review.openstack.org/57670202:30
openstackgerritChason Chan proposed openstack/nova master: Fix the duplicated config options of api_database and placement_database  https://review.openstack.org/57702302:41
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add instance.unlock notification  https://review.openstack.org/57670303:01
*** gjayavelu has joined #openstack-placement03:17
*** tssurya has joined #openstack-placement03:28
*** tssurya has quit IRC03:33
*** rpifan has quit IRC03:50
*** gjayavelu has quit IRC04:04
*** gjayavelu has joined #openstack-placement04:11
openstackgerritDinesh Bhor proposed openstack/nova master: Report PCPU to placement  https://review.openstack.org/57703804:17
openstackgerritTetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate  https://review.openstack.org/56548704:23
openstackgerritTetsuro Nakamura proposed openstack/nova master: Fix nits in placement-return-all-resources series  https://review.openstack.org/57360404:28
*** dansmith has quit IRC04:29
*** gjayavelu has quit IRC04:49
*** tssurya has joined #openstack-placement04:52
*** gjayavelu has joined #openstack-placement04:56
*** tssurya has quit IRC04:57
openstackgerrithyunsik Yang proposed openstack/nova-specs master: Specification for support pxe/IPMI boot for instance  https://review.openstack.org/57704805:08
*** e0ne has joined #openstack-placement05:13
*** e0ne has quit IRC05:14
*** dansmith has joined #openstack-placement05:31
*** dansmith is now known as Guest8832005:32
*** tssurya has joined #openstack-placement05:53
*** alex_xu has quit IRC05:57
*** tssurya has quit IRC05:58
*** nicolasbock has joined #openstack-placement06:00
*** alex_xu has joined #openstack-placement06:03
openstackgerrithyunsik Yang proposed openstack/nova-specs master: Specification for support pxe/IPMI boot for instance  https://review.openstack.org/57704806:15
openstackgerritChen proposed openstack/osc-placement master: Fix docstring for delete allocation method  https://review.openstack.org/57707506:17
openstackgerritgaryk proposed openstack/nova master: VMware: populate datastore refs at init  https://review.openstack.org/57468806:24
openstackgerritChen proposed openstack/nova master: Specify default update_resources_interval  https://review.openstack.org/57686406:29
openstackgerrithyunsik Yang proposed openstack/nova-specs master: Specification for support pxe/IPMI boot for instance  https://review.openstack.org/57704806:51
*** tssurya has joined #openstack-placement07:08
*** peereb has joined #openstack-placement07:15
*** tssurya_ has joined #openstack-placement07:16
*** tssurya_ has quit IRC07:21
*** efried has quit IRC07:28
*** efried has joined #openstack-placement07:28
*** e0ne has joined #openstack-placement07:40
*** ttsiouts has joined #openstack-placement07:48
*** belmoreira has joined #openstack-placement07:48
*** gjayavelu has quit IRC07:49
*** gjayavelu has joined #openstack-placement07:50
*** ttsiouts has quit IRC07:52
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add InstanceGroupPolicy object  https://review.openstack.org/57362807:54
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy to InstanceGroup object and api models.  https://review.openstack.org/56337507:54
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object  https://review.openstack.org/56340107:54
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy  https://review.openstack.org/57116607:54
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model  https://review.openstack.org/57146507:54
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group  https://review.openstack.org/56753407:54
*** ttsiouts has joined #openstack-placement08:23
*** tssurya_ has joined #openstack-placement08:39
*** tssurya_ has quit IRC08:44
openstackgerritVladyslav Drok proposed openstack/nova master: ironic: Report resources as reserved when needed  https://review.openstack.org/51792108:49
openstackgerritGergely Csatari proposed openstack/nova master: Adding oslo.service to test-requirements  https://review.openstack.org/57709508:51
*** e0ne has quit IRC09:01
*** gjayavelu has quit IRC09:08
*** tetsuro has quit IRC09:10
*** tetsuro has joined #openstack-placement09:10
*** tetsuro has quit IRC09:10
*** tetsuro has joined #openstack-placement09:13
openstackgerritChris Dent proposed openstack/nova master: Cleanup nits in placement database changes  https://review.openstack.org/57710709:27
*** takashin has left #openstack-placement09:30
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Fix all invalid obj_make_compatible test case  https://review.openstack.org/57424009:31
openstackgerritVishakha Agarwal proposed openstack/nova master: No requirement of –all-tenants while listing servers  https://review.openstack.org/57711109:41
*** e0ne has joined #openstack-placement09:55
*** cdent has joined #openstack-placement10:00
openstackgerritVishakha Agarwal proposed openstack/nova master: No requirement of –all-tenants while listing servers  https://review.openstack.org/57711110:10
*** e0ne has quit IRC10:10
*** e0ne has joined #openstack-placement10:12
*** ttsiouts has quit IRC10:15
*** e0ne has quit IRC10:16
*** ttsiouts has joined #openstack-placement11:10
*** tetsuro has quit IRC11:29
*** e0ne has joined #openstack-placement11:58
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge extended availability zone controller  https://review.openstack.org/50285912:25
*** ttsiouts has quit IRC12:29
*** ttsiouts has joined #openstack-placement12:31
*** edmondsw has joined #openstack-placement12:58
*** ttsiouts has quit IRC12:59
openstackgerritjiang wei proposed openstack/nova master: Add action initiator attribute to the update payload  https://review.openstack.org/53624313:08
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge server create schema for legacy BDM extension  https://review.openstack.org/57715213:09
*** ttsiouts has joined #openstack-placement13:12
openstackgerritEric Fried proposed openstack/nova master: WIP: Tighten up ReportClient use of generation  https://review.openstack.org/55666913:14
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge extended availability zone controller  https://review.openstack.org/50285913:15
belmoreiraedleafe: efried: I'm having issues when trying to use a custom resource class for a GPU13:21
*** jaypipes has joined #openstack-placement13:21
efriedbelmoreira: tell me.13:21
belmoreirathe resource class is defined but after adding it to the resource_provider inventory it gets removed by the resource tracker update13:22
efriedbelmoreira: You add it to the inventory how?  Via CLI?13:22
cdentanybody noodling summit submissions?13:22
belmoreiraefried: yes13:23
efriedcdent: I just saw that email, haven't started thinking about it yet.13:23
* cdent nods13:23
efriedbelmoreira: Okay, so the virt driver is reporting the inventory it thinks it has - which doesn't include your GPU.13:23
efriedbelmoreira: and on RT update, we flush what the virt driver reports back to placement.13:23
belmoreirabut being a custom resource class it shouldn't care13:24
efriedThat's an interesting way to look at it.  It's not how it works at the moment.13:24
efriedWe don't just let you add random inventory to the compute from the CLI.13:24
cdentit's something we've discussed, but yeah, it's not currently supported13:25
efriedbelmoreira: Are you trying to do passthrough of physical GPU device?13:25
belmoreiraefried: thanks. I was understanding custom to "anything" that we would like to define: colour, ...13:26
efriedYou can certainly *define* anything you want.  But we're still pretty restricted in terms of how you can use it.13:26
belmoreiraefried: yes, some are for passthrough but we also have the use case for vgpus with P10013:27
efriedbelmoreira: vGPU stuff is partially supported and making progress - bauzas is on the bleeding edge of that effort.13:27
efriedbelmoreira: For traditional passthrough, you should still be able to use the legacy [pci]passthrough_whitelist BS.13:28
cdentfor later: I reckon we probably need a way to figure out how to apply non-virt-driver managed resource classes. We've planned it for traits, right?13:28
efriedcdent: Yes to the latter.  For the former, the plan is to allow non-virt-driver to manage providers elsewhere in the tree.13:29
belmoreiraefried: in our test infrastructure we were using the legacy [pci]passthrough_whitelist. But in that case we also need to enable the pci filter13:29
efriedcdent: In fact, as soon as nrp-in-alloc-cands lands, you could probably get away with doing that via the CLI.13:30
cdentefried: i'm unable to translate what you just said. What I'm trying to say is "I think there ought to be a way for belmoreira to do what's being described: out of band custom resource classing that isn't happening on the nova-compute itself"13:30
efriedincluding belmoreira's use case.13:30
cdent"just said being ^^, not ^"13:30
belmoreirathe issue is that we only have few nodes with GPUs isolated in one cell and I would like to avoid to enable the pci filter in the entire cloud because with cellsV2 scheduler is global13:30
efriedcdent: Yes, and I'm saying we don't plan to allow out-of-band custom resource classes (or any inventory changes) to be injected into providers that are *owned by compute*.  Virt will always blow those away, as belmoreira is seeing here.13:31
cdentokay, I'm saying we should allow htat13:31
efriedcdent: But for e.g. neutron & cyborg the plan is to allow nested providers.13:31
cdentIt seems sane to be able to say my computer resource provider has stuff13:32
cdentdoing it just by nested is fine, I guess, but I'm pathalogically meh about nested :)13:32
efriedcdent: So once we get nrp-in-alloc-cands, that starts to be possible - but there's nothing special about neutron & cyborg that you couldn't just do the same via the CLI.13:32
efriedbelmoreira: How willing are you to run with unmerged code?  :)13:32
belmoreiraefried: ahahah13:33
efriedbelmoreira: I think we have proposed code that will allow you to get that GPU *allocated* - but note that said code still won't actually *attach* the GPU to your VM.13:33
efried...which is presumably what you actually want.13:34
efriedcdent: I don't agree that we should allow entity X to mess with inventory on a provider owned by entity Y.  That way lies madness.13:35
cdentthe thing that owns all the entities is the human running the installation13:35
efried(If you recall, I also didn't want entity X messing with *traits* on a provider owned by entity Y, but I got overruled on that one.)13:36
belmoreiraefried: what I was expecting was to have something similar to ironic. A resource class is defined and the corresponding flavor. When using the flavor placement will provide only those RP to scheduler13:36
efriedbelmoreira: Ironic's custom resource classes are pushed by the virt driver.13:36
efriedand also represent an entire compute node, IIUC.13:37
belmoreiraefried: true13:37
edleafeso why isn't the virt driver able to "see" the GPU?13:38
efriededleafe: Cause it's not programmed to.13:38
edleafeIs that what bauzas is working on?13:39
bauzasmmm ?13:39
efriedno, bauzas is working on *v*GPU13:39
bauzasare you discussing about GPU passthrough ?13:39
efriedbelmoreira: As it happens, I care very much about this subject.  You could even say my paycheck arrives twice a month because I've promised to get generic GPU passthrough working for Power.  So watch this space - it's coming soon.13:40
edleafeI thought that the virt driver is able to understand all its hardware, including how they get virtualized13:40
efriededleafe: In theory, sure.  But at the moment if it reports hardware through get_available_resource, the only way it can get allocated is if it's a) in one of the special resource classes (VCPU, MEMORY_MB, DISK_GB), or b) via traditional PCI subsystem craziness.13:41
openstackgerritLee Yarwood proposed openstack/nova master: libvirt: Log breadcrumb for known encryption bug  https://review.openstack.org/57716413:42
efriededleafe: Stuff it reports through get_inventory/update_provider_tree gets allocated generically, but at the moment only libvirt is reporting anything other than those three, and I believe it's only reporting VGPU.13:42
belmoreiraso for know the only workaround that I see is to play with placement aggregates like we are doing for cells and have one aggregate per GPU type13:42
edleafeefried: thanks for the explanation. So yeah, I would much rather see the virt driver be the only thing that messes with compute node inventory.13:43
efriedbelmoreira: You mean so that you can use PCI passthrough without enabling it across the whole cloud?13:43
belmoreiraefried: yes13:44
efriedbelmoreira: I suppose.13:44
belmoreiraefried: edleafe: what I don't understand is why is not a good idea to define directly a RC in a RP13:45
belmoreiraI though that was the point of custom RC13:45
edleafebelmoreira: a custom RC is just something that isn't standard across all OpenStack deployments13:46
efriedbelmoreira: Okay, let's say hypothetically that you did that.13:46
edleafeAn ironic node type, for example13:46
efriedYou defined CUSTOM_GPU and gave your compute node inventory of 2 units of that.13:46
edleafeThe thing is that the virt driver is the authority on what is on a compute node13:46
efriedNow you create a VM and ask for resources:CUSTOM_GPU=113:47
edleafeIf it can't report it, it essentially isn't there13:47
efriedAnd verily, we claim that and land you on that host and build your VM.13:47
efriedSomething has to know what to *do* with that CUSTOM_GPU=1 in the allocation.13:47
efriedThat something is... the virt driver.13:47
cdentnot necessarily13:47
efriedAnd nobody ever told the virt driver what to do with that thing.13:47
cdentthere could be a third party which cares about that. the only thing that matters is counting them up and down by claiming and releasing allocations13:48
efriedcdent: What, you want the user to manually attach the GPU after the fact?13:48
cdenti think gpu is a bad example because it is a piece of hardware13:48
efriedbad example?  It's the use case we're trying to solve right now.13:48
cdentbut there could be lots of things that are inventory that is cared about by the compute node but do not need to be attached to the vm13:48
cdentthe general question is "why can't I have a custom rc, right?"13:49
cdentfor gpu's yes, they need to be managed by the virt driver13:49
cdentlet's say, for example, that you wanted to limit the total number of absolute instances on a host13:50
edleafecdent: then what would be an example of the sort on non-virt-driver-managed thing you have in mind?13:50
efriedor by cyborg, or whatever.13:50
cdentCUSTOM_INSTANCES: total 4513:50
cdentwhen we get to zero you can't use this host any more13:50
cdentwhen an instance goes away we make room for one more13:50
edleafeugh - implementing scheduler filters?13:50
cdentoriginally the entire idea was to get rid of all scheduler filters13:51
cdentI'm not saying the above is a good idea, but is an example of the concept13:51
cdent(first thing that came to mind in the inevitable painful rush of irc conversation)13:51
edleafeso... you're saying that the virt driver should only be authoritative for *some* RCs, and not others?13:52
efriedwell, to be fair, that's what we've said for traits.13:52
efriedwhich, once again, I think sucks.13:52
efriedthough I see why it's necessary (sort of).13:53
cdentedleafe: I'm not making a categorical statement one way or another. I'm merely supporting the idea that counting things is what resource classes provide, and some people may want to count in ways that we're not currently thinking of and I'm trying to think it through rather than dismiss it.13:54
cdentit could be a horrible way to do things, but I'm not currently sure13:54
edleafecdent: sure; I'mm trying to think it through too13:54
belmoreirain my case I was just trying to avoid enable a filter because few nodes13:55
efriedIn my case I want the whole PCI subsystem to die in a fire.13:55
edleafeI can certainly support things like "CUSTOM_AVAILABLE_VMS" as a resource in theory13:56
edleafe... except that if you don't want to have a limit, you'll have to set it to float(inf)  :)13:56
edleafeefried: we all do13:57
efriededleafe, cdent: For such things, ironic has been using the model of having the user define them in such a way that the virt driver picks them up.13:57
cdentyes, but that works because of a special relationship between ironic and nova13:57
efriedlike there's some kind of ironic API or CLI where you have to set custom traits so that they get picked up and *reported* by virt.13:57
edleafeefried: yeah - the node has to have its resource_class defined13:57
efriedI was talking about the traits thing, not the custom resource classes for compute nodes.13:58
cdentwe have, in the past, talked about a compute node being able to define itself (it's traits and classes) at startup13:58
edleafethat was made mandatory in Ocata, I think13:58
edleafebut yeah, that is a one-off solution between ironic and nova14:00
efriedThe traits thing isn't14:00
efriedit was designed to be more generic.14:00
* edleafe goes for more coffee14:01
cdentto distract us, I have a question about consumer generation conflict error messages. If you send a numeral when null/None is expected you get "expected None but got 1"14:02
cdentNone is python-speak, not JSON speak, does that matter?14:02
efriedI guess I would prefer if it said null.  That would require a special case in the code, right?14:02
efried"expected %s but got %s" % ('null' if expected is None else expected, 'null' if actual is None else actual)14:03
efriedkind of thing14:03
efriedwhich is ew, but technically more correct.14:04
cdentit's already special cased: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/util.py#L64314:05
openstackgerritBalazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object  https://review.openstack.org/56884014:07
openstackgerritBalazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec  https://review.openstack.org/56726714:07
openstackgerritBalazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes  https://review.openstack.org/57084714:07
openstackgerritBalazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler  https://review.openstack.org/56726814:07
openstackgerritBalazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding  https://review.openstack.org/56945914:07
openstackgerritBalazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request  https://review.openstack.org/57331714:07
cdentefried: unless you've already pushed that on to your magic stack, I'll go ahead and make a reminder bug so I don't switch out of what I'm up to now14:09
efriedcdent: Oh, didn't realize it was that easy.  I can push up a patch.14:10
efriedcdent: Doesn't even look like we have gabbits testing for it, so it'll be a one word change.14:11
cdentsince i'm nearly done with the bug anyway, I'll continue, then you have something to link to14:11
efriedcdent: Unless you want me to write a gabbi, which prolly wouldn't be the worst idea.14:11
belmoreirathanks all for the help14:12
cdenta gabbi thing is probably nice, becuase why not? https://bugs.launchpad.net/nova/+bug/177807114:12
openstackLaunchpad bug 1778071 in OpenStack Compute (nova) "The placement consumer generation conflict error message can be misleading" [Low,Triaged]14:12
efriedcdent: ack, on it.14:12
cdentbelmoreira: are you any further along than when you started or did we go off into abstraction too much?14:13
efriedcdent, edleafe: btw, real example of that "traits owned outside of virt" thing happened yesterday - see https://review.openstack.org/#/c/576877/ and its predecessor (linked in the commit message)14:15
belmoreiracdent: the abstraction was good because I was expecting to use custom RC as I wish.14:15
cdentbelmoreira: cool, good14:15
belmoreirabut still don't understand why we have a CLI to add custom RC to an inventory if they need to be set by the libvirt14:17
openstackgerritEric Fried proposed openstack/nova master: placement: s/None/null/ in consumer conflict msg  https://review.openstack.org/57717114:18
efriedcdent: ^14:18
cdentbelmoreira: I think that discussion didn't reach its finish. There's more to talk about.14:18
efriedbelmoreira: Because Nova is (theoretically) not the only thing that can use placement.14:18
efriedbelmoreira: In the case of Nova, it was decided that the virt driver needs to be the source of truth for the compute node's inventory.14:19
efriedbelmoreira: With nested providers, we're edging toward a world where entities outside of Nova are allowed to be the source of truth for providers *underneath* the compute node provider.14:19
efriedbut we're not quite there yet.14:20
belmoreiraefried: ok, thanks a lot14:20
efriedbelmoreira: In this case, the fact remains that you would still need some way to attach the allocated GPU to the instance.  If you wanted to do that by hand, we could actually make this thing work.14:20
efriedBecause you *can* attach custom *traits* to an instance.  So you could stuff CUSTOM_HAS_GPU on your compute node, just to make sure your instance landed there, and then do your manual attach.14:21
efriedcourse, then you don't know when you've run out.14:21
belmoreiraefried: yes14:23
efriedbelmoreira: if you're attaching GPUs by hand anyway, then you'll know when you've used the last of 'em, and you can remove the trait from the compute node :)14:27
efriedhackisher and hackisher...14:27
*** ttsiouts has quit IRC14:32
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge server create schema for BDM v2 extension  https://review.openstack.org/57718514:38
openstackgerritGhanshyam Mann proposed openstack/nova master: Merge server create schema for BDM v2 extension  https://review.openstack.org/57718514:40
openstackgerritChris Dent proposed openstack/nova master: WIP: [placement] Add /reshaper handler for POST  https://review.openstack.org/57692714:49
openstackgerritChris Dent proposed openstack/nova master: [placement] Extract create_allocation_list  https://review.openstack.org/57720014:49
cdentefried: the bottom of that ^ is some method extraction/refactoring for the thing above it. Your input appreciated on how to make further cleanups in the least annoying way. The inventory handling in reshaping duplicates a bit of code. The allocation handling doesn't need to.14:50
cdentedleafe: if you're around for real today, you too14:50
efriedack14:51
*** peereb has quit IRC14:51
*** ttsiouts has joined #openstack-placement14:55
*** ttsiouts has quit IRC15:03
*** ttsiouts has joined #openstack-placement15:09
*** gjayavelu has joined #openstack-placement15:14
edleafecdent: about 50% today. I'll try to get to it.15:22
cdentthanks15:22
openstackgerritBalazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding  https://review.openstack.org/56945915:24
openstackgerritBalazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request  https://review.openstack.org/57331715:24
*** ttsiouts has quit IRC15:28
*** ttsiouts has joined #openstack-placement15:29
*** ttsiouts has quit IRC15:33
*** Guest88320 is now known as dansmith15:59
openstackgerritEric Fried proposed openstack/nova master: Address nits from consumer generation  https://review.openstack.org/57722716:11
openstackgerritEric Fried proposed openstack/nova master: WIP: deduplicate {placement|api}_database help text  https://review.openstack.org/57722916:28
*** e0ne has quit IRC16:30
openstackgerritStephen Finucane proposed openstack/nova master: objects: Remove NUMATopologyLimits.obj_from_db_obj  https://review.openstack.org/53741216:38
openstackgerritStephen Finucane proposed openstack/nova master: objects: Remove legacy '_to_dict' functions  https://review.openstack.org/53741316:38
openstackgerritStephen Finucane proposed openstack/nova master: network: Unchain '_get_phynet_info' from '_get_port_vnic_info'  https://review.openstack.org/56444316:38
openstackgerritStephen Finucane proposed openstack/nova master: network: Rename 'create_pci_requests_for_sriov_ports'  https://review.openstack.org/57548816:38
openstackgerritStephen Finucane proposed openstack/nova master: network: Always retrieve network information if available  https://review.openstack.org/56444416:38
openstackgerritStephen Finucane proposed openstack/nova master: network: Retrieve tunneled status in '_get_network_info'  https://review.openstack.org/56444516:38
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NUMACell.network_info  https://review.openstack.org/56443916:38
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add NUMATopologyLimits.networks  https://review.openstack.org/57548616:38
openstackgerritStephen Finucane proposed openstack/nova master: hardware: Start accounting for networks in NUMA placement  https://review.openstack.org/56444816:38
openstackgerritStephen Finucane proposed openstack/nova master: objects: Add RequestSpec.numa_networks  https://review.openstack.org/56444216:38
openstackgerritStephen Finucane proposed openstack/nova master: scheduler: Start utilizing RequestSpec.numa_networks  https://review.openstack.org/56445216:38
openstackgerritStephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options  https://review.openstack.org/56444016:38
openstackgerritStephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_info field  https://review.openstack.org/56444116:38
openstackgerritStephen Finucane proposed openstack/nova master: network: Rename '_get_phynet_info' to '_get_network_info'  https://review.openstack.org/57723316:38
openstackgerritEric Fried proposed openstack/nova master: WIP: deduplicate {placement|api}_database help text  https://review.openstack.org/57722916:45
openstackgerritZack Cornelius proposed openstack/nova master: Fix nits from change I676291ec0faa1dea0bd5050ef8e3426d171de4c6  https://review.openstack.org/57698516:45
openstackgerritZack Cornelius proposed openstack/nova master: Implement discard for file backed memory  https://review.openstack.org/57699416:45
*** gjayavelu has quit IRC16:56
*** gjayavelu has joined #openstack-placement17:09
cdentnice digging on that aggregate stuff efried17:23
efriedcdent: Ugh, thanks.  Fixing it (everywhere) is going to be uuuuugly.17:23
efriedcdent: But at least it validates the need to have put generations on aggregates.17:24
*** e0ne has joined #openstack-placement17:30
openstackgerritEric Fried proposed openstack/nova master: WIP: Tighten up ReportClient use of generation  https://review.openstack.org/55666917:40
*** tssurya has quit IRC17:54
openstackgerritEric Fried proposed openstack/nova master: Tighten up ReportClient use of generation  https://review.openstack.org/55666917:58
efriedcdent: In case you're following - that oughtta do it ^17:58
cdentthat's in my queue, but slogging through meeting at the moment. mleh17:59
efriedfurther work is needed to resolve the actual race.  Possibly with retries.17:59
efriedThink I'm gonna take the afternoon off, in case anyone asks.18:17
*** efried is now known as efried_pto18:20
cdentgood choice efried_pto . you work too much.18:20
efried_ptoso says my wife18:20
cdentshe's right18:20
cdentand you can tell her I said so18:21
*** tssurya has joined #openstack-placement18:44
*** tssurya has quit IRC18:48
*** e0ne has quit IRC19:17
*** tssurya has joined #openstack-placement19:19
*** gjayavelu has quit IRC19:33
openstackgerritLenny Verkhovsky proposed openstack/nova master: WIP workaround for RHEL7.5 qemu 2.0  https://review.openstack.org/57728019:35
*** gjayavelu has joined #openstack-placement20:33
*** takashin has joined #openstack-placement20:43
openstackgerritMatt Riedemann proposed openstack/nova master: Address nits from consumer generation  https://review.openstack.org/57722720:44
openstackgerritChris Dent proposed openstack/nova master: WIP: [placement] Add /reshaper handler for POST  https://review.openstack.org/57692720:50
*** gjayavelu has quit IRC20:52
*** cdent has quit IRC20:53
openstackgerritTakashi NATSUME proposed openstack/nova-specs master: Create specs directory for Stein  https://review.openstack.org/57360221:15
*** tssurya has quit IRC21:35
*** nicolasbock has quit IRC21:54
*** tssurya has joined #openstack-placement21:56
*** tssurya has quit IRC22:00
*** edmondsw has quit IRC22:24
*** edmondsw has joined #openstack-placement22:26
*** edmondsw has quit IRC22:35
*** edmondsw has joined #openstack-placement22:35
*** tssurya has joined #openstack-placement22:38
openstackgerritMerged openstack/nova master: Add check if neutron "binding-extended" extension is available  https://review.openstack.org/52354822:39
*** edmondsw has quit IRC22:40
*** tssurya has quit IRC22:43
openstackgerritZack Cornelius proposed openstack/nova master: Fix nits from change Ia7cf4414feb335b3c2e863b4c8b4ff559b275c34  https://review.openstack.org/57731422:44
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (4)  https://review.openstack.org/56846223:58
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py  https://review.openstack.org/56615323:59
openstackgerritTakashi NATSUME proposed openstack/nova master: Remove mox in nova/tests/unit/virt/xenapi/stubs.py  https://review.openstack.org/56841223:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!