Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-11-20
20:44:51 efried no. GZIP_CAPABLE stays. Not sure we have to retrait every time we reprogram.
20:45:32 efried but if we do, the GZIP_IS_ON_THIS_THING_AT_THE_MOMENT trait would be separate, and have separate meaning, than the GZIP_CAPABLE.
20:45:57 dansmith you're just providing no abstraction there
20:46:02 efried the former would be used only as an optimization, if/when we have "preferred traits", to avoid reprogramming if there's one that's already set up.
20:46:30 jaypipes premature optimization...
20:47:03 efried I'd be happy if we skipped that whole bit for the first pass and just used the *_CAPABLE traits.
20:47:14 efried Programming gets done after the claim, if and as necessary.
20:47:23 jaypipes I'd be happy if we just skipped everything other than just using custom resource classes.
20:47:46 efried jaypipes: So preprogram everything?
20:48:05 dansmith pretty sure we said the first step was assuming everything was static, no?
20:48:22 dansmith except for the "user will program it themselves" case of course
20:48:44 jaypipes dansmith++ again.
20:49:21 dansmith when we talked about this in (denver I think?) I think the overwhelming majority of cases where this really applies is the pre-programmed case,
20:49:39 dansmith because it provides for locality in certain FGPAs that have one code region and multiple execution contexts,
20:50:11 dansmith such that if you co-locate a GZIP and a TLS, they both can't use the same FPGA, but if you get two GZIP tenants on the same box, they can
20:50:41 dansmith and I thought we agreed to avoid boiling the ocean with "everything is completely dynamic all the time forever" until we could do, you know, fucking anything :)
20:50:56 efried I can buy it for a first pass. Long-term, that seems like not very cloudy. Though I suppose if the "pre"programming is done by a higher orchestrator, it could fly.
20:50:58 dansmith maybe that's just me missing something, but..
20:51:14 efried okay, thanks for the fresher.
20:51:15 dansmith efried: well, if you do it with inventories, you can actually count usage of those things,
20:51:33 dansmith and then your pre-programming workflow can ensure that X% of GZIP is available based on current demand
20:51:41 dansmith but if you do it with traits that seems a lot messier
20:52:04 dansmith usage and capacity I mean
20:52:07 slaweq mriedem: thx for taking care of this issue
20:52:37 dansmith efried: the orchestrator that maintains a certain amount of available inventory of stuff, I mean
20:52:40 jaypipes I was under the impression Cyborg was gonna contain that "pre-orchestrator/pre-programming" thing...
20:53:05 dansmith jaypipes: yeah I dunno, could be, or could be a cron job you run once an hour that makes sure the capacity is within limits
20:53:17 dansmith meaning I dunno if that level of cron is something cyborg was going to do or not,
20:53:22 efried "load balancing" your accelerators
20:53:51 efried guess that implies moving them around, which isn't what we're talking about
20:53:54 efried but yeah, I get the idea.
21:46:53 jaypipes sean-k-mooney: questions for you on https://review.openstack.org/#/c/602384/ please
22:09:15 openstackgerrit Chris Dent proposed openstack/nova master: Use external placement in functional tests https://review.openstack.org/617941
22:09:43 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Delete the placement code https://review.openstack.org/618215
22:13:53 mriedem efried: turns out https://review.openstack.org/#/c/619061/ does fix the pg thing
22:14:03 mriedem also, wee lots of red http://logs.openstack.org/05/613305/7/check/tempest-full/999ec9f/controller/logs/screen-n-api.txt.gz?level=ERROR
22:14:08 mriedem unrelated regression
22:14:37 efried ack x2
22:15:35 efried mriedem: What were the func test failures? Actual differences in results?
22:15:56 mriedem dansmith: looks like an unintended side effect of using the scatter_gather_single_cell for nova show is the scatter thing logs errors from the query^ even for things we expect
22:16:07 mriedem efried: yeah, got 5 rows when 1 expected
22:16:12 mriedem efried: pull it down and try it out
22:16:17 efried okay, must be aggregate functions
22:16:27 efried nah, higher priorities.
22:16:28 dansmith mriedem: oh yeah, I think I called that out initially and then totally forgot :(
22:16:34 dansmith it's spewing errors to the logs right?
22:16:36 mriedem yes
22:16:39 mriedem i'll open a bug
22:16:40 efried If it fixes the problem, let's roll with it.
22:19:09 mriedem https://bugs.launchpad.net/nova/+bug/1804325
22:19:09 openstack Launchpad bug 1804325 in OpenStack Compute (nova) "InstanceNotFound traceback errors in n-api logs while polling for server delete" [High,Triaged]
22:20:16 mriedem so, i think we just remove that exception line since the caller can get the actual exception type now
22:20:20 mriedem and decide if it needs to log
22:20:33 mriedem dansmith: you want it or shall i?
22:20:55 dansmith I don't want it
22:21:03 dansmith I'm "finishing one email" away from disappearing
22:21:09 mriedem are you around tomorrow?
22:21:42 mriedem air quotes is acceptable
22:23:03 dansmith heh
22:23:12 dansmith I am but I have a few things going on
22:23:19 dansmith but if you lay on the guilt extra thick I might do something productive
22:30:01 mriedem i'm going to be working hard at trying to figure out this espresso maker i bought
22:30:18 mriedem it's like if dr seuss tried to make coffee
22:32:42 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove exception logging from scatter_gather_cells https://review.openstack.org/619110
22:39:40 openstackgerrit Matt Riedemann proposed openstack/nova master: Add HPET timer support for x86 guests https://review.openstack.org/605902
23:44:14 openstackgerrit Takashi NATSUME proposed openstack/nova stable/rocky: Add description of custom resource classes https://review.openstack.org/619122
23:57:59 mriedem hello friends, could use some core reviews on this pretty simple straight forward spec https://review.openstack.org/#/c/612531/
#openstack-nova - 2018-11-21
00:06:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Add functional regression recreate test for bug 1790204 https://review.openstack.org/619123
00:06:54 openstack bug 1790204 in OpenStack Compute (nova) "Allocations are "doubled up" on same host resize even though there is only 1 server on the host" [Medium,Triaged] https://launchpad.net/bugs/1790204
00:41:09 openstackgerrit Takashi NATSUME proposed openstack/nova stable/queens: Add description of custom resource classes https://review.openstack.org/619125
02:00:02 openstackgerrit Merged openstack/nova master: Consider root id is None in the database case https://review.openstack.org/613305
02:39:47 jascott1 hi, im trying to understand how and under what conditions openstack attaches config-drive to a VM. I have one distro that has the config drive and one that doesnt. Any help appreciated
03:54:50 openstackgerrit Yikun Jiang proposed openstack/nova master: WIP: Remove live_migration_progress_timeout config https://review.openstack.org/619142
03:54:51 openstackgerrit Yikun Jiang proposed openstack/nova master: WIP: Add live migration timeout action https://review.openstack.org/619143
04:02:00 openstackgerrit Takashi NATSUME proposed openstack/nova master: Add descriptions about microversions https://review.openstack.org/619144
04:18:46 bhagyashris Atom: Hi,
04:20:52 bhagyashris artom: Hi, Sorry for spelling ^ because of auto correction
04:44:22 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Fix a type of block_device_mapping_v2 in a comment https://review.openstack.org/619145
05:35:45 openstackgerrit Merged openstack/nova master: Remove v1 check in Cinder client version lookup https://review.openstack.org/617927
07:05:05 openstackgerrit Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142
07:26:13 openstackgerrit Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629
07:32:18 openstackgerrit Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142
07:37:54 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Amend the detach-boot-volume design https://review.openstack.org/619161
07:38:59 openstackgerrit Zhenyu Zheng proposed openstack/nova-specs master: Make scheduling weight more granular https://review.openstack.org/599308
07:53:09 openstackgerrit Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142
07:53:09 openstackgerrit Yikun Jiang proposed openstack/nova master: Add live migration timeout action https://review.openstack.org/619143
08:22:25 openstackgerrit Silvan Kaiser proposed openstack/nova master: Added mount fstype based validation of Quobyte mounts https://review.openstack.org/522245
08:34:28 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add method to allow fetch root_volume BDM by instance_uuid https://review.openstack.org/614672
09:04:09 openstackgerrit Yikun Jiang proposed openstack/nova master: Add live migration timeout action https://review.openstack.org/619143
09:12:01 frickler melwitt: how long do we need to wait for further reviews on https://review.openstack.org/616580 ? this is a severe issue for us, currently running with a local patch. should I propose stable backports already or wait for the patch being merged in master?
09:46:02 kashyap frickler: Hi, Mel is away on PTO, so a core reviewer can just do +W there, since it already has two +2s
09:49:01 cdent ah yes, Thanskgiving, a time of review drought and dry turkeys.
09:50:13 melwitt frickler: I'm not sure. I would ask mriedem since he left a comment about waiting a bit before approval
10:29:49 openstackgerrit Chris Dent proposed openstack/nova master: Use external placement in functional tests https://review.openstack.org/617941
10:29:49 openstackgerrit Chris Dent proposed openstack/nova master: WIP: Delete the placement code https://review.openstack.org/618215
12:01:51 openstackgerrit Jens Harbott (frickler) proposed openstack/nova stable/queens: Make supports_direct_io work on 4096b sector size https://review.openstack.org/619220
12:06:49 openstackgerrit Elod Illes proposed openstack/nova master: Transform scheduler.select_destinations notification https://review.openstack.org/508506
12:10:01 openstackgerrit Jan Gutter proposed openstack/nova-specs master: Spec to implement os-vif generic datapath offloads https://review.openstack.org/607610
12:26:14 mnaser https://review.openstack.org/#/c/615134/
12:26:43 mnaser this has been sitting for a little while and it's very trivial

Earlier   Later