*** fanzhang has joined #openstack-placement | 00:57 | |
*** Nel1x has quit IRC | 01:33 | |
*** tetsuro has joined #openstack-placement | 01:49 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: add zvm into support matrix https://review.openstack.org/532720 | 02:01 |
---|---|---|
*** efried1 has joined #openstack-placement | 02:26 | |
*** efried has quit IRC | 02:28 | |
*** efried1 is now known as efried | 02:28 | |
*** tetsuro has quit IRC | 03:26 | |
openstackgerrit | Merged openstack/nova master: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/590538 | 03:58 |
openstackgerrit | Merged openstack/nova master: Trivial fix to remove extra 'will' on microversion doc https://review.openstack.org/591235 | 03:58 |
openstackgerrit | Merged openstack/nova master: Adds a test for _get_provider_ids_matching() https://review.openstack.org/590150 | 03:58 |
openstackgerrit | Merged openstack/nova master: Test case for multiple forbidden traits https://review.openstack.org/591070 | 03:58 |
openstackgerrit | Merged openstack/nova master: placement: use simple code paths when possible https://review.openstack.org/590388 | 04:05 |
openstackgerrit | Merged openstack/nova master: Handle unicode characters in migration params https://review.openstack.org/591080 | 04:05 |
openstackgerrit | Merged openstack/nova master: improve migration script https://review.openstack.org/591030 | 04:05 |
*** fanzhang has left #openstack-placement | 04:18 | |
*** takashin has quit IRC | 04:24 | |
*** alex_xu has joined #openstack-placement | 05:03 | |
*** openstackgerrit has quit IRC | 05:18 | |
*** takashin has joined #openstack-placement | 05:35 | |
*** takashin has quit IRC | 05:41 | |
*** takashin has joined #openstack-placement | 06:04 | |
*** openstackgerrit has joined #openstack-placement | 06:20 | |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Revert "libvirt: slow live-migration to ensure network is ready" https://review.openstack.org/591275 | 06:20 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform volume.usage notification https://review.openstack.org/580345 | 07:34 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform compute_task notifications https://review.openstack.org/482629 | 07:39 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform libvirt.error notification https://review.openstack.org/484851 | 07:44 |
openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova-specs master: Placement model for PCI devices https://review.openstack.org/591037 | 07:57 |
*** e0ne has joined #openstack-placement | 08:20 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP scheduler: check request_spec.instance_group before updating https://review.openstack.org/590733 | 08:34 |
*** tssurya has joined #openstack-placement | 08:34 | |
*** giblet_off is now known as gibi | 08:55 | |
openstackgerrit | garyk proposed openstack/nova master: Docs: remove invalid link for remote debugging https://review.openstack.org/591316 | 09:18 |
*** cdent has joined #openstack-placement | 09:25 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] api-ref: Add missing aggregates example https://review.openstack.org/591332 | 10:12 |
openstackgerrit | Rajesh Tailor proposed openstack/nova master: Fix case-sensitivity for metadata keys https://review.openstack.org/504885 | 10:25 |
*** mriedem has joined #openstack-placement | 11:36 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (4) https://review.openstack.org/568462 | 11:52 |
openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: VMware: fix TypeError while get console log https://review.openstack.org/591365 | 11:54 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | 12:00 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (5) https://review.openstack.org/570842 | 12:05 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (6) https://review.openstack.org/571330 | 12:10 |
openstackgerrit | Chris Dent proposed openstack/nova master: DNM: Add placement perf info gathering to end of nova-next https://review.openstack.org/591367 | 12:13 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992 | 12:14 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993 | 12:18 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in unit/network/test_neutronv2.py (3) https://review.openstack.org/574104 | 12:22 |
*** nicolasbock has joined #openstack-placement | 12:48 | |
*** jroll has quit IRC | 12:59 | |
*** jroll has joined #openstack-placement | 13:00 | |
*** mriedem has quit IRC | 13:03 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] api-ref: Add missing aggregates example https://review.openstack.org/591332 | 13:10 |
efried | cdent: How did adding aggs to the mix change the results? | 13:16 |
*** SteelyDan is now known as dansmith | 13:25 | |
*** jaypipes has joined #openstack-placement | 13:26 | |
jaypipes | efried: I won't be at the scheduler meeting today, sorry | 13:26 |
efried | jaypipes: ack. Anything for me to report/convey? | 13:27 |
jaypipes | I don't have much to add other than I'll try to get through reviews this morning, so feel free to stack some priority items up for me if you'd like. | 13:27 |
efried | jaypipes: Roger that. We can start landing Stein code at this point, so let's get the reshaper series in. (<== gibi) | 13:27 |
gibi | efried: ack | 13:28 |
cdent | efried: very little, but I was only testing on recent master, not the split code | 13:35 |
efried | cdent: I feel like the split code will still be very efficient there, because now we're front-loading the aggregate filter. | 13:36 |
cdent | efried: I've started working on a gate performance check, but it hasn't complete a run yet, and I assume it will fail spectacularly the first time | 13:36 |
efried | cdent: Are you putting an actual time limit on it? | 13:36 |
cdent | no, it's just info | 13:36 |
cdent | what I mean is that there's no way to test that the script even runs correctly, except in the gate | 13:37 |
cdent | s/way/easy way/ | 13:37 |
efried | I see | 13:38 |
cdent | hmmm, yeah, I got a few different steps wrong | 13:40 |
* cdent tries agai | 13:40 | |
gibi | efried: I only have nits in the first reshaper patch so +2 | 13:46 |
efried | gibi: Great, thanks. Yeah, I think we're going to end up with a cumulative nitbuster patch on top. | 13:47 |
openstackgerrit | Chris Dent proposed openstack/nova master: DNM: Add placement perf info gathering to end of nova-next https://review.openstack.org/591367 | 13:47 |
efried | cdent: I'm throwing you under the bus^W^W^W^W^Wadding perf/scale testing in the gate to the meeting agenda. | 13:49 |
cdent | great | 13:49 |
efried | :P | 13:50 |
efried | Scheduler meeting in 10 minutes in #openstack-meeting-alt | 13:50 |
cdent | this is so far away from real scale testing, I'm not sure we should even suggest it in the same breath | 13:50 |
efried | Baby steps cdent. Baby steps. | 13:53 |
efried | What you're doing is better than what we had before (which is nothing) by a long shot. | 13:54 |
cdent | sure | 13:54 |
cdent | I worry, a lot, about people jumping to the wrong conclusions or thinking of stuff as "done" | 13:54 |
cdent | when we are miles fro | 13:54 |
cdent | m | 13:54 |
efried | is anything "done"? | 13:56 |
efried | anything at all? | 13:56 |
cdent | don't add to my existential dread please | 13:57 |
cdent | but yes, we often have people who claim "done" way too early | 13:57 |
*** tetsuro__ has joined #openstack-placement | 13:59 | |
*** tetsuro__ has quit IRC | 14:02 | |
*** tetsuro has joined #openstack-placement | 14:03 | |
jaypipes | cdent: any perf differences to speak of w.r.t. https://github.com/cdent/placeload/commit/0c07d42c88516ab27fe11acf250dcd05414774fa? | 14:07 |
cdent | jaypipes: no, not really | 14:07 |
cdent | jaypipes: but i haven't done super complete testing yet | 14:08 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840 | 14:46 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267 | 14:46 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847 | 14:46 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | 14:46 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459 | 14:46 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | 14:46 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667 | 14:56 |
*** takashin has left #openstack-placement | 15:01 | |
cdent | jaypipes: can you, at your leisure, expand on [t 2Jku] ? | 15:04 |
purplerbot | <jaypipes> it *seem* like nova == placement for all intents and purposes. [2018-08-13 14:56:03.893742] [n 2Jku] | 15:04 |
*** tetsuro has quit IRC | 15:05 | |
cdent | jaypipes: I gotta run sarah to physio so no rush if now is bad, but it will help me to shape my thinking | 15:05 |
jaypipes | cdent: mostly what I was trying to convey is that the non-technical arguments for keeping placement inside nova have always been that nova has been seen (since this has been the reality for 2+ years) as placement's only real client, and the calling code inside of nova out to a placement service is very Nova-centric (consumers == instance or migration for example). this perception probably influences opinions on whether it is appropriate | 15:07 |
jaypipes | (regardless of technical details) for placement to be separate from nova. | 15:07 |
openstackgerrit | Merged openstack/nova master: add zvm into support matrix https://review.openstack.org/532720 | 15:08 |
cdent | jaypipes: sure, but that calling is over a hard boundary | 15:09 |
cdent | and has always been over a hard boundary. the thing the is permeable is the people doing the reviewing | 15:10 |
openstackgerrit | Eric Fried proposed openstack/nova master: DNM: Remove blacklisted py3 xen tests https://review.openstack.org/591419 | 15:17 |
openstackgerrit | Eric Fried proposed openstack/nova master: DNM: Remove blacklisted py3 xen tests https://review.openstack.org/591419 | 15:18 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Enable nested allocation candidates in scheduler https://review.openstack.org/585672 | 15:19 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Functional test for booting with nested resources https://review.openstack.org/527728 | 15:19 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Functional test for moving with nested resources https://review.openstack.org/587350 | 15:19 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add request_spec.RequestGroup versioned object https://review.openstack.org/568840 | 15:35 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add requested_resources field to RequestSpec https://review.openstack.org/567267 | 15:35 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add bandwidth related standard resource classes https://review.openstack.org/570847 | 15:35 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transfer port.resource_request to the scheduler https://review.openstack.org/567268 | 15:36 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send resource allocations in the port binding https://review.openstack.org/569459 | 15:36 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test boot with more ports with bandwidth request https://review.openstack.org/573317 | 15:36 |
*** e0ne has quit IRC | 16:03 | |
openstackgerrit | Chris Dent proposed openstack/nova master: DNM: Add placement perf info gathering to end of nova-next https://review.openstack.org/591367 | 16:24 |
*** e0ne has joined #openstack-placement | 16:28 | |
*** e0ne has quit IRC | 17:01 | |
openstackgerrit | Merged openstack/nova master: [placement] api-ref: Add missing aggregates example https://review.openstack.org/591332 | 17:03 |
*** openstackgerrit has quit IRC | 17:19 | |
*** tssurya has quit IRC | 17:36 | |
*** stephenfin_ has quit IRC | 17:42 | |
*** stephenfin has joined #openstack-placement | 17:45 | |
cdent | efried, jaypipes, edleafe : started https://etherpad.openstack.org/p/placement-extract-stein but note the big caveat on the top. Despite the fact that I've been playing with this idea for quite some time, I caution us to not overestimate my vision. There are plenty of things I simply do not care about that plenty of other people do. | 18:01 |
efried | cdent: ack, thanks. | 18:02 |
*** e0ne has joined #openstack-placement | 18:30 | |
*** tssurya has joined #openstack-placement | 18:39 | |
*** openstackgerrit has joined #openstack-placement | 19:14 | |
openstackgerrit | Eric Fried proposed openstack/nova master: DNM: Remove blacklisted py3 xen tests https://review.openstack.org/591419 | 19:14 |
*** mgagne_ has joined #openstack-placement | 19:24 | |
*** mgagne has quit IRC | 19:25 | |
*** purplerbot has quit IRC | 19:27 | |
openstackgerrit | Chris Dent proposed openstack/nova master: DNM: Add placement perf info gathering to end of nova-next https://review.openstack.org/591367 | 19:45 |
*** e0ne has quit IRC | 19:52 | |
*** efried has quit IRC | 19:58 | |
*** tssurya has quit IRC | 20:07 | |
*** efried has joined #openstack-placement | 20:13 | |
cdent | efried, jaypipes: this is kinda dead in the water: https://review.openstack.org/#/c/567191/ how much does it matter? | 20:14 |
openstackgerrit | melanie witt proposed openstack/nova master: Add functional test for affinity with multiple cells https://review.openstack.org/585073 | 20:14 |
openstackgerrit | melanie witt proposed openstack/nova master: Make scheduler.utils.setup_instance_group query all cells https://review.openstack.org/540258 | 20:14 |
efried | cdent: Um, we should do it. | 20:14 |
efried | cdent: At least the normalization in the handler. | 20:15 |
efried | Seems like an easy one for someone to pick up. I bet takashin would rip it out overnight if we asked him. | 20:16 |
cdent | will you see him soon? I tend to only see him in morning email detritus | 20:17 |
*** e0ne has joined #openstack-placement | 20:18 | |
jaypipes | cdent: I agree we should do it. | 20:24 |
*** purplerbot has joined #openstack-placement | 20:25 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Improve debug logs for GET /allocation_candidates https://review.openstack.org/591510 | 20:25 |
efried | jaypipes, cdent: ^ | 20:25 |
efried | jaypipes: threw that one on top of Gigantor to demo what I'm rambling on about in my review comments. | 20:26 |
efried | cdent: Is it urgent? I can probably knock it out pretty quick. | 20:27 |
efried | cdent: I actually want to say somebody (gibi?) already did it for at least some of the operations | 20:27 |
cdent | efried: if it were urgent someone would have fixed it by now. That's why I'm asking about it. If people aren't rushing around trying to fix it, it may not matter. I'm aware that it came up in a vmware situation | 20:28 |
cdent | but the people who were concerned about it there did something else | 20:28 |
cdent | gibi fixed some tests and associated things where we were using non-uuids and only warning, instead of erroring | 20:29 |
cdent | if we honestly think it is low hanging fruit, we should leave it low hanging and get on with the hard stuff, yeah? | 20:29 |
cdent | otherwise we aren't leaving any holes for poeple to fill | 20:30 |
efried | wfm | 20:31 |
efried | bbiab | 20:31 |
*** e0ne has quit IRC | 20:33 | |
cdent | man, tempest are ess ell double oh | 20:56 |
*** cdent has quit IRC | 21:06 | |
*** e0ne has joined #openstack-placement | 21:50 | |
*** e0ne has quit IRC | 21:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!