| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-14 | |||
| 18:26:17 | jaypipes | :) | |
| 18:26:31 | jaypipes | dansmith: and no on the second (no rewriting table) | |
| 18:26:40 | dansmith | okay | |
| 18:27:43 | openstackgerrit | Pushkar Umaranikar proposed openstack/nova-specs master: Stop using Neutron admin token https://review.openstack.org/439890 | |
| 18:36:31 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | |
| 18:45:25 | openstackgerrit | Maxim Nestratov proposed openstack/nova-specs master: libvirt: reintroduce runtime image type detection https://review.openstack.org/363547 | |
| 18:48:08 | dansmith | jaypipes: so.. that doesn't seem to work.. when I'm reading the result of the sql create, I don't have a mapped field, and if I try to read it, sqla lazy-loads (and fails in this case) | |
| 18:50:49 | openstackgerrit | dane-fichter proposed openstack/nova master: Use cursive for signature verification https://review.openstack.org/351232 | |
| 18:50:52 | dansmith | jaypipes: also, the oslo.db model sync checker does't seem to support sever_default | |
| 18:51:09 | dansmith | jaypipes: http://pastebin.com/ckkJXUMi | |
| 18:52:51 | jaypipes | dansmith: guh. ok... lemme +2 that patch then. | |
| 18:53:09 | dansmith | jaypipes: I'm kina disappointed, because server_default is what I want | |
| 18:53:16 | jaypipes | ya | |
| 18:53:32 | jaypipes | dansmith: vote changed :) | |
| 18:54:20 | dansmith | that probably needs fixing in oslo.db | |
| 19:04:41 | dansmith | gibi: around by chance? | |
| 19:04:50 | openstackgerrit | sean mooney proposed openstack/nova master: remove flake8-import-order for test requirements https://review.openstack.org/445622 | |
| 19:06:49 | openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | |
| 19:06:50 | openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | |
| 19:08:05 | openstackgerrit | Dan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | |
| 19:09:26 | cdent | johnthetubaguy: I've started but not made much progress: https://review.openstack.org/#/c/409340/ (and one above it). I keep asking for volunteers from the reading audience of the placement update emails, but I think labor is oversubscribed | |
| 19:13:16 | melwitt | dansmith: just looked at the update. what I was missing was that you needed the entire cell objects in _get_computes_for_cells, and not just the uuids | |
| 19:13:37 | bauzas | johnthetubaguy: good points about https://review.openstack.org/#/c/440580/5 I missed them | |
| 19:13:37 | dansmith | okay | |
| 19:14:03 | melwitt | if you only needed the uuids, then it would have saved something but since you need the whole objects, it doesn't | |
| 19:19:00 | openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Add live-migration-force-after-timeout spec https://review.openstack.org/438467 | |
| 19:55:23 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in volumes.py https://review.openstack.org/445637 | |
| 19:55:56 | jroll | mriedem: all the tooz traffic on the ML reminds me to bug you about this again, you had a question there, I answered it, what say you? https://review.openstack.org/#/c/399022/ | |
| 20:11:12 | efried | mriedem Re: "...how the libvirt driver handles connection failures, it will auto-disable the compute service record in the database, and when it reconnects it will auto-enable the compute service record in the DB." - can you give me a pointer to some code to look at (and shamelessly mimic)? | |
| 20:18:44 | openstackgerrit | Juan Antonio Osorio Robles proposed openstack/nova master: Make versioned notifications topics configurable https://review.openstack.org/444947 | |
| 20:30:43 | openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | |
| 20:32:35 | openstackgerrit | Anusha Unnam proposed openstack/nova master: POC: add descriptions to server tags https://review.openstack.org/434842 | |
| 20:50:45 | mriedem | dansmith: i failed to mention the van in the yard covered in snow | |
| 20:51:01 | dansmith | mriedem: not that scary | |
| 20:51:04 | mriedem | dansmith: the government center can't do the thing i need, but gene fuckin' turner gets it done | |
| 20:51:10 | dansmith | did it have wheels? | |
| 20:51:15 | dansmith | hah | |
| 20:51:16 | mriedem | gray-faced old ass pug met me at the door | |
| 20:51:27 | mriedem | lots of cat food cans and cat hair but no cat in sight | |
| 20:51:33 | mriedem | the van had wheels | |
| 20:51:35 | mriedem | and rust | |
| 20:51:41 | mriedem | gene mothertruckin' turner | |
| 20:51:57 | dansmith | nice | |
| 20:52:01 | openstackgerrit | Anusha Unnam proposed openstack/nova master: Add descripiton to policies in virtual_interfaces.py https://review.openstack.org/445652 | |
| 20:52:07 | mriedem | it was like i saw into jay's future | |
| 20:52:18 | dansmith | haha | |
| 20:52:40 | mriedem | smelled like the mrs was in the kitchen cooking up that cat food | |
| 20:52:41 | dansmith | jay's present is jay's future | |
| 20:56:32 | mriedem | edleafe: this is the callback method in the libvirt driver https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L458 | |
| 20:56:41 | mriedem | called from the event listener in nova/virt/libvirt/host.py | |
| 20:57:08 | mriedem | then the magic happens here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L3396 | |
| 20:57:11 | antwash | Hey nova team, I have a review for you that's related to the move of policy in code and add more descriptive policies for deployers. https://review.openstack.org/#/c/439070/ | |
| 20:57:22 | antwash | ^ feedback is always welcomed :) | |
| 20:58:00 | edleafe | mriedem: thx - will dig into that in a bit | |
| 20:58:16 | antwash | ^ johnthetubaguy -- I think this patch is cleaned up nice now lol | |
| 21:02:13 | openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in aggregates.py https://review.openstack.org/445230 | |
| 21:05:18 | openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in admin_password.py https://review.openstack.org/443957 | |
| 21:05:21 | openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in admin_password.py https://review.openstack.org/443957 | |
| 21:07:32 | mriedem | jroll: i have spoken | |
| 21:07:52 | jroll | joy! | |
| 21:08:09 | jroll | mriedem: nice, thank you much | |
| 21:08:17 | mriedem | i meh'ed all over that thing | |
| 21:11:35 | jroll | same | |
| 21:48:02 | efried | edleafe I suspect mriedem's pointers to _set_host_enabled were for me ;-) | |
| 21:48:14 | efried | If not, I'm interested in talking to you... | |
| 21:49:18 | efried | mriedem Hypothetically, if I wanted to use that thing, could I move it out of libvirt and into someplace common? (Rather than duping it) | |
| 21:49:53 | efried | (Looks like vmware has a [simpler] version of same) | |
| 21:52:41 | mriedem | ha yes they were | |
| 21:52:43 | mriedem | sorry ed | |
| 21:53:08 | mriedem | efried: so vcenter has a way for active session on startup with the backing vcenter? | |
| 21:53:11 | mriedem | *wait | |
| 21:53:37 | efried | mriedem Well, they have a _set_host_enabled that does the same thing as libvirt's. Haven't yet drilled down into how it's used. | |
| 21:53:44 | mriedem | efried: i think we need something like that, otherwise the powervm driver init_host is blocking the nova-compute process from being 'started' for potentially what 10 minutes? | |
| 21:53:57 | efried | mriedem Well, yeah, but... | |
| 21:54:09 | mriedem | it does seem sort of weird to start the nova-compute service but automatically disable it, | |
| 21:54:17 | mriedem | as noted in the review, i'm not sure if the vcenter driver does anything similar | |
| 21:54:23 | mriedem | or how other people feel about this | |
| 21:54:25 | efried | mriedem That service *should* start in under a minute, unless the admin misconfigured the nvl (compute host VM) | |
| 21:54:43 | efried | And I might have other ways of figuring out if that service is trying to start, or dead in the water. | |
| 21:55:04 | ZZelle | Hi everyone | |
| 21:55:06 | efried | (ugly-ass native dbus calls equivalent to systemctl status polling) | |
| 21:55:17 | ZZelle | Who should i ask to review VMware related changes? | |
| 21:55:30 | ZZelle | (in nova of course :)) | |
| 21:56:03 | efried | mriedem So... if I could get that polling down to, say, two minutes, with a more reliable dead-in-the-water detection, could I get away with avoiding this subthread compute service disable/enable thing? | |
| 21:56:17 | mriedem | ZZelle: generally you should start with the vmware subteam, which i think right now is browne | |
| 21:56:52 | ZZelle | mriedem, only one person in VMware subteam? | |
| 21:56:56 | efried | mriedem ...which, as you say, seems a bit weird - and fairly involved to validate | |
| 21:57:01 | browne | ZZelle: sure, just put me on the patch to review | |
| 21:57:24 | ZZelle | browne, ok :) | |
| 21:57:41 | edleafe | efried: ah, ok. Thought it was related to another libvirt thing mriedem and I were discussing | |
| 21:57:57 | efried | ZZelle https://etherpad.openstack.org/p/pike-nova-priorities-tracking has a list of subteam members, FYI. | |
| 21:58:06 | mriedem | browne: since you're here, do you know if the vcenter driver has to wait at all to make a connection to the backing vcenter on startup? | |
| 21:58:12 | efried | ZZelle Not sure if your patch would be appropriate for inclusion in that subteam review list. | |
| 21:58:44 | openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: add some documentation around quotas https://review.openstack.org/417563 | |
| 21:59:34 | browne | mriedem: i believe there was a bug related to that. where startup of nova-compute was slow because it was connecting to vcenter on startup to retrieve all objects or some effect like that | |
| 21:59:40 | browne | let me do a search for it | |
| 21:59:42 | efried | mriedem https://github.com/openstack/nova/blob/edf51119fa59ff8a3337abb9107a06fa33d3c68f/nova/virt/vmwareapi/host.py#L49-L118 | |
| 21:59:45 | efried | and browne ^^ | |
| 22:00:13 | ZZelle | efried, i should add my changes in "Ready for subteam review:" part? | |
| 22:00:17 | efried | mriedem browne Looks like on both init and get_host_stats, it checks and updates the service state. | |