| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-06-21 | |||
| 16:35:08 | mriedem | kfox1111: ^ is what you want | |
| 16:35:18 | mriedem | or as close as virtfs has gotten to nova land | |
| 16:35:40 | kfox1111 | yeah something like that. | |
| 16:35:43 | kfox1111 | thanks for the link. | |
| 16:39:25 | sfinucan | efried: Yeah, saw that. Wasn't sure if/how I could fix it | |
| 16:39:42 | sfinucan | Maybe your patch addresses it - I haven't checked yet | |
| 16:40:06 | efried | sfinucan Yes, mine does address it, but manually by setting box heights. The other option is as discussed above, adding an autosize facility. | |
| 16:40:14 | efried | ...to your monkeypatchage. | |
| 16:43:48 | sfinucan | efried: Gotcha. Maybe we could set the height attribute for that last box while I work on auto-sizing? | |
| 16:44:04 | sfinucan | I'll let you do that, if so. It's home time for me | |
| 16:44:46 | efried | sfinucan That would work. Probably not urgent. Have a good evening, hit me up tomorrow if you like. | |
| 16:45:00 | sfinucan | (y) | |
| 16:51:27 | jaypipes | w00t. only 168 unit test failures to go... | |
| 17:16:31 | openstackgerrit | Chris Friesen proposed openstack/python-novaclient master: 2.47: Show flavor info in server details https://review.openstack.org/435141 | |
| 17:16:56 | cfriesen_ | mriedem: this ^ has the --minimal changes | |
| 17:20:04 | openstackgerrit | Chris Friesen proposed openstack/python-novaclient master: 2.47: Show flavor info in server details https://review.openstack.org/435141 | |
| 17:20:04 | ildikov | mriedem: did you have a chance to check the live_migrate patch: https://review.openstack.org/#/c/463987/ ? | |
| 17:21:24 | ildikov | mriedem: I also saw you mentioned China yesterday, when will you be there exactly? | |
| 17:40:44 | smcginnis | ildikov: Friday through next Saturday. | |
| 17:41:11 | ildikov | smcginnis: tnx :) | |
| 17:44:55 | dansmith | mriedem: stalker alert | |
| 17:49:51 | openstackgerrit | Rafael Folco proposed openstack/nova master: Delete all inventory has its own method DELETE https://review.openstack.org/460147 | |
| 17:54:20 | jaypipes | guh, the scheduler unit tests just make me want to shoot myself. | |
| 17:56:53 | melwitt | :( | |
| 17:57:33 | mriedem | ildikov: it's on the list - my only tab open right now | |
| 17:57:35 | melwitt | the gate seems really angry right now. getting weird novnc tempest failures and "no route to host" | |
| 17:58:28 | ildikov | mriedem: I'm guessing that's a good sign, plz don't close it until you reviewed :) | |
| 17:59:12 | ildikov | mriedem: added a few tests to the swap patch | |
| 18:00:00 | ildikov | mriedem: should I/we involve more people from the Nova team to ensure we're progressing fast enough? | |
| 18:00:53 | mriedem | ildikov: more nova people don't exist | |
| 18:01:21 | ildikov | mriedem: hmm, then what now? | |
| 18:01:26 | ildikov | mriedem: should we clone you? | |
| 18:01:31 | mriedem | i sugggest a rebranding in the governance repo to recruit new nova people | |
| 18:03:08 | ildikov | that'll need some coaching and mentoring | |
| 18:04:04 | ildikov | I'm constantly trying to recruit, will need to do a better job... | |
| 18:09:18 | mriedem | ildikov: i was just joking | |
| 18:09:46 | mriedem | melwitt: jaypipes: dansmith: $.05 to whoever knows the answer to: what is the type of the object returned from driver.block_device_info_get_mapping() | |
| 18:09:57 | ildikov | mriedem: well, every joke has some truth in it :) | |
| 18:10:16 | melwitt | mriedem: I'm gonna guess a dict | |
| 18:10:17 | jaypipes | mriedem: no clue. | |
| 18:10:29 | mriedem | melwitt: bzzt | |
| 18:10:40 | ildikov | mriedem: and I would really love to have the new attach stuff landed in Pike :) | |
| 18:10:42 | melwitt | dangit. | |
| 18:10:46 | mriedem | hint: it's some type of list | |
| 18:11:02 | mriedem | don't let the singular in the name fool you | |
| 18:11:11 | melwitt | BlockDeviceMappingList | |
| 18:11:21 | melwitt | hah | |
| 18:11:53 | mriedem | is that your final answer? | |
| 18:12:14 | melwitt | sure | |
| 18:12:21 | mriedem | WRONG | |
| 18:12:25 | mriedem | BZZT | |
| 18:12:31 | melwitt | gah | |
| 18:12:44 | mriedem | it's a list of nova.virt.block_device.DriverBlockDevice objects | |
| 18:12:49 | mriedem | s/objects/subclasses/ | |
| 18:12:52 | mriedem | which are wrappers of, | |
| 18:12:54 | mriedem | you guessed it, | |
| 18:13:02 | mriedem | nova.objects.BlockDeviceMapping | |
| 18:13:05 | melwitt | lol | |
| 18:13:16 | melwitt | um, yeah. would never have guessed that | |
| 18:13:29 | mriedem | mdbooth would | |
| 18:13:31 | mriedem | he dreams about it | |
| 18:13:47 | mriedem | that and haggas | |
| 18:13:55 | mriedem | *haggis | |
| 18:13:57 | melwitt | lol! | |
| 18:28:37 | mriedem | we don't have a ListOfUUIDsField in o.vo?! | |
| 18:36:50 | jaypipes | mriedem: nope. | |
| 18:50:09 | openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971 | |
| 19:02:58 | mriedem | ildikov: ok here you go https://review.openstack.org/#/c/463987/ | |
| 19:03:04 | mriedem | stvnoyes already skedaddled for the day | |
| 19:03:17 | mriedem | i have some bad news | |
| 19:03:49 | mriedem | ^ is only implemented for the libvirt driver, but needs to also be handled for xen and hyperv | |
| 19:04:14 | ildikov | mriedem: thanks for the review | |
| 19:04:30 | ildikov | mriedem: can the other hypervisor be covered in follow-up patches? | |
| 19:05:42 | ildikov | mriedem: also if you feel like adding a zillion comments to the swap patch that's very welcomed as well ^^ | |
| 19:05:56 | ildikov | mriedem: just to keep me busy while you're partying in China :) | |
| 19:09:41 | mriedem | I suppose the other virt drivers, or all virt drivers, can be done in a series of changes since none of this does anything with the new flow until we start attaching with the new style at the very end | |
| 19:10:20 | ildikov | ok, that was my thinking too | |
| 19:28:39 | openstackgerrit | Merged openstack/nova master: hypervisor_hostname must match get_available_nodes https://review.openstack.org/475926 | |
| 19:34:08 | openstackgerrit | Merged openstack/nova master: Provide a hint when _verify_response fails https://review.openstack.org/475512 | |
| 19:55:31 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix redundant BDM lookups during rebuild https://review.openstack.org/476258 | |
| 19:59:29 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Explain why API services are filtered out of GET /os-services https://review.openstack.org/476189 | |
| 20:00:08 | mriedem | dansmith: https://review.openstack.org/#/c/475067/ won't pass mitaka without us doing some things | |
| 20:00:14 | mriedem | some dirty deeds | |
| 20:08:22 | sdague | mriedem: honestly, just make all the mitaka tempest jobs non-voting | |
| 20:08:35 | sdague | and trust people read the results | |
| 20:10:26 | mriedem | i'm checking with lbragstad if the roles api actually changed it's response code at some point | |
| 20:17:14 | cfriesen_ | mriedem: do you think it would make sense to submit https://review.openstack.org/#/c/468203 for review before https://review.openstack.org/#/c/461456 ? I'm a bit worried about the second one getting blocked and preventing the first from being considered. | |
| 20:18:47 | mriedem | cfriesen_: i don't have any idea about that code | |
| 20:22:53 | dansmith | mriedem: oh, this is that thing, right.. I just blindly rechecked it | |
| 20:23:18 | mriedem | yeah trying to figure out if keystone changed their api since mitaka | |
| 20:23:49 | dansmith | tests not passing indicate that it should be taken out back and shot, IMHO | |
| 20:24:26 | mriedem | keystone? | |
| 20:25:10 | dansmith | no mitaka | |
| 20:25:26 | mriedem | oh, yeah, well it's already after it's eol date | |
| 20:25:32 | dansmith | right | |
| 20:29:48 | cfriesen_ | mriedem: dansmith: given the -1 to https://review.openstack.org/#/c/300616/, should I submit a patch to disallow swap-to-noswap resizes? | |
| 20:31:16 | mriedem | i don't have sufficient context on that | |
| 20:31:25 | cfriesen_ | bug is https://bugs.launchpad.net/nova/+bug/1552777 | |
| 20:31:25 | openstack | Launchpad bug 1552777 in OpenStack Compute (nova) "resizing from flavor with swap to one without swap puts instance into Error status" [Medium,In progress] - Assigned to Kam Nasim (knasim-wrs) | |
| 20:31:33 | mriedem | let me rephrase | |