| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-10-30 | |||
| 10:35:01 | openstackgerrit | wangqiang-bj proposed openstack/nova-specs master: fix wrong spelling of "unnecessary" https://review.openstack.org/614173 | |
| 10:35:05 | pvc | hi | |
| 10:35:06 | pvc | anyone successfully setup cyborg on openstack and attach an accelerator on an instance? | |
| 10:37:38 | openstackgerrit | wangqiang-bj proposed openstack/nova-specs master: fix wrong spelling of "unnecessary" https://review.openstack.org/614173 | |
| 10:39:35 | openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: Force refresh instance info_cache during heal https://review.openstack.org/591607 | |
| 10:39:36 | openstackgerrit | Maciej Jozefczyk proposed openstack/nova master: WIP: Add fill_virtual_interface_list online_data_migration script https://review.openstack.org/614167 | |
| 10:53:38 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add Unified Limits Spec https://review.openstack.org/602201 | |
| 11:02:53 | openstackgerrit | wanghui proposed openstack/nova-specs master: Fix a typo of 'keystoneauth1' https://review.openstack.org/614187 | |
| 11:10:07 | openstackgerrit | Gorka Eguileor proposed openstack/nova master: Fix unnecessary locking when connecting volumes https://review.openstack.org/614190 | |
| 11:12:24 | openstackgerrit | wangqiang-bj proposed openstack/nova master: fix wrong spelling of "explicit" https://review.openstack.org/614191 | |
| 13:16:42 | openstackgerrit | Merged openstack/nova master: Remove restart_scheduler_service() method https://review.openstack.org/613348 | |
| 13:33:08 | openstackgerrit | Gorka Eguileor proposed openstack/nova master: Fix unnecessary locking when connecting volumes https://review.openstack.org/614190 | |
| 13:34:04 | efried | bauzas: Did you see http://lists.openstack.org/pipermail/openstack-dev/2018-October/136123.html ? | |
| 13:42:07 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125 | |
| 13:42:32 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267 | |
| 13:42:32 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840 | |
| 13:42:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | |
| 13:42:33 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847 | |
| 13:42:34 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | |
| 13:42:34 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459 | |
| 14:06:12 | jaypipes | lbragstad: never got around to quota stuff yesterday, sorry. day ended up completely consumed by chasing down Chef bugs. | |
| 14:06:19 | jaypipes | lbragstad: will try again this afternoon. | |
| 14:09:52 | mriedem | gibi: you rebased while i was reviewing https://review.openstack.org/#/c/568840/24 | |
| 14:09:56 | mriedem | but you can remove those todos now | |
| 14:10:44 | efried | melwitt: Would you mind giving https://review.openstack.org/#/c/610174/ another pass today please? (<== edmondsw) | |
| 14:11:36 | efried | jaypipes: Since https://review.openstack.org/#/c/468560/ was +A before and the rebase was trivial (uuidsentinel moved to oslo), any chance of a fast approve? | |
| 14:12:11 | jaypipes | efried: done | |
| 14:12:15 | efried | jaypipes: Thanks! | |
| 14:21:34 | lbragstad | jaypipes sounds good | |
| 14:21:43 | lbragstad | i'll be around | |
| 14:32:04 | fghaas | Hi kashyap, assuming you're back and recovered from Edinburgh, could I bother you to take another peek at https://review.openstack.org/#/c/609788/ ? | |
| 14:32:19 | kashyap | fghaas: Hi, I'm also recovering from "other things", will look. :-) | |
| 14:32:34 | gibi | mriedem: thanks for the review. | |
| 14:32:47 | fghaas | Perfect. No rush, just wanted to make sure it didn't fall off the table. :) | |
| 14:33:39 | kashyap | fghaas: Ah, you've answered the questions. And have added live migration sections | |
| 14:35:04 | johnthetubaguy | jaypipes: are you planning work on placement quotas, just saw your tidy up patches | |
| 14:35:50 | johnthetubaguy | so I should have read the scrollback | |
| 14:36:10 | johnthetubaguy | I was planning on looking at this stuff ASAP: https://review.openstack.org/#/c/602201 | |
| 14:39:12 | johnthetubaguy | jaypipes: I see your thread now, quota classes I think come from some out of tree rackspace idea that never made it downstream. | |
| 14:40:01 | johnthetubaguy | jaypipes: the idea was a project was assigned a class, if not assigned you fallback to the default. i.e. you could upgrade someone to the I_HAVE_BIG_CREDIT_LIMIT quota class, etc. A bit like flavors. | |
| 14:40:25 | gaudenz | Hi | |
| 14:40:51 | mriedem | gibi: a few nits and questions in https://review.openstack.org/#/c/567267/ | |
| 14:41:37 | sean-k-mooney | johnthetubaguy: is that related to the idea of qoutas for flavors and generic placement resouce class quotas | |
| 14:41:55 | johnthetubaguy | sadly no, that's different | |
| 14:42:17 | johnthetubaguy | this is more a groups of limits, rather than limits on flavors | |
| 14:42:18 | sean-k-mooney | oh ok i like that usecase but never herard of quota classes until now | |
| 14:42:33 | johnthetubaguy | its half lurking in the code, and was never finished | |
| 14:42:39 | johnthetubaguy | best ignored | |
| 14:42:47 | openstackgerrit | Jack Ding proposed openstack/nova-specs master: High Precision Event Timer (HPET) on x86 guests https://review.openstack.org/607989 | |
| 14:43:00 | jaypipes | johnthetubaguy: or you could just have quota templates internally and assign a new project the values of that quota template... quota classes never needed to be in the API. | |
| 14:43:00 | sean-k-mooney | ok ill continue to do so | |
| 14:43:33 | johnthetubaguy | jaypipes: yeah, totally, that is the current plan for unified limits. its crazy pants stuff | |
| 14:43:40 | jaypipes | in a similar way that "registered quotas" in the new keystone limits API doesn't need to be exposed to users at all. | |
| 14:43:54 | jaypipes | johnthetubaguy: ++ | |
| 14:46:06 | johnthetubaguy | jaypipes: did you get chance to look at the unified limits spec I am looking at working on with alex_xu? | |
| 14:48:01 | mriedem | bauzas: do you remember this? https://review.openstack.org/#/c/582417/1/nova/conductor/tasks/migrate.py@a287 | |
| 14:48:31 | mriedem | because i'm having a hell of a time figuring out why i can't pass a RequestSpec object to the compute prep_resize RPC API but we can pass request spec to other things like select_destinations and build_and_run_instance | |
| 14:49:28 | jaypipes | johnthetubaguy: in my queue. | |
| 14:49:57 | johnthetubaguy | jaypipes: cool, just making sure to avoid duplicate work | |
| 14:55:30 | kashyap | fghaas: One latest remark in the change; regardless, I think it can be merged now. And can fine-tune in further patches. | |
| 14:59:43 | bauzas | mriedem: looking | |
| 15:00:01 | bauzas | currently on an internal priority bug hence my absence today | |
| 15:01:37 | bauzas | mriedem: oh I remember the tz issue, but I don't remember how I fixed it | |
| 15:01:54 | bauzas | mriedem: that's only a test issue, right? | |
| 15:03:12 | mriedem | it's not fixed | |
| 15:03:25 | mriedem | there was an oslo.messaging bug for it against the fake driver, but was invalidated | |
| 15:03:52 | mriedem | i'm just not sure why the prep_resize cast is a problem, but passing request spec to other rpc methods isn't a problem | |
| 15:06:12 | melwitt | efried: sure | |
| 15:06:19 | efried | thanks! | |
| 15:07:01 | bauzas | mriedem: I think I fixed this using the call_for_cast for my tests | |
| 15:07:06 | bauzas | IIRC | |
| 15:09:14 | mriedem | fghaas: a couple of questions in https://review.openstack.org/#/c/609788/2 | |
| 15:10:01 | wangy | Is there anyone can help me？I used rally to boot server with concurrency 50， | |
| 15:10:02 | wangy | I got error {u'message': u'No valid host was found. There are not enough hosts available.', u'code': 500,} My platform version is stable/pike, and nova-placement-api.log showed “DB error: (pymysql.err.InternalError) (1213, u'Deadlock found when trying to get lock;” | |
| 15:12:26 | mriedem | wangy: you might need https://github.com/openstack/nova/commit/c66ae65775bb9d885fac059847063fee70617bc5#diff-a040eef16b56cc7862fb2a334c757dea | |
| 15:12:38 | mriedem | since you didn't mention where the deadlock is occurring, it's hard to say | |
| 15:12:46 | mriedem | there was also https://github.com/openstack/nova/commit/e001b0538a9d413f864d39584c2aea0b9e200e4b#diff-a040eef16b56cc7862fb2a334c757dea but that was fixed in pike | |
| 15:13:27 | wangy | The db deadlock was caused by "INSERT INTO allocations " | |
| 15:14:28 | fghaas | mriedem, kashyap: all good points, thanks — will push update momentarily | |
| 15:15:01 | wangy | mriedem: thanks,I had added retry decorator, it did not help | |
| 15:15:52 | mriedem | wangy: seems that would be covered with https://github.com/openstack/nova/commit/e001b0538a9d413f864d39584c2aea0b9e200e4b#diff-a040eef16b56cc7862fb2a334c757dea | |
| 15:16:42 | mriedem | do you see "Performing DB retry for function" in the placement api logs? | |
| 15:16:50 | mriedem | if so, that means it's retrying on deadlock | |
| 15:17:03 | mriedem | if you see "DB exceeded retry limit" that means it hit the retry limit | |
| 15:17:24 | mriedem | and probably means you need more resources / more placement workers in your env | |
| 15:18:10 | wangy | mriedem: yes, I see the logs, thanks for your opinion, I will try it. | |
| 15:19:33 | openstackgerrit | Theodoros Tsioutsias proposed openstack/nova-specs master: Enable rebuild for instances in cell0 https://review.openstack.org/554218 | |
| 15:21:40 | Shilpa | mriedem: HI. | |
| 15:21:56 | Shilpa | mriedem:Can you please help me to understand in case of nova evacuate, why we are restoing allocations at source? | |
| 15:22:34 | Shilpa | is it for rollback? | |
| 15:23:04 | Shilpa | or in case of evacuate, in case of success, how allocaions at source are cleared off? Is any process set for it? | |
| 15:23:48 | mriedem | i'm not sure what you mean by "restoring allocations at source" | |
| 15:24:16 | mriedem | if the evacuated source nova-compute service starts up again and instances have been successfully migrated from it, the allocations are cleaned up for the source node during that startup | |
| 15:24:46 | mriedem | here https://github.com/openstack/nova/blob/d1e38f06bb5282dd9bf81d979926a0ebbfd93caa/nova/compute/manager.py#L712 | |
| 15:25:43 | mriedem | if the evacuation fails, the source node allocations should still exist in placement | |
| 15:26:19 | Shilpa | mriedem: yes in case of failure, observe source node allocations are still there in placement | |
| 15:26:38 | mriedem | if the evacuation fails, the destination node allocations, which are created by the scheduler, should be cleaned up here https://github.com/openstack/nova/blob/d1e38f06bb5282dd9bf81d979926a0ebbfd93caa/nova/compute/manager.py#L3044 or here https://github.com/openstack/nova/blob/d1e38f06bb5282dd9bf81d979926a0ebbfd93caa/nova/compute/manager.py#L3058 | |
| 15:26:55 | Shilpa | mriedem: will check for evacuated source nova-compute service starts up again, and what happens there | |
| 15:26:59 | gibi | mriedem: thanks for the review on https://review.openstack.org/#/c/567267/ I have to get back to these patches next week. Now I'm in a heavy preparation for our bandwidth demo for the summit | |
| 15:27:30 | openstackgerrit | sean mooney proposed openstack/nova master: harden placement init under wsgi https://review.openstack.org/610034 | |