| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-15 | |||
| 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. | |
| 11:46:50 | mdbooth | Yeah, just other priorities :( | |
| 11:47:09 | jaypipes | rpodolyaka: so, you finish the placement client plugin in OSC yet? :P | |
| 11:47:22 | jaypipes | cdent: :) | |
| 11:59:15 | mdbooth | lyarwood: Patch 1 done. I should hopefully pick up the pace a bit with growing context :) | |
| 12:07:56 | mnestratov | mdbooth: one more concern about ignoring images_type parameter. If we implement multibackend support, then it would be hard to report free disk space to scheduler. Is there a way to report free space for all supported backends? | |
| 12:13:26 | mdbooth | mnestratov: Free space needs to be reported per backend anyway. | |
| 12:14:55 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Fix formatting errors and warnings https://review.openstack.org/407364 | |
| 12:14:56 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: tox: Build docs with Python 2.7 https://review.openstack.org/445906 | |
| 12:14:56 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Treat doc build warnings as errors https://review.openstack.org/439883 | |
| 12:15:12 | sfinucan | johnthetubaguy: mriedem had. I've put them in a series now ^ | |
| 12:15:37 | sfinucan | bauzas: Easy +2s for your newfound spec powers above ^ :) | |
| 12:17:31 | johnthetubaguy | sfinucan: sweet, I want to see jenkins +2 all that first | |
| 12:17:49 | johnthetubaguy | sfinucan: did that last patch miss a link it your commit message? | |
| 12:18:07 | arslanq | rpodolyaka: I just tries this manually and i came to know that this happens only when i block migrate. "Live Migration failure: Operation not supported: qemu does not support drive-mirror command" | |
| 12:18:29 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: tox: Build docs with Python 2.7 https://review.openstack.org/445906 | |
| 12:18:52 | sfinucan | johnthetubaguy: Oops. Fixed | |
| 12:19:08 | arslanq | rpodolyaka: when i just use live migration it says that live migration is not possible without shared storage. | |
| 12:21:00 | mdbooth | I've stared at this long enough now that I assume I've just become blind to a typo or something. Could any who's got a second tell me what's causing the TypeError here: http://logs.openstack.org/42/445142/3/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/07e282a/logs/screen-n-api.txt.gz?level=ERROR | |
| 12:21:35 | mdbooth | Thank you #openstack-nova for helping me spot that typo | |
| 12:21:43 | mdbooth | You can ignore the above | |
| 12:24:17 | mdbooth | Also, mock-- for hiding an incorrect use of an interface | |
| 12:24:29 | johnthetubaguy | lyarwood: mdbooth: did you see the discussion over here: https://bugs.launchpad.net/nova/+bug/1672624 | |
| 12:24:29 | openstack | Launchpad bug 1672624 in OpenStack Compute (nova) "Ceph volumes attached to local deleted instance could not be correctly handled" [Undecided,New] | |
| 12:25:11 | mdbooth | johnthetubaguy: I didn't. Can look after lunch, though. | |
| 12:26:49 | openstackgerrit | Matthew Booth proposed openstack/nova master: Make server_external_events use an optimal number of db queries https://review.openstack.org/445142 | |
| 12:28:36 | mdbooth | johnthetubaguy: Do we want to have api punt 'local_delete' to conductor and have it be a bit more thorough? | |
| 12:28:43 | mnestratov | mdbooth: yeah I know, but IFAIK it's impossible now | |
| 12:29:58 | mdbooth | johnthetubaguy: That's very similar to the evacuate use case where you want to cleanup a volume attachment when the compute host isn't available. | |
| 12:30:21 | mdbooth | biab | |
| 12:38:35 | openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add extra-specs-image-meta-conflicts spec https://review.openstack.org/382382 | |
| 12:40:49 | lyarwood | johnthetubaguy: just back from lunch, that isn't a bug for me, more of an operator failure tbh. ./me adds a note to the bug | |
| 12:41:32 | johnthetubaguy | lyarwood: hmm, curious what you mean by that | |
| 12:42:23 | lyarwood | johnthetubaguy: while the compute service is down the original instance / domain is still active and connected to the backing rbd volume | |
| 12:42:37 | lyarwood | johnthetubaguy: so any attempt to remove the volume from the rbd pool is going to fail | |
| 12:42:49 | johnthetubaguy | lyarwood: ah, yes, it should do, good point | |
| 12:43:01 | johnthetubaguy | lyarwood: you have to wait for clean up on the compute node, which they haven't done | |
| 12:43:14 | lyarwood | johnthetubaguy: that or manually fence it | |
| 12:43:21 | lyarwood | johnthetubaguy: get it into a known state | |
| 12:45:29 | johnthetubaguy | lyarwood: yeah, we might need better tooling for that long term, post cinder API fix ups | |
| 12:45:47 | alex_xu | nova api meeting is 15 mins at openstack-meeting-4 | |
| 12:45:57 | johnthetubaguy | alex_xu: yeah, I should get me some lunch quickly! | |
| 12:46:08 | alex_xu | johnthetubaguy: :) | |
| 12:47:29 | alex_xu | johnthetubaguy: maybe we can move the meeting early 15mins, then you can get a better lunch | |
| 12:48:46 | johnthetubaguy | alex_xu: nah, that just means I am hungry during the meeting | |
| 12:49:04 | alex_xu | :) | |
| 12:50:59 | alex_xu | johnthetubaguy: sdague experiment for removing stevedore https://review.openstack.org/#/c/445864/ | |
| 12:55:47 | sdague | alex_xu: that looks really neat | |
| 12:57:29 | alex_xu | sdague: thanks, make me have more confidence | |
| 12:57:30 | sdague | alex_xu: and *so* much more clear | |
| 12:59:06 | alex_xu | sdague: just follow the placement api pattern, explicitly list the routes | |
| 13:00:16 | alex_xu | nova api meeting is running | |
| 13:12:09 | openstackgerrit | Mikhail Feoktistov proposed openstack/nova master: Add is_vz_container function https://review.openstack.org/445947 | |
| 13:26:22 | openstackgerrit | Rafael Folco proposed openstack/nova master: DELETE all inventory for a resource provider https://review.openstack.org/416669 | |
| 13:26:23 | openstackgerrit | Rafael Folco proposed openstack/nova master: [placement] Raising http codes on old microversion https://review.openstack.org/444497 | |
| 13:28:34 | rpodolyaka | jaypipes: heh, you'll be the first one to know, when the placement client plugin is done ;) | |
| 13:29:50 | dansmith | gibi: can you look at this? https://review.openstack.org/#/c/445697/ | |
| 13:30:13 | dansmith | gibi: I found another case where we were lazy-loading that from an instance on compute and so I think we probably want to go a little more aggressive on it | |
| 13:32:25 | jaypipes | rfolco: quick request on https://review.openstack.org/#/c/444497/. if you could handle that, I'll re-review quickly. | |
| 13:33:02 | jaypipes | rfolco: besides that, I think we should be able to merge your two patches in that series today. | |
| 13:33:22 | rfolco | jaypipes.requests --> priority -1 | |
| 13:33:27 | cdent | i concur, dr pipes | |
| 13:33:39 | jaypipes | rfolco: :) | |
| 13:34:22 | stvnoyes1 | i was writing a tempest test for swap disk (update volume attachment). The test runs ok on a running server, but I get a libvirt exception (Domain not found: no domain with matching uuid) when run against a stopped server. Is swap not supported on a stopped server? I didn't see anything in the doc about a restriction like that. thanks... | |
| 13:36:08 | mdbooth | stvnoyes1: It is not supported, no. | |
| 13:36:41 | jaypipes | stvnoyes1: when you say "you get a libvirt error", are you saying that that libvirt error is returned to you as a *user*, or that you see that error in the nova-compute log file? | |
| 13:36:56 | stvnoyes1 | in the log | |
| 13:37:14 | mdbooth | johnthetubaguy: ^^^ Incidentally, what stvnoyes1 said is why I was talking about the 'did_copy' response in the swap_volume callback to cinder | |
| 13:38:07 | johnthetubaguy | mdbooth: ack, makes sense, but we can't fix everything in one go | |
| 13:38:35 | mdbooth | johnthetubaguy: Yep, but if we could stick it in the api then we can fix it later more easily. | |
| 13:38:42 | mriedem | jaypipes: if you haven't seen yet, i had some questions in your ironic get_inventory patch | |
| 13:38:50 | mriedem | only thing i have left to review there is the RT functional test | |
| 13:38:57 | jaypipes | mriedem: yup, thanks much for your review and questions. answering now. | |
| 13:39:59 | mriedem | fwiw dansmith explained a bit of earlier conversations to me last night | |
| 13:40:30 | jaypipes | mriedem: dansmith is good like that. :) | |
| 13:43:34 | openstackgerrit | Dan Smith proposed openstack/nova master: Avoid lazy-loading projects during flavor notification https://review.openstack.org/445697 | |
| 13:48:14 | mriedem | johnthetubaguy: you were +2 on this before the cleanups https://review.openstack.org/#/c/429569/ | |
| 13:50:25 | mriedem | bauzas: unless you have a reason not to, i'm going to open a bug about the fact that we never delete request_specs when we delete an instance, | |
| 13:50:28 | mriedem | i'm not sure if that's by design | |
| 13:51:08 | mriedem | maybe i'll start in the ML | |
| 13:51:15 | mriedem | since we wouldn't want to do this for soft delete | |
| 13:53:41 | dansmith | mriedem: we could also do with a bug to clean up all the conversion back and forth to/from the legacy version that never got cleaned up | |
| 13:53:45 | stvnoyes1 | jaypipes: i was interested to see what would happen doing a swap to a stopped vm with the cli. the cli command returned without an error. The volume status of both vols were unchanged. And there was that Domain not found error in the logs. I found a similar non-error using the API, the test only failed because the new volume never became 'in-use'. | |
| 13:53:47 | bauzas | mriedem: I have a branch for that | |
| 13:54:14 | bauzas | mriedem: that's one of the things I have in my plate and never had time for that in Ocata | |
| 13:54:17 | mriedem | dansmith: yeah bauzas has a blueprint for that from ocata, | |
| 13:54:29 | mriedem | i don't remember seeing anything about actually deleting these things though | |
| 13:54:34 | bauzas | mriedem: hence me planning to fix it by Pike | |
| 13:54:41 | mriedem | i'll brain dump in the ML | |