| *** efried has quit IRC | 01:45 | |
| openstackgerrit | Merged openstack/placement master: Remove a redundant test https://review.opendev.org/665463 | 06:06 |
|---|---|---|
| *** helenafm has joined #openstack-placement | 07:23 | |
| *** ttsiouts has joined #openstack-placement | 07:53 | |
| *** takashin has left #openstack-placement | 08:02 | |
| *** ttsiouts has quit IRC | 08:14 | |
| *** ttsiouts has joined #openstack-placement | 08:15 | |
| *** ttsiouts has quit IRC | 08:19 | |
| *** ttsiouts has joined #openstack-placement | 08:20 | |
| *** tssurya has joined #openstack-placement | 08:44 | |
| *** tssurya has quit IRC | 08:55 | |
| *** cdent has joined #openstack-placement | 09:35 | |
| *** finucannot is now known as stephenfin | 09:36 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Uniquify allocation mappings https://review.opendev.org/665638 | 10:19 |
| *** ttsiouts has quit IRC | 11:08 | |
| *** ttsiouts has joined #openstack-placement | 11:09 | |
| *** ttsiouts has quit IRC | 11:13 | |
| *** ttsiouts has joined #openstack-placement | 11:39 | |
| *** edleafe has joined #openstack-placement | 12:02 | |
| *** mriedem has joined #openstack-placement | 13:19 | |
| cdent | gibi: a light but was found in the mappings stuff, should be fixed at https://review.opendev.org/#/c/665638/ | 13:35 |
| gibi | cdent: ack. I have to go back to the nested magic discussion as well | 13:35 |
| * gibi building a todo list :D | 13:35 | |
| *** efried has joined #openstack-placement | 13:39 | |
| efried | o/ from coffee shop | 13:39 |
| efried | cdent: Is there anything we can talk about that will make you less 'if we make this go' and more 'merge this'? | 13:41 |
| efried | other than tweaks/cleanups as noted | 13:41 |
| efried | which I will get to work on now | 13:41 |
| cdent | the "if we make this go" is just to make clear that this isn't a fait accompli, we need more input from at least gibi and tetsuro | 13:41 |
| efried | tetsuro seemed on board with the latest tweaks to definitions/terms. | 13:44 |
| efried | that's what I gathered from his most recent comments on the code & spec anyway | 13:44 |
| cdent | if you're most intersted in assuaging my anxieties, the thing to do is make sure that I don't forget (every time I look) why we need this particular thing. by which I mean not telling _me_ but telling the commit | 13:48 |
| efried | ack | 13:49 |
| efried | cdent: Is there a way to phrase "So I land on a *host* that can FOO (or not FOO)" in a placement-ish (as opposed to nova-ish) way that isn't too mealy? | 13:50 |
| *** belmoreira has joined #openstack-placement | 13:50 | |
| cdent | are you looking for a substitute for "host"? | 13:50 |
| efried | yeah | 13:50 |
| efried | cause I thought you said something about the way it was currently phrased being a solution rather than a use case | 13:51 |
| efried | --^ is definitely a use case | 13:51 |
| efried | but it's a nova use case. | 13:51 |
| cdent | it's not a nova use case, it's a solution to a nova use case | 13:52 |
| cdent | the question is why "do I need to express queries _this_ way, as opposed to the previous ways I made queries, and why do I need to make that query at all". I want to land on host that has FOO is a query. Why that query? | 13:53 |
| cdent | It helps to contextualize the solution in the real world to make sure that we are thinking about it the real way | 13:53 |
| cdent | It's _also_ useful for us to be able to say "we have trees with roots in placement and we need to be able to select by traits on the roots of those trees" as a general statement of functionality that ought to be possible. So: this general thing can be used to do ________ | 13:54 |
| efried | ...model my resource provider root as being able to FOO and then place my resource consumer on a tree with such a root, without necessarily wanting resources from that root itself | 13:58 |
| efried | cdent: I may not be understanding the distinction. | 14:01 |
| efried | Without getting more specific: "Land me on a host that can MULTI_ATTACH" or "Land me on a host not reserved for WINDOWS_LICENSE" | 14:02 |
| cdent | efried: A use case is something an operator does: As an operator I want to __________ | 14:02 |
| cdent | use cases _are_ specific | 14:02 |
| efried | okay, so --^ | 14:03 |
| efried | augmented with: | 14:03 |
| efried | ...without having to know any more about the tree model than "host-level traits are always put on the root provider" | 14:03 |
| cdent | and where you need the specificity here, is the situations in which required doesn't work for that | 14:03 |
| cdent | which is when inventory is all somewhere else | 14:03 |
| efried | right, that last clause is the clincher | 14:03 |
| cdent | which is NUMA | 14:03 |
| * cdent nods | 14:03 | |
| *** e0ne has joined #openstack-placement | 14:21 | |
| *** belmoreira has quit IRC | 14:24 | |
| *** e0ne has quit IRC | 14:26 | |
| *** belmoreira has joined #openstack-placement | 14:28 | |
| gibi | cdent, efried: the new request-wide-required key works for me | 14:29 |
| efried | thanks gibi | 14:29 |
| gibi | efried: thanks for your perseverance | 14:30 |
| efried | likewise | 14:30 |
| openstackgerrit | Eric Fried proposed openstack/placement master: research_context._get_roots_with_traits() https://review.opendev.org/665491 | 14:32 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Refactor anchors_for_sharing_providers https://review.opendev.org/665561 | 14:32 |
| openstackgerrit | Eric Fried proposed openstack/placement master: WIP: Microversion 1.35: root_required https://review.opendev.org/665492 | 14:32 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Miscellaneous doc/comment/log cleanups https://review.opendev.org/665691 | 14:32 |
| efried | cdent: rebased, split out cleanups. Just saving my place cause I'm about to do the (harder) split of RequestWideParams | 14:32 |
| efried | so no need to review yet (unless you want to look at the cleanups split) | 14:33 |
| cdent | in the midst of starting on a nested perfload, so will catch up when it is more later | 14:33 |
| efried | ack | 14:33 |
| *** e0ne has joined #openstack-placement | 14:36 | |
| *** belmoreira has quit IRC | 14:39 | |
| *** belmoreira has joined #openstack-placement | 14:44 | |
| *** dklyle has joined #openstack-placement | 14:55 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: WIP: Nested performance testing https://review.opendev.org/665695 | 14:56 |
| openstackgerrit | Chris Dent proposed openstack/placement master: WIP: Nested performance testing https://review.opendev.org/665695 | 14:56 |
| openstackgerrit | Chris Dent proposed openstack/placement master: WIP: Nested performance testing https://review.opendev.org/665695 | 14:57 |
| openstackgerrit | Chris Dent proposed openstack/placement master: WIP: Nested performance testing https://review.opendev.org/665695 | 14:59 |
| cdent | efried: I've got family stuff to do now, but will be catching up before going to bed, will lay in some more commentary then, but if gibi and tetsuro are basically on board, I think we're likely good | 15:01 |
| efried | ack, thx cdent | 15:03 |
| *** cdent has quit IRC | 15:09 | |
| *** e0ne has quit IRC | 15:20 | |
| openstackgerrit | Eric Fried proposed openstack/placement master: WIP: Microversion 1.35: root_required https://review.opendev.org/665492 | 15:26 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Miscellaneous doc/comment/log cleanups https://review.opendev.org/665691 | 15:26 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Introduce RequestWideParams https://review.opendev.org/665712 | 15:26 |
| *** belmoreira has quit IRC | 15:28 | |
| *** temka is now known as artom | 15:35 | |
| *** helenafm has quit IRC | 15:47 | |
| *** ttsiouts has quit IRC | 15:54 | |
| *** ttsiouts has joined #openstack-placement | 15:55 | |
| *** ttsiouts has quit IRC | 15:59 | |
| openstackgerrit | Merged openstack/placement master: Uniquify allocation mappings https://review.opendev.org/665638 | 16:26 |
| *** artom has quit IRC | 17:26 | |
| openstackgerrit | Eric Fried proposed openstack/placement master: research_context._get_roots_with_traits() https://review.opendev.org/665491 | 17:46 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Refactor anchors_for_sharing_providers https://review.opendev.org/665561 | 17:46 |
| openstackgerrit | Eric Fried proposed openstack/placement master: RequestWideParams and RequestWideSearchContext https://review.opendev.org/665712 | 17:46 |
| openstackgerrit | Eric Fried proposed openstack/placement master: WIP: Microversion 1.35: root_required https://review.opendev.org/665492 | 17:46 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Miscellaneous doc/comment/log cleanups https://review.opendev.org/665691 | 17:46 |
| *** alex_xu has quit IRC | 17:54 | |
| *** alex_xu has joined #openstack-placement | 18:02 | |
| *** bauzas has quit IRC | 18:21 | |
| *** bauzas has joined #openstack-placement | 18:27 | |
| *** artom has joined #openstack-placement | 18:32 | |
| *** efried has quit IRC | 19:19 | |
| *** cdent has joined #openstack-placement | 20:01 | |
| *** e0ne has joined #openstack-placement | 20:07 | |
| *** e0ne has quit IRC | 20:07 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: WIP: Nested performance testing https://review.opendev.org/665695 | 20:10 |
| *** artom has quit IRC | 20:59 | |
| *** Sundar has joined #openstack-placement | 21:16 | |
| *** efried has joined #openstack-placement | 21:19 | |
| efried | cdent: I'm not ignoring all those comments - I'm working on them in stages | 21:26 |
| efried | Posting new change sets when I do something pretty complicated to the stack - like the split-outs - where the likelihood of me f'ing them up locally and having to start over is (historically speaking) high. | 21:27 |
| cdent | acking them in the comments (with plans) helps people who are following along from the sides (including me) not have to read your mind | 21:27 |
| efried | I thought I mentioned somewhere that that's what I was up to; anyway, will do. | 21:27 |
| cdent | it also avoids me redudantly wondering "did he see that"?" | 21:27 |
| efried | Also there was no internet at the DMV... | 21:27 |
| cdent | bummer | 21:28 |
| efried | but I'm back home now, so things should return to relatively normal. | 21:28 |
| cdent | any other damage besides power? | 21:28 |
| efried | fridge | 21:28 |
| efried | as in, everything in fridge needs to be chucked and replaced. | 21:28 |
| efried | Power was out for like 18h | 21:28 |
| efried | ...yeah, I have like a dozen draft comments on the WIP patch that I didn't get a chance to post yet | 21:29 |
| cdent | aight | 21:29 |
| efried | cdent: biggest thing in the works is the user doc, which I'll post in the next few minutes | 21:30 |
| efried | Still need to finish test cases | 21:30 |
| cdent | i'll be abed very soon | 21:30 |
| cdent | but will look at it all again 1st thing in the morn | 21:30 |
| efried | coo | 21:37 |
| *** cdent has quit IRC | 22:04 | |
| *** ChanServ has quit IRC | 22:09 | |
| *** ChanServ has joined #openstack-placement | 22:14 | |
| *** niven.freenode.net sets mode: +o ChanServ | 22:14 | |
| *** mriedem has quit IRC | 22:27 | |
| openstackgerrit | Eric Fried proposed openstack/placement master: RequestWideParams and RequestWideSearchContext https://review.opendev.org/665712 | 22:43 |
| openstackgerrit | Eric Fried proposed openstack/placement master: WIP: Microversion 1.35: root_required https://review.opendev.org/665492 | 22:43 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Miscellaneous doc/comment/log cleanups https://review.opendev.org/665691 | 22:43 |
| *** artom has joined #openstack-placement | 23:43 | |
| openstackgerrit | sean mooney proposed openstack/os-traits master: add libvirt image metadata traits https://review.opendev.org/652996 | 23:51 |
| openstackgerrit | sean mooney proposed openstack/os-traits master: add GOP, NONE and VIRTIO grapics models https://review.opendev.org/665830 | 23:51 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!