| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-11-20 | |||
| 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 | openstack | Launchpad bug 1804325 in OpenStack Compute (nova) "InstanceNotFound traceback errors in n-api logs while polling for server delete" [High,Triaged] | |
| 22:19:09 | mriedem | https://bugs.launchpad.net/nova/+bug/1804325 | |
| 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 | 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:06:54 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional regression recreate test for bug 1790204 https://review.openstack.org/619123 | |
| 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: Add live migration timeout action https://review.openstack.org/619143 | |
| 07:53:09 | openstackgerrit | Yikun Jiang proposed openstack/nova master: Remove live_migration_progress_timeout config https://review.openstack.org/619142 | |
| 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: WIP: Delete the placement code https://review.openstack.org/618215 | |
| 10:29:49 | openstackgerrit | Chris Dent proposed openstack/nova master: Use external placement in functional tests https://review.openstack.org/617941 | |
| 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 | |
| 12:34:59 | jangutter | ^^ impressive amount of CI checks against that one. | |
| 12:36:41 | sean-k-mooney | mnaser: so the logic is really old flavor could only be plublic so default to public? if that is the case then yes this makes sense | |
| 12:37:49 | mnaser | jangutter: i pride myself in checking my stuff, hah | |
| 12:38:12 | mnaser | sean-k-mooney: well we either have to assume its public or its not. given that we default to public=True, and that field is missing, it probably is true | |
| 12:38:50 | sean-k-mooney | ok i can see the logic in that argument. | |
| 12:57:47 | sean-k-mooney | jaypipes: https://review.openstack.org/#/c/602384/ responces in line. ill respin once you decide if you would like to see addtional changes but i think the typo was the only thing you wanted adressed in the patch correct | |
| 13:01:05 | jaypipes | sean-k-mooney: yup. was just curious about the other questions... | |
| 13:14:07 | sean-k-mooney | cool in that case ill fix the typo. can you cast your eye over the follow up patch when you have a chance | |
| 13:49:46 | mriedem | dansmith: lyarwood: https://review.openstack.org/#/q/topic:bug/1799892+(status:open+OR+status:merged) on stable should be good to go now | |
| 13:50:33 | mriedem | other cores, the hpet change just needs a final +2 https://review.openstack.org/#/c/605902/ | |