| mattoliverau | clayg: yeah and put them in contraints or something | 00:09 |
|---|---|---|
| *** annegentle has joined #openstack-swift | 00:16 | |
| *** annegentle has quit IRC | 00:32 | |
| clayg | mattoliverau: yeah... | 00:35 |
| *** openstackgerrit has quit IRC | 00:35 | |
| *** openstackgerrit has joined #openstack-swift | 00:35 | |
| mattoliverau | sigh.. looks like due to the gerrit maintenence looks like alot of patches are queued in the check gate and patch 165188 has been paused for over 2 hours not running at all. Guess i'll run tests locally and decide on a score based on that. | 00:46 |
| patchbot | mattoliverau: https://review.openstack.org/#/c/165188/ | 00:46 |
| *** tsg_ has quit IRC | 01:13 | |
| clayg | torgomatic: this is not a good sign - now I managed to get a segfault :\ | 01:24 |
| *** annegentle has joined #openstack-swift | 01:32 | |
| *** openstack has joined #openstack-swift | 01:39 | |
| clayg | anyone know how to get ahold of tsg and kevin? https://gist.github.com/clayg/2860667dc8a7d3a31e73 | 01:52 |
| clayg | I can think of lots of reasonable things that might happen if you run that text through a python inturpreture - segfaulting is not one of them | 01:53 |
| clayg | also be useful if someone could confirm... | 01:53 |
| *** barra204 has joined #openstack-swift | 01:56 | |
| *** shakamunyi has joined #openstack-swift | 01:58 | |
| clayg | torgomatic: didn't you say something about whitespace? i was able to reduce the reprotest case a little bit... | 01:59 |
| *** thumpba_ has joined #openstack-swift | 02:11 | |
| *** jamielennox|away is now known as jamielennox | 02:11 | |
| *** thumpba has quit IRC | 02:14 | |
| *** thumpba_ has quit IRC | 02:16 | |
| *** thumpba has joined #openstack-swift | 02:19 | |
| *** annegentle has joined #openstack-swift | 02:25 | |
| *** annegentle has quit IRC | 02:50 | |
| *** thumpba has quit IRC | 03:03 | |
| openstackgerrit | Clay Gerrard proposed openstack/swift: Fix AttributeError and logging in ECAppIter https://review.openstack.org/168543 | 03:11 |
| openstackgerrit | Clay Gerrard proposed openstack/swift: Add Fragment Index filter support to ssync https://review.openstack.org/165188 | 03:20 |
| openstackgerrit | Clay Gerrard proposed openstack/swift: wip: ec reconstructor probe test https://review.openstack.org/164291 | 03:20 |
| openstackgerrit | Clay Gerrard proposed openstack/swift: Erasure Code Reconstructor https://review.openstack.org/131872 | 03:20 |
| clayg | notmyname: I believe the word should have been "released" | 03:40 |
| clayg | should we make it an alias now - like opposite ways to spell the same thing --verbose=false and --quiet=true | 03:41 |
| clayg | available? | 03:42 |
| *** Akshat has joined #openstack-swift | 03:42 | |
| *** Akshat has quit IRC | 03:45 | |
| mattoliverau | clayg: I'll +A patch 164950 then, then you can make a depend patch if you want. Let's merge the sucker.. unless there is another core around who wants to take a look. | 04:01 |
| patchbot | mattoliverau: https://review.openstack.org/#/c/164950/ | 04:01 |
| mattoliverau | clayg: in my mind, verbose is higher logging level (INFO and DEBUG), whereas quite would mean less (or none, or maybe just ERROR).. but maybe that's just me. | 04:03 |
| mattoliverau | *quiet | 04:03 |
| clayg | yeah that was probably not the best example - there's some syntax in argparse for like store_true and store_false - but you can have them both go into the same thing | 04:20 |
| clayg | so like you have a --on or --off and you can even have a default but you can spell it either way | 04:21 |
| tdasilva | mattoliverau, clayg : re 164950 clayg added a comment that he would change the behavior in server.py to return a 503 instead of returning the default policy, do you know if the plan is to do on this patch or a seperate one? | 04:21 |
| clayg | mattoliverau: i was just thinking about the "deprecated" option being available as "available" as an alias (just opposite) | 04:21 |
| clayg | i thought i just typed git review | 04:21 |
| mattoliverau | clayg: ahh | 04:21 |
| clayg | wtf is that other terminal doing | 04:22 |
| openstackgerrit | Clay Gerrard proposed openstack/swift: Guard against invalid Policy https://review.openstack.org/168548 | 04:22 |
| clayg | tdasilva: ^ opp forgot to type 'yes' | 04:22 |
| clayg | i mean i squash them or whatever | 04:22 |
| clayg | notmyname wasn't around to day so I just decided we're not gunna go all crazy merging stuff - we'll merge what's ready | 04:22 |
| clayg | i'm going to start the rebase for master work - and i'm sure there's some good code in the feature branch somewhere that we can propose to ec-review or whatever when he gets that setup on Monday | 04:23 |
| clayg | hopefully by tuesday or wednesday well have the whole patch chain up | 04:23 |
| tdasilva | ok, cool, sounds good! | 04:23 |
| clayg | paul is going to be disappointed if I don't do any work on the object fixer while he was out | 04:24 |
| clayg | but com'n notmyname and peluse can't both go away the last two days then come back on Monday - and be like - what it's not done yet!? | 04:24 |
| mattoliverau | true | 04:24 |
| tdasilva | lol | 04:24 |
| clayg | that being said - if anyone around between now and monday wants to merge aything on the ec review list that's *great* | 04:25 |
| clayg | I think there's lots of good stuff that's ready to go on - and there's lots of nits and cleanups that can happen in the review against master | 04:25 |
| clayg | we'll have almost two weeks to nit - and I'll be rebasing the whole time - so it'll be fun to torture me! | 04:25 |
| mattoliverau | clayg: want to sqaush the changes then? 168548 i mean? screw it we'll just merge em both.. found a comment that needs to be updated in a test, but we can leave that for someone to get a free ATC ticket to summit :P | 04:26 |
| clayg | mattoliverau: yeah squash, push over, +2 - it's whatever you are feeling brother! | 04:26 |
| tdasilva | clayg: I thought peluse said he would reward you with scotch for your torture | 04:26 |
| tdasilva | lol | 04:27 |
| clayg | yeah that's all ready to go acctually if I can make it through! | 04:27 |
| tdasilva | i think i saw that comment too, little mispelling | 04:27 |
| clayg | what comment - you guys know i can't spell! | 04:28 |
| mattoliverau | I just meant you mention in a test that no policy will return a 400 and then you assert 503 :P | 04:29 |
| tdasilva | pleanty of nodes | 04:29 |
| mattoliverau | tdasilva: lol, i apparently don't read so good :P | 04:29 |
| clayg | damnit - what test!? | 04:31 |
| mattoliverau | I left a comment | 04:31 |
| clayg | oh pfhpphththt | 04:31 |
| clayg | hey it says "should" not "must" | 04:31 |
| mattoliverau | yeah, sorry, I can push it for ya | 04:31 |
| mattoliverau | lol | 04:31 |
| mattoliverau | true, +2 then :P | 04:32 |
| clayg | sure thanks - i'm done | 04:32 |
| clayg | ... for tonight | 04:32 |
| tdasilva | me too....brain is dead...talk to you guys later | 04:32 |
| mattoliverau | clayg: if you +2 then that last one, I'll merge it once jenkins gets back to us successfully. | 04:33 |
| mattoliverau | night both of you.. I'll probably head soon, its getting late afternoon on a Saturday.. need to spend time with the wife :) | 04:33 |
| clayg | i'll +2 anything on feature/ec - even shit i wrote - you don't even nkow | 04:34 |
| mattoliverau | clayg: lol, thanks. Now go sleep, that's for all the hard work so late into the night! | 04:35 |
| notmyname | hello | 04:46 |
| notmyname | clayg: don't go blame not merging code on me not being here ;-) | 04:46 |
| mattoliverau | notmyname: hey hey | 04:46 |
| notmyname | mattoliverau: are you in the middle of an archery range? | 04:46 |
| mattoliverau | clayg: he will and he has :P | 04:46 |
| mattoliverau | notmyname: nope not today, been at my desk :( | 04:47 |
| notmyname | well, let's go see what's landed today | 04:47 |
| notmyname | not much | 04:47 |
| mattoliverau | things seem to be queued up on gerrit too. | 04:49 |
| notmyname | ya, looks like a few are in the gate too | 04:49 |
| mattoliverau | there was some gerrit downtime today for maintenence.. but wonder if that caused the gerrit backlog. | 04:50 |
| notmyname | I'm a little disconcerted by the number of unit test runs | 04:54 |
| notmyname | err...how many failed | 04:54 |
| *** tsg has joined #openstack-swift | 05:01 | |
| notmyname | a lot of ValueError: invalid literal for int() with base 16: '' | 05:14 |
| notmyname | which I thought torgomatic had fixed | 05:14 |
| notmyname | (and everything is passing on my saio | 05:14 |
| openstackgerrit | Merged openstack/swift: Extract EC PUT to ECObjController https://review.openstack.org/164950 | 05:22 |
| *** thumpba has joined #openstack-swift | 05:22 | |
| mattoliverau | Sigh.. clayg's 503 change (patch 168548).. and coused some unit tests to fail, fixed all but 1, but that last one is a little more annoying.. solveable but I need to go out for some errands, then dinner (it is approaching saturday evening.. will take a closer look at this later. | 05:28 |
| patchbot | mattoliverau: https://review.openstack.org/#/c/168548/ | 05:28 |
| notmyname | I am seeing unittest errors locally on https://review.openstack.org/#/c/168543/ but not on https://review.openstack.org/#/c/168185 | 05:34 |
| notmyname | but let me rerun with the current patch that just landed | 05:34 |
| notmyname | bah! merge conflicts | 05:34 |
| notmyname | torgomatic: merge conflicts on https://review.openstack.org/#/c/168185 that look slightly more complex than I want do deal with at this hour | 05:37 |
| *** tsg has quit IRC | 05:42 | |
| *** haigang has joined #openstack-swift | 05:46 | |
| *** haigang has quit IRC | 05:49 | |
| *** thumpba has quit IRC | 06:11 | |
| *** rdaly2 has joined #openstack-swift | 06:17 | |
| *** rdaly2 has quit IRC | 06:21 | |
| *** silor has joined #openstack-swift | 08:36 | |
| *** tsg has joined #openstack-swift | 08:52 | |
| *** tsg has quit IRC | 09:01 | |
| *** welldannit has quit IRC | 09:27 | |
| *** jamielennox is now known as jamielennox|away | 09:56 | |
| *** jasondotstar has quit IRC | 10:37 | |
| *** jasondotstar has joined #openstack-swift | 10:38 | |
| *** rdaly2 has joined #openstack-swift | 11:22 | |
| *** rdaly2 has quit IRC | 11:26 | |
| *** silor has quit IRC | 11:50 | |
| *** remix_tj has quit IRC | 12:21 | |
| *** Gu_______ has joined #openstack-swift | 12:23 | |
| *** remix_tj has joined #openstack-swift | 12:23 | |
| *** rdaly2 has joined #openstack-swift | 12:26 | |
| *** rdaly2 has quit IRC | 12:26 | |
| *** haomaiwang has joined #openstack-swift | 12:59 | |
| *** silor has joined #openstack-swift | 13:14 | |
| *** Gu_______ has quit IRC | 13:51 | |
| *** silor has quit IRC | 14:56 | |
| *** tsg has joined #openstack-swift | 15:16 | |
| openstackgerrit | Samuel Merritt proposed openstack/swift: EC: don't mix different fragment archives https://review.openstack.org/168185 | 16:07 |
| *** thumpba has joined #openstack-swift | 16:21 | |
| *** thumpba has quit IRC | 16:25 | |
| torgomatic | my patch generated a bunch of ValueError: invalid literal for int() with base 16: ''; the fix turned out to be cleaning up error-limits in the proxy so they didn't leak between test runs | 16:40 |
| *** silor has joined #openstack-swift | 16:40 | |
| torgomatic | and by that, I mean a tearDown method in my test case that emptied out the error limits | 16:40 |
| notmyname | ah | 16:40 |
| notmyname | thanks | 16:40 |
| torgomatic | sure | 16:40 |
| notmyname | running tests on it now | 16:40 |
| torgomatic | tox run is going now; after that I'll type "git review" and shut my laptop for the day | 16:41 |
| torgomatic | what you have will fai | 16:41 |
| torgomatic | notmyname: ^^ | 16:41 |
| notmyname | ah ok | 16:41 |
| openstackgerrit | Samuel Merritt proposed openstack/swift: EC: don't mix different fragment archives https://review.openstack.org/168185 | 16:42 |
| torgomatic | and with that, I'm off to prepare for my kids' birthday party | 16:42 |
| notmyname | :-) | 16:43 |
| notmyname | heh. I'll miss "test_crazytown" | 16:43 |
| notmyname | (but the new one is better for maintenance) | 16:44 |
| *** silor has quit IRC | 17:10 | |
| *** jrichli has joined #openstack-swift | 17:57 | |
| *** thumpba has joined #openstack-swift | 19:26 | |
| *** thumpba has quit IRC | 19:31 | |
| *** jamielennox|away is now known as jamielennox | 21:20 | |
| openstackgerrit | Janie Richling proposed openstack/swift: WIP - Provides a simple skeleton of middleware for encryption feature. https://review.openstack.org/157907 | 21:22 |
| *** swat30 has quit IRC | 21:30 | |
| *** peluse has quit IRC | 21:30 | |
| *** swat30 has joined #openstack-swift | 21:31 | |
| *** peluse has joined #openstack-swift | 21:31 | |
| *** jrichli has quit IRC | 21:32 | |
| *** Gues_____ has joined #openstack-swift | 21:44 | |
| *** tsg_ has joined #openstack-swift | 22:35 | |
| *** tsg has quit IRC | 22:38 | |
| *** shakamunyi has quit IRC | 22:49 | |
| *** barra204 has quit IRC | 22:49 | |
| openstackgerrit | Matthew Oliver proposed openstack/swift: Guard against invalid Policy https://review.openstack.org/168548 | 22:54 |
| *** barra204 has joined #openstack-swift | 23:02 | |
| *** shakamunyi has joined #openstack-swift | 23:02 | |
| openstackgerrit | Merged openstack/swift: EC: don't mix different fragment archives https://review.openstack.org/168185 | 23:25 |
| *** tsg_ has quit IRC | 23:32 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!