| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-14 | |||
| 17:56:40 | dansmith | and I've already changed it | |
| 17:57:09 | dansmith | there's not a double iteration if I put the conditional in the caller, I just thought the point was to avoid the conditional, | |
| 17:57:21 | dansmith | and if not, then I don't get the point of moving | |
| 17:57:37 | dansmith | but it's done anyway | |
| 17:58:57 | melwitt | hm, okay. I thought the point was to avoid the skip all but one in the specific cell case. but I'm probably missing something else | |
| 17:59:52 | johnthetubaguy | sfinucan: did you have some pbr fix for this: https://review.openstack.org/#/c/407364 | |
| 18:00:39 | openstack | Launchpad bug 1672624 in OpenStack Compute (nova) "Ceph volumes attached to local deleted instance could not be correctly handled" [Undecided,New] | |
| 18:00:39 | melwitt | mriedem: it looks like https://bugs.launchpad.net/nova/+bug/1672624 is another regression. I'm working on a local repro | |
| 18:03:39 | dansmith | melwitt: should we halt this and put it in the issues like the other? https://review.openstack.org/#/c/445575/ | |
| 18:05:29 | melwitt | dansmith: like the other "fix coming soon" one? argh, maybe. I'm not 100% of when this regressed and I haven't repro-d it yet. from the code, it doesn't look like it should be happening | |
| 18:05:43 | dansmith | yeah, that | |
| 18:06:45 | melwitt | I guess I hesitate about mentioning it in there when I haven't repro'd it yet | |
| 18:07:24 | melwitt | I just tried it on an older devstack ceph I have and the local delete worked fine. working on a try with master | |
| 18:20:16 | johnthetubaguy | cdent: jaypipes: I guess we don't have an api-ref site for placement yet? | |
| 18:20:37 | jaypipes | johnthetubaguy: not yet, no. one is needed for sure. | |
| 18:24:00 | johnthetubaguy | mriedem: would be nice to get this policy spec moving, folks seem generally happy with the approach now: https://review.openstack.org/#/c/433010/ | |
| 18:24:41 | dansmith | jaypipes: if I make that server_default=0 thing can I assume that just reading existing records will give me zero? | |
| 18:24:52 | dansmith | jaypipes: and, does that require writing the table again, or is it smart? | |
| 18:25:20 | jaypipes | dansmith: adding server_default="0" will allow you to make the field non-nullable from the start. | |
| 18:25:56 | dansmith | jaypipes: I assume that means "yes" to the first part | |
| 18:26:15 | jaypipes | dansmith: yes | |
| 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. | |