| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-11 | |||
| 16:40:31 | sean-k-mooney | cool no worries i should have the review up in the next 15 mins or so | |
| 16:41:22 | sean-k-mooney | it was driving nuts trying to figure out why my test were not running lol i should start testing with py35 by default | |
| 16:52:15 | openstackgerrit | sean mooney proposed openstack/os-vif master: fix tox py27 job https://review.openstack.org/567895 | |
| 16:58:36 | PaulN | Hi, quick question, I'm trying to create a stack via Heat Templates containing a OS::Nova::ServerGroup in which I want to use the soft-anti-affinity policy which was added in nova compute api microversion 2.15. I can create it via the CLI by specifying --os-compute-api-version 2.15, but I can't create it via heat. I can't seem to find any information on how one specifies the nova api microversion from a Heat Template. | |
| 17:00:26 | PaulN | This in on an older OpenStack cluster that is only running Newton. | |
| 17:03:17 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Report the virtual size of RAW disks https://review.openstack.org/567899 | |
| 17:06:43 | jgwentworth | PaulN: I think you'll need to try the heat channel or tripleo channel. not sure anyone here will know how to specify microversions via heat templates | |
| 17:06:44 | PaulN | Yeah I have the Heat API version set to the value appropriate for Newton, but thats just the Heat Template version, doesn't really seem to have any correlation to what Nova API version is used. | |
| 17:06:57 | cfriesen_ | PaulN: based on https://bugzilla.redhat.com/show_bug.cgi?id=1447798 it seems heat support for nova microversions was only added in Ocata | |
| 17:07:06 | jgwentworth | nvm | |
| 17:07:08 | PaulN | Okay, I can try over in the heat channel. | |
| 17:07:51 | PaulN | Ugh, based on that seems I may be unable to use it via Heat on this cluster. | |
| 17:08:18 | openstack | bugzilla.redhat.com bug 1447798 in openstack-heat "soft-anti-affinity is not accepted by python-openstackclient but is with python-novaclient" [High,Closed: duplicate] - Assigned to zbitter | |
| 17:11:38 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Report the virtual size of RAW disks https://review.openstack.org/567899 | |
| 17:20:00 | prometheanfire | mriedem_lunch: the new keystonemiddleware breaks nova tests | |
| 17:20:03 | prometheanfire | http://logs.openstack.org/49/567449/1/check/cross-nova-py27/c2d43ef/testr_results.html.gz | |
| 17:35:10 | jgwentworth | cdent ^ you might understand what's going on there. only difference is the string quotes? | |
| 17:38:02 | cdent | jgwentworth: huh, wacky. | |
| 17:38:10 | jgwentworth | yeah | |
| 17:41:07 | cdent | jgwentworth: it's this https://review.openstack.org/#/c/559925/ | |
| 17:41:24 | jgwentworth | aaahhh | |
| 17:42:00 | cdent | so the test was relying on apparently not quite correct behavior by the middleware | |
| 17:42:20 | cdent | so it failing now makes sense | |
| 17:42:24 | jgwentworth | good find | |
| 17:42:26 | cdent | I guess we can just change the test to be a bit more flexible | |
| 17:42:44 | cdent | the format of the output isn't really the important part | |
| 17:42:54 | jgwentworth | true | |
| 17:43:06 | cdent | i'm sort of in the middle of the something at the moment, but can do it a bit later if someone reminds me? | |
| 17:43:19 | jgwentworth | yeah, np | |
| 17:45:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test raw image live migration with vfat config drive https://review.openstack.org/567860 | |
| 17:50:34 | prometheanfire | mriedem: I'd like that :D | |
| 17:50:52 | mriedem | like what? | |
| 17:51:10 | openstackgerrit | Jay Pipes proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958 | |
| 17:51:11 | mriedem | https://www.youtube.com/embed/bsB7UQ8BlE0 | |
| 17:51:12 | openstackgerrit | Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | |
| 17:51:12 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405 | |
| 17:51:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | |
| 17:51:29 | prometheanfire | mriedem: the config drive live migration | |
| 17:51:39 | mriedem | leakypipes: i want to see you yell that at your dogs ^ | |
| 17:51:42 | mriedem | cousins style | |
| 17:53:40 | leakypipes | mriedem: :) | |
| 17:56:27 | openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Fix HTTP500 error of changes-since on v2.0 API https://review.openstack.org/567916 | |
| 17:56:57 | cdent | jgwentworth: that thing I was in the midst of has utterly failed to do the right thing so needs to think about what it did, so I can fix the keystone thing now. Should I make a bug or just fix it or something else? | |
| 17:58:02 | jgwentworth | prometheanfire: do you have a bug for the cross-nova-py27 fail? cdent is gonna fix it | |
| 18:00:11 | prometheanfire | jgwentworth: I don't have a bug yet, it's just part of the release review | |
| 18:00:19 | prometheanfire | https://review.openstack.org/567449 | |
| 18:00:36 | jgwentworth | k | |
| 18:01:54 | jgwentworth | cdent: I'll open a bug | |
| 18:02:05 | prometheanfire | thanks | |
| 18:02:28 | cdent | thanks jgwentworth and prometheanfire . should have something pretty quick | |
| 18:03:56 | prometheanfire | ya, it didn't look bad | |
| 18:06:18 | jgwentworth | cdent: https://bugs.launchpad.net/nova/+bug/1770718 | |
| 18:06:20 | openstack | Launchpad bug 1770718 in OpenStack Compute (nova) "test_auth_middleware_factory fails with new keystonemiddleware 5.1.0" [Undecided,New] - Assigned to Chris Dent (cdent) | |
| 18:06:32 | cdent | thanks | |
| 18:06:46 | jgwentworth | thanks for fixing it :) | |
| 18:18:43 | openstackgerrit | Chris Dent proposed openstack/nova master: Flexibly test keystonmiddleware in placement stack https://review.openstack.org/567923 | |
| 18:18:57 | cdent | jgwentworth, prometheanfire ^ that oughta do the trick (and work for old and new) | |
| 18:19:46 | jgwentworth | saweet | |
| 18:20:47 | openstackgerrit | sean mooney proposed openstack/os-vif master: add noop plugin https://review.openstack.org/567924 | |
| 18:20:58 | prometheanfire | nice | |
| 18:32:18 | mriedem | blarg, if the libvirt driver raises RequestedVRamTooHigh during spawn we'll reschedule to at least 2 other nodes and fail on the same thign | |
| 18:32:19 | mriedem | *thing | |
| 18:33:59 | mriedem | also, double blarg, this is something we could validate in the API | |
| 18:34:36 | jgwentworth | yeah, I was going to ask, do we validate the other extra specs in the API or where | |
| 18:36:13 | openstackgerrit | Merged openstack/nova master: libvirt: always pass emulator threads policy https://review.openstack.org/561913 | |
| 18:39:14 | openstackgerrit | Merged openstack/nova master: Add connection_parameters to list of items copied from database https://review.openstack.org/567248 | |
| 18:42:26 | mriedem | jgwentworth: validating this in the api for server create and rebuild would be trivial, | |
| 18:42:51 | mriedem | only question about that is currently only the libvirt driver supports the image property and flavor extra spec, but i guess that just means - you shouldn't be using these if yo'ure not using libvirt, | |
| 18:43:09 | mriedem | and if you're doing mixed compute, make sure you use host aggregates to pin the image/flavor to libvirt hosts | |
| 18:45:03 | jgwentworth | you mean as far as where the checks belong in the api since they're driver-specific? | |
| 18:45:14 | jgwentworth | s/where/whether/ | |
| 18:47:11 | mriedem | kind of yeah | |
| 18:47:20 | artom | Expose it as traits? | |
| 18:47:21 | mriedem | my point being, the admin controls the flavors, | |
| 18:47:37 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't reschedule on RequestedVRamTooHigh errors https://review.openstack.org/567929 | |
| 18:47:38 | mriedem | and if the cloud doesn't have libvirt hosts to support this image property, then don't put that extra spec in the flavor | |
| 18:47:48 | mriedem | and if you have mixed compute, you would need to isolate using aggregates | |
| 18:48:16 | mriedem | artom: that's way more work than i want to put into this :) | |
| 18:48:41 | artom | mriedem, that's fair - I was only saying that as a way to procrastinate from doing real work :D | |
| 18:49:24 | mriedem | you thought, "how can i rope placement into this?" | |
| 18:49:40 | jgwentworth | yeah | |
| 18:50:13 | mriedem | next time i have weird muscle pain i'm going to ask how placement can fix it for me | |
| 18:50:31 | jgwentworth | solid plan | |
| 18:51:28 | artom | Isn't that what chiropractors do? Place your joints and muscles and tendons in all the right, err, places? | |
| 18:52:15 | mriedem | for about 3 days? | |
| 18:52:16 | mriedem | that's what i've heard | |
| 18:52:29 | mriedem | them your body has a periodic task that goofs it all up again | |
| 18:52:32 | mriedem | *then | |
| 18:52:42 | mriedem | similar to our resource tracker | |
| 18:52:43 | mriedem | :) | |
| 18:52:50 | prometheanfire | could be, I noticed that posture and back exercises help a bunch | |
| 18:53:01 | mnaser | so um we're going queens for control plane this weekend most likely | |
| 18:53:06 | mnaser | anything fun to watch out for? | |
| 18:53:19 | prometheanfire | mnaser: it's working fine here | |
| 18:54:05 | mnaser | prometheanfire: upgrade from pike? | |
| 18:56:32 | prometheanfire | ya | |
| 18:56:42 | prometheanfire | old grizzly install originally | |
| 18:59:14 | mriedem_afk | vdrok: recreated that live migration failure http://logs.openstack.org/60/567860/4/check/nova-live-migration/8993c7b/logs/subnode-2/screen-n-cpu.txt.gz?level=TRACE | |
| 18:59:38 | artom | mriedem_afk, huh, so https://github.com/openstack/nova/blob/18.0.0.0b1/nova/virt/libvirt/driver.py#L4703-L4708 is pretty straightforward and doens't need to be in libvirt | |
| 19:00:14 | artom | mriedem_afk, I'm wondering if we can't forgo your small fix and jump straight to in-API validation? It's just comparing image props and flavor extra specs, both are accessible to the API... | |