| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-11-20 | |||
| 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/ | |
| 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 | |
| 14:13:30 | frickler | mriedem: o.k., so I'll do pike but not ocata. pike is the release most of our deployments are currently running on. | |
| 14:14:37 | openstackgerrit | Jens Harbott (frickler) proposed openstack/nova stable/rocky: Make supports_direct_io work on 4096b sector size https://review.openstack.org/619251 | |
| 14:17:48 | mriedem | stable cores: we have quite a few rocky backports that need a final +2 https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/rocky+label:Code-Review=2 | |
| 14:18:37 | openstackgerrit | Jens Harbott (frickler) proposed openstack/nova stable/pike: Make supports_direct_io work on 4096b sector size https://review.openstack.org/619254 | |
| 14:31:21 | gary_perkins | Hi! I'm not sure if this would be the best channel to ask, but I ask anyway :) I have a kolla-ansible deployment of Queens. I have set a nova policy rule "os_compute_api:servers:create:zero_disk_flavor": "rule:admin_api" to stop any image-backed instances with a zero root-disk from being created. I want to only allow cinder-volume backed instances. But, for some reason, this policy change has had no noticable effect. I'm expecting my " | |
| 14:31:22 | gary_perkins | openstack server create" to return an error, but instead it creates an instance consuming local storage. Is there something I'm missing? | |
| 14:31:49 | gary_perkins | TIA | |
| 15:12:17 | sean-k-mooney | so rather then ask this in the olso changel again.. the stien schdule does not list the nova specs freeze. is it m2 | |
| 15:15:34 | cdent | sean-k-mooney: last I heard, yes, but I don't have a good memory for arbitrary deadlines | |
| 15:17:01 | sean-k-mooney | cdent: ya that i why i check the scduel. m2 is still the first week of january so that means really the dealin is decemebr ish before people evaporate | |
| 15:17:47 | Sundar | Can somebody tell me how to derive a SQLalchemy model (table) from an OVO, or vice versa? | |
| 15:18:18 | dansmith | Sundar: they're not related | |
| 15:18:19 | jangutter | rule of thumb, the spec freeze is about two weeks before the first possible free time of the reviewers who can approve them. | |