Friday, 2015-10-02

*** garthb has quit IRC00:02
*** nadeem has quit IRC00:03
*** diogogmt has quit IRC00:03
*** Kennan has quit IRC00:16
*** CR7 has quit IRC00:20
*** chlong has joined #openstack-swift00:21
*** lyrrad has quit IRC00:23
*** bill_az has quit IRC00:25
*** wbhuber has quit IRC00:29
*** km has quit IRC00:29
*** wbhuber has joined #openstack-swift00:32
*** CR7 has joined #openstack-swift00:36
*** CR7 has quit IRC00:40
kota_morning00:40
*** m_kazuhiro has joined #openstack-swift00:48
*** wbhuber has quit IRC00:54
*** sanchitmalhotra has joined #openstack-swift00:55
*** sanchitmalhotra1 has quit IRC00:56
*** gyee has quit IRC00:59
*** ccavanna has quit IRC01:00
*** ccavanna has joined #openstack-swift01:05
*** sanchitmalhotra1 has joined #openstack-swift01:11
*** sanchitmalhotra has quit IRC01:13
*** amit213 has quit IRC01:24
*** amit213 has joined #openstack-swift01:24
*** hrou has joined #openstack-swift01:56
openstackgerritOpenStack Proposal Bot proposed openstack/swift: Updated from global requirements  https://review.openstack.org/8873601:59
*** jrichli has joined #openstack-swift02:01
*** wbhuber has joined #openstack-swift02:19
claygwhere's peluse been?02:21
*** chlong has quit IRC02:27
mattoliveraukota_: morning02:28
mattoliverauacoles_: retested patch, working well, just +A'ed.02:29
*** bill_az_ has quit IRC02:40
*** chlong has joined #openstack-swift02:41
*** wbhuber_ has joined #openstack-swift03:02
*** wbhuber has quit IRC03:05
*** wbhuber_ has quit IRC03:07
*** wbhuber has joined #openstack-swift03:08
*** wer has quit IRC03:09
*** wer has joined #openstack-swift03:10
openstackgerritMerged openstack/swift: Fix copy requests to service accounts in Keystone  https://review.openstack.org/22713303:10
*** wbhuber has quit IRC03:12
*** trifon has joined #openstack-swift03:25
*** jrichli has quit IRC03:25
notmynamehello, world03:26
*** links has joined #openstack-swift03:31
*** trifon has quit IRC03:44
*** kei_yama has quit IRC04:02
*** kei_yama has joined #openstack-swift04:03
*** klrmn1 has quit IRC04:22
*** kei_yama has quit IRC04:35
*** kei_yama has joined #openstack-swift04:36
claygok, one more run of the probetests I think04:42
claygacoles_: good news!  my diff on patch 138498 is < 1000 lines!04:43
patchbotclayg: https://review.openstack.org/#/c/138498/ - Add POST capability to ssync for .meta files04:43
*** mahatic has joined #openstack-swift04:43
notmynameclayg: I'm just about finished going through it, I think04:44
notmynamewell, "finished" is generous04:44
claygnotmyname: I started with tcpdump and worked my way backwards from probetests to implementation to unittests ;)04:46
claygnotmyname: acoles_ is good - it's a big change and there's really not that much slop04:46
notmynamethat is more thorough than I've done ;-)04:47
claygfuck04:47
notmynamefind something else?04:47
claygthinking of slop I realized I factored out some methods to functions and didn't add explict unittests for them :\04:47
claygnah - just that constant nagging feeling I'm not at good as software development as most of the people around me04:47
notmynameI very strongly sympathize with that04:48
clayg*as04:48
claygwell, let me slap a couple tests on there04:48
mahatichrou: hello! thanks for the review, if I understood it correcctly, you're already working on correcting the encoding way on this 194191 per timburke's comments and you want to see if that's gonna resolve?04:49
*** trifon has joined #openstack-swift04:51
hroumahatic, yep !  I think I'll need to do more to resolve it though, timburke made some good points in a comment as well, so I'm going to look into those (but yea in other words don't worry about that failing test, and keep it using utf804:51
mahatichrou: I'm happy to continue looking into that once you're done with 19419104:52
hroumahatic, awesome great !  Yea I should have something for that patch tomr04:53
mahatichrou: cool, I'm looking forward to that change04:55
mahatichrou: and we could really use acoles_ opinion here like you said04:56
claygand I found a bug in my fucking code too04:59
hroumahatic, yep, well, mind you that was before tim replied : )04:59
*** m_kazuhiro has quit IRC04:59
mahatichrou: yes, I missed that one actually05:00
claygweird that probetests didn't catch it - better run 'em again05:00
mahatictimburke: thanks for the review!05:01
*** mragupat has joined #openstack-swift05:06
mahaticclayg: "most people" (re s/w dev)  is a bit of a stretch ;)05:08
notmynameclayg: I'm ready to look at your patch set whenever you push it05:08
*** m_kazuhiro has joined #openstack-swift05:16
claygnotmyname: i had it right the first time!05:23
openstackgerritClay Gerrard proposed openstack/swift: Add POST capability to ssync for .meta files  https://review.openstack.org/13849805:31
*** mahatic_ has joined #openstack-swift05:31
notmynamewheeee05:31
claygmy +2 is only sorta tenative - I'd like to get @acoles approval before it +A's if at all possible - I think he should be online soon?05:32
notmynameyeah. 6:30am for him right now05:33
*** mahatic has quit IRC05:33
clayggreat!  ok, I'll pull down the next one :P05:37
claygi guess I should rebase it really05:37
claygoh god this one is bigger than the other one!05:38
notmynameyeah, fixing POSTs was just to get ready for that one ;-)05:38
notmynamehowever, most of the raw line count is from the tests, and they're pretty well-formatted IIRC05:39
notmynameclayg: what do you mean by hoping nobody runs with post-as-copy=false?05:41
claygi forget - i do sorta remember saying that tho05:41
notmynameyeah, I'm looking over your comments in the previous patch set05:42
claygoh, something about how we let object server's put datafiles under metadata files05:42
claygand metadata files only exist with wiht post-as-copy = false so it's only *barely* a behavior change because it's a change in the behavior of the non-default configuration05:42
openstackgerritMahati Chamarthy proposed openstack/swift: Guess content type on PUT and probe tests  https://review.openstack.org/22089705:43
claygi mean it's only a backend behavior - it'll be fine05:43
clayg... probably05:43
claygthis rebase is screwed :\05:43
*** trifon has quit IRC05:43
claygoh wait - i see it now05:44
*** trifon has joined #openstack-swift05:48
claygshit - half the stuff I cleaned up in the last patch were stuff acoles fixed in *this* patch!05:48
claygbah w/e05:48
notmynamebah! there was somethign else I was supposed to do tonight too.05:49
claygjok, i say "half" - i't more like two05:49
claygi used the old gerrit view just long enough that everything in the old view is in the wrong place now :'(05:51
openstackgerritClay Gerrard proposed openstack/swift: Enable object server to return non-durable data  https://review.openstack.org/21527605:52
clayg...05:52
*** hrou has quit IRC06:01
claygok, so this didn't quite make it to X-Backend-If-Not-Frag-Timestamps - it seems like the proxy is still explicitly asking for things...06:03
clayginstead of explicitly asking for anything but something that would be useless06:03
notmynameclayg: this is what I get for the difference between acoles_'s patch sets and the 2 you just pushed https://gist.github.com/notmyname/c9cc9a9506430fe8834306:06
claygI expect it to be like06:09
clayg--- good acoles code06:10
clayg+++ crappy clayg code06:10
claygdid you just diff the sha's from the gerrit page?  it's good that he like rebased recently I guess06:11
claygnotmyname: but yeah that looks about right to me?06:11
notmynameI had already pulled down the 2 patches locally into one branch. did the same after you pushed and diffed the 2 branches06:11
claygnotmyname:  ah - that's handy then06:12
notmynameso if this looks sane to me, then I can look at the final without worry about the changes you've made and take them as part of the whole06:12
claygyeah i really don't think any of my changes were significant?  I probably should have just done it all in follow up :\06:12
notmynameat least, I hope. this whole thing is rather mind-warping. I feel like I'm coming into all this, from a deep code level, late, so I've got catch-up to do :-/06:13
claygi mean the docstring stuff was good06:13
claygi think pulling the 4 encode/decode methods out to module level functions will help with maintance and some initail grok06:13
notmynameyeah. I've already been going back and forth between the 2 files to see each :-)06:14
claygfixing the TODO/meta stuff *would* have been significant it acoles hadn't already done in the non-durable patch06:14
notmynameso yeah figuring out your 36 stream-of-consciousness review comments, though... ;-)06:15
claygtotally useless right - you have to just ignore me - they're notes for me06:15
notmyname(it's great. not a complaint)06:16
notmynamethey're good, because it helps understand why you were thinking stuff06:16
notmynamedo we really not have an overview doc on ssync? ie "here's what the protocol is"06:18
claygif you follow the eveloper docs through replicator and read the Sender and Reciver you can get quite a bit out of there06:18
notmynameyeah, I know. I was hoping for the picture-book version ;-)06:19
claygNICE!06:19
*** mahatic_ has quit IRC06:20
*** mahatic has joined #openstack-swift06:21
*** dmorita has joined #openstack-swift06:22
claygcharz: notmyname: can we retrigger the cluster-swift-tox-func tests on patch 13849806:25
patchbotclayg: https://review.openstack.org/#/c/138498/ - Add POST capability to ssync for .meta files06:25
notmynameclayg: I think a recheck comment will trigger that set of tests too06:26
charzclayg: checking06:26
claygah!  very well then06:29
claygok i'ma sign off for awhile06:29
claygacoles_: hopefully catch you in my am06:29
claygis tomorrow Friday already!?06:29
charzclayg: the jobs is in queue.06:29
notmynamehopefully that means until the morning ;-)06:29
claygnotmyname: well see06:29
notmynameyeah. already06:29
notmynameI'll try to give this patch set a go and see where I get06:29
openstackgerritMahati Chamarthy proposed openstack/swift: Guess content type on PUT and probe tests  https://review.openstack.org/22089706:30
*** mragupat has quit IRC06:30
notmynamets. sometimes a tombstone. sometimes a timestamp. good luck06:36
notmynameeg ts_meta and ts_data. surprise! this time it's a timestamp06:37
*** sanchitmalhotra has joined #openstack-swift06:43
*** itlinux has joined #openstack-swift06:43
*** sanchitmalhotra1 has quit IRC06:44
*** sanchitmalhotra1 has joined #openstack-swift06:50
*** sanchitmalhotra has quit IRC06:52
*** wbhuber has joined #openstack-swift06:58
*** mahatic has quit IRC07:01
*** wbhuber has quit IRC07:02
*** rledisez has joined #openstack-swift07:08
*** tr_ has joined #openstack-swift07:29
*** geaaru has joined #openstack-swift07:29
*** chlong has quit IRC07:31
*** kei_yama has quit IRC07:36
*** itlinux has quit IRC07:38
*** kei_yama has joined #openstack-swift07:40
*** acoles_ is now known as acoles07:40
acolesclayg: yeah i'll around later07:41
*** itlinux has joined #openstack-swift07:41
acolesnotmyname: its obvious , ts is a tombstamp :)07:42
*** haomaiwang has joined #openstack-swift07:44
acolesmattoliverau: thanks!07:45
*** openstackgerrit has quit IRC07:46
*** openstackgerrit has joined #openstack-swift07:46
*** jordanP has joined #openstack-swift07:47
*** sanchitmalhotra has joined #openstack-swift07:51
*** sanchitmalhotra1 has quit IRC07:53
acolescharz: thanks for doing a recheck on 138498. i'm interested to see if that passes on QA cluster before merging.07:54
acolesnotmyname: seriously, i had in mind to change ts to tomb where appropriate. obviously my mind is not the best place to make that kind of note.07:56
*** haomaiwang has quit IRC08:01
*** haomaiwa_ has joined #openstack-swift08:01
*** eranrom has joined #openstack-swift08:09
*** kei_yama has quit IRC08:18
*** kei_yama has joined #openstack-swift08:18
*** jistr has joined #openstack-swift08:24
*** peterlisak has joined #openstack-swift08:25
*** tr_ has quit IRC08:30
*** tab_ has joined #openstack-swift08:30
*** cazino has joined #openstack-swift08:32
acolescharz: yay it passed this time08:35
charzacoles: COOL!!!08:36
claygohai!08:37
acolesclayg: THANKS for all the reviewing! I am just going through, looks like we have ssync fix to ready land :D08:40
acolescharz: yay for the QA cluster :)08:40
charzLOL08:40
acolesclayg: oh, sorry :/ i left a comment somewhere that those TODO's were fixed in the next patch, must have got hidden by new versions on gerrit08:43
claygs'ok this way they merge even if we can't the fix-all-the-things-proxy patch in08:44
claygacoles: I am sorta sniffing around the idea that now that we know *roughly* what the finished product will look like - we could try to split up the non-durable from the alt-frags change without too much worry that doing the first half without the second will paint us into a corner?08:46
claygbut reading over your diff, there's still a number of TODO's in the grand-unified-fix-everything-we-could-possibly-want-to-fix-in-the-proxy patch08:46
claygnotmyname probably won't cut anything release looking until Monday, but realistically today is probably the day - not sure what you've got already on your plate with the service token stuff?08:47
acolessure, its good to have those meta+tomb anomalies fixed.08:48
claygI also just remembered patch 181407 which ... would be very nice to have08:48
patchbotclayg: https://review.openstack.org/#/c/181407/ - Don't ssync fragments that conflict with another f...08:48
acolesclayg: service token fix is merged08:49
claygwhooo hoo!08:49
acoleskerching08:49
claygpatch 227133 wasn't crossed off on the wiki08:49
patchbotclayg: https://review.openstack.org/#/c/227133/ - Fix copy requests to service accounts in Keystone08:49
acolesso that leaves all day to fix the whole world08:49
claygnotmyname: cc^ slacking08:49
acolesi'll cross it off08:49
*** links has quit IRC08:50
clayga'ight well - i'm not going to be able to get a proper review of patch 215276 in tonite08:50
patchbotclayg: https://review.openstack.org/#/c/215276/ - Enable object server to return non-durable data08:50
acolesk so i can think about how dropping the alt frags may simplify 215276 patch and i will look at 181407 again08:51
acolesclayg: u go sleep, i will still be here when you start your friday08:51
claygshit patch 222799 has a unittest failure :\08:52
patchbotclayg: https://review.openstack.org/#/c/222799/ - Validate against duplicate device part replica ass...08:52
*** aix has joined #openstack-swift08:54
acolesclayg: do i need to review that ^^ or do we have eyes on it? have to confess, the ring would be a learning curve for me.08:54
*** m_kazuhiro has quit IRC09:00
*** haomaiwa_ has quit IRC09:01
claygacoles: torgomatic has the ball09:01
*** haomaiwang has joined #openstack-swift09:01
claygacoles: and cschwede maybe?09:01
claygacoles: mattoliverau looked at it09:01
claygacoles: I'd prefer if you could stay on the proxy/diskfile stuff09:02
acolesclayg: oh phew. i love team effort.09:02
acolesyeah not yielding hash when only a .meta makes a lot of sense. good call.09:02
openstackgerritClay Gerrard proposed openstack/swift: Validate against duplicate device part replica assignment  https://review.openstack.org/22279909:02
claygwho else is up?09:03
claygkota_: mattoliverau: ???09:03
acolesthat would be a bit of ahospital pass09:03
acolesyielding the meta file only that is09:04
clayggoogle didn't know what ahospital was either :\09:06
*** [1]eranrom has joined #openstack-swift09:09
*** eranrom has quit IRC09:09
*** [1]eranrom is now known as eranrom09:09
*** tdasilva has quit IRC09:10
clayghah!  screw all you write-leader-electing-concensus-getting-consistent-promising distributed systems => http://danluu.com/limplock/09:10
*** Guest64825 has quit IRC09:14
*** dmsimard has quit IRC09:14
*** dmorita has quit IRC09:14
*** openstack has joined #openstack-swift09:23
*** amandap has joined #openstack-swift09:23
*** openstackstatus has joined #openstack-swift09:23
*** ChanServ sets mode: +v openstackstatus09:23
*** marzif has joined #openstack-swift09:29
*** tdasilva has joined #openstack-swift09:32
*** ChanServ sets mode: +v tdasilva09:32
*** mahatic has joined #openstack-swift09:33
*** cazino has left #openstack-swift09:34
*** mahatic has quit IRC09:36
*** marzif has quit IRC09:37
*** marzif has joined #openstack-swift09:37
*** haomaiwang has quit IRC10:01
*** haomaiwang has joined #openstack-swift10:01
*** daemontool__ has joined #openstack-swift10:06
*** daemontool_ has quit IRC10:07
*** [1]eranrom has joined #openstack-swift10:09
*** eranrom has quit IRC10:09
*** [1]eranrom is now known as eranrom10:09
openstackgerritPeter Lisák proposed openstack/swift: swift-init return codes  https://review.openstack.org/23035210:09
*** pcaruana has joined #openstack-swift10:13
*** cazino has joined #openstack-swift10:18
*** sanchitmalhotra1 has joined #openstack-swift10:31
*** sanchitmalhotra has quit IRC10:33
*** mahatic has joined #openstack-swift10:38
mahaticacoles: hello10:39
mahaticacoles: re: probe tests comment - won't unit tests need to skip too wrt crypto on/off?10:47
mahaticacoles: ok, i don't think they do. My other question would be - are we not going to do any probe tests? there was a card in trello - which lead me to start these10:53
*** ig0r_ has joined #openstack-swift10:55
acolesmahatic: yeah, sorry, i only just realised that some of the testing could perhasp be done using the unit test pattern in test_server. IDK if there may still be some that need probe tests.10:55
*** ig0r_ has quit IRC10:56
acolesclayg: you didn't add yourself as co-author on patch 13849810:59
patchbotacoles: https://review.openstack.org/#/c/138498/ - Add POST capability to ssync for .meta files10:59
*** haomaiwang has quit IRC11:01
mahaticacoles: no problem. Also, you did trace it right in obj controller file - so if i wrap it and put it in request_helpers, you mean we could use that in future as well? also change it in controller/obj.py?11:01
*** haomaiwang has joined #openstack-swift11:01
acolesmahatic: i was thinking one function in request_helpers that's then called from obj.py and encrypter11:02
acolesmahatic: but I didn't think about it for too long, so please double check my logic!11:02
acolesmahatic: sorry i didn't have much time to review it today.11:02
*** cazino has quit IRC11:05
mahaticacoles: sure, will look into that. thank you!11:06
mahaticacoles: and np, thanks for the review - I didn't expect it being friday today (for L) you'd look at it :)11:08
*** eranrom has quit IRC11:08
*** eranrom has joined #openstack-swift11:09
acolesmahatic: sadly friday is included in the working week ;P have a good weekend11:15
*** delatte has joined #openstack-swift11:16
*** cdelatte has joined #openstack-swift11:16
mahaticacoles: I know ;). Thanks, you have a  good weekend too11:16
*** cazino has joined #openstack-swift11:24
openstackgerritAlistair Coles proposed openstack/swift: Add POST capability to ssync for .meta files  https://review.openstack.org/13849811:24
*** ccavanna has quit IRC11:25
acolesclayg: nm i implicated you. great improvements, thanks.11:26
*** pcaruana has quit IRC11:47
*** wbhuber has joined #openstack-swift11:52
*** wbhuber has quit IRC11:54
*** haomaiwang has quit IRC12:01
*** haomaiwang has joined #openstack-swift12:01
*** zul has quit IRC12:01
*** zul has joined #openstack-swift12:04
*** cazino has quit IRC12:05
*** cazino has joined #openstack-swift12:05
*** [1]eranrom has joined #openstack-swift12:09
*** eranrom has quit IRC12:09
*** [1]eranrom is now known as eranrom12:09
*** haomaiwang has quit IRC12:10
*** 20WAB697A has joined #openstack-swift12:11
*** 20WAB697A has quit IRC12:25
*** thumpba has joined #openstack-swift12:34
*** delatte has quit IRC12:36
*** cdelatte has quit IRC12:36
*** delatte has joined #openstack-swift12:37
*** cdelatte has joined #openstack-swift12:37
*** MVenesio has joined #openstack-swift12:43
*** MVenesio_ has joined #openstack-swift12:47
*** thumpba_ has joined #openstack-swift12:48
*** MVenesio has quit IRC12:50
*** thumpba has quit IRC12:52
*** MVenesio_ has quit IRC12:54
*** MVenesio has joined #openstack-swift12:55
*** dustins has joined #openstack-swift12:57
*** proteusguy_ has joined #openstack-swift13:01
*** [1]eranrom has joined #openstack-swift13:09
*** eranrom has quit IRC13:09
*** [1]eranrom is now known as eranrom13:09
*** peterlisak has quit IRC13:10
*** itlinux has quit IRC13:15
*** itlinux has joined #openstack-swift13:15
*** jrichli has joined #openstack-swift13:20
*** tab_ has quit IRC13:21
*** CR7 has joined #openstack-swift13:25
*** haomaiwa_ has joined #openstack-swift13:25
*** CR7 has quit IRC13:25
*** hrou has joined #openstack-swift13:31
*** cazino has quit IRC13:35
*** cazino has joined #openstack-swift13:36
*** haomaiwa_ has quit IRC13:37
*** haomaiwang has joined #openstack-swift13:38
*** haomaiwang has quit IRC13:43
*** haomaiwang has joined #openstack-swift13:44
*** itlinux has quit IRC13:50
openstackgerritMerged openstack/swift: Add POST capability to ssync for .meta files  https://review.openstack.org/13849813:54
*** alejandrito has joined #openstack-swift13:55
*** kei_yama has quit IRC13:59
*** haomaiwang has quit IRC14:01
*** lcurtis has joined #openstack-swift14:03
*** haomaiwa_ has joined #openstack-swift14:04
*** lcurtis has quit IRC14:04
*** lcurtis has joined #openstack-swift14:04
*** pberis has joined #openstack-swift14:06
*** thurloat_isgone is now known as thurloat14:07
*** thurloat is now known as thurloat_isgone14:09
*** mahatic has quit IRC14:10
*** haomaiwa_ has quit IRC14:12
*** elmo_ is now known as elmo14:12
*** haomaiwa_ has joined #openstack-swift14:13
*** eranrom has quit IRC14:13
*** thumpba_ has quit IRC14:14
*** cazino has quit IRC14:23
*** jbonjean has joined #openstack-swift14:26
*** trifon has quit IRC14:27
*** jlhinson has joined #openstack-swift14:31
*** haomaiwa_ has quit IRC14:33
*** wbhuber has joined #openstack-swift14:35
*** haomaiwang has joined #openstack-swift14:35
openstackgerritOpenStack Proposal Bot proposed openstack/swift: Updated from global requirements  https://review.openstack.org/8873614:36
*** cazino has joined #openstack-swift14:36
*** haomaiwang has quit IRC14:40
notmynamegood morning14:43
acolesnotmyname: hi14:45
notmynamehttps://wiki.openstack.org/wiki/Swift/PriorityReviews updated :-)14:45
acolesnotmyname: 2 patches merged today: service token fix and ssync post fix14:46
notmynameyup :-)14:46
notmynameacoles: thanks for those14:46
notmynamewhere I left patch 215276 last night was agreeing with clayg's comments and realizing that there's another patch14:47
patchbotnotmyname: https://review.openstack.org/#/c/215276/ - Enable object server to return non-durable data14:47
acolesno, thanks you and clayg for a late night reviewing it seems14:47
acolesanother?14:47
notmynamemaybe? looking at it now, it doesn't seem I got too far on that one :-)14:50
notmynameso ignore what I just said14:50
acolesnotmyname: clayg mentioned patch 181407 to me earlier so i have been breathing life back into that14:52
patchbotacoles: https://review.openstack.org/#/c/181407/ - Don't ssync fragments that conflict with another f...14:52
acolesnotmyname: so far so good and its not that huge14:52
acolesnotmyname: but is based on 215276 in order to take advantage of the diskfile changes in there14:53
acolesso i am wondering whether to split out some of the diskfile stuff from 215276 into a common base patch. and then there is clayg's comment about reducing scope of 215276 to just the non-durable getting14:54
acolesi've not really had chance to consider that too much14:55
*** minwoob has joined #openstack-swift15:00
*** Fin1te has joined #openstack-swift15:09
*** chlong has joined #openstack-swift15:10
claygacoles: I was hoping you wouldn't notice :\15:14
acolesclayg: notice what?15:15
claygacoles: i wan't implicated15:16
*** trifon has joined #openstack-swift15:16
acolesclayg: oh right. heh15:16
notmynameoh, we all noticed ;-)15:16
acolesclayg: i feel i made a real *hash* of that patch :D15:17
notmynameacoles: which "part"? we should set up a timestone15:18
acoleslol15:20
acolesnotmyname: as i heard said just this week, 'this code was hard to write, it should be hard to understand' :)15:21
claygacoles: ^ that's a redbo-ism!15:23
acoleswell redbo is famous worldwide then15:24
claygok, I need to get the kids ready for school15:24
claygi'll rebase patch 215276 and probably try to split the .durable from the alt-frags stuff15:24
patchbotclayg: https://review.openstack.org/#/c/215276/ - Enable object server to return non-durable data15:24
*** lyrrad has joined #openstack-swift15:24
claygwe may have to cut the rc w/o any of it?15:24
acolesnotmyname: clayg is it worth doing a hangout to discuss where we go with 215276?15:25
acoleslater that is15:25
notmynameyeah, I think so15:25
claygif next week while testing I feel like I can make a case that we *need* one or both - i'll have the split up15:25
claygif you're still on in a few hours - i need to get off for now15:25
acolesk15:26
claygfew/couple15:26
acolesi'll be here fo a bit longer15:26
claygyou can also type up some stuff15:26
claygafk15:26
acolesi think i'll have patch 181407 in good shape15:26
patchbotacoles: https://review.openstack.org/#/c/181407/ - Don't ssync fragments that conflict with another f...15:26
*** chsc has joined #openstack-swift15:28
notmynameI need to get ready too15:29
timburkegood morning15:35
*** cazino has quit IRC15:35
*** klrmn1 has joined #openstack-swift15:36
*** thumpba has joined #openstack-swift15:38
*** thumpba has quit IRC15:38
*** nadeem has joined #openstack-swift15:39
*** thumpba has joined #openstack-swift15:39
*** nadeem has quit IRC15:40
*** nadeem has joined #openstack-swift15:41
*** janonymous_ has joined #openstack-swift15:44
*** klrmn1 has quit IRC15:47
*** cazino has joined #openstack-swift15:47
*** silor has joined #openstack-swift15:52
*** pberis has quit IRC15:59
*** silor has quit IRC16:05
*** jordanP has quit IRC16:09
*** CR7 has joined #openstack-swift16:10
*** jordanP has joined #openstack-swift16:11
*** rledisez has quit IRC16:18
claygacoles: you need to type yes or something on patch 18140716:23
patchbotclayg: https://review.openstack.org/#/c/181407/ - Don't ssync fragments that conflict with another f...16:23
claygacoles: idk how it's gunna work out - I have a commute followed by an eng meeting - i'm probably still ~2hrs from skyping16:24
acolesclayg: as in yes i am working on it?16:25
acolesclayg: k, well i may commute home and can hangout later16:25
claygacoles: !16:25
*** pberis has joined #openstack-swift16:25
acolesclayg: where do i need to type yes?16:25
claygacoles: I saw "i think i have patch 181407 in good shape"16:25
patchbotclayg: https://review.openstack.org/#/c/181407/ - Don't ssync fragments that conflict with another f...16:25
claygand when I saw it was still in merge conflict I thought you forgot to push16:26
claygbut you're just being optomistic about the future - THAT'S GOOD!16:26
claygok, i'm heading in - afk again16:26
acolesclayg: oic. yes! just putting a probe test in16:26
claygacoles: hopefully talk to you later - but if not next week!16:26
claygl8416:26
acolesiirc you wrote the probe test16:26
*** cazino has left #openstack-swift16:27
*** diogogmt has joined #openstack-swift16:32
*** jordanP has quit IRC16:36
*** pberis has quit IRC16:39
*** pberis has joined #openstack-swift16:39
*** wbhuber_ has joined #openstack-swift16:41
*** janonymous_ has quit IRC16:47
*** diogogmt_ has joined #openstack-swift16:48
*** wbhuber_ has quit IRC16:48
*** diogogmt has quit IRC16:50
*** diogogmt_ has quit IRC16:53
openstackgerritAlistair Coles proposed openstack/swift: Don't ssync fragments to handoffs if they conflict  https://review.openstack.org/18140716:56
*** jlhinson_ has joined #openstack-swift16:56
openstackgerritAlistair Coles proposed openstack/swift: Enable object server to return non-durable data  https://review.openstack.org/21527616:56
*** Fin1te has quit IRC16:57
acolesclayg: i typed yes. 181407 depends on 215276 because it uses the DiskFile.fragments property, so we may end up wanting to rip some of the diskfile changes out of 216276 into a separate patch and base 181407 on that.16:57
*** jlhinson has quit IRC16:59
acolesclayg: and you are an author on 181407 because you wrote the probe test. i took it from patch 17640316:59
patchbotacoles: https://review.openstack.org/#/c/176403/ - WIP Revert to another handoff when primary is unav...16:59
acoles^^ that is probably now redundant16:59
*** diogogmt has joined #openstack-swift17:00
*** Fin1te has joined #openstack-swift17:02
*** jistr has quit IRC17:10
*** geaaru has quit IRC17:11
hrouHey timburke, have some time to chat about the encoding issue you pointed out in the review of patch 194191 ?  I can just leave a comment in the review if you'd find that easier.17:11
patchbothrou: https://review.openstack.org/#/c/194191/ - Encrypter and Decrypter middleware - object body only17:11
*** jrichli has quit IRC17:12
*** garthb has joined #openstack-swift17:13
* notmyname back online17:14
*** Fin1te has quit IRC17:16
*** klrmn1 has joined #openstack-swift17:16
timburkehrou: sure! i've got a meeting in about 15min, so it might be better to wait until after that, though.17:17
hroutimburke, awesome thanks!  Yea lets wait till after you're meeting.17:18
timburkebut it couldn't hurt to talk, and give me something to mull over instead of paying attention to the meeting :P17:18
hroutimburke, sure in that case : ), here goes:17:18
hroutimburke, So firstly, thanks again for pointing it out, it will indeed cause problems with py3.17:18
hroutimburke, Now, you've suggested we serialize the crpyto meta to a json formatted string, for the IV (which is a bytes object), we will first base64 encode it.17:19
hroutimburke, just wanted to point out one minor issue with this (but it has an easy fix); json.dumps expects a string (unicode) not bytes (which is the output of encode), so a small fix to this is to take the encoded binary decode it in utf-8 (i.e. decode('utf-8') ).17:19
timburke(or even the default, us-ascii. it sure *better* be ascii!)17:20
hroutimburke, so all of the above will work in py2 and 3 just fine.  But leads to my actual question, what's the advantage of JSON'ing the object, an alternative (which you hinted at too ; ) is to simply encode crypto_meta[iv], which is a bytes object in 3.0 (i.e. the encoding will work).17:21
hroutimburke, haha hmm I thought the default was utf-8 was the default actually, I'll check.17:22
timburkewe've got structure that we're encoding, too, though. or are you thinking of using separate headers for iv and cipher? that'd be fine, too17:23
hroutimburke, ah wait I see you're suggesting just leave it to the default which would naturally be utf-8 in py3, yep that makes sense !17:23
hroutimburke, well the debate is do we need to encode the entire structure ?17:24
hroutimburke, the rest is 100% ascii, but maybe to future proof (name the crypto algo) its better to encode the entire thing.17:24
timburkeand maybe we don't. i wasn't sure how much else you might want to stuff in there, so was looking for ways to still just use the one header17:25
hroutimburke, yea I think I'd rather leave it as one header for now, so maybe the encoding route is better.17:25
timburkeincluding the cipher definitely seems like a good idea. surely at some point we'll want the new AAES_CTR_1024 or some madness, and we'll want a way to distinguish them without relying on defaults17:28
hrouYep absolutely !17:28
hrouSo I'm liking the proposal of json'ing the object and encoding the IV.17:28
*** garthb has quit IRC17:29
*** daemontool__ has quit IRC17:29
timburke(imagine if we'd done this in the long long ago, and now had to deal with headers.get(cipher_header, 'DES_ECB') code... bleh!)17:29
timburkecool. sounds good.17:29
timburkesee, 15mins was perfect!17:30
*** daemontool__ has joined #openstack-swift17:30
hrouhaha yea that'd be pretty terrible.17:30
hrouAnd that was no coincidence at all ; )  Thanks so much timburke !  It was neat ,got to learn more about py3 though this, which is always fun.17:31
timburkeyeah... i'm kinda dreading swift actually trying to support it. the fact that there's *still* unicode bugs in swiftclient doesn't make me feel good17:32
hrouyea they kinda of jump out at you once you've dealt with one, ah interesting enough it seems they dealt with the issue I cited above in the nova client, patch 9562317:33
patchbothrou: https://review.openstack.org/#/c/95623/ - In Py3 decode the output of base64.decode17:33
*** ChanServ has quit IRC17:37
acolesnotmyname: clayg i am heading home, back on line later (maybe around noon your time?)17:38
*** ChanServ has joined #openstack-swift17:40
*** cameron.freenode.net sets mode: +o ChanServ17:40
*** acoles is now known as acoles_17:41
*** gyee has joined #openstack-swift17:50
notmynameacoles_: ok, great. we'd love to get on the phone when you have time17:57
*** delatte has quit IRC18:06
*** cdelatte has quit IRC18:06
*** nadeem has quit IRC18:06
*** garthb has joined #openstack-swift18:06
*** jrichli has joined #openstack-swift18:07
*** janonymous_ has joined #openstack-swift18:10
*** wbhuber has quit IRC18:18
*** wbhuber has joined #openstack-swift18:18
*** fbo has joined #openstack-swift18:21
*** daemontool__ has quit IRC18:22
*** pberis has quit IRC18:28
*** CR7 has quit IRC18:32
claygacoles_: shit you had to rebase patch 181407 on patch 215276 :'(18:40
patchbotclayg: https://review.openstack.org/#/c/181407/ - Don't ssync fragments to handoffs if they conflict18:40
patchbotclayg: https://review.openstack.org/#/c/215276/ - Enable object server to return non-durable data18:40
claygtorgomatic: hey whenever you're looking at patch 222799 - if you need to high bandwidth anything just let me know?18:41
patchbotclayg: https://review.openstack.org/#/c/222799/ - Validate against duplicate device part replica ass...18:41
*** eranrom has joined #openstack-swift18:46
*** pberis has joined #openstack-swift18:53
*** sanchitmalhotra has joined #openstack-swift18:57
*** ccavanna has joined #openstack-swift18:59
*** sanchitmalhotra1 has quit IRC18:59
*** Fin1te has joined #openstack-swift19:01
*** jrichli has quit IRC19:02
*** Fin1te has quit IRC19:03
*** marcusvrn_ has quit IRC19:05
*** acoles_ is now known as acoles19:09
*** wolsen has quit IRC19:10
claygacoles: hey!19:13
acolesclayg: hi, i'm back19:13
notmynamegreat19:14
*** nadeem has joined #openstack-swift19:14
notmynameacoles: clayg: want to get on video chat and talk about these last patches?19:14
acolesnotmyname: sure19:14
*** daemontool_ has joined #openstack-swift19:14
claygnotmyname: sure!19:15
claygnotmyname: but I don't know how :'(19:15
notmynameI can. let me finish shoveling this food in my mouth19:15
claygnom nom nom19:15
notmynamenoms done19:16
acolesi can wait if you guys are lunching19:16
notmynameI'm done. and clayg absorbs nutrients from the air, I think19:16
claygthe code - it feeds me19:17
*** nadeem has quit IRC19:17
*** wolsen has joined #openstack-swift19:17
*** nadeem has joined #openstack-swift19:17
acoles'tis manna19:17
*** jrichli has joined #openstack-swift19:21
*** subscope has joined #openstack-swift19:24
*** subscope_ has joined #openstack-swift19:25
*** marcusvrn_ has joined #openstack-swift19:26
acolesi just saw the t shirt19:26
*** CR7 has joined #openstack-swift19:27
*** david-lyle has quit IRC19:35
*** david-lyle has joined #openstack-swift19:35
*** CR7 has quit IRC19:38
jrichliacoles: did I miss a link?  I checked twitter ... nothing but the 'hint' ...19:39
acolesits on notmyname's body19:39
jrichlioh!  you guys are ... having dinner?19:40
*** [1]eranrom has joined #openstack-swift19:43
*** eranrom has quit IRC19:43
*** [1]eranrom is now known as eranrom19:43
acolesjrichli: skype19:44
*** david-ly_ has joined #openstack-swift19:45
jrichlioh right ... i guess i saw that you guys might be on a call later19:45
jrichliso, how does it look???19:46
jrichliis there a bird?  is it top secret?19:46
acolesjrichli: i am sworn to secrecy19:46
jrichlii figured ;-)19:46
acolesit has short sleeves19:46
jrichligee, thx19:47
acolescotton i am guessing19:47
acolesdark, but we knew that19:47
notmynamelol19:47
notmynameactually only partly cotton. it's a blend. super soft19:47
acolesthey could have chosen a better model :P :P19:48
notmynameTRUE THAT!19:48
acolesoooh super soft. well i couldn't actualy touch it. and that would have been weird anyway.19:48
*** david-lyle has quit IRC19:49
*** david-ly_ has quit IRC19:49
jrichlimaybe i could find a source from swiftstack who hasn't been sworn to secrecy yet ...19:49
acolesclayg: oh i forgot to say that i have a follow on for 215276 that removes the durable requirement completely. but, later...19:49
acolesjrichli: actually i'm thinking the whole thing is a decoy19:50
jrichliI actually pictured a shirt with a big duck on it19:50
acolesjrichli: it did kinda look like pieces of sticky tape on a plain tee19:50
jrichlibut i know what u mean ;-)19:51
notmynamethey were packed up and shipping to japan today. I just grabbed one early :-)19:51
acolesjrichli: quick, get to the port and intercept...19:52
notmynameok, so acoles and clayg and I just chatted about the outstanding patches. for those who are curious, here's the plan:19:52
notmynamewe need to land patch 222799 today19:52
patchbotnotmyname: https://review.openstack.org/#/c/222799/ - Validate against duplicate device part replica ass...19:52
acolesi'm curious ;)19:52
*** thumpba has quit IRC19:53
notmynameand python 215276 will not land today. we'll keep working on it, though19:53
acolespython!!19:53
notmynameinstead, clayg is working on a new simple patch to increase the quorum for durables19:53
notmynamethat combined with the other changes that have gone in recently should seriously mitigate any concerns with but 145243119:54
notmynamebug 145243119:54
openstackbug 1452431 in OpenStack Object Storage (swift) "some parts replicas assigned to duplicate devices in the ring" [Critical,Confirmed] https://launchpad.net/bugs/1452431 - Assigned to Samuel Merritt (torgomatic)19:54
notmynamealthough, the bug isn't completely closed now, the likelyhood is much much less, we suspect, of happening19:54
notmynameso with clayg's new patch and patch 222799 and my patch for the authors/changelog update, we'll have our liberty release19:54
patchbotnotmyname: https://review.openstack.org/#/c/222799/ - Validate against duplicate device part replica ass...19:54
notmynameand at leat the 3 of us feel pretty much ok with it :-)19:55
*** CR7 has joined #openstack-swift19:55
eranromI have been looking at it for the past few days, and confess I am only starting to figure out that builder code.19:56
eranromSo I take it is no longer required (FWIW)19:56
eranromI am referring to  https://review.openstack.org/#/c/222799/19:56
acoleseranrom: that IS required19:57
acoleseranrom: patch 215276 is deferred19:57
patchbotacoles: https://review.openstack.org/#/c/215276/ - Enable object server to return non-durable data19:57
eranromacoles: So should I coninue reviewing the duplicate devices then?19:58
eranromagain FWIW19:58
*** thumpba has joined #openstack-swift19:59
acoleseranrom: i was just clarifying that 222799 is still in scope for the release, notmyname: clayg should answer if they need your review19:59
*** mac_ified has joined #openstack-swift19:59
eranromok thanks!20:00
*** acoles is now known as acoles_20:02
notmynameyeah, any reviews on patch 222799 or the one clayg will push up later will be very helpful20:05
patchbotnotmyname: https://review.openstack.org/#/c/222799/ - Validate against duplicate device part replica ass...20:05
eranromok, sure20:07
notmynameI just saw this article about how swift is used to help make movies/tv shows. http://dailycloud.info/somewhere-over-the-rainbow-cloud-instances-shine/20:09
*** CR7 has quit IRC20:11
janonymous_link is awesome : http://lmgtfy.com/?q=pigeonhole+principle :P20:12
*** CR7 has joined #openstack-swift20:17
*** thumpba has quit IRC20:17
*** pberis has quit IRC20:18
*** david-lyle has joined #openstack-swift20:24
*** Fin1te has joined #openstack-swift20:25
*** CR7 has quit IRC20:30
*** CR7 has joined #openstack-swift20:30
*** Fin1te has quit IRC20:36
*** [1]eranrom has joined #openstack-swift20:43
*** eranrom has quit IRC20:43
*** [1]eranrom is now known as eranrom20:43
claygoh yeah, hrmm... I sorta expected I might get dinged for that in the review being as that's it's *kinda* snarky - but oh well - maybe we're all cool with a having a little bit of fun with stuff like that?20:44
claygnotmyname: wow - *one* test failed the durable quorum change20:44
notmynamenot bad!20:45
notmynameor terrifying20:45
janonymous_clayg : It's cool :)20:47
janonymous_I was having a look at swift vids and i saw this: https://www.openstack.org/summit/vancouver-2015/summit-videos/presentation/what-and-039s-up-swift-the-globally-distributed-community-behind-swift :)20:50
claygjanonymous_: man - just *look* at those handsome blokes!20:52
claygwe were so young20:52
notmynameEC has aged us20:54
janonymous_clayg: Hahaa. It was a nice talk :)20:54
*** ccavanna has quit IRC20:55
torgomaticSome people age like fine wines; I age more like leftover pizza.20:57
claygwe *still* have that stupid ChunkWriteTimeout popping up in tests?  rly?20:58
claygI'm about to go stuck a calling frame on that fucker and find out who made him20:59
openstackgerritClay Gerrard proposed openstack/swift: Make sure we have enough .durable's for GETs  https://review.openstack.org/23064120:59
claygyou know... when you type out a description of the change like that it sounds *entirely* too reasonable :\21:00
notmynameyup. looks very reasonable when looking at the diff21:03
*** dustins has quit IRC21:03
notmynameafter I put down the other patch, I'll run tests on this one and leave a reveiw21:03
claygnotmyname: you may wait and see if the probetests will pass :\21:04
notmynameanyone have in-progress reviews on python 222799? I wanted to update the commit message21:04
notmynameclayg: of course. I'll run the tests :-)21:04
claygi don't *think* probetests really do anything to much with timeouts on .durables tho21:05
claygand we require the same number of 201's we always did *really* (at least for the frag archives themselves)21:05
clayga'ight ChunkWriteTimeout - let's see if you can hide *now*21:05
*** MVenesio has quit IRC21:06
notmynameyeah. probably a place for some additional probe tests21:06
claygnext step is to hack nosetest runner to sleep 60 before exiting and doing a binary search module by module21:06
claygwell that or a sysexit handler in eventlet that will wait for the hub to clear it's timeouts...21:07
claygit's gotta be in a spawn_n somewhere21:07
claygFile "/vagrant/swift/swift/proxy/controllers/obj.py", line 1407, in put_fragments_in_queue\n    for fragment in frag_iter:\n', '  File "/vagrant/swift/swift/proxy/controllers/base.py", line 881, in iter_bytes_from_response_part\n    self.app.client_timeout21:08
notmynamedidn't I just add some spawn_ns to swift?21:08
claygfigues - the stack is totally disconnected from the test that spawned it (stupid greenlets)21:08
claygnotmyname: i don't think so - you had regular spawns?21:09
notmynamehmm..yeah. earlier versions used spawn_n21:09
notmynameI think21:09
notmyname..back to rings21:09
notmynamethe comments in test_warn_at_risk() and test_no_warn_when_balanced are really helpful!21:10
*** thumpba has joined #openstack-swift21:10
claygthat seems so unlikely...21:10
claygoh those21:11
claygyeah kota tried to add those comments and like - gah - it's really hard to explain21:11
*** wbhuber has quit IRC21:13
*** trifon has quit IRC21:17
*** jrichli has quit IRC21:18
clayg... this may not work - but I've attached the getter that's firing the timeout to the timeout so in addition to the stack frame I can include some of the getter's attributes like req_path, and statuses - see if it gives me a hint which test is starting up this thing21:20
*** diogogmt_ has joined #openstack-swift21:26
*** diogogmt has quit IRC21:29
*** diogogmt_ is now known as diogogmt21:29
*** nadeem has quit IRC21:35
*** jlhinson has joined #openstack-swift21:39
*** jlhinson_ has quit IRC21:39
*** [1]eranrom has joined #openstack-swift21:44
*** eranrom has quit IRC21:44
*** [1]eranrom is now known as eranrom21:44
*** marcusvrn_ has quit IRC21:45
*** thumpba has quit IRC21:49
*** jlhinson has quit IRC21:50
*** janonymous_ has quit IRC21:51
*** pgbridge has quit IRC21:55
*** subscope_ has quit IRC21:56
*** pgbridge has joined #openstack-swift21:59
*** CaioBrentano has quit IRC22:01
notmynameso set the subtier to the smaller of the roomiest tier or fudgiest tier, then figure out the sibling tiers and put it in one that is unused. set the tier to the subtier and loop22:03
notmynameit's all so clear now22:03
*** hrou has quit IRC22:06
*** minwoob_ has joined #openstack-swift22:12
eranromnotmyname: I ask myself if there is a point in looking at non direct siblings in this case.22:13
*** minwoob has quit IRC22:13
notmynameeranrom: isn't it like 1am for you?22:14
eranromthat is in the case where all siblings already hold a replica of the partition22:14
eranromit is :-)22:14
claygalright!  now I have some details /v1/a/c/o - so *that* narrows it down22:15
claygeranrom: back tracking would be an improvement; but not going down the wrong path would be a solution22:16
claygeranrom: the *real* weeds start to get pulled up when you realize your problem isn't in _reassign_parts it's in *gather* parts - then you say f' it - I'll ship this and loop back around next week ;)22:17
clayg'Range': 'bytes=0-1927' < grep for 1927 didn't turn up anything22:18
eranromclayg, well, I still did not get to that (gather) part yet in my exploration of the code...22:19
claygeranrom: heh - i'm just funnin' - I don't think that change acctually touches gather22:20
eranromit does not22:20
notmynameclayg: I thought you had added a check that you couldn't actually have a ring at all with less devices than parts22:21
openstackgerritZack M. Davis proposed openstack/swift: replace use of deprecated rfc822.Message with a helper utility  https://review.openstack.org/20330422:21
claygnotmyname: turned it into a warning22:21
notmynameok22:21
notmynameand there are *lots* of warnings22:21
claygnotmyname: since we're not going to restrict double subscribing devices to different replicas of the same part there didn't seem like a strong reason to make it blow up hard22:22
claygnotmyname: I think that one *could* be made an error tho - since we allow set_replicas - and reducing the replica count is cheaper than growing new replicas of parts22:22
eranromI will post a short review comment on what I have so far (haven't gone through the test/unit/common/ring/test_builder.py) will continue tomorrow assuming it will still be relevant...22:24
notmynameeranrom: perhaps, but unlikely. I'm hoping to get this landed tonight22:25
notmynameI'm also hoping that torgomatic is looking at it22:25
eranromnotmyname,ok, then hopefully what I have is helpful in any way :-)22:26
notmynameeranrom: thanks22:26
*** lcurtis has quit IRC22:26
*** subscope has quit IRC22:30
claygeranrom: yeah I'm sure it'll be great!22:30
*** chsc has quit IRC22:33
*** eranrom has quit IRC22:39
clayghrmm.... it might be an interesting fallback to pick just *any* other devices w/ort tier - basically throw out dispersion and balance in favor of not doing something totally stupid - seems like a cludge - picking another device on the same server is mostly benign22:43
*** david-lyle has quit IRC22:45
*** CR7 has quit IRC22:46
*** david-lyle has joined #openstack-swift22:47
claygso my nose trick didn't work - but I'm not sure if it's because the gt that spawned the thing was closed and gc shut something down before my sleep - or if it was just cause my sleep wasn't green so the hub wasn't able to spin?22:49
*** ajiang has quit IRC22:49
clayg... either way I pulled it out and I'm trying to wait in test.unit.proxy.test_server.teardown22:49
*** marzif has quit IRC22:49
clayg... I'm pretty sure it's in proxy.test_server22:49
*** bhakta has quit IRC22:49
*** itlinux has joined #openstack-swift22:53
claygshit - i think i added it with the ec-disconn tests :\22:57
*** itlinux has quit IRC22:57
notmynameI think half the problem with understanding ring patches is the variable naming and nested data structures23:00
claygnotmyname: YES!23:01
claygi was *thinking* about that!23:01
notmynamewhat's tier in this context? what's that nested list data structure look like? whoa! here comes a tuple!23:01
claygI think ... well i have some thoughts23:01
notmynameand why are we talking about fudge? and where can I get some?23:02
notmynameclayg: I suspect you may have written all the "fudgy" stuff after visiting your grandmother for the holidays or something23:03
notmynamehere's the fudge tiers. over there are the pralines. here's the pecan pie23:03
*** gyee has quit IRC23:04
*** alejandrito has quit IRC23:10
claygnotmyname: no i didn't write any of that!23:11
notmynameheh23:11
*** wbhuber has joined #openstack-swift23:12
claygi'm mostly free of git blame under swift.common.ring - well... maybe that's not true because of dispersion reports23:12
notmynameyeah, looks like torgomatic23:12
*** itlinux has joined #openstack-swift23:14
notmynameI wonder if mattoliverau is doing anything this saturday am23:15
*** MVenesio has joined #openstack-swift23:17
*** MVenesio has quit IRC23:18
*** itlinux has quit IRC23:19
mattoliveraunotmyname: I'm brunching, but not leaving for about 1/2 hour.. Want me to attempt to look at the ring change again?23:21
notmynameit would be helpful23:21
notmynamemattoliverau: also, https://review.openstack.org/#/c/230641/ is short and needs to land in lieu of patch 21527623:22
patchbotnotmyname: https://review.openstack.org/#/c/215276/ - Enable object server to return non-durable data23:22
mattoliverauAlso, yes, fudge factor is, I'm sure useful, but a little confusing and not at all as delicious as it sounds23:22
mattoliverauHai23:22
*** garthb has quit IRC23:34
*** nadeem has joined #openstack-swift23:41
openstackgerritClay Gerrard proposed openstack/swift: Validate against duplicate device part replica assignment  https://review.openstack.org/22279923:42
clayglol - no it's not tasty at all23:43
claygidk, maybe it would have been useful - what if it's not obvious why you can't have three replicas on two devices?23:47
notmynameno23:49
notmyname:-)23:49
*** diogogmt_ has joined #openstack-swift23:51
*** diogogmt has quit IRC23:52
*** diogogmt_ is now known as diogogmt23:52
*** diogogmt has quit IRC23:54
*** diogogmt has joined #openstack-swift23:56

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!