| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-14 | |||
| 10:04:08 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | |
| 10:18:46 | sean-k-mooney | bauzas: johnthetubaguy could ye take a look at this trival backport for os-vif stable/queens branch when ye have time. https://review.openstack.org/#/c/567942/ | |
| 11:18:46 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Escalate UUID validation warning to error in test https://review.openstack.org/540386 | |
| 11:18:47 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Suppress UUID warning in map_instance unit tests https://review.openstack.org/568263 | |
| 11:58:09 | openstackgerrit | Julia Kreger proposed openstack/nova master: ironic: add instance_uuid before any other spawn activity https://review.openstack.org/563722 | |
| 12:02:48 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make nova service-list use scatter-gather routine https://review.openstack.org/568271 | |
| 12:10:49 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | |
| 12:10:50 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider cadidates https://review.openstack.org/567150 | |
| 12:10:51 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508 | |
| 12:16:50 | openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | |
| 12:28:27 | gyankum | ping openstack-nova, How I can enable connection_trace for nova in RHOSP8. When I am setting connection_trace=true in nova.conf under Database section and restarting the nova-scheduler via pacemaker, nova-scheduler and nova-conduction services are not coming up. | |
| 12:54:22 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make nova service-list use scatter-gather routine https://review.openstack.org/568271 | |
| 13:14:36 | openstackgerrit | Eric Fried proposed openstack/nova master: add lower-constraints job https://review.openstack.org/555961 | |
| 13:18:25 | efried | finucannot: you around? | |
| 13:41:13 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test raw image live migration with vfat config drive https://review.openstack.org/567860 | |
| 13:54:00 | openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675 | |
| 13:54:01 | openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043 | |
| 13:54:02 | openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326 | |
| 13:58:21 | melwitt | efried: finucannot is on PTO today | |
| 13:58:49 | efried | melwitt: ack, thx | |
| 14:12:46 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add contributor docs on deprecating and removing compute REST APIs https://review.openstack.org/567687 | |
| 14:19:38 | mriedem | alex_xu: can you take a look at this spec again? https://review.openstack.org/#/c/546925/ | |
| 14:20:53 | openstackgerrit | Bob Ball proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | |
| 14:21:09 | alex_xu | mriedem: yea, I will try to review tomorrow | |
| 14:21:37 | mriedem | alex_xu: thanks | |
| 14:21:45 | alex_xu | mriedem: np | |
| 14:21:47 | mriedem | mdbooth: are you going to be able to address review comments on this series soon? https://review.openstack.org/#/c/526347/ | |
| 14:22:22 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | |
| 14:22:48 | mdbooth | mriedem: I'm about half way through. | |
| 14:22:58 | mdbooth | Probably be a day or so. | |
| 14:23:13 | mdbooth | I was expecting to get bumped to the back of the runway if that's what you're asking. | |
| 14:23:19 | mriedem | mdbooth: ok, yeah | |
| 14:29:11 | openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Update docs, API samples for noVNC 1.0.0 https://review.openstack.org/550173 | |
| 14:29:11 | mriedem | mdbooth: anyway, regardless of the runway, ping me when those 3 or so bottom changes are updated | |
| 14:29:32 | mdbooth | mriedem: Will do, thanks. | |
| 14:39:15 | kashyap | If anyone wants to take a look a simple clean-up: https://review.openstack.org/#/c/567258/ -- "libvirt: Remove usage of migrateToURI2() API" | |
| 15:02:31 | mriedem | lyarwood: when I +2ed https://review.openstack.org/#/c/567899/ i had a false positive on the live migration recreate patch sitting on top of it; i fixed that this morning and now the job is still failing even with your patch | |
| 15:02:49 | mriedem | this is when force_config_drive=true, images_type=raw, config_drive_format=vfat | |
| 15:02:51 | mriedem | vdrok: ^ | |
| 15:03:13 | vdrok | hrm | |
| 15:03:25 | mriedem | http://logs.openstack.org/60/567860/6/check/nova-live-migration/702f809/logs/screen-n-cpu.txt.gz?level=TRACE#_May_14_14_39_42_271510 | |
| 15:03:57 | vdrok | different error though, that's good :) | |
| 15:05:17 | mriedem | tonyb: kashyap: remember https://bugs.launchpad.net/nova/+bug/1246201 at all and the related libvirt bug? i couldn't actually find a specific libvirt bug/patch in the related libvirt ML threads, but i'm wondering which version of libvirt fixes that so we can deprecate the workaround of using config_drive_format=vfat | |
| 15:05:18 | openstack | Launchpad bug 1246201 in OpenStack Compute (nova) juno "Live migration fails when the instance has a config_drive_format=iso9660" [High,Fix released] - Assigned to Matt Riedemann (mriedem) | |
| 15:05:20 | openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler client allow multiple member_of query parameters https://review.openstack.org/568313 | |
| 15:05:30 | mriedem | "If your hypervisor driver is libvirt, and you want live migrate to work without shared storage, then use VFAT." | |
| 15:05:42 | kashyap | Hey there | |
| 15:06:18 | kashyap | mriedem: Where are you quoting that from? | |
| 15:06:21 | mriedem | that should probably read, "and you want live migrate to work with a config drive but without shared storage" | |
| 15:06:29 | mriedem | the "config_drive_format" option help text | |
| 15:07:09 | kashyap | mriedem: So, looking at the Launchpad bug, it redirects me to this upstream libvirt bug that I asked to file: | |
| 15:07:12 | kashyap | https://bugzilla.redhat.com/show_bug.cgi?id=1203032 | |
| 15:07:13 | openstack | bugzilla.redhat.com bug 1203032 in libvirt "Enhancing block/disk migration in libvirt" [Low,Closed: nextrelease] - Assigned to pboldin | |
| 15:07:18 | BobBall | hi melwitt - thought I'd join here rather than discussing on the ticket. It seems that tempest is using a flavour with root disk = 0 (flavor ref 42 which is new and called m1.nano) | |
| 15:07:23 | kashyap | mriedem: And the FixedInVersion field says: libvirt-1.2.17 | |
| 15:07:49 | BobBall | melwitt: Because we are requesting a 0-size root disk, it means that we don't try and resize, hence the success | |
| 15:08:55 | mriedem | kashyap: hmm, presumably we can do away with some workaround code then | |
| 15:09:21 | kashyap | mriedem: Yeah, definitely. Can add it to my list if you like | |
| 15:09:28 | kashyap | If you are already on it, won't step on your toes | |
| 15:09:31 | mriedem | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7495 | |
| 15:09:34 | melwitt | BobBall: yeah, just saw your reply. that's news to me that tempest uses a root_gb=0 flavor for resizes ... although, is it resizing from a 0 disk to a different 0 disk with different cpu/ram? | |
| 15:10:03 | BobBall | melwitt: Yes - that's right - the resize target appears to be 84 (I think) which is m1.micro: root_gb=0 but more RAM | |
| 15:10:20 | vdrok | mriedem: tho that error happens on spawn, not on migration now | |
| 15:11:00 | mriedem | vdrok: yeah | |
| 15:11:01 | vdrok | yeah we can do iso9660 now i think but we need to pass a new param to libvirt-python migrate method, migrate_disks, specifying cdrom device iiuc | |
| 15:11:19 | mriedem | vdrok: i think kashyap removed that | |
| 15:11:31 | melwitt | BobBall: I see. seems like that would be problematic in tempest in general and miss covering functionality in drivers if it's not doing any disk resize | |
| 15:11:34 | kashyap | mriedem: Just to note my availability, this week until next Monday, I will be "sparsely available" due to a prior committment. From Monday I should be more available. | |
| 15:11:43 | kashyap | (But I will be responsive on IRC, though) | |
| 15:12:02 | kashyap | vdrok: Since Rocky onwards | |
| 15:12:11 | melwitt | BobBall: thanks for tracking that down | |
| 15:12:13 | mriedem | vdrok: i might be thinking of this https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L6910 | |
| 15:12:15 | BobBall | melwitt: This appears to be set up in devstack's tempest plugin... So yes, I believe this is a common failing | |
| 15:12:17 | kashyap | ... the libvirt migrate API needs the 'params' dict passed. | |
| 15:13:47 | vdrok | ah so most of the things are in place already | |
| 15:13:51 | kashyap | Yeah | |
| 15:14:01 | kashyap | And note: this in-flight patch will remove completely the old API | |
| 15:14:14 | melwitt | BobBall: ack. I'll look into whether we could do minimal disk resizing in tempest | |
| 15:14:40 | kashyap | vdrok: As it's not needed, and the new one variant satsified by Nova's current min libvirt version is backwards compatible w/ the older variant: | |
| 15:14:43 | kashyap | https://review.openstack.org/#/c/567258/ | |
| 15:14:44 | melwitt | BobBall: the other question I had was, is the new image handler thing the only way to fix resize for xenapi? I didn't get how that's connected | |
| 15:15:44 | BobBall | melwitt: The fix for resize is common to all image handlers for XenServer - it's not related to the new image handler and could justifiably be split out into a separate patch. We just kept it here because this is where we were doing the manual testing | |
| 15:16:55 | melwitt | BobBall: okay. I think it would be best to open a bug and fix the breakage separately rather than roll it into the blueprint work | |
| 15:17:45 | BobBall | melwitt: Bah :D I knew you were going to say that :D I'll do it now so it doesn't have to wait for tomorrow with PRC's timezone. | |
| 15:18:05 | melwitt | I'm too predictable | |
| 15:18:47 | kashyap | Unrelated: Does anyone here use "lp-tools" to file Launchpad bugs via command-line? | |
| 15:18:59 | kashyap | (Launch, manage, query, etc.) | |
| 15:19:22 | kashyap | Which reminds me ... some projects have moved to Storyboard. One day, probably Nova will too | |
| 15:19:38 | melwitt | but yeah, I'd like to get the driver fixed ASAP and not have to wait behind the feature work that's in-progress | |
| 15:21:43 | BobBall | melwitt: We were hoping that patch for the feature was literally about to get merged once the tests all pass :) | |
| 15:22:06 | mriedem | vdrok: so we used to use this utility method to call mkfs.vfat https://review.openstack.org/#/c/527510/3/nova/utils.py | |
| 15:22:41 | mriedem | i don't see anything checking to see if the path exists first or creates the path first | |
| 15:23:16 | mriedem | reading https://linux.die.net/man/8/mkfs.vfat i'm not sure if -C should be specified or not | |
| 15:24:15 | mriedem | http://manpages.ubuntu.com/manpages/xenial/man8/mkfs.vfat.8.html | |
| 15:25:38 | mriedem | unless it's complaining that /opt/stack/data/nova/instances/51858976-6a84-467a-895b-39a22b43b27f/ doesn't exist, but i'm pretty sure it does at this point | |
| 15:25:51 | mriedem | or mkfs.vfat doesn't exist | |
| 15:26:31 | mriedem | trying something | |
| 15:26:57 | vdrok | yeah I see 'unable to open <path>' when specifying non-existing path | |
| 15:28:05 | vdrok | I was testing this on pike when privsep stuff did not exist | |
| 15:28:07 | mriedem | i figured it was because we don't install dosfstools in devstack | |