Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-11-20
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
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/
13:53:33 openstackgerrit Hervé Beraud proposed openstack/nova stable/rocky: Stop to mocking oslo.service.loopingcall._ThreadingEvent since this class was removed https://review.openstack.org/619246
13:58:03 frickler mriedem: do you want to continue waiting on https://review.openstack.org/616580 ? it would be great if we could get that fix into the next round of stable releases, not sure whether there is something scheduled for that yet
14:00:16 mriedem frickler: i was waiting for more core reviews, but they didn't come, so approved
14:00:21 mriedem you'll need to backport to stable/rocky
14:00:39 mriedem we likely won't be doing a series of stable releases until after the US thanksgiving holiday (which is tomorrow)
14:02:53 openstackgerrit Jens Harbott (frickler) proposed openstack/nova stable/rocky: Make supports_direct_io work on 4096b sector size https://review.openstack.org/619251
14:04:16 frickler mriedem: stable/rocky is a simple cherry-pick from master, which I did now. stable/queens required editing because the function moved its location, which is why I prepared that patch earlier
14:08:12 frickler mriedem: do you think that it should also go into pike and ocata? seems those would require more editing, but the bug is present in those branches, too.
14:08:29 openstackgerrit Hervé Beraud proposed openstack/nova stable/rocky: Stop to mocking oslo.service.loopingcall._ThreadingEvent since this class was removed https://review.openstack.org/619246
14:09:32 mriedem frickler: can you re-cherry-pick to rocky using the command line?
14:09:35 mriedem with the -x option
14:09:46 mriedem the gerrit UI won't do that until the change you're cherry picking *from* has merged
14:10:28 mriedem frickler: ocata is in extended maintenance mode and i'm not really sure this needs to go that far back upstream
14:10:47 mriedem EM means it *could* go that far back, but i'm not sure it's high enough priority
14:10:57 mriedem s/priority/severity/
14:12:28 frickler mriedem: oh, I was hoping gerrit would be the better option than command-line. but does it do anything other than add the "cherry picked from" line to the commit message? in that case I'd just add that line manually

Earlier   Later