| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-15 | |||
| 08:19:44 | jichen | alex_xu: great, thanks a lot~ | |
| 08:19:51 | jichen | I will take a look :) | |
| 08:19:57 | alex_xu | jichen: np :) | |
| 09:00:04 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: utils.get_disk return the target dev of the root device https://review.openstack.org/402716 | |
| 09:00:04 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Make guest.BlockDevice only use the target dev of a disk https://review.openstack.org/402163 | |
| 09:39:20 | openstackgerrit | jichenjc proposed openstack/nova master: WIP: allow owner to retrieve some data by default https://review.openstack.org/445844 | |
| 09:41:33 | openstackgerrit | jichenjc proposed openstack/nova master: WIP: allow owner to retrieve some data by default https://review.openstack.org/445844 | |
| 09:44:53 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add InstanceList.get_by_instance_mapping_list() https://review.openstack.org/445141 | |
| 09:51:23 | mdbooth | lyarwood: Morning. Can you point me at your series again? | |
| 09:53:37 | lyarwood | mdbooth: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bug/1489581 & https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/cinder-new-attach-apis | |
| 09:54:15 | lyarwood | mdbooth: the first two are just nice to have, we've been trying to get a uuid added for bdms for ages | |
| 09:54:52 | mdbooth | lyarwood: K. I'll ask a series of dumb questions throughout the morning in all likelihood | |
| 09:55:06 | openstackgerrit | Matthew Booth proposed openstack/nova master: Make server_external_events use an optimal number of db queries https://review.openstack.org/445142 | |
| 09:55:54 | lyarwood | mdbooth: np and thanks for looking at this | |
| 10:05:25 | openstackgerrit | Sergey Nikitin proposed openstack/nova-specs master: TinyFix: Changed type of CPU ID from 'String' to 'Integer' in diagnostics spec https://review.openstack.org/445855 | |
| 10:19:11 | ildikov | lyarwood: hi | |
| 10:19:45 | ildikov | lyarwood: are the BDM patches ready for review? | |
| 10:20:10 | lyarwood | ildikov: yes, mdbooth is going over them now | |
| 10:20:31 | ildikov | lyarwood: I saw on the priority etherpad that they are still listed under subteam review, but if you fixed Matt's comments, etc then I'll move them to ready | |
| 10:20:48 | ildikov | lyarwood: great, I'll move them on the etherpad too | |
| 10:20:56 | ildikov | lyarwood: tnx | |
| 10:21:42 | mdbooth | lyarwood: Do you happen to know why we don't refer to bdms by the primary key (id)? | |
| 10:22:04 | mdbooth | We don't do that anywhere, so this isn't an outlier | |
| 10:22:10 | mdbooth | Just wondering if you know why | |
| 10:23:04 | lyarwood | I wasn't aware that we ever did that so no, if we did I don't know why we stopped | |
| 10:24:31 | mdbooth | I suppose the db's primary key isn't universally unique... | |
| 10:24:52 | lyarwood | yeah, as we are moving towards multiple dbs etc I guess we needed uuids | |
| 10:27:32 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Deprecate the Cinder API v2 support https://review.openstack.org/444253 | |
| 10:27:47 | openstackgerrit | jichenjc proposed openstack/nova master: remove keypair server extension https://review.openstack.org/396611 | |
| 10:27:57 | openstackgerrit | Alex Xu proposed openstack/nova master: Another poc for removing stevedore for nova api https://review.openstack.org/445864 | |
| 10:29:35 | arslanq | hi guys, i am confused about live migration of VMs. What is the requirement of live migration does it require shared storage ? | |
| 10:32:08 | rpodolyaka | arslanq: it does not. libvirt can 'block migrate' the storage for you | |
| 10:32:22 | rpodolyaka | arslanq: that being said, shared storage is highly recommended | |
| 10:33:16 | MasterofJOKers | https://docs.openstack.org/admin-guide/compute-configuring-migrations.html#block-migration | |
| 10:37:58 | arslanq | rpodolyaka: Thanks, i am facing some errors on block migration. i'll get back to you | |
| 10:50:43 | openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225 | |
| 11:04:05 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | |
| 11:04:05 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | |
| 11:04:06 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | |
| 11:04:07 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | |
| 11:04:11 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | |
| 11:04:12 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | |
| 11:04:13 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | |
| 11:09:53 | asettle | Morning bauzas and johnthetubaguy - Brian seems to be having issues with his CentOS install (which, he htinks is environment related). Is there anyone in here that could test on CentOS or RHEL for us? | |
| 11:09:58 | asettle | We're 2/3rds there | |
| 11:17:11 | openstackgerrit | Maxim Nestratov proposed openstack/nova-specs master: libvirt: reintroduce runtime image type detection https://review.openstack.org/363547 | |
| 11:26:51 | mnestratov | mdbooth: hi | |
| 11:27:01 | mnestratov | regarding ^^ | |
| 11:27:22 | mdbooth | mnestratov: Yeah, I don't think I'm ever going to like that :( | |
| 11:28:09 | mdbooth | I'm very much in favour of multi-backend support, though. | |
| 11:28:14 | mnestratov | and I repeat what I just aksed in review - how do we change current default behavir? | |
| 11:28:14 | mdbooth | Just not with static config. | |
| 11:28:54 | mnestratov | *behavior | |
| 11:29:28 | mdbooth | 2 things: associate the backend in use somehow with the bdm, so it lives with the instance not with the host config | |
| 11:29:49 | mnestratov | I'm fine with it | |
| 11:30:10 | mdbooth | Second: configure a *default* somehow, or pass it in the api in a driver-agnostic way | |
| 11:30:32 | mdbooth | Advantage of doing the former is that you can change your mind about the latter | |
| 11:30:38 | mnestratov | are we going to ignore images_type config parameter? | |
| 11:30:51 | mdbooth | I would love to be able to do that, yes | |
| 11:30:57 | mdbooth | It's a bug | |
| 11:31:02 | mnestratov | me too | |
| 11:31:16 | mnestratov | are we going to do this? | |
| 11:31:58 | mdbooth | Given infinite time I will hopefully get to it eventually :) | |
| 11:32:18 | mdbooth | If you can get it done now I'll definitely support you in that. | |
| 11:32:34 | arslanq | rpodolyaka: i am running block migration and hitting with this issue: Error LiveMigrateException: Live Migration failed: Migration complete but instance did not change host: compute7 | |
| 11:32:34 | mnestratov | it doesn't solve my problems | |
| 11:33:15 | mnestratov | but ok I'll try to figure out how to do this via DBM | |
| 11:33:42 | rpodolyaka | arslanq: are there any additional errors on both source and destination compute nodes? nova-compute / libvirt logs | |
| 11:34:09 | mnestratov | I mean BDM :) | |
| 11:35:21 | arslanq | rpodolyaka: i found this error log in nova logs: Live Migration failure: Operation not supported: qemu does not support drive-mirror command | |
| 11:36:02 | johnthetubaguy | asettle: do you have more details on the issue he is hitting at all? | |
| 11:37:51 | asettle | johnthetubaguy: From Brian this morning "And all the steps work correctly on CentOS, I just have communication problems between the compute and the controller nodes. At this point I’m thinking it is a problem with CentOS packages or perhaps my environment." | |
| 11:37:58 | asettle | I don't have much more than that, unfortunately (as in, no error logs) | |
| 11:38:06 | mdbooth | mnestratov: Sorry, got disconnected. Scrolling back. | |
| 11:39:04 | rpodolyaka | arslanq: hmm, this sounds similar to https://bugs.launchpad.net/nova/+bug/1441054 | |
| 11:39:04 | openstack | Launchpad bug 1441054 in OpenStack Compute (nova) "live-migration --block-migrate fails with default libvirt flags" [Medium,Invalid] - Assigned to Mathieu Rohon (mathieu-rohon) | |
| 11:39:35 | rpodolyaka | arslanq: I remember there were some issues with block migration on some versions of libvirtd | |
| 11:40:01 | rpodolyaka | arslanq: specifically, we could not block migrate instances with block devices attached (e.g. config drives) | |
| 11:40:14 | mdbooth | mnestratov: Moving this into the BDM makes it unambiguous. It opens up the possibility of safely having a heterogenous storage config. | |
| 11:40:17 | rpodolyaka | arslanq: but, unfortunately, I do not remember details :( | |
| 11:40:20 | arslanq | rpodolyaka: Note: i don't have shared storage..... | |
| 11:40:36 | mdbooth | It means operators can change storage config after deployment without breaking all their existing instances. | |
| 11:40:38 | rpodolyaka | arslanq: anyway, this must have been fixed in the latest versions of libvirt/qemu | |
| 11:41:12 | mdbooth | It means migrating between hosts with different storage becomes unambiguous. | |
| 11:41:44 | arslanq | rpodolyaka: right now i am on openstack newton branch. Can i get some proper document of this issue? It will be very helpful for me. | |
| 11:42:12 | mdbooth | mnestratov: There are a bunch of advantages beyond just your use case. | |
| 11:42:31 | mdbooth | Whilst also fixing some edge cases where we currently have to skirt around security issues. | |
| 11:43:17 | mnestratov | mdbooth: what would you say if we will not introduce a new config parameter but instead just change the behavior. Like if images_type is none or default then image backend is chosen according to image disk_format metadata and stored in instance' BDM | |
| 11:44:37 | mdbooth | mnestratov: The behaviour should be that if it's present in instance BDM, that should always be used. We should only consult a config/whatever if it's not present, and then we should use that to set it in the bdm. | |
| 11:44:53 | jaypipes | morning folks | |
| 11:45:18 | mdbooth | mnestratov: That gets around the issue of changing config/different config between hosts. | |
| 11:45:27 | mdbooth | jaypipes: morning | |
| 11:45:27 | jaypipes | alex_xu, melwitt, dansmith, Vek: review bait: https://review.openstack.org/399679 | |
| 11:45:55 | mnestratov | mdbooth: it demands changing the way you boot your instances | |
| 11:46:12 | jaypipes | mdbooth: hey, morning. so how's it going with the libvirt storage pools patch series? haven't looked at it in a month or so. Need to get back on that. | |
| 11:46:15 | arslanq | rpodolyaka: i have the following libvirt version: libvirtd (libvirt) 2.0.0 | |
| 11:46:23 | mdbooth | jaypipes: I haven't done anything with it in that time, tbh. | |
| 11:46:40 | jaypipes | mdbooth: you planning to? or other priorities getting in the way? | |
| 11:46:44 | mdbooth | jaypipes: I need to make a new start on it. It's still on my list, though. | |
| 11:46:47 | cdent | I'm not coping well with jaypipes being 4 hours off instead of the usual 5. It's making me all confused and discombobulated. | |