| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-03 | |||
| 17:24:28 | openstackgerrit | Matthew Booth proposed openstack/nova master: fdatasync() downloaded images before use https://review.openstack.org/441246 | |
| 17:25:28 | lyarwood | mdbooth: thanks for the review, would you mind if refactor is in a follow up patch to keep the backports cleaner? | |
| 17:25:51 | mdbooth | lyarwood: Patch looked good, so not at all | |
| 17:26:32 | lyarwood | mdbooth: cool thanks | |
| 17:30:58 | mdbooth | mriedem: I literally just grepped for 'GlanceImageService', and it's no longer present in the codebase anywhere, so unless we're doing some funky class name munging somewhere, I don't think anything can instantiate it. | |
| 17:46:47 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Check if domain is persistent before detaching volume https://review.openstack.org/441204 | |
| 17:55:09 | openstackgerrit | Andreas Jaeger proposed openstack/nova master: Use Sphinx 1.5 warning-is-error https://review.openstack.org/441257 | |
| 18:15:24 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: refresh connection_info during hard_reboot https://review.openstack.org/441269 | |
| 18:16:51 | openstackgerrit | Matthew Booth proposed openstack/nova master: Remove GlanceImageService https://review.openstack.org/441241 | |
| 18:16:52 | openstackgerrit | Matthew Booth proposed openstack/nova master: fdatasync() downloaded images before use https://review.openstack.org/441246 | |
| 18:45:29 | superdan | melwitt: boom. down to 28 fails: http://logs.openstack.org/94/436094/11/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/f2d1af0/logs/testr_results.html.gz | |
| 18:45:59 | superdan | nine of those are os-hypervisors related I think | |
| 18:46:19 | melwitt | nice | |
| 19:15:23 | mriedem | mdbooth_: $GLANCE_V1_ENABLED in devstack is what i was thinking of i think | |
| 19:29:07 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | |
| 19:29:07 | openstackgerrit | Dan Smith proposed openstack/nova master: Target cell in super conductor operations https://review.openstack.org/438022 | |
| 19:29:08 | openstackgerrit | Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023 | |
| 19:29:08 | openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | |
| 19:29:11 | openstackgerrit | Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024 | |
| 19:29:15 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | |
| 19:29:15 | openstackgerrit | Dan Smith proposed openstack/nova master: Teach simple_tenant_usage about cells https://review.openstack.org/441388 | |
| 19:39:54 | openstackgerrit | Danil Akhmetov proposed openstack/nova master: Use proper user and tenant in the owner section of libvirt.xml. https://review.openstack.org/399679 | |
| 19:40:40 | mriedem | cdent: jay wanted to do an os-traits release after https://review.openstack.org/#/q/project:openstack/os-traits+branch:master+topic:normalize was done right? | |
| 19:40:58 | cdent | mriedem: I believe so, yes | |
| 19:41:03 | mriedem | ok | |
| 19:41:29 | cdent | I'm not sure how much of the storage related stuff is done or needs to be done for that to happen | |
| 19:42:04 | mriedem | i'll get the release request teed up and jay can ack it | |
| 19:48:09 | artom | Maybe someone here knows - when running tox -e api-samples SomeTest, it doesn't generate the sameples in doc/ | |
| 19:48:36 | artom | I could run tox -e api-samples with no test specified, but that's like 20 minutes. What class/test do I need to add to SomeTest for it to generate the doc/ samples | |
| 19:49:33 | sdague | artom: I don't think it takes a dedicated class | |
| 19:50:23 | artom | sdague, weird. I've added my new .json.tpl file, but when I run the test no doc/ sample is generated | |
| 19:50:50 | artom | Or maybe... | |
| 19:51:15 | melwitt | in the past, there was an env var you had to set to make it generate samples | |
| 19:52:47 | artom | Bleah, I have to create the version directory manually | |
| 19:52:52 | artom | Apologies for the noise | |
| 20:08:11 | mriedem | melwitt: tox -e api-samples has that built in | |
| 20:08:32 | melwitt | yeah, after I said that, I checked and say that in tox.ini | |
| 20:08:37 | melwitt | *saw | |
| 20:09:06 | melwitt | I'm clearly not in the know anymore on api-samples | |
| 20:30:34 | openstackgerrit | Chris Friesen proposed openstack/nova master: Add comment to instance_destroy() https://review.openstack.org/441460 | |
| 20:30:50 | cfriesen_ | mriedem: ^ that comment for instance_destroy() we talked about yesterday | |
| 20:31:36 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | |
| 20:31:36 | openstackgerrit | Dan Smith proposed openstack/nova master: Target cell in super conductor operations https://review.openstack.org/438022 | |
| 20:31:37 | openstackgerrit | Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023 | |
| 20:31:38 | openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | |
| 20:31:41 | openstackgerrit | Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024 | |
| 20:31:44 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | |
| 20:31:45 | openstackgerrit | Dan Smith proposed openstack/nova master: Make consoleauth target the proper cell https://review.openstack.org/441461 | |
| 20:31:52 | cfriesen_ | artom: did you find that documented somewhere or did you have to figure it out? | |
| 20:32:16 | mriedem | cool | |
| 20:54:20 | superdan | melwitt: I think one of these fails is due to quota: http://logs.openstack.org/94/436094/11/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/f2d1af0/logs/testr_results.html.gz | |
| 20:54:29 | superdan | melwitt: down near the bottom, test_update_instance_port_admin_state | |
| 20:54:53 | superdan | runs in isolation, but not as part of the larger thing | |
| 20:55:22 | superdan | I'm wondering if we're reserving in cell0 and claiming in cell1 or something like that | |
| 20:56:20 | melwitt | superdan: ah, okay. yeah, I think it would work that way. the quota commit (done by compute) would probably end up in the compute cell's db, so that would end up not freeing the reservation at the top | |
| 20:56:30 | superdan | right | |
| 20:57:02 | superdan | is reservations where that gets made in the api? | |
| 20:57:14 | superdan | I thought it was something else | |
| 20:57:23 | superdan | but it seems that I don't know at all where these things go | |
| 20:57:27 | melwitt | what gets made? | |
| 20:57:53 | melwitt | the quota reserve happens in the api, which will create the reservation record | |
| 20:58:25 | superdan | what I mean is.. we create reservation in ... the reservations table? | |
| 20:58:34 | melwitt | oh, yes | |
| 20:58:37 | superdan | where does that get converted to a claim or whatever? | |
| 20:59:04 | melwitt | it will create a record in the reservations table and then also update records in the quota_usages table to mark resources as 'reserved' | |
| 20:59:28 | melwitt | then the quota commit will delete the reservation record from the reservations table and update the quota_usages records to unmark them as 'reserved' and mark them as 'in use' | |
| 20:59:29 | superdan | okay I have things in nova_cell0.reservations | |
| 20:59:54 | superdan | 21 in reservations and 15 in quota_usages, in cell0 interestingly | |
| 21:00:00 | melwitt | quota_usages records have a in_use field and reserved field which represent int counts | |
| 21:00:17 | superdan | ugh | |
| 21:00:25 | melwitt | the commit will subtract from reserve and increment in_use, if it's an alloc | |
| 21:00:33 | melwitt | yeah. | |
| 21:00:51 | superdan | I heard "monday, monday. then monday and monday, followed by monday" | |
| 21:04:19 | melwitt | lol | |
| 21:04:22 | melwitt | that's fair | |
| 21:05:31 | siva_krishnan | mriedem: Looked at your bug report https://bugs.launchpad.net/nova/+bug/1315201. I have been hitting this issue for one of my patches. Any suggestion/help on how to debug these tempest test failures? | |
| 21:05:31 | openstack | siva_krishnan: Error: Could not gather data from Launchpad for bug #1315201 (https://launchpad.net/bugs/1315201). The error has been logged | |
| 21:06:41 | mriedem | siva_krishnan: that's super old | |
| 21:07:11 | mriedem | siva_krishnan: see my last comment in there? need to debug what's going on in the service logs | |
| 21:09:14 | superdan | melwitt: actually, maybe it should be working because we hard target the context, which seems to be how the quota stuff works | |
| 21:10:30 | siva_krishnan | mriedem: I am getting NoValidHost: No valid host was found. There are not enough hosts available in n-cpu | |
| 21:10:48 | superdan | oh, but. | |
| 21:10:53 | superdan | we don't have an instance at that point | |
| 21:10:55 | mriedem | siva_krishnan: that could be caused by many different things | |
| 21:11:07 | mriedem | siva_krishnan: usually check n-sch, n-cond and/or n-cpu logs | |
| 21:11:20 | superdan | melwitt: so we kinda have to have your counting quotas thing before we can do this... | |
| 21:12:22 | melwitt | superdan: this is where the reserve happens https://github.com/openstack/nova/blob/master/nova/compute/api.py#L312 | |
| 21:12:27 | sneti | Hi alex_xu, can you please look at this when you have time: https://review.openstack.org/#/c/392305/. Current fix doesn't work with query param validation . Do you suggest any other way to fix this bug. | |
| 21:13:59 | siva_krishnan | mriedem: apart from the above error other thing which I could find in logs is this one "SSLError: SSL exception connecting to https://10.28.135.25/placement/resource_providers/dba7216f-e694-49ee-821b-098e692a64c2/allocations: ("bad handshake: SysCallError(104, 'ECONNRESET')",)" | |
| 21:14:02 | superdan | melwitt: which is before we know what cell | |
| 21:14:31 | mriedem | siva_krishnan: do you have a link to the patch or job that failed? | |
| 21:14:46 | siva_krishnan | mriedem: https://review.openstack.org/#/c/434870/3 | |
| 21:16:17 | melwitt | superdan: yeah. I think what I have up right now for quotas works, so it would be interesting to rebase on that and see what happens | |
| 21:16:39 | superdan | melwitt: like passing tempest? | |
| 21:16:57 | melwitt | the todos I have for it have to do with changing how to deal with quota count races and not counting flavor blob in instance_extra | |
| 21:16:58 | melwitt | yeah | |
| 21:17:18 | superdan | okay | |
| 21:17:26 | melwitt | well, I guess it has a merge conflict now. | |
| 21:17:32 | melwitt | sorry, I forgot that | |
| 21:17:52 | superdan | melwitt: so this? | |