| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-03 | |||
| 16:41:46 | mriedem | i think they are running libvirt 1.2.17 or 2.0.0 but not entirely sure since i couldn't find the libvirt version in the compute logs (i think that might be logged at debug, which they don't have enabled) | |
| 16:42:12 | lyarwood | yeah, we should really log that at INFO at the start, it's only one line | |
| 16:42:46 | kashyap | True, that's the first thing I sort of look, and waste 5 minutes trying it fish it out | |
| 16:42:54 | kashyap | s/look/look for/ | |
| 16:43:45 | kashyap | Oh, it's the 'mirror' element patch for block operation completion | |
| 16:44:14 | kashyap | mriedem: But in CI gate we should have those logs | |
| 16:44:38 | kashyap | Assuming you quoted from this job failure (gate-tempest-dsvm-cells-ubuntu-trusty) | |
| 16:45:01 | lyarwood | kashyap: the snippet isn't from CI | |
| 16:46:26 | kashyap | lyarwood: Then, from where is it? | |
| 16:46:47 | kashyap | mriedem: Can we get the exact 'block_resize' command that was sent to see what's causing the mess? | |
| 16:46:53 | lyarwood | kashyap: a downstream customer of mriedem, sorry if that wasn't clear | |
| 16:46:57 | kashyap | But I think you are also looking for the same | |
| 16:47:23 | mriedem | yeah sorry not a ci issue, | |
| 16:47:26 | kashyap | mriedem: Normally I have these log filters that I use for canned bugzilla responses -- https://kashyapc.fedorapeople.org/virt/openstack/request-nova-libvirt-qemu-debug-logs.txt | |
| 16:47:32 | mriedem | i'm just fishing for obvious 'oh yeah i know about that' | |
| 16:47:54 | mriedem | kashyap: thanks i can pass that on | |
| 16:48:21 | kashyap | mriedem: Also, tell them to not miss the NOTE at the end of that text file :-) | |
| 16:49:07 | openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova-specs master: Enable SR-IOV NIC offload feature discovery https://review.openstack.org/435954 | |
| 16:53:13 | mriedem | sileht: superdan: that devstack cells fix is merged now | |
| 16:53:25 | superdan | [08:21:31] <superdan>mriedem: merged | |
| 16:53:44 | mriedem | superdan: i thought you were talking about jay's os-traits patch | |
| 16:53:51 | superdan | ah, no :) | |
| 16:53:57 | superdan | I'm rebasing my stuff on top right now | |
| 16:53:57 | mriedem | https://review.openstack.org/#/c/440656/ | |
| 16:54:00 | mriedem | you should +W that anyway | |
| 16:55:33 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: fix default values reporting infra worker https://review.openstack.org/441227 | |
| 16:55:42 | siva_krishnan | johnthetubaguy: why do we allow to live migrate a paused instance ? Shouldn't it be restricted ? | |
| 16:56:20 | johnthetubaguy | siva_krishnan: it should just work, I would have thought? so thats fine to allow it | |
| 16:56:31 | sileht | mriedem, thanks a lot ! | |
| 16:57:30 | mriedem | johnthetubaguy: siva_krishnan: it does just work, at least with libvirt | |
| 16:57:37 | mriedem | we also test live migrating paused instances in the gate | |
| 16:57:45 | mriedem | that's been in nova for several years now | |
| 16:57:48 | siva_krishnan | johnthetubaguy: mriedem it doesn't work when post copy is enabled. | |
| 16:58:27 | johnthetubaguy | siva_krishnan: why? | |
| 17:00:21 | superdan | mriedem: whoa I have core there? | |
| 17:00:24 | johnthetubaguy | siva_krishnan: I am curious, how/when does it fail? | |
| 17:00:26 | siva_krishnan | johnthetubaguy: fails with this error http://paste.openstack.org/show/601362/ | |
| 17:00:55 | johnthetubaguy | siva_krishnan: hmm, interesting, work a bug report I think | |
| 17:01:28 | siva_krishnan | johnthetubaguy: sure. | |
| 17:01:41 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Make BlockDeviceMapping object support uuid https://review.openstack.org/242603 | |
| 17:01:45 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Add uuid column to BlockDeviceMapping https://review.openstack.org/242602 | |
| 17:02:55 | johnthetubaguy | siva_krishnan: probably should not request post_copy if vm is paused, that should fix it | |
| 17:03:55 | siva_krishnan | yup | |
| 17:05:27 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Check if domain is persistent before detaching volume https://review.openstack.org/441204 | |
| 17:09:28 | mdbooth | lyarwood: Swift work with the transient domain patch :) | |
| 17:10:00 | mdbooth | lyarwood: I wonder, though, if it's kinda weird we're getting the persistent state from guest, then passing it straight back. | |
| 17:10:41 | mdbooth | Intuitively I think guest.detach_device_with_retry() should just do the right thing? | |
| 17:10:42 | lyarwood | mdbooth: weird in the sense it might change in the meantime | |
| 17:10:52 | mdbooth | Nah, I think that's ok | |
| 17:11:18 | lyarwood | mdbooth: yeah I see your point, why allow the assumption | |
| 17:11:23 | mdbooth | I just wonder if the default behaviour of that persistent argument should be neither True nor False, but 'whatever will work' | |
| 17:11:50 | mdbooth | There may be a reason to override it somewhere, but if not we should probably also just remove the argument | |
| 17:12:39 | mdbooth | Anyway, patch looks good. That's a stylistic thing I guess. | |
| 17:12:40 | kashyap | mdbooth: Yeah, during the downstream bug triage, Lee already pointed the fix on IRC | |
| 17:12:45 | kashyap | So yeah, lyarwood++ | |
| 17:13:47 | mriedem1 | lyarwood: i guess i'm going to have to do that here too https://review.openstack.org/#/c/349014/ | |
| 17:14:57 | openstackgerrit | Danil Akhmetov proposed openstack/nova master: Use proper user and tenant in the owner section of libvirt.xml. https://review.openstack.org/399679 | |
| 17:15:28 | mdbooth | mriedem lyarwood: Yeah, I think a sane default could prevent a proliferation of unhandled edge cases | |
| 17:15:40 | lyarwood | mriedem: yeah I think so | |
| 17:16:10 | mriedem | i'm fine with that | |
| 17:16:15 | mriedem | then i don't have to duplicate it | |
| 17:17:31 | mriedem | johnthetubaguy: am i overthinking the need for a unique constraint across volume_id and attachment_id here? https://review.openstack.org/#/c/437597/4/nova/db/sqlalchemy/models.py | |
| 17:18:01 | mdbooth | lyarwood: +1 with a suggestion from me | |
| 17:19:45 | johnthetubaguy | mriedem: hmm, good question, in theory its unique globally in the table I guess? | |
| 17:19:47 | openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in admin_actions. https://review.openstack.org/440829 | |
| 17:19:57 | mriedem | the attachment id? | |
| 17:20:08 | mriedem | well, i wasn't sure if we could have multiple bdms with the same attachment id | |
| 17:20:24 | mriedem | the attachment is scoped to the volume and instance | |
| 17:20:54 | mriedem | and i don't think we'd have a case where we have like, 2 bdms with the same volume and attachment ids but difference device names | |
| 17:20:55 | mriedem | for example | |
| 17:21:07 | mriedem | that would mean the same attachment is mounted in two places which is bonkers | |
| 17:22:08 | openstackgerrit | Matthew Booth proposed openstack/nova master: Remove GlanceImageService https://review.openstack.org/441241 | |
| 17:22:54 | mriedem | heh that was on my todo list | |
| 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: Target cell in super conductor operations https://review.openstack.org/438022 | |
| 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: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: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: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: Teach simple_tenant_usage about cells https://review.openstack.org/441388 | |
| 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: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 | |