Earlier  
Posted Nick Remark
#openstack-nova - 2018-01-17
17:31:18 cdent jaypipes: is the expectation that in provider summaries, anything with a parent will have parent_provider_uuid field?
17:33:01 jaypipes cdent: yup, zactly.
17:33:17 cdent roger, con aye, ten degrees down bubble
17:37:14 openstackgerrit Jackie Truong proposed openstack/nova master: Account for deprecation of personality files https://review.openstack.org/534849
17:51:33 cdent jaypipes: another, more theorectical, question. sean-k-mooney has a various times talked about moving vcpu and memory to the nested numa node, in which case the allocation records in allocation_requests will be against that. when that happens are we assuming that the nova-scheduler will climb the provider tree hierarchy...
17:52:01 cdent ...recreated from provider summaries, to find the compute node where the actual spawn will happen?
17:53:33 jaypipes cdent: yes
17:53:42 cdent cool
18:04:16 sean-k-mooney cdent: well we schedule resouce based on the leaf nodes to select the set of roots nodes that can fufil the request so yes.
18:04:20 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
18:06:37 openstackgerrit Aditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371
18:08:07 openstackgerrit Jay Pipes proposed openstack/nova master: fix _get_trees_matching_all() when sum children https://review.openstack.org/534866
18:08:17 sean-k-mooney cdent: before we can move the vcpu or other resouce under a numa node i belive there was a distance/ajacency concept that was required also to allow affinity/antiaffinity to be expressed. that is out of scope for the current nested prodiers in queens right jaypipes?
18:08:22 jaypipes efried, gibi: eyeballs needed on the patch ^^
18:08:44 jaypipes sean-k-mooney: totally out of scope, yes
18:09:22 sean-k-mooney jaypipes: and also a prerequec for numa in placement or do you think they can be independent
18:09:41 jaypipes sean-k-mooney: pre-req
18:09:48 jaypipes sean-k-mooney: well....
18:10:05 jaypipes sean-k-mooney: pre-req if we want placement to do certain numa placement calculations...
18:10:31 jaypipes sean-k-mooney: I'm still not entirely convinced that placement should do that for NUMA nodes (as opposed to a weigher in scheduler doing it)
18:10:35 sean-k-mooney hehe pre-req or ptg topic :), i guess you could delegate to the numa topology filter
18:11:17 jaypipes sean-k-mooney: if you can only imagine the SQL in https://review.openstack.org/#/c/534866/1/nova/objects/resource_provider.py getting 10X more complicated ...
18:12:59 sean-k-mooney jaypipes: ya i have not looked at how nrp has evovled both the sql and usage of placement to have developed an intuition regarding if the logic should be kept in the schduler and just track numa info in placement
18:13:41 jaypipes ++
18:15:28 openstackgerrit Merged openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512
18:15:38 openstackgerrit Merged openstack/nova master: make unit tests compatible with os-vif 1.8.0 https://review.openstack.org/533766
18:17:01 sean-k-mooney oh ^ merged cool. now just waiting for https://review.openstack.org/#/c/534839/ and we are set for os-vif
18:18:07 cdent jaypipes: is your work also dealing with adding support for 'required' or is that rocky?
18:18:21 jaypipes cdent: for n-r-p?
18:18:44 cdent traits in n-r-p, yeah
18:19:17 sean-k-mooney cdent: im not sure you need special logic unless you want traits to inherit down the tree
18:19:48 cdent as far as I can tell 'required' is not merged in any fashion, yet, n-r-p or not
18:19:59 cdent (at the HTTP level)
18:21:11 sean-k-mooney cdent: i think alex xu was mainly working on required/traits
18:21:27 cdent yeah, just pulling up his pending stuff
18:21:51 sean-k-mooney cdent: :) you mean https://review.openstack.org/#/q/owner:hejie.xu%2540intel.com+status:open
18:22:05 cdent yes
18:22:57 cdent Basically I'm trying to figure out how much is not done
18:23:33 cdent and it looks like maybe it could just be a matter of it not being turned on
18:24:48 sean-k-mooney cdent: i think this is one of the main blockers for traits https://review.openstack.org/#/c/479776/
18:27:13 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
18:27:49 sean-k-mooney cdent: im not sure how much of that has been pulled into https://review.openstack.org/#/q/topic:bp/granular-resource-requests+(status:open+OR+status:merged) however
18:31:49 sean-k-mooney efried: does https://review.openstack.org/#/c/517757/1 superceed https://review.openstack.org/#/c/479776 i belive the numbered RequireN sysntax does
18:36:51 mgoddard efried: whilst trying to get myself out of a hole caused by something a bit like https://bugs.launchpad.net/nova/+bug/1714248, I have hit this odd issue: http://paste.openstack.org/show/646537/hl
18:36:53 openstack Launchpad bug 1714248 in OpenStack Compute (nova) pike "Compute node HA for ironic doesn't work due to the name duplication of Resource Provider " [High,In progress] - Assigned to Matt Riedemann (mriedem)
18:37:30 mgoddard efried: can't delete a resource provider because it has a foreign key that points back to itself
18:38:03 mgoddard efried: I did try this via placement API first, with the same result
18:42:04 cdent mgoddard: that's new and different and fixed by...
18:42:16 cdent https://review.openstack.org/#/c/529519/6
18:42:23 sean-k-mooney mgoddard: well the root node of a tree should have itself as a root_provider_id
18:43:16 sean-k-mooney cdent: :) i was just going to say you could do an update of the root_provider_id then delete but thats a better solution
18:46:14 mgoddard cdent: thanks
19:18:34 cdent jaypipes, sean-k-mooney, efried : this explains some of my confusion: most of the code was already there https://bugs.launchpad.net/nova/+bug/1743860
19:18:35 openstack Launchpad bug 1743860 in OpenStack Compute (nova) "allocation candidates db backend can handle non-nested traits but the HTTP front end is not turned on" [Undecided,New]
19:19:23 jaypipes cdent: ack
19:23:19 openstackgerrit Chris Dent proposed openstack/nova master: DNM: demo making 'required' work on /allocation_candiates https://review.openstack.org/534886
19:24:36 cdent and that's ^ the demo making it go
20:26:34 spotz Anyone about who can answer a question about an OLD blueprint?
20:28:15 cdent how old?
20:29:11 spotz cdent juno:) https://specs.openstack.org/openstack/nova-specs/specs/juno/approved/selecting-subnet-when-creating-vm.html - someonoe was asking how to add a new instance on a subnet and came across this. Doesn't seem dooable with Oopenstack cli or nova client though
20:29:35 cdent juno is before my nova time
20:30:18 cdent i would guess some kind of server group affinity is the way to do that, but I have no real clue. jaypipes and sean-k-mooney were around a while ago and they may know
20:30:45 spotz Yeah and johnthetubaguy approved the spec but I suspect he's asleep
20:31:12 spotz poor jaypipes:(
20:32:55 cdent john being potentially asleep reminds me that I'm in the same time zone as him, time to exit
20:33:20 spotz heheh
22:20:35 openstackgerrit sean mooney proposed openstack/os-vif master: Fix VF-rep lookup routine to use parent PF number https://review.openstack.org/531674
22:29:56 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3) https://review.openstack.org/526557
22:33:15 openstackgerrit sean mooney proposed openstack/os-vif master: Configure privsep binary https://review.openstack.org/531358
22:41:10 efried edleafe qq, in alt hosts, is there a conf option to tweak how many alt hosts I get?
22:44:28 edleafe efried: CONF.scheduler.max_attempts
22:53:22 efried edleafe Thanks.
22:53:27 efried mdrabe ^
23:01:50 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform instance.resize_confirm notification https://review.openstack.org/482557
23:09:27 openstackgerrit Jay Pipes proposed openstack/nova master: placement: _get_trees_matching_all_resources() https://review.openstack.org/531512
23:09:28 openstackgerrit Jay Pipes proposed openstack/nova master: tests for alloc candidates with nested and traits https://review.openstack.org/531899
23:09:28 openstackgerrit Jay Pipes proposed openstack/nova master: add test for scenario with sum of child resources https://review.openstack.org/534339
23:09:29 openstackgerrit Jay Pipes proposed openstack/nova master: _get_trees_matching_all_resources() sum children https://review.openstack.org/534866
23:09:30 openstackgerrit Jay Pipes proposed openstack/nova master: add _get_usages_from_trees() function https://review.openstack.org/534967
23:09:30 openstackgerrit Jay Pipes proposed openstack/nova master: WIP - populate alloc cands for trees https://review.openstack.org/534968
#openstack-nova - 2018-01-18
01:23:05 openstackgerrit wangdequn proposed openstack/nova master: correct referenced url in comments https://review.openstack.org/534987
01:36:47 openstackgerrit Chen Hanxiao proposed openstack/nova master: _cleanup_resize: remove rbd swap disk if resize to flavor without swap https://review.openstack.org/534711
01:37:20 Spazmotic Morning
01:38:06 Spazmotic Here's to another day of praying for some XenAPI code review
01:54:23 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: [WIP]Fix update server's wrong meta when update concurrently https://review.openstack.org/534682
02:20:56 openstackgerrit lei zhang proposed openstack/nova master: Fix openstackdocstheme options for api-ref https://review.openstack.org/534999
02:21:53 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
02:35:12 openstackgerrit Merged openstack/nova master: Fix fake libvirt XML generation for disks https://review.openstack.org/531165
02:35:19 openstackgerrit Merged openstack/nova master: Account for deprecation of personality files https://review.openstack.org/534849
02:52:25 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API https://review.openstack.org/524506
03:12:01 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/535030
03:22:00 openstackgerrit OpenStack Proposal Bot proposed openstack/os-traits master: Updated from global requirements https://review.openstack.org/533994
03:22:11 openstackgerrit OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/533918
03:30:09 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/535121
04:06:52 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: [WIP]Fix update server's wrong meta when update concurrently https://review.openstack.org/534682
04:36:41 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823
05:13:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (1) https://review.openstack.org/531258
05:47:36 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266
06:09:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280
06:16:19 diga jaypipes: hi

Earlier   Later