| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-05-11 | |||
| 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 | |
| 19:15:22 | openstackgerrit | Jay Pipes proposed openstack/nova master: rework allocation handler _allocations_dict() https://review.openstack.org/565407 | |
| 19:15:23 | openstackgerrit | Jay Pipes proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604 | |
| 19:47:40 | fried_rice | Hey folks, was pip10 blacklisted in queens? | |
| 19:48:01 | fried_rice | never mind | |
| 20:19:03 | jgwentworth | guh, forgot to update my instance group patch | |
| 20:26:05 | mriedem_afk | cfriesen: that's why i left the api validation to a separate change | |
| 20:26:11 | mriedem_afk | but yes it would be create, resize and rebuild i think | |
| 20:32:31 | arvindn05 | mriedem: can you add https://review.openstack.org/#/c/560596/ to your review list? | |
| 20:33:10 | arvindn05 | i got efried to approve the changes...so need 1 more :) | |
| 20:33:46 | mriedem | vdrok: i'll rebase the recreate patch on top of lyarwood's change once the other thing underneath mine is merged | |
| 20:35:53 | mriedem | arvindn05: yes | |
| 20:41:57 | openstackgerrit | sean mooney proposed openstack/os-vif stable/queens: fix tox py27 job https://review.openstack.org/567942 | |
| 20:53:03 | openstackgerrit | Merged openstack/nova master: Fix irrelevant-files in nova-dsvm-multinode-base https://review.openstack.org/567884 | |
| 20:55:24 | mriedem | arvindn05: done | |
| 20:59:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test raw image live migration with vfat config drive https://review.openstack.org/567860 | |
| 21:05:06 | arvindn05 | mriedem: awesome....thank you :) | |
| 21:05:37 | mriedem | now get cracking on that rebuild patch | |
| 21:05:39 | mriedem | and docs | |
| 21:06:00 | arvindn05 | yep...will have the rebuild patch sometime next week | |
| 21:15:38 | openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: xenapi: drop deprecated vif_driver config option https://review.openstack.org/567877 | |
| 21:32:18 | openstackgerrit | Merged openstack/os-vif master: fix tox py27 job https://review.openstack.org/567895 | |
| 21:50:21 | openstackgerrit | Eric Fried proposed openstack/nova master: Granular requests to get_allocation_candidates https://review.openstack.org/515811 | |
| 22:00:42 | openstackgerrit | Merged openstack/nova master: Address feedback from instance_list smart-cell behavior https://review.openstack.org/567643 | |