| *** ttsiouts has quit IRC | 00:03 | |
| *** ttsiouts has joined #openstack-placement | 00:35 | |
| *** ttsiouts has quit IRC | 00:40 | |
| *** ttsiouts has joined #openstack-placement | 01:15 | |
| *** bhagyashris has joined #openstack-placement | 01:20 | |
| *** ttsiouts has quit IRC | 01:49 | |
| *** altlogbot_2 has quit IRC | 02:12 | |
| *** altlogbot_3 has joined #openstack-placement | 02:13 | |
| *** ttsiouts has joined #openstack-placement | 02:55 | |
| *** ttsiouts has quit IRC | 03:29 | |
| *** ttsiouts has joined #openstack-placement | 04:02 | |
| *** ttsiouts has quit IRC | 04:07 | |
| *** ttsiouts has joined #openstack-placement | 04:39 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/placement master: Trivial: Fix comment for LEFT join https://review.opendev.org/661182 | 04:58 |
|---|---|---|
| *** ttsiouts has quit IRC | 05:11 | |
| *** ttsiouts has joined #openstack-placement | 06:27 | |
| *** ttsiouts has quit IRC | 06:58 | |
| gibi | efried: ack on the nested magic spec, I'm planning to review it soonish. | 07:10 |
| *** tssurya has joined #openstack-placement | 07:15 | |
| *** ttsiouts has joined #openstack-placement | 07:30 | |
| *** mnaser has quit IRC | 07:35 | |
| *** helenafm has joined #openstack-placement | 07:36 | |
| *** guilhermesp has quit IRC | 07:37 | |
| *** guilhermesp has joined #openstack-placement | 07:43 | |
| *** mnaser has joined #openstack-placement | 07:43 | |
| *** e0ne has joined #openstack-placement | 07:51 | |
| *** mnaser has quit IRC | 08:02 | |
| *** mnaser has joined #openstack-placement | 08:05 | |
| *** ttsiouts has quit IRC | 08:35 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Don't run functional.db tests in nova functional run https://review.opendev.org/661235 | 08:36 |
| *** bhagyashris has quit IRC | 09:03 | |
| *** ttsiouts has joined #openstack-placement | 09:11 | |
| *** ttsiouts has quit IRC | 09:15 | |
| *** cdent has joined #openstack-placement | 09:44 | |
| *** ttsiouts has joined #openstack-placement | 09:52 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Avoid traversing summaries in _check_traits_for_alloc_request https://review.opendev.org/660691 | 09:57 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Use trait strings in ProviderSummary objects https://review.opendev.org/660692 | 09:57 |
| openstackgerrit | Chris Dent proposed openstack/placement master: DNM: perfload with written allocations https://review.opendev.org/660754 | 10:00 |
| cdent | gibi could you have a quick look at https://review.opendev.org/#/c/661131/ that stack fixes a gate blocker (my fault, basically) | 10:24 |
| *** ttsiouts has quit IRC | 10:25 | |
| gibi | cdent: looking.. | 10:48 |
| cdent | thanks | 10:48 |
| gibi | cdent: +Ad | 10:49 |
| cdent | thanks | 10:49 |
| cdent | this is the lowest I've seen it so far: http://logs.openstack.org/92/660692/3/check/placement-perfload/ef919db/logs/placement-perf.txt | 11:32 |
| cdent | not that it is that meaningful: the variance in the gate nodes is _huge_ | 11:33 |
| *** ttsiouts has joined #openstack-placement | 11:33 | |
| *** cdent has quit IRC | 11:35 | |
| *** cdent has joined #openstack-placement | 11:36 | |
| *** cdent has quit IRC | 11:40 | |
| *** cdent has joined #openstack-placement | 11:55 | |
| *** ttsiouts has quit IRC | 12:06 | |
| *** dims has left #openstack-placement | 12:20 | |
| *** ttsiouts has joined #openstack-placement | 12:35 | |
| *** ttsiouts has quit IRC | 12:39 | |
| *** e0ne has quit IRC | 12:52 | |
| *** e0ne has joined #openstack-placement | 13:01 | |
| openstackgerrit | Merged openstack/placement master: Bump os-resource-classes requirements https://review.opendev.org/661131 | 13:04 |
| openstackgerrit | Merged openstack/placement master: Fix typo in usage.yaml and usage-policy.yaml https://review.opendev.org/661044 | 13:04 |
| openstackgerrit | Merged openstack/placement master: Canary test for os-traits version https://review.opendev.org/661132 | 13:04 |
| *** mriedem has joined #openstack-placement | 13:12 | |
| *** ttsiouts has joined #openstack-placement | 13:12 | |
| *** ttsiouts has quit IRC | 13:40 | |
| *** ttsiouts has joined #openstack-placement | 13:40 | |
| gibi | cdent, efried: I went through the nested magic spec, left some comments inside | 13:42 |
| cdent | thanks gibi | 13:42 |
| efried | thanks gibi | 13:43 |
| gibi | please don't hate me about my rant over the subtree terminology | 13:43 |
| gibi | I think what we want to achive is good but we might need some extra definition what we mean by two RPs is in the same subtree | 13:44 |
| * gibi goes to brew a tree to recharge batteries | 13:44 | |
| cdent | a whole tree! | 13:47 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Avoid traversing summaries in _check_traits_for_alloc_request https://review.opendev.org/660691 | 13:50 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Use trait strings in ProviderSummary objects https://review.opendev.org/660692 | 13:50 |
| *** ttsiouts has quit IRC | 13:59 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Trivial: Fix comment for LEFT join https://review.opendev.org/661182 | 14:00 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Don't run functional.db tests in nova functional run https://review.opendev.org/661235 | 14:00 |
| efried | cdent: on the nested magic spec, the template shuffle led me to wonder some things, like | 14:08 |
| efried | Q: should we have a template at all? A: yeah, of course, still helpful for authors. | 14:08 |
| efried | Q: should we enact template enforcement like nova has? A: no, that's annoying and I should get to use my discretion. That said, as a reviewer, I wouldn't necessarily notice that a section was missing, thereby making it harder for me to know that everything has been considered and addressed. Any way to make the template checker available but, like, non-voting? | 14:08 |
| efried | Q: should we shuffle the template itself around? i.e. how much of the shuffling I did is sensible in general? A: profoundly dunno, but probably so low priority that it'll never float to the top of my to-do list. | 14:08 |
| cdent | efried: I think having the template is a useful aid but that's all it needs to be. no checker required, author's discretion on how to they want to do things. | 14:10 |
| efried | ++ | 14:13 |
| *** stephenfin is now known as finucannot | 14:14 | |
| gibi | s/tree/tea/ bahh :D | 14:15 |
| cdent | efried, gibi, edleafe : should this extension to perfload happen: https://review.opendev.org/660754 ? | 14:31 |
| gibi | cdent: do I understand the test result in ^^ that this proves that having allocations does not really affect the a_c query times? | 14:38 |
| cdent | gibi: pretty much | 14:38 |
| edleafe | cdent: perf all the things! | 14:38 |
| cdent | The question that was raised was "does joining on an allocations table with stuff in it change the performance profile compared to an empty allocations table" | 14:39 |
| *** ttsiouts has joined #openstack-placement | 14:39 | |
| cdent | so now the question is "now that we know, should we maintain perfload in this form (until the next way it gets changed)?" | 14:40 |
| edleafe | cdent: I'm not sure why you wouldn't | 14:41 |
| cdent | cuz it's long and noisy? | 14:41 |
| edleafe | But it gives more realistic results, no? | 14:42 |
| cdent | yes | 14:42 |
| efried | ++ | 14:42 |
| gibi | cdent: If this does not make the pref job run significantly longer then I think we should merge it | 14:42 |
| cdent | adds about 5 minutes, but still under 10 minutes | 14:42 |
| efried | even if it did | 14:42 |
| cdent | i'll fix up the change so it runs all the tests and push it back up | 14:42 |
| efried | but -2 for trailing whitespace where it doesnt' matter. | 14:43 |
| *** ttsiouts has quit IRC | 14:44 | |
| gibi | :) | 14:44 |
| cdent | i've fixed that | 14:44 |
| openstackgerrit | Chris Dent proposed openstack/placement master: perfload with written allocations https://review.opendev.org/660754 | 14:44 |
| cdent | bbl | 14:46 |
| *** cdent has quit IRC | 14:46 | |
| *** Sundar has joined #openstack-placement | 14:47 | |
| *** e0ne has quit IRC | 15:08 | |
| *** e0ne has joined #openstack-placement | 15:11 | |
| *** e0ne has quit IRC | 15:14 | |
| *** e0ne has joined #openstack-placement | 15:15 | |
| *** ttsiouts has joined #openstack-placement | 15:20 | |
| *** cdent has joined #openstack-placement | 15:33 | |
| *** wwriverrat has quit IRC | 15:40 | |
| *** helenafm has quit IRC | 15:43 | |
| *** ttsiouts has quit IRC | 15:54 | |
| *** e0ne has quit IRC | 16:04 | |
| *** cdent has quit IRC | 16:25 | |
| *** tssurya has quit IRC | 16:44 | |
| *** ttsiouts has joined #openstack-placement | 16:56 | |
| *** ttsiouts has quit IRC | 17:30 | |
| *** e0ne has joined #openstack-placement | 17:54 | |
| *** ttsiouts has joined #openstack-placement | 17:57 | |
| *** ttsiouts has quit IRC | 18:02 | |
| *** finucannot is now known as stephenfin | 18:05 | |
| *** e0ne has quit IRC | 18:26 | |
| *** ttsiouts has joined #openstack-placement | 18:29 | |
| *** e0ne has joined #openstack-placement | 18:31 | |
| *** Sundar has quit IRC | 18:43 | |
| *** ttsiouts has quit IRC | 19:03 | |
| openstackgerrit | Merged openstack/placement master: Trivial: Fix comment for LEFT join https://review.opendev.org/661182 | 19:13 |
| openstackgerrit | Merged openstack/placement master: Don't run functional.db tests in nova functional run https://review.opendev.org/661235 | 19:14 |
| *** ttsiouts has joined #openstack-placement | 20:10 | |
| *** aspiers has quit IRC | 20:38 | |
| *** amodi has quit IRC | 20:38 | |
| *** ttsiouts has quit IRC | 20:43 | |
| *** aspiers has joined #openstack-placement | 20:48 | |
| *** mriedem has quit IRC | 21:13 | |
| *** ttsiouts has joined #openstack-placement | 21:13 | |
| *** ttsiouts has quit IRC | 21:17 | |
| *** e0ne has quit IRC | 21:40 | |
| *** ttsiouts has joined #openstack-placement | 21:42 | |
| openstackgerrit | Eric Fried proposed openstack/placement master: Add NUMANetworkFixture for gabbits https://review.opendev.org/657463 | 22:40 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Gabbi test cases for can_split https://review.opendev.org/658192 | 22:40 |
| *** ttsiouts has quit IRC | 23:53 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!