| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-16 | |||
| 15:05:33 | dims | thanks jaypipes ! | |
| 15:05:39 | jaypipes | any time! :) | |
| 15:11:10 | jaypipes | cdent: k, I have read your email about can_host. | |
| 15:11:25 | jaypipes | cdent: you want me to respond or should we discuss in hangout? | |
| 15:11:32 | jaypipes | cdent: or both? :) | |
| 15:11:39 | cdent | both would be awesome | |
| 15:11:42 | jaypipes | k. | |
| 15:11:52 | cdent | because in the hangout it will just be whoever is there | |
| 15:12:09 | cdent | although, if your answer is sufficiently informative in writing I'm not sure we need the hangout. that was mriedem's idea | |
| 15:12:36 | bauzas | cdent: jaypipes: just a question, should we need to wait for edleafe since he's on PTO ? | |
| 15:12:56 | mriedem | we're not going to wait no | |
| 15:13:01 | mriedem | jay and i are both out next week | |
| 15:14:15 | cdent | mriedem: can you (re-)share the hangout link | |
| 15:14:27 | bauzas | okay | |
| 15:14:39 | bauzas | I'm fine with that, just asking | |
| 15:14:45 | mriedem | alaski: happy birthday :) | |
| 15:15:02 | mriedem | MLK steals your thunder | |
| 15:15:22 | mriedem | cdent: https://hangouts.google.com/hangouts/_/calendar/bWF0dHJpZWQyMkBnbWFpbC5jb20.uve66g1e93k1ooj42jr29g1bug?authuser=0 | |
| 15:15:27 | cdent | thanks | |
| 15:16:05 | jaypipes | bauzas: no rest for the weary I'm afraid. | |
| 15:32:35 | openstackgerrit | sahid proposed openstack/nova: libvirt: rework _destroy method https://review.openstack.org/416995 | |
| 15:32:35 | openstackgerrit | sahid proposed openstack/nova: libvirt: make coherent logs when reboot success https://review.openstack.org/416994 | |
| 15:32:36 | openstackgerrit | sahid proposed openstack/nova: libvirt: rework hard reboot https://review.openstack.org/376350 | |
| 15:35:30 | yellow | ping #openstack-nova: Looking for the best way to clean a nova db. I see alot of error messages where the domain UUID is not found .. | |
| 15:35:52 | ildikov | mriedem: johnthetubaguy: hi. just a quick check, will you be able to make it to the Cinder-Nova meeting today? | |
| 15:36:06 | mriedem | probably | |
| 15:36:11 | yellow | On mitaka I see the addition of nova-manage db command but its not useful for my situation since the UUID is not null | |
| 15:36:46 | ildikov | mriedem: cool, tnx | |
| 15:52:35 | mriedem | jaypipes: this is what i was talking about https://github.com/openstack/nova/blob/7e0edc67108b3b9d23549a6d3c0d3106dfc2acec/nova/objects/resource_provider.py#L563 - but as noted on the call, it's not in allowed_filters (or the schema now) | |
| 15:53:34 | jaypipes | mriedem: ah. well, yeah, it could never have been added via the API or exposed to it. | |
| 15:54:03 | mriedem | jaypipes: you said you need to rebase this? https://review.openstack.org/#/c/407309/ | |
| 15:54:22 | jaypipes | mriedem: yeah. | |
| 15:55:19 | openstackgerrit | Pawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | |
| 15:55:21 | openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML https://review.openstack.org/286744 | |
| 15:55:21 | openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: pass libvirt reference methods to migration https://review.openstack.org/328997 | |
| 15:58:42 | bauzas | jaypipes: refrain your pencil, I'm litterally rebasing now | |
| 15:59:10 | bauzas | jaypipes: that will still make Jenkins unhappy, but at least you'd get a fresh copy | |
| 15:59:27 | cdent | I'm going to emboss "refrain your pencil" somewhere. Has a very nice ring to it. | |
| 16:04:22 | openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | |
| 16:04:22 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Add ComputeNode.get_by_uuid method https://review.openstack.org/418134 | |
| 16:04:34 | bauzas | jaypipes: mriedem: there it is | |
| 16:06:32 | mriedem | bauzas: ok issue in the bottom change | |
| 16:06:42 | bauzas | snap snap | |
| 16:07:20 | bauzas | mriedem: roger, this. | |
| 16:07:23 | bauzas | mriedem: on it | |
| 16:08:37 | mriedem | do we have a migration that added a uuid to every existing compute_nodes record? | |
| 16:08:55 | mriedem | cdent: in honor of you calling out my haggard looks i've brewed another cup of coffee | |
| 16:09:19 | cdent | I am your enabler? | |
| 16:09:46 | mriedem | yes | |
| 16:09:58 | cdent | Is that a paid position? | |
| 16:10:09 | mriedem | now when i get the caffeine shakes i know who to blame | |
| 16:10:25 | openstackgerrit | Andrey Volkov proposed openstack/nova: Add nova-manage cell_v2 create_cell command https://review.openstack.org/332713 | |
| 16:11:12 | openstackgerrit | Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | |
| 16:11:13 | openstackgerrit | Vasyl Saienko proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | |
| 16:11:40 | mriedem | ah migration 330 | |
| 16:11:42 | mriedem | cool | |
| 16:12:44 | bauzas | mriedem: oh oops, sorry missed your question | |
| 16:12:50 | bauzas | mriedem: yeah, it was done AFAIK | |
| 16:13:41 | openstackgerrit | Andrey Volkov proposed openstack/nova: Add nova-manage cell_v2 create_cell command https://review.openstack.org/332713 | |
| 16:15:35 | melwitt | mriedem: I had thought the map_cell0 name/connection was intentionally based on the api db because they're expected to be co-located together | |
| 16:19:55 | openstackgerrit | Sylvain Bauza proposed openstack/nova: Add ComputeNode.get_by_uuid method https://review.openstack.org/418134 | |
| 16:19:56 | openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | |
| 16:20:04 | gibi | mriedem: hi! I managed finish the notification lazy load field bugfix. I didn't manage to split it up nicely. If you have time could you please check it? https://review.openstack.org/#/c/415857/ | |
| 16:23:05 | openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers https://review.openstack.org/312488 | |
| 16:28:05 | openstackgerrit | Vasyl Saienko proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | |
| 16:28:05 | openstackgerrit | Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | |
| 16:29:53 | mriedem | melwitt: i'm not sure, we'd probably have to ask alaski if he remembers a specific reason for calling it nova_api_cell0 even though it's using the main db schema | |
| 16:31:14 | melwitt | that was my understanding anyway | |
| 16:31:58 | openstackgerrit | Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | |
| 16:31:59 | openstackgerrit | Vasyl Saienko proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | |
| 16:36:26 | mriedem | idk | |
| 16:36:26 | mriedem | melwitt: still, co-locating the nova_api db and cell0 doesn't mean we should name it nova_api_cell0 right? unless it just meant something like, cell0 is only used at the api/controller level | |
| 16:36:30 | mriedem | it's probably confusing either way | |
| 16:37:10 | mriedem | we should probably figure out what we want though before all of dansmith's changes for it go in | |
| 16:39:43 | melwitt | mriedem: yeah, I could see the name being confusing. I'm trying to think if there's any caveat to making it default to the main db connection. I thought the idea was it's an api cell construct, so default to the connection of the other api cell db | |
| 16:41:01 | mriedem | melwitt: you want to comment on https://review.openstack.org/#/c/420439/ ? | |
| 16:41:07 | mriedem | it's held up by dependencies right now | |
| 16:41:16 | melwitt | okay | |
| 16:41:37 | mriedem | it would be nice if we didn't have to make the change because it's also in devstack and grenade, and potentially backports | |
| 16:42:19 | mriedem | but this comment in the code was wrong: "The cell0 database will use the same database scheme and netloc as the API database" | |
| 16:42:27 | mriedem | which leads me to believe it was just a mistake from the beginning | |
| 16:51:51 | melwitt | mriedem: I was thinking about that and I think it meant scheme as in, url format. not database schema | |
| 16:52:05 | melwitt | I left a comment on the review, fwiw | |
| 16:52:08 | mriedem | oh maybe | |
| 16:53:32 | mdbooth | http://eventlet.net/doc/modules/greenthread.html#eventlet.greenthread.spawn | |
| 16:53:50 | mdbooth | Is there any way to deterministically get a GreenThread object before it starts running? | |
| 16:54:10 | mdbooth | I want to do: | |
| 16:54:19 | mdbooth | x = GreenThread() | |
| 16:54:34 | mdbooth | ...register thread somewhere... | |
| 16:54:39 | mdbooth | x.run() | |
| 16:55:14 | mriedem | mnaser: i'm assumed at your '99% of this volume multiattach stuff is done on the cinder side' comment in the ops list :) | |
| 16:55:30 | mnaser | mriedem hi | |
| 16:55:41 | mnaser | yeah, i think cinder side has multiattachments already setup | |
| 16:55:50 | mnaser | APIs have all that stuff already wired up | |
| 16:56:06 | mnaser | i think the part that never merged was the nova side that made the appropriate api calls and interacted with it | |
| 16:56:23 | mriedem | mnaser: yes i'm fully aware of the history | |
| 16:56:31 | mriedem | but it's not that simple | |
| 16:56:54 | mnaser | mriedem: of course it's not :) it's probably why it never got finished up, i'm very understanding of that | |
| 16:57:21 | mnaser | sorry if that came off as "nova team, finish your job" .. i understand there's complexities involved but i meant it more of "half of the battle is done" | |
| 16:57:51 | mriedem | yeah i know, i've already replied to that thread with details | |