| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-11 | |||
| 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... | |
| 19:00:54 | cfriesen | artom: at what point would you check them? boot/resize/rebuild ? | |
| 19:01:01 | artom | cfriesen, yes. | |
| 19:01:03 | artom | >;) | |
| 19:01:41 | vdrok | mriedem_afk: yup, that's the one. lyarwood 's patch seems to solve it | |
| 19:01:44 | cfriesen | I think those are the applicable three since we need to have both flavor/image to get a full picture | |
| 19:01:50 | artom | cfriesen, actually yeah, good point. We'd need to pick up on every time the image/flavor combo changes. | |
| 19:03:18 | artom | Which isn't a massive deal - I think boot+resize+rebuild actually covers it. But yeah, not as trivial as I thought. | |
| 19:15:19 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: always create consumer records https://review.openstack.org/567678 | |
| 19:15:20 | openstackgerrit | Jay Pipes proposed openstack/nova master: add consumers generation field https://review.openstack.org/557958 | |
| 19:15:21 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: Allocation.consumer field https://review.openstack.org/565405 | |