Earlier  
Posted Nick Remark
#openstack-nova - 2017-04-25
15:31:39 mriedem sean-k-mooney: we don't have allocations for numa yet
15:31:41 mriedem in placement
15:31:58 mriedem i believe that's nested resource providers territory
15:32:13 sean-k-mooney oh ok i understand the gap now. ya that was covered by nested providers
15:32:43 mriedem so if you're an NFV shop and we don't have nested resource providers yet, then you'll need to enable retries yes
15:32:56 mriedem which means you're not using the whole cells v2 superconductor deployment model
15:33:04 bauzas mriedem: dansmith: I can start pushing code, fersür
15:33:48 bauzas the spec, IHMO, is in a good shape with good work items, so that's a cool start
15:34:08 bauzas now that said, I need to disappear for now
15:34:18 bauzas ttyl folks
15:39:08 openstackgerrit Daniel Pawlik proposed openstack/nova master: Added kvm_hidden feature https://review.openstack.org/459753
15:40:15 dansmith sean-k-mooney: if we ran filters twice or something maybe
15:41:01 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676
15:42:19 openstackgerrit Chris Dent proposed openstack/nova master: Register osapi_compute when nova-api is wsgi https://review.openstack.org/457283
15:42:20 openstackgerrit Chris Dent proposed openstack/nova master: Add pbr-installed wsgi application for metadata api https://review.openstack.org/459413
15:46:22 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification https://review.openstack.org/455801
15:52:24 sean-k-mooney dansmith: perhaps though honestly if the only main gap is the nfv constartisn of numa,cpu pinning,hugepages and pci devices it sounds like that will be addressed in time by nested resouce providers.
15:52:41 dansmith sean-k-mooney: indeed, that's the plan
15:52:58 sean-k-mooney dansmith: a validate call to the filters that accpeted a claim would also work but its a lot of work that may not be worth it
15:53:17 dansmith sean-k-mooney: yeah, I'd rather focus on the right fix
15:53:35 mriedem i haven't thought yet about how we replace the affinity filters with placement
15:54:03 mriedem i guess sum the reqs for all members of an affinity group and find a provider for them
15:54:23 sean-k-mooney the affinity filters in general or the server group affinity filters
15:54:27 dansmith mriedem: needs more placement support I think.. jaypipes has waxed poetic about it in the past
15:54:29 mriedem server group
15:54:44 sean-k-mooney mriedem: traits maybe?
15:55:00 mriedem anti-affinity, you could get RPs where len(RPs) > anti-affinity server group members, and then have to fit each member into a unique provider
15:55:04 mriedem but idk
15:55:12 mriedem sean-k-mooney: traits aren't the answer to everything :)
15:55:26 dansmith lol
15:55:36 mriedem when i give the project roadmap thing for R,
15:55:53 mriedem i think i'm going to say, "fix traits which has become the new bdm/quotas/local delete quagmire"
15:56:27 dansmith lol
15:56:31 mriedem roadmap for R: turn traits into CIMaaS
15:56:36 jaypipes mriedem: server groups are on my list of "figure out how to port this over to placement" things, yeah....
15:56:39 sean-k-mooney true i was thinking more of taging a resouce prvider in some way and saying only condider providers with this tag
15:56:55 jaypipes mriedem: for the immediate future, they will need to stay in the scheduler filters/weighers.
15:57:00 mriedem sean-k-mooney: that's more like an aggregate to me
15:57:06 jaypipes mriedem: right.
15:57:08 mriedem jaypipes: yeah that's fine
15:57:26 jaypipes mriedem: the problem with server groups has *always* been how to handle changes in the group over time.
15:57:43 sean-k-mooney mriedem: yes more of dynamcally created one though
15:58:07 mriedem sean-k-mooney: now you're just talking sci fi
15:58:13 mriedem :)
15:58:19 sean-k-mooney :)
15:58:30 mriedem don't forget we have to add support for migrating affinity groups as a unit
15:59:28 sean-k-mooney mriedem: that would be an entirely new feature right? currently you cant migrate a group just the instance of a group
16:00:04 sean-k-mooney that is going to create a lot of headaches if one of the member fails to migrate
16:00:14 mriedem sean-k-mooney: yes entirely new
16:00:29 mriedem sean-k-mooney: let me find some light reading for your evening
16:01:02 mriedem sean-k-mooney: https://review.openstack.org/#/c/440580/
16:01:35 mriedem basically we can spew out scheduler hints from the API so external systems can attempt to coordinate things, or we build something into nova
16:02:41 sean-k-mooney i see are the sceduler hints corrent stored in the db and this is just exposing them or are we also going start storeing them
16:02:50 sean-k-mooney just reading over it now
16:03:00 mriedem they are stored
16:03:03 mriedem we don't expose them
16:03:27 openstackgerrit Daniel Pawlik proposed openstack/nova master: Added kvm_hidden feature https://review.openstack.org/459753
16:03:39 sean-k-mooney ah first line in promblem discription well that make upgrades simplar as we still have the info for exisint server groups
16:05:09 sdague dansmith / jaypipes - easy +2
16:05:12 sdague https://review.openstack.org/#/c/459735/
16:06:55 openstackgerrit Matt Riedemann proposed openstack/nova master: api-ref: move createBackup to server-actions https://review.openstack.org/452971
16:06:55 openstackgerrit Matt Riedemann proposed openstack/nova master: Deprecate Multinic, floatingip action and os-virtual-interface API https://review.openstack.org/457181
16:06:56 openstackgerrit Matt Riedemann proposed openstack/nova master: 2.43: Remove Location header from createImage and createBackup responses https://review.openstack.org/455443
16:07:29 mriedem sdague: https://review.openstack.org/#/c/457181/
16:07:30 mriedem for 2.44
16:07:39 openstackgerrit Balazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667
16:07:39 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077
16:07:51 sdague mriedem: loking
16:08:32 openstackgerrit Matt Riedemann proposed openstack/nova master: 2.45: Remove Location header from createImage and createBackup responses https://review.openstack.org/455443
16:09:42 jaypipes sdague: done
16:11:21 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.volume_attach.error notification https://review.openstack.org/455801
16:14:01 sdague mriedem: done
16:14:09 sdague mriedem: one issue on your 2.45 patch I think
16:14:16 sdague if you want to go look, just left review comments
16:17:29 mriedem ok
16:19:02 mriedem replied, i can make a change
16:20:00 sdague mriedem: cool
16:25:04 efried mriedem These, whenever you get a moment: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/powervm-nova-compute-driver+is:mergeable
16:25:32 efried please and thanks
16:25:38 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077
16:25:52 openstackgerrit Balazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667
16:28:06 openstackgerrit Matt Riedemann proposed openstack/nova master: 2.45: Remove Location header from createImage and createBackup responses https://review.openstack.org/455443
16:28:10 mriedem sdague: ^
16:34:46 sdague mriedem: you want to also assert the body is empty? We probably should have done that before
16:35:04 sdague which I was thinking, but apparently forgot to write
16:35:59 mriedem i suppose i can do that
16:36:54 sdague just so that the 2.45 code doesn't slip back into there
16:37:15 sdague then, I'm +2 for reals
16:44:22 openstackgerrit Matt Riedemann proposed openstack/nova master: 2.45: Remove Location header from createImage and createBackup responses https://review.openstack.org/455443
16:44:24 mriedem sdague: done ^
16:54:37 openstackgerrit Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677
16:55:12 openstackgerrit Eric Fried proposed openstack/nova master: WIP/PoC: nova.utils.get_service_url(group) https://review.openstack.org/458257
16:55:52 efried mordred jaypipes johnthetubaguy If y'all have a few minutes, I'd like to chat about the direction of this ^^
16:56:04 jaypipes efried: yuppers. it's on my list for today.
16:56:34 efried jaypipes Cool; just updated with some fixes, did some live testing, and I think it's working the way I want, as far as it goes.
16:57:01 efried Before I get any further, I've got some outstanding questions.
16:57:04 johnthetubaguy efried: yeah, my take was lets make all the bits of Nova consistent in our usage, where possible, but it looks like mordred has some alternative ideas
16:57:29 johnthetubaguy efried: I am about to go and make my dinner in a few mins though, could answer some quick questions
16:57:29 efried johnthetubaguy I think I've isolated mordred's alternatives down to the guts of one private method.
16:57:47 efried What I gathered was that those suggestions involved pieces that weren't quiiite ready for primetime usage.

Earlier   Later