| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-15 | |||
| 08:14:24 | alex_xu | jichen: i think no... | |
| 08:14:29 | jichen | alex_xu: yeah, I knew this discussion | |
| 08:14:41 | jichen | ok, I will give another thought and see how to proceed this | |
| 08:14:44 | alex_xu | johnthetubaguy: ^ I think we needn't waiting for cap API, right? | |
| 08:15:19 | alex_xu | jichen: thanks, if johnthetubaguy also ensure this needn't waiting for cap API, then ping me when the patch ready | |
| 08:15:32 | jichen | ok | |
| 08:15:42 | jichen | another question | |
| 08:15:54 | alex_xu | jichen: yea, please | |
| 08:16:41 | jichen | https://review.openstack.org/307675, it's abandoned but this seems discussed before and a potential item? any discussion on PTG about it ? | |
| 08:17:15 | jichen | Change subject: Make API errors conform to API-WG schema | |
| 08:17:16 | alex_xu | jichen: there is no discussion about this in the PTG | |
| 08:17:40 | jichen | ok... I went through the ethpad and didn't find anything.... | |
| 08:17:58 | alex_xu | jichen: yea, I didn't remember we bring that idea into the api meeting before PTG also | |
| 08:18:05 | jichen | so it might be restore back later after P , Q ... :) ? | |
| 08:18:20 | alex_xu | jichen: yea, first ensure someone commit to work on it | |
| 08:18:28 | jichen | I can't remember clearly but seems I saw it's discussed somewhere | |
| 08:18:43 | alex_xu | jichen: there is proposal add that into placement API | |
| 08:18:54 | jichen | ok, can you point me the link? | |
| 08:19:02 | alex_xu | searching the link | |
| 08:19:08 | jichen | ok, thanks~ | |
| 08:19:25 | alex_xu | jichen: https://review.openstack.org/#/c/418393/ | |
| 08:19:36 | alex_xu | maybe co-work with that spec, and migrate the solution into the nova api | |
| 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: Make guest.BlockDevice only use the target dev of a disk https://review.openstack.org/402163 | |
| 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: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: Delete nic metadata when detaching interface https://review.openstack.org/435387 | |
| 11:04:05 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | |
| 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 | mdbooth | Just not with static config. | |
| 11:28:14 | mnestratov | and I repeat what I just aksed in review - how do we change current default behavir? | |
| 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 | mnestratov | it doesn't solve my problems | |
| 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: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 | 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:04 | rpodolyaka | arslanq: hmm, this sounds similar to https://bugs.launchpad.net/nova/+bug/1441054 | |
| 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) | |