| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-11-20 | |||
| 20:44:01 | efried | But then a resource provider representing a blank (as-yet-unprogrammed) FPGA would have to show inventories of multiple resource classes, and then when one of those is consumed, we would have to nix the other resource classes (or do the reserved=total trick for them). | |
| 20:44:17 | efried | which is racy, as well as being ew. | |
| 20:44:19 | dansmith | same for the trait right? | |
| 20:44:32 | dansmith | you say FPGA=1, trats=GZIP,TLS,BITCOIN | |
| 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 | |