| *** ttsiouts has quit IRC | 00:04 | |
| *** ttsiouts has joined #openstack-placement | 00:05 | |
| *** ttsiouts has quit IRC | 00:09 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/placement master: Adds check for duplicate alloc_cands https://review.openstack.org/638936 | 02:56 |
|---|---|---|
| openstackgerrit | Tetsuro Nakamura proposed openstack/placement master: in_tree alloc_cands support with microversion 1.31 https://review.openstack.org/635723 | 03:52 |
| *** bhagyashris has joined #openstack-placement | 05:29 | |
| *** tetsuro has joined #openstack-placement | 05:56 | |
| *** ttsiouts has joined #openstack-placement | 07:29 | |
| *** belmoreira has joined #openstack-placement | 07:40 | |
| *** ttsiouts has quit IRC | 07:52 | |
| *** ttsiouts has joined #openstack-placement | 07:52 | |
| *** ttsiouts has quit IRC | 07:57 | |
| *** takashin has left #openstack-placement | 07:59 | |
| *** helenafm has joined #openstack-placement | 08:36 | |
| *** rubasov has joined #openstack-placement | 08:40 | |
| *** tssurya has joined #openstack-placement | 08:48 | |
| *** e0ne has joined #openstack-placement | 08:51 | |
| *** giblet is now known as gibi | 08:55 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Update alloc-candidates-in-tree https://review.openstack.org/639033 | 08:56 |
| *** ttsiouts has joined #openstack-placement | 09:00 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/placement master: in_tree alloc_cands support with microversion 1.31 https://review.openstack.org/635723 | 09:07 |
| *** tetsuro has quit IRC | 09:07 | |
| *** cdent has joined #openstack-placement | 09:47 | |
| *** cdent has quit IRC | 09:56 | |
| *** bhagyashris has quit IRC | 10:02 | |
| *** cdent has joined #openstack-placement | 10:43 | |
| *** ttsiouts has quit IRC | 11:23 | |
| *** ttsiouts has joined #openstack-placement | 11:36 | |
| *** finucannot is now known as stephenfin | 11:40 | |
| *** cdent has quit IRC | 12:17 | |
| *** ttsiouts has quit IRC | 12:31 | |
| *** ttsiouts has joined #openstack-placement | 12:32 | |
| *** cdent has joined #openstack-placement | 12:34 | |
| *** ttsiouts has quit IRC | 12:37 | |
| cdent | If we want normalize_name in os-resource-classes we need to merge it asap: https://review.openstack.org/#/c/634258/ | 12:47 |
| *** e0ne has quit IRC | 12:47 | |
| *** ttsiouts has joined #openstack-placement | 12:48 | |
| cdent | jaypipes, fried_rice ^ | 13:26 |
| *** e0ne has joined #openstack-placement | 13:35 | |
| jaypipes | cdent: done | 13:36 |
| cdent | thanks | 13:36 |
| cdent | I'll do a release before thursay | 13:36 |
| openstackgerrit | Merged openstack/os-resource-classes master: Add normalize_name utility https://review.openstack.org/634258 | 13:45 |
| cdent | fried_rice: you in the house? | 13:58 |
| cdent | scheduler placement meeting in #openstack-meeting-alt now ish | 13:59 |
| *** fried_rice is now known as efried | 14:04 | |
| *** ttsiouts has quit IRC | 14:04 | |
| *** ttsiouts has joined #openstack-placement | 14:05 | |
| *** ttsiouts has quit IRC | 14:06 | |
| *** ttsiouts has joined #openstack-placement | 14:06 | |
| openstackgerrit | Eric Fried proposed openstack/placement master: Add DISK_GB to compute in SharedStorageFixture https://review.openstack.org/638929 | 14:43 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Prepare in_tree allocation candidates https://review.openstack.org/635722 | 14:43 |
| openstackgerrit | Eric Fried proposed openstack/placement master: in_tree alloc_cands support with microversion 1.31 https://review.openstack.org/635723 | 14:43 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Add second shared provider to SharedStorageFixture https://review.openstack.org/638712 | 14:43 |
| efried | cdent: I'm happy with this series ^ | 14:46 |
| cdent | roger, moving location, but will go back to it when home | 14:46 |
| efried | jaypipes: you may want to have a look also ^ | 14:46 |
| *** mriedem has joined #openstack-placement | 14:47 | |
| efried | (It's still tetsuro's; I just fixed a typo and rebased) | 14:47 |
| * efried fuels | 14:47 | |
| *** cdent has quit IRC | 14:53 | |
| *** e0ne has quit IRC | 15:03 | |
| *** e0ne has joined #openstack-placement | 15:05 | |
| jaypipes | efried: will try. | 15:32 |
| efried | jaypipes: If you're overloaded, you can probably leave it alone and cdent will get it. But it's a pretty small and simple change, should be kind of fun to review if you need a "break" from more challenging things :) | 15:40 |
| jaypipes | efried: ya, I'll try. gibi's bw patches are top prio right now | 15:45 |
| *** ttsiouts has quit IRC | 16:11 | |
| *** ttsiouts has joined #openstack-placement | 16:12 | |
| *** ttsiouts has quit IRC | 16:16 | |
| *** cdent has joined #openstack-placement | 16:18 | |
| *** e0ne has quit IRC | 16:26 | |
| *** e0ne has joined #openstack-placement | 16:31 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add 'flavor-extra-spec-image-property-validation' spec https://review.openstack.org/638734 | 16:40 |
| *** dklyle_ has quit IRC | 16:40 | |
| *** helenafm has quit IRC | 16:44 | |
| *** ttsiouts has joined #openstack-placement | 16:51 | |
| *** dklyle has joined #openstack-placement | 16:55 | |
| *** ttsiouts has quit IRC | 16:59 | |
| *** ttsiouts has joined #openstack-placement | 16:59 | |
| *** ttsiouts has quit IRC | 17:04 | |
| *** e0ne has quit IRC | 17:18 | |
| *** IvensZambrano has joined #openstack-placement | 17:26 | |
| openstackgerrit | Eric Fried proposed openstack/placement master: Fix a bad granular gabbi test https://review.openstack.org/639168 | 18:12 |
| efried | cdent: interesting thing I found while doing something unrelated ^ | 18:12 |
| cdent | hmmm. yeah | 18:13 |
| efried | It would be nice if duplicate keys were detected and warned somehow (and warnings made errors). But I suspect the duplicate keys are folded together by the yaml parser, so gabbi might not be able to help us here. | 18:13 |
| cdent | like I said in my response on one of tetsuro's patches: I think we are likely to find warts and sundry | 18:13 |
| cdent | yeah, it's yaml | 18:13 |
| efried | perhaps there's a 'strict' yaml parser option? | 18:13 |
| cdent | not yet | 18:13 |
| cdent | there will be soon | 18:13 |
| efried | okay. | 18:13 |
| efried | well, in the meantime, there. | 18:14 |
| * cdent does some googling | 18:14 | |
| cdent | efried: yeah, known bug: https://github.com/yaml/pyyaml/issues/165 | 18:15 |
| efried | noyce. I'll add that to the commit message. | 18:16 |
| cdent | the workarounds are more cumbersom than I'd like to put in gabbi given that I ought to be able to pressure the new(ish) maintainer of pyyaml | 18:16 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Fix a bad granular gabbi test https://review.openstack.org/639168 | 18:16 |
| efried | done | 18:16 |
| efried | oh, yeah, that ought to be at least an *option* to catch at the yaml parser levle. | 18:17 |
| efried | level | 18:17 |
| cdent | ah, you know what efried | 18:22 |
| efried | sometimes | 18:22 |
| cdent | query_parameters does support duplicate keys in a sense | 18:22 |
| efried | what? | 18:22 |
| efried | oh? | 18:22 |
| cdent | you ought to be able to do: member_of2: | 18:23 |
| cdent | - $ENVIRON['AGGA'] | 18:23 |
| efried | o | 18:23 |
| * efried tries... | 18:23 | |
| cdent | if it's value is a list, it will do the right thing | 18:23 |
| cdent | I think | 18:23 |
| cdent | which makes sense because the python notion of encoding query params is a dict of lists | 18:24 |
| efried | it won't do ?member_of2=[$agga, in:$aggb,$aggc] ? | 18:24 |
| cdent | (and I have collegue from long ago who would never have let me get away with not supporting it) | 18:24 |
| cdent | try it, should be quick, right? | 18:25 |
| efried | Yeah. works. | 18:25 |
| efried | thanks cdent, will update the patch, cause that's prettier. | 18:25 |
| * cdent nods | 18:25 | |
| cdent | heh, it's even in the docs: (search for query_parameters) https://gabbi.readthedocs.io/en/latest/example.html | 18:26 |
| * cdent has a _great_ memory | 18:26 | |
| cdent | not | 18:26 |
| cdent | efried: note the "not necessarily in that order" if that matters to you | 18:27 |
| efried | it matters that the order shouldn't matter, so it would be nice if it were deterministic, but I don't think I care enough to make it so atm. | 18:28 |
| efried | because if it were deterministic, I should really dup the test case with them in both orders. | 18:28 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Optionally migrate database at service startup https://review.openstack.org/619050 | 18:29 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Fix a bad granular gabbi test https://review.openstack.org/639168 | 18:31 |
| efried | cdent: done, much better, thank you. | 18:31 |
| cdent | yay! | 18:31 |
| efried | cdent: "The other option would be to generation two different clauses and then only and them once the where is called." -- Absent typeface distinctions, "and then only and them once the where is called" would be pretty cryptic to the uninitiated :) | 18:36 |
| openstackgerrit | Merged openstack/placement master: Add DISK_GB to compute in SharedStorageFixture https://review.openstack.org/638929 | 18:36 |
| cdent | indeed | 18:37 |
| efried | cdent: Do you want me to fix https://review.openstack.org/#/c/635722/ or wait for tetsuro? | 18:38 |
| cdent | no need to wait, I thought about doing it myself, but was trying to do a full run through of outstanding patches, so if you're there, do it | 18:39 |
| cdent | jaypipes, mriedem : easy clean up needs another +2 : https://review.openstack.org/#/c/636221/ | 18:40 |
| cdent | and https://review.openstack.org/#/c/636236/ | 18:40 |
| *** IvensZambrano has quit IRC | 18:42 | |
| *** e0ne has joined #openstack-placement | 18:44 | |
| cdent | efried: I'm struggling to re-convince myself that the gabbi integration tests started at https://review.openstack.org/#/c/601614/ are worth it. Could you please reflect back to me why you think it is a good idea? | 18:46 |
| efried | cdent: You had me convinced when you said they weren't strictly redundant with tempest-full | 18:47 |
| efried | ...which I agree we should be running. | 18:47 |
| efried | Let me put it this way: The coverage might be redundant. But the ability to write these tests in gabbi is super useful IMO. | 18:47 |
| *** tssurya has quit IRC | 18:47 | |
| efried | so it seems a shame to throw away this work just because we haven't *yet* written any tests that are unique to the gabbi framework. | 18:48 |
| efried | I'm not going to go to war for it though. | 18:48 |
| * cdent hopes no one goes to war | 18:48 | |
| cdent | thanks | 18:49 |
| cdent | i reckon that's sufficient | 18:49 |
| * cdent will do it after dinner | 18:52 | |
| openstackgerrit | Eric Fried proposed openstack/placement master: Prepare in_tree allocation candidates https://review.openstack.org/635722 | 19:01 |
| openstackgerrit | Eric Fried proposed openstack/placement master: in_tree[N] alloc_cands with microversion 1.31 https://review.openstack.org/635723 | 19:01 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Add second shared provider to SharedStorageFixture https://review.openstack.org/638712 | 19:01 |
| efried | cdent: ^ done | 19:02 |
| *** IvensZambrano has joined #openstack-placement | 19:40 | |
| melwitt | efried: re: bug smash, I think it's a fine idea. don't feel you need to wait for me to organize it, I'd be happy if you wanted to. but if you'd like me to organize it, I can. just let me know | 19:59 |
| openstackgerrit | Merged openstack/placement master: Adds tests for granular single shared rp request https://review.openstack.org/638935 | 20:04 |
| openstackgerrit | Merged openstack/placement master: Adds check for duplicate alloc_cands https://review.openstack.org/638936 | 20:04 |
| efried | melwitt: It was cdent's idea! Mea non culpa! | 20:12 |
| melwitt | heh | 20:13 |
| efried | I'll talk to cdent and we'll figure something out. I think a bug smash is appropriate regardless of what we end up doing for bug tracking, but I wanted to be able to use the excuse of waiting until that was sorted... | 20:14 |
| *** IvensZambrano has quit IRC | 20:15 | |
| melwitt | :) makes sense. sounds good | 20:15 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Optionally migrate database at service startup https://review.openstack.org/619050 | 20:19 |
| cdent | melwitt, efried : bah! It wasn't my idea. I never used the term "bug smash" that was efried! | 20:20 |
| * cdent passes the buck another time | 20:20 | |
| melwitt | haha | 20:21 |
| cdent | efried: in_tree +W up to your added test | 20:30 |
| efried | cdent: woot, thx | 20:30 |
| efried | I guess the added test needs another reviewer. | 20:31 |
| efried | Let's see, who spoke last? | 20:31 |
| efried | melwitt! | 20:31 |
| melwitt | lol | 20:32 |
| efried | kidding, it's not really your wheelhouse, but if you feel like it: https://review.openstack.org/#/c/638712 | 20:32 |
| efried | just don't retaliate by asking me to review something quota. | 20:32 |
| melwitt | heh | 20:38 |
| cdent | efried: just to be clear on the dependency idea | 20:57 |
| cdent | all the short jobs would still run in parallel | 20:57 |
| cdent | but you'd know within 5 minutes of a failure | 20:58 |
| cdent | rather than needing to wait on grenade and tempest | 20:58 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Initial integration test using gabbits https://review.openstack.org/601614 | 21:05 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Use gabbi-tempest job from gabbi-tempest plugin https://review.openstack.org/607508 | 21:05 |
| cdent | efried: I believe that ^ gets some of the concerns mentioned, but it probably ought to wait until it runs through the gate as I tweaked some things for clarity and haven't been able to test it | 21:06 |
| efried | cdent: It was my understanding that you were saying "wait to start long jobs; if short jobs fail, bail out and don't start the long jobs at all" | 21:06 |
| cdent | that's correct, yes, but "short jobs all run at the same time" | 21:07 |
| efried | like they do today | 21:07 |
| cdent | yes | 21:07 |
| efried | dandy | 21:07 |
| cdent | the expectation is that if a unit or functional fails, then grenade and tempest will too | 21:07 |
| cdent | so why wait | 21:07 |
| efried | oh, because a) not necessarily - in fact quite often it's just the test that's broken; and b) the failure will manifest in quite different ways that aren't unit tested, which is why we need them. The latter is the not-wanting-to-iterate thing. | 21:09 |
| melwitt | yeah, I must confess I'm guilty of working on my first draft of patches that fail unit/func tests but pass tempest, letting zuul verify that things are working | 21:10 |
| cdent | If a) then the result is the same: it's still a wasted grenade and tempest run, which costs node time and your time. | 21:10 |
| cdent | melwitt: yeah, that makes quite a lot of sense in nova where there are many pieces | 21:10 |
| melwitt | instead of spending time tweaking unit/func tests between each iteration. I could have run tempest locally with devstack to verify things instead but I didn't | 21:11 |
| cdent | but in placement, where it is basically just the api, and the functional tests cover that whole thing, functional tests end up being the "am I doing it right" filter | 21:11 |
| mriedem | i *never* run tempest locally | 21:11 |
| melwitt | yeah, I was thinking in placement-land might not be nearly as compelling to do that way | 21:11 |
| efried | Yeah, so running tempest locally - or even running full tox - is a nonstarter for me. My laptop melts through my desk. | 21:11 |
| cdent | efried: in nova, sure | 21:11 |
| efried | (well, full tox in placement is okay) | 21:11 |
| efried | yeah. | 21:11 |
| cdent | this is why I didn't put [all] in the subject line | 21:12 |
| melwitt | yeah, makes sense | 21:12 |
| cdent | placement is fairly unique in this regard | 21:12 |
| efried | yeah, in placement there's not really an excuse for pushing up changes that fail py/func | 21:12 |
| cdent | it has tests that are both very complete and very fast | 21:12 |
| cdent | (only "complete" becuase there's not that much there to complete) | 21:12 |
| melwitt | yeah, that's fair | 21:13 |
| openstackgerrit | Merged openstack/placement master: tox: Don't write byte code (maybe) https://review.openstack.org/636995 | 21:13 |
| openstackgerrit | Merged openstack/placement master: Use tox 3.1.1 fixes https://review.openstack.org/636997 | 21:13 |
| cdent | (not because we are like supercoders (ahoy!) or something) | 21:13 |
| melwitt | :) | 21:13 |
| * cdent makes yet another tshirt | 21:13 | |
| cdent | mriedem: if you're conscious you want to get https://review.openstack.org/#/c/638712 if you're feeling it? Clears out that stack | 21:14 |
| mriedem | maybe | 21:18 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Test for multiple limit/group_policy qparams https://review.openstack.org/636221 | 21:21 |
| *** e0ne has quit IRC | 21:21 | |
| efried | jaypipes, cdent: Easy manual rebase on ^ if you want to re+A quick. | 21:22 |
| cdent | yeah, just confirming it | 21:22 |
| cdent | I think for my next trick I need a local pypi mirror | 21:24 |
| mriedem | cdent: efried: comments/questions in https://review.openstack.org/#/c/638712/ | 21:33 |
| mriedem | clearly i'm late to the GET /a_c?in_tree party | 21:33 |
| cdent | those are why you get invited | 21:35 |
| mriedem | well i think i figured out my confusion in the test | 21:36 |
| mriedem | numbered vs unnumbered request groups with numbered in_tree filters | 21:36 |
| cdent | the numbered request stuff causes rather an explosion of confusion | 21:37 |
| mriedem | yeah, hence https://review.openstack.org/#/c/635723/9/api-ref/source/parameters.yaml@70 | 21:38 |
| cdent | My expectation would be that that comes later, when something is actually using this stuff. Much of placement is out of sync with the actual using reality | 21:39 |
| efried | mriedem: responded. Do you want me to fix those things here or throw a fup? | 21:40 |
| mriedem | efried: i'm getting really tired of tracking follow ups at this point :) | 21:40 |
| cdent | Only later because then it could be done in a concrete fashion | 21:40 |
| efried | okay, will respin, stand by. | 21:40 |
| mriedem | i'm not sure what concrete fashion has to do with later | 21:40 |
| mriedem | seems like a chicken and egg issue to me, | 21:41 |
| mriedem | nova is going to want to use the in_tree thing for stuff like forced host scheduling | 21:41 |
| mriedem | but nova, as the api user/client, needs to now how the api works | 21:41 |
| cdent | sure | 21:41 |
| mriedem | without digging into code and gabbits... | 21:41 |
| mriedem | just playing devil's advocate and saying i'm a new contributor and all | 21:41 |
| cdent | I hear ya | 21:42 |
| mriedem | i guess granular in_tree is going to be more of a cyborg thing | 21:42 |
| mriedem | to which i'll happily punt for now | 21:42 |
| cdent | probably. and at this point even reading the phrase "granular in_tree" makes me go "yeah, okay, mmm hmm" and want to wait and see how it progress before trying to evaluate documentation about it | 21:43 |
| cdent | but on the flip side of that: if tetsuro is coming along with fixes for that stuff, the last thing we want to do is discourage them | 21:43 |
| efried | mriedem: I don't agree that it's just a cyborg thing. If we ever want to e.g. migrate an instance with shared storage while the storage stays put, we'll want to say "give me allocation candidates where VCPU/MEMORY_MB are anywhere, and DISK_GB stays in SS1". | 21:45 |
| efried | mriedem: I mean, yeah, I guess we could force the VCPU/MEMORY_MB resources into a numbered group so the shared storage could be wedged into the unnumbered group so we wouldn't have to use in_treeN. But that would suck worse than using in_treeN. And what if multiple shared storages? | 21:46 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Initial integration test using gabbits https://review.openstack.org/601614 | 21:50 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Use gabbi-tempest job from gabbi-tempest plugin https://review.openstack.org/607508 | 21:50 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Add second shared provider to SharedStorageFixture https://review.openstack.org/638712 | 21:52 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Test for multiple limit/group_policy qparams https://review.openstack.org/636221 | 21:52 |
| efried | mriedem: ^ | 21:52 |
| efried | cdent: in case you want another look | 21:52 |
| *** e0ne has joined #openstack-placement | 21:54 | |
| *** e0ne has quit IRC | 21:55 | |
| *** takashin has joined #openstack-placement | 21:56 | |
| *** e0ne has joined #openstack-placement | 22:05 | |
| *** e0ne has quit IRC | 22:08 | |
| *** s10 has joined #openstack-placement | 22:11 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove redundant second cast to int https://review.openstack.org/639203 | 22:12 |
| efried | cdent: wanna re+A https://review.openstack.org/#/c/638712/ and https://review.openstack.org/#/c/636221/ before you go to bed (which you should do soon)? | 22:28 |
| cdent | i'm looking at that now, gonna put a -.5 on it | 22:28 |
| *** avolkov has quit IRC | 22:30 | |
| efried | one step forward, two steps back | 22:39 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Add second shared provider to SharedStorageFixture https://review.openstack.org/638712 | 22:40 |
| openstackgerrit | Eric Fried proposed openstack/placement master: Test for multiple limit/group_policy qparams https://review.openstack.org/636221 | 22:40 |
| efried | cdent: ^ | 22:40 |
| cdent | ayup | 22:40 |
| mriedem | doh i knew i should have -1ed over that punctuation | 22:47 |
| cdent | efried: here we go. those integrated gabbits have just revealed a real bug that is tested by neither standard tempest, nor the functional tests. you win. | 22:50 |
| cdent | you can also have a does of "you're right" because the issue is with another instance of func.sum | 22:50 |
| cdent | (which suggests that when those merge, we should add a full spectrum run of the api as a gate/gabbit ) | 22:54 |
| efried | cdent: I love it when a plan comes together. | 22:58 |
| * cdent lights efried's cigar | 22:58 | |
| efried | funny, these days "Hannibal" brings Anthony Hopkins to mind. | 22:58 |
| * cdent pours the chianti | 23:01 | |
| openstackgerrit | Chris Dent proposed openstack/placement master: Cast Usage.usage to int https://review.openstack.org/639216 | 23:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Initial integration test using gabbits https://review.openstack.org/601614 | 23:09 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Use gabbi-tempest job from gabbi-tempest plugin https://review.openstack.org/607508 | 23:09 |
| cdent | efried: if we're in the business tonight of crushing pending reviews: three more on https://review.openstack.org/#/q/status:open+project:openstack/placement+branch:master+topic:cd/less-ovo | 23:13 |
| cdent | all have jay's +2 | 23:13 |
| efried | cdent: ack, getting there, but it may not be tonight. | 23:13 |
| efried | Need to rebuild my janky kitchen | 23:14 |
| * cdent wants to wake up tomorrow to all the placement code merged | 23:14 | |
| cdent | did you unbuild it? | 23:14 |
| efried | getting new countertops. When we removed the old ones, one cabinet disintegrated. And today when they installed, another one was found to be about an inch lower than its counterpart (heh) on the other side of the dishwasher, so much shimming was necessary. We have to go make that permanent. | 23:16 |
| cdent | good luck | 23:16 |
| efried | yeah, thanks. Loathe carpentry. | 23:16 |
| cdent | we're blowing out the back of the house later this year (if all goes to plan). that part of the house is the kitchen. I expect to find bad mojo lurking. | 23:17 |
| *** cdent has left #openstack-placement | 23:19 | |
| *** cdent has joined #openstack-placement | 23:19 | |
| *** s10 has quit IRC | 23:34 | |
| efried | cdent: https://review.openstack.org/#/c/636807/ has some things that could be tweaked, but you can do 'em in a separate patch if you like. lmk if you want me to +W. | 23:43 |
| cdent | I'll fix those in that, the _updates one is especially ick | 23:44 |
| efried | ight | 23:45 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Don't use OVO with ResourceProvider and ResourceProviderList https://review.openstack.org/636807 | 23:49 |
| cdent | efried: that ^ should do it. I hope | 23:50 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Initial integration test using gabbits https://review.openstack.org/601614 | 23:56 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Use gabbi-tempest job from gabbi-tempest plugin https://review.openstack.org/607508 | 23:56 |
| openstackgerrit | Chris Dent proposed openstack/placement master: Remove redundant second cast to int https://review.openstack.org/639203 | 23:58 |
| cdent | efried: I think you were right about the antibiotics making me new | 23:59 |
| efried | cdent: that's great to hear. | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!