Wednesday, 2015-03-04

claygyeah00:01
tdasilvaclayg: sorry, I see what you mean, will change the code00:04
*** ho has joined #openstack-swift00:04
hogood morning guys!00:08
claygpeluse: acoles_away: I'm looking at 159637 - since it's not marked WIP00:13
peluseclayg, cool.  its ready for eyes00:15
pelusethe other one is a little bit of a mess due to changes needed because of the multi-phase put stuff00:15
claygand i'm gathering we're just not there on the etag stuff - tdasilva seems to have a patch against feature/ec that starts that work?00:17
peluseclayg, container DB update using correct etag was the only thing I knew of00:18
claygpeluse: the proxy also returns some random FA's etag on PUT00:19
tdasilvaclayg, peluse: i'm having some issues with my dev. environment with ec work, so i'm trying to get that resolved00:19
claygpeluse: i found it easiest to see with a upload via curl using -T /path/to/local/file - the md5 will come back different every time00:20
peluseclayg, yeah, well I meant the only thing I knew about until you mentioned the md5 test earlier00:22
pelusebut right now my whole dev env is jacked all to hell also so I'm ready to throw something00:22
claygpeluse: oic, ok well those two things are the only ones I know about00:22
tdasilvapeluse: mmm...you wouldn't be running into issues with ec master, would you?00:24
peluseno, my older VM had an old eventlet that didn't have the changes tsg made and the requirements file on feature/ec isn't up to snuff there yet00:26
peluserunning tests now....00:26
tdasilvaok00:26
*** lpabon has joined #openstack-swift00:28
*** lpabon has quit IRC00:28
claygpeluse: acoles_away: twice now reading through this diff I've thought that it seems like we should just have storage policies define a diskfile - you guys should probably start now building a case against it - I tend to make my mind up about stuff and with out a good argument to contrarty I'll probably have to code it up00:29
peluseclayg, yeah that was discussed way back when for sure and we know a lot more now than we did then...00:36
openstackgerritTim Burke proposed openstack/python-swiftclient: Use proper checksum algorithm for large objects  https://review.openstack.org/16104300:38
claygI think we originally didn't iagmine all this fragment archive and .durable stuff smushing all up against the object's storage path00:38
pelusebut I dunno... something we should talk about more but unfortunately I have nasty business travel (back to the freezing f'ing cold near tdasilva) so the rest of my week is nearly shot to hell00:38
clayglol00:38
tdasilvahaha00:39
tdasilvaare you headed to Boston?00:39
peluseis it snowing up there? - yes Boston00:39
peluseleave tomroor like 10 min after the IRC meeting00:39
tdasilvayeah, 2-3 inches overnight, another 4 inches on thu00:40
tdasilvathe thursday snow should break the record for all time high for boston, so you will be here to experience history in the making :D00:40
pelusesheeeeeeeeit00:41
tdasilvapeluse: are you attending Vault conference next week?00:45
*** nexusz99 has joined #openstack-swift00:45
mattoliverauho: morning (sorry for late response)00:46
pelusethought about it but too much going on and I probably would have gotten a "no" anyhow with the summit coming up soon00:46
*** gyee has quit IRC00:51
*** jrichli has joined #openstack-swift00:51
*** gyee has joined #openstack-swift01:04
*** annegentle has quit IRC01:08
tdasilvaclayg: re: the issue with etag that you found, I'm changing it so that the proxy returns the correct etag in the response to the client01:09
openstackgerritpaul luse proposed openstack/swift: Ssync fix for EC PUT path  https://review.openstack.org/16105701:11
pelusetdasilva, BTW, feature/ec is working OK for me now that I fixed my env (eventlet)01:11
*** annegentle has joined #openstack-swift01:12
tdasilvayeah...that was my problem too...once you mentioned eventlet and I looked at tushars patch and noticed that it fixed exactly the issue I was running into:  send_hundred_continue_response01:13
tdasilvapeluse: ^01:13
pelusecool01:13
claygtdasilva: well good!01:15
pelusei gotta go get some grub.....01:16
*** nexusz99 has quit IRC01:16
*** annegentle has quit IRC01:19
*** cebruns has joined #openstack-swift01:38
openstackgerritThiago da Silva proposed openstack/swift: WIP: Use Container override header to update etag and size  https://review.openstack.org/16103001:41
claygredbo: I was wondering if as alternative to implementing a python vm in go so you can update and sync hashes.pkl - if we could move to a more portable format?01:43
redboOh I'm off the python vm.  I implemented pickles in go.01:46
claygredbo: I know you were thinking about doing a per-part sqlite database - but before that if the REPLICATE verb would send json, and there was some sort of upgrade path for hash invalidation in the object server from a pickle to... idk - berkley db?  seems like you get get all the pickles out of your go's01:46
claygoh... well if that works i suppose pickle it is01:46
redboThere's also https://github.com/hydrogen18/stalecucumber but it's kind of slow.01:47
claygstill i wonder if a there's a better serialization format for updating key's in a dictionary than a deserialize/modify/serialize pickle01:47
claygoh nice01:48
claygok then - sounds like pickle is gunna turn out to be a portable data format after all - wtg standards evolving from implementation!01:48
redboWell just adding a log to that will help a lot.  Like append suffixes to a file that says "next time you recalculate hashes, invalidate this one".01:49
redboThat will lighten the load for PUTs and DELETEs01:49
redboand push it on replication where it belongs01:50
clayghashes.pkl.pending ?01:50
clayg:P01:50
redbo:)01:52
claygso are you rewriting the replicators in go too?  or did you get nerd sniped trying to get feature parity in the proxy?01:53
redboWe're playing around with a replicator.  I don't think we'll ever get a full proxy, but if we could get a simple proxy that serves a couple of our big use cases (like serving CDN traffic) it'd be interesting.01:57
claygi think i like the .pending idea - if the object server could just start appending invalidates on writes - you could convert the hashes data format however you might like during replication01:57
redbooh, true.  But I'd like it to append "add file X/remove file Y" :)01:58
claygsounds awesome, keep me posted, let me know if there's anything I can do to help (besides... you know... breaking handoff partition cleanup :\)01:58
claygredbo: one thing at a time?01:58
clayg:P01:58
*** jamielennox is now known as jamielennox|away01:59
*** reed has quit IRC02:00
claygoh.. no, that was *one* thing - instead of append "suff x, suff x, suff x" over and over you'd go ahead and put the name an op in there - but then still do the "apply .pending" as read all the objects, squash them down to the list of suffix to invalidate, and update the hashes.pkl02:00
claygredbo: you have to go more slowly with me02:00
redbohaha yeah... just to future proof it.  Okay, $lady is here for dinner.02:02
openstackgerritSamuel Merritt proposed openstack/swift: Fix EC chunking  https://review.openstack.org/16107002:02
*** dmsimard_away is now known as dmsimard02:08
redboI mean.. don't tell her she's a variable and not a defined constant.02:11
*** timburke has joined #openstack-swift02:13
*** nexusz99 has joined #openstack-swift02:17
*** annegentle has joined #openstack-swift02:20
*** nexusz99 has quit IRC02:21
claygoh my02:24
claygtimburke: welcome to the shit!  https://review.openstack.org/#/c/161043/02:25
*** annegentle has quit IRC02:25
claygho: mattoliverau: who were the guys talking about slo md5's at the hack-a-thon?02:25
hoclayg: miyahara-san, I think. I will check first name. Please wait...02:26
hoclayg: his first name is Kazuhiro02:27
mattoliverauclayg: yeah what ho said.. but I dont know his irc nick02:27
mattoliveraudmorita: what's Kazuhiro irc nick ^^02:27
*** jamielennox|away is now known as jamielennox02:37
dmoritamattoliverau: sorry i do not know his nick but i can forward a message to him :)02:37
mattoliveraudmorita: no probs :)02:38
dmoritaAhh, patch 161043 is relating to Kazuhiro's suggestion! I will tell him this patch.02:42
claygdmorita: thanks man!02:49
*** jamielennox is now known as jamielennox|away02:52
*** jamielennox|away is now known as jamielennox02:55
*** jamielennox is now known as jamielennox|away02:56
*** jamielennox|away is now known as jamielennox02:57
*** annegentle has joined #openstack-swift03:05
*** annegentle has quit IRC03:07
*** nexusz99 has joined #openstack-swift03:09
*** gyee has quit IRC03:29
*** tsg has joined #openstack-swift03:30
claygpeluse: ok, i read over 159637 - i'm trying to think of how to exercise some of it functionaly...03:43
claygmaybe i write some scripts to play around with the suffix pickles and replication requests... maybe if I nuter _finalize_put so it doesn't call hash_cleanup_listdir I can get some interesting hashdirs03:44
*** jrichli has quit IRC03:47
*** nexusz99 has quit IRC04:00
*** haomaiwang has joined #openstack-swift04:04
*** ho__ has joined #openstack-swift04:06
*** ho has quit IRC04:06
*** dmsimard is now known as dmsimard_away04:29
*** ho has joined #openstack-swift04:34
*** ho__ has quit IRC04:35
*** openstackgerrit has quit IRC04:46
*** kkkkkkk has joined #openstack-swift04:49
*** openstackgerrit has joined #openstack-swift04:53
*** bkopilov has joined #openstack-swift05:05
*** kkkkkkk has quit IRC05:08
*** ppai has joined #openstack-swift05:17
*** zaitcev has quit IRC05:24
*** jamielennox is now known as jamielennox|away05:32
*** openstack has joined #openstack-swift05:36
openstackgerritHisashi Osanai proposed openstack/swift: Enable Object Replicator's failure count in recon  https://review.openstack.org/13834205:46
*** silor has joined #openstack-swift06:05
*** bsdkurt has quit IRC06:14
*** Sanchit has quit IRC06:17
*** bsdkurt has joined #openstack-swift06:25
*** SkyRocknRoll has joined #openstack-swift06:39
*** btorch has quit IRC06:45
*** eikke has quit IRC07:19
openstackgerritPrashanth Pai proposed openstack/swift: fsync() on directories  https://review.openstack.org/12692307:25
*** david-lyle_afk has joined #openstack-swift07:33
*** openstackgerrit has quit IRC07:49
*** openstackgerrit has joined #openstack-swift07:50
*** zacksh has quit IRC07:52
*** mmcardle has joined #openstack-swift07:59
*** rledisez has joined #openstack-swift08:17
*** zacksh has joined #openstack-swift08:17
*** chlong has quit IRC08:20
*** geaaru has joined #openstack-swift08:22
*** mmcardle has quit IRC08:25
*** mmcardle has joined #openstack-swift08:29
-openstackstatus- NOTICE: Zuul check queue stuck due to reboot maintenance window at one of our cloud providers - no need to recheck changes at the moment, they won't move forward.08:32
*** ChanServ changes topic to "Zuul check queue stuck due to reboot maintenance window at one of our cloud providers - no need to recheck changes at the moment, they won't move forward."08:32
*** jordanP has joined #openstack-swift08:48
*** jistr has joined #openstack-swift08:50
*** ho has quit IRC08:58
*** km has quit IRC09:04
*** openstack has joined #openstack-swift15:28
*** btorch has joined #openstack-swift15:38
*** mattoliverau has quit IRC15:43
*** eikke has joined #openstack-swift16:03
*** Bsony has quit IRC16:03
*** mattoliverau has joined #openstack-swift16:05
notmynamegood morning16:07
tdasilvanotmyname: morning! :)16:12
*** tsg has quit IRC16:12
mahaticgood morning!16:14
openstackgerritStuart McLaren proposed openstack/python-swiftclient: Allow reading from object body on download  https://review.openstack.org/15529116:14
openstackgerritStuart McLaren proposed openstack/python-swiftclient: Retry download of object body  https://review.openstack.org/15920816:14
openstackgerritThiago da Silva proposed openstack/swift: WIP: Use Container override header to update etag and size  https://review.openstack.org/16103016:15
notmynamemahatic: feeling better today?16:15
mahaticnotmyname, yup, quite a bit. thanks for checking :)16:16
notmynametimburke: you submitted a patch! awesome!16:16
* notmyname didn't even know you were working on one :-)16:17
notmynamelooks like the community qa cluster is back online. yay16:20
*** annegentle has quit IRC16:21
*** annegentle has joined #openstack-swift16:22
*** rdaly2_ has joined #openstack-swift16:25
*** rdaly2 has quit IRC16:29
*** mahatic has quit IRC16:42
*** mrda has quit IRC16:43
*** mahatic has joined #openstack-swift16:44
*** btorch has quit IRC16:52
*** btorch has joined #openstack-swift16:52
notmynamemahatic: yesterday I think you said you had some questions?17:02
notmynameeikke: so I see that you're working on some py26 compat issues in your diskfile adapter. how important is py26 to you?17:04
mahaticnotmyname, yeah. I'm still looking actually (at the patch). I'm doing some more clarifications. Can I ping you back in a bit?17:06
notmynameeikke: I ask because there's been some talk about dropping py26 compat post-kilo. most recently because of https://bugs.launchpad.net/swift/+bug/142577617:06
openstackLaunchpad bug 1425776 in OpenStack Object Storage (swift) "ACL tests fails on fresh Ubuntu 14.04" [Undecided,In progress] - Assigned to Christian Schwede (cschwede)17:06
notmynamemahatic: sure17:07
mahaticthanks.17:07
*** tsg has joined #openstack-swift17:12
*** jistr has quit IRC17:13
notmynametsg: good morning! (I have questions for you)17:15
-openstackstatus- NOTICE: Issue solved, gate slowly digesting accumulated changes17:17
*** nellysmitt has quit IRC17:18
*** mmcardle has quit IRC17:23
tsgnotmyname: morning! how is it going17:26
notmynametsg: good17:26
notmynametsg: as we're getting closer to EC being done, getting pyeclib available on pypi is getting more important. so 2 questions:17:27
notmynametsg: when will the current (1.0RC?) be available in pypi?17:28
notmynametsg: I'm told that the current version on pypi (0.9.8) still does external calls on install. is that true? if so, when will that be fixed?17:28
tsgnotmyname: to get pypi version up-to-date, there is a dependency on liberasurecode - that needs to be added to jenkins slaves or else pyeclib 1.0 will not work.  liberasurecode is available as a Debian package now - I have been trying to understand from zigo what's needed from a package availability standpoint17:30
notmynamegot it. thanks17:31
tsgif the package has to be pulled into ubuntu repos to be a part of jenkins slaves ..17:31
notmynametsg: and thanks for working on it17:31
mahaticnotmyname, I have a basic question: Shouldn't the ring take care of all the servers no matter which storage policy?17:32
notmynametsg: I'd love to see some sort of resolution this week so that there are fewer barriers for testing the EC work17:32
tsgnotmyname: no problem - zigo not being reachable for the past few days has made it go a bit slow17:32
tsgnotmyname: trying my best17:32
tsgwill talk to clarkb on some alternatives17:32
notmynamemahatic: there's a different object ring for each storage policy. I'd love to see swift-recon be able to identify the storage policy and then it will pick up the appropriate ring17:33
notmynametsg: thanks17:33
mahaticoh okay17:33
notmynamemahatic: and it looks like ho and dmorita are working on some recon updates (they're both in japan and not online right now)17:34
mahaticnotmyname, https://review.openstack.org/#/c/138697/ This is one is in progress only for auditor-check I believe?17:34
notmynamemahatic: ya, there are actually 2 parts to policy-aware recon17:35
mahaticnotmyname, oh17:35
notmynamemahatic: one, like dmorita is working on there, is to have the recon middleware report back on info found for any storage policy on a given host17:35
notmynamemahatic: the other is on the CLI side knowing which host to actually query. that's where the different rings come into play17:36
notmynamedoes that make sense?17:37
mahaticnotmyname, okay. But through the first part you mentioned, it would also report the host right?17:38
*** rledisez has quit IRC17:38
notmynameI don't follow what you mean17:39
mahaticnotmyname, yeah, my question was wrong. I sort of understood what you said. But I do see a huge gap in "finding out which host to query"17:41
notmynameI think with dmorita's patch right now it will only find the per-policy auditor info for hosts that happen to be in the default storage policy 0 and also in some other policy17:42
notmynamebut if there are hosts that are not in the legacy storage policy at all, then swift-recon has no way to query them17:43
notmynameI think17:43
mahaticI think i'm really missing something. Is it not that, you can only have one storage policy as a default?17:43
mahaticoh17:44
mahaticno, object can have one, container can have another policy, etc correct?17:44
notmynameno17:45
notmynamepolicies are only for objects17:45
notmynamebut the "legacy" policy is storage policy 0 (that's what I meant by default. sorry that was the wrong word)17:45
*** gyee has joined #openstack-swift17:47
*** Nadeem has joined #openstack-swift17:47
mahaticokay. the thing you said about dmorita's patch makes sense.17:48
notmynamemahatic: so as I see it right now, there's not a way to do a `swift-recon` with anything other than storage policy 0. ie the one with the object.ring.rz. you can't load and query hosts that are referenced in object-N.ring.gz17:48
acolespeluse: clayg: around?17:49
mahaticright17:49
mahaticnotmyname, I understand on a higher level, but not so much on the technical one. I'll need to look. But I got what you mean17:50
notmynamemahatic: then you're at the same place as me :-)17:51
mahaticlol17:51
mahaticnotmyname, well, I'll try to get ahead of you on that :)17:52
mahaticnotmyname, what I also meant was, I haven't really looked at how rings for different policies are done and/or accessed. I'll start from there.17:53
*** erlon has joined #openstack-swift18:00
mahaticnotmyname, This is sure to keep me busy tomorrow! I'm calling it a night for now. Will check the meeting log later.18:01
notmyname:-)18:01
notmynameso at this week's team meeting (in 1 hour), we'll discuss EC progress. and we should make a call on if we are committed to an EC beta in Kilo or not18:04
mahaticyeah, gotta sleep before my running nose gets the better of me :D18:04
*** mahatic has quit IRC18:06
*** ChanServ changes topic to "Review Dashboard: http://goo.gl/uRzLBX | Overview Dashboard: http://goo.gl/9EI0Sz | Priority Reviews: https://wiki.openstack.org/wiki/Swift/PriorityReviews | Ideas: https://wiki.openstack.org/wiki/Swift/ideas"18:07
tsgnotmyname: just talked to clarkb on -infra.  It appears that the package has to be a part of the standard Ubuntu repos.18:09
tsgnotmyname: typically Debian packages are syned to Ubuntu on every (dot) release - doesn't look like liberasurecode was synced on 14.04.2 (that went out on 02/19) though.  I just filed an Ubuntu bug to sync the package.18:09
notmynamehmm18:09
notmynametsg: so what if it's not? does that mean we can't use pyeclib and therefore can't test any ec changes in the gate?18:10
notmynametsg: also, that makes me wonder what ec testing is on the feature/ec branch today18:10
tsgnotmyname: 0.9.8 is working well right now18:10
notmynameok18:10
tsgwe are all testing using 0.9.818:10
tsg(and Kevin/I/Kota with 1.0 also, ofcourse)18:11
notmynametsg: ah, ok. so 0.9.8 is in pypi. and the right c library for that is in repos? it's just updated versions?18:11
notmynameor is there no c library component in 0.9.8?18:11
tsgnotmyname: the C code is all part of pyeclib in 0.9.8, not split out into liberasurecode18:12
notmynameoh ok18:12
tsgnotmyname: that was done on Debian/RH guys' request18:12
tsgplus it was also good to create a generic C API library as well18:13
notmynameok18:13
notmynametsg: and the question around the external calls?18:13
notmynameduring install18:13
tsgthat's the other thing we changed in liberasurecode18:13
notmynamethe split of the c library has the change that removes the external calls at install?18:14
tsgwe designed it with dl so that the compilation will not require pulling all the dependencies of the Internet18:14
tsgyes18:14
notmynameok18:14
notmynamewhat's "dl"?18:14
tsgdynamic loader18:15
notmynameah ok18:15
notmynametsg: how worried should I be about this impacting an EC release schedule? :-)18:15
notmyname(cause it sounds troublesome)18:15
tsgnotmyname: because the Debian package for liberasurecode and jerasure is already, it shouldn't be18:17
tsgI will find out asap18:17
*** logan2 has quit IRC18:18
notmynametsg: is my understanding correct? EC in swift will require pyeclib 1.0 and 0.9.8 will be unsuitable. and therefore the risk is that it will not be synced from debian to ubuntu in time?18:18
notmyname(/me thinks the remote loading in 0.9.8 is "unsuitable")18:19
tsgnotmyname: hm clarkb didn't seem completely against it18:19
*** jordanP has quit IRC18:20
tsgnotmyname: but yes, we should move to 1.0 asap given the API changes/fixes it brings18:20
notmynametsg: ok18:20
notmynametsg: we have 4 weeks to get this packaging/dependency stuff resolved, if we are going to make kilo. do you think that's reasonable?18:22
notmynametsg: oh, actually less than that. that is for our swift code. the feature freeze is before that. let me find that date18:22
tsgnotmyname: I think so, if I can get hold of zigo :)18:23
tsgnotmyname: I will try to talk to ubuntu folks today18:23
notmynamemarch 19 for the openstack dependency freeze18:23
notmynameso 15 calendar days18:23
notmyname2 weeks18:23
notmynametsg: pyeclib 0.9.8 is in global requirements right now and installed on the openstack CI boxes?18:26
tsgnotmyname: yes - it is installed on CI slaves18:26
notmynameok, thanks18:26
tsg(what our EC tests have been using)18:27
tsgbrb18:27
*** tsg has left #openstack-swift18:27
*** tsg has joined #openstack-swift18:27
*** geaaru has quit IRC18:28
*** acoles is now known as acoles_away18:29
*** zhill has joined #openstack-swift18:34
openstackgerritThiago da Silva proposed openstack/swift: Use Container override header to update etag and size  https://review.openstack.org/16103018:48
*** gyee has quit IRC18:49
*** tab____ has joined #openstack-swift18:50
*** cutforth has joined #openstack-swift18:52
*** rdaly2_ has quit IRC18:55
*** kota_ has joined #openstack-swift18:57
*** zhill_ has joined #openstack-swift18:58
mattoliveraumorning18:58
notmynamemattoliverau: hello18:58
kota_morning18:59
notmynamemattoliverau: thanks for waking up early :-)18:59
notmynamekota_: wow. you're up really early too!18:59
notmynamemore than mattoliverau18:59
kota_notmyname: :)18:59
mattoliverauyeah, go kota_18:59
zaitcevTZ=Japan date  Thu Mar  5 03:59:57 JST 201519:00
notmynameok, meeting time in #openstack-meeting19:00
zaitcevtaihensoudesu19:00
kota_zaitcev: no worries :)19:01
notmynameclayg: torgomatic: acoles_away: cschwede: dfg: redbo: meeting ping19:01
notmynamepeluse: ^19:02
*** zhill_ has quit IRC19:03
notmynametsg: ^19:03
*** acoles_away is now known as acoles19:05
*** lpabon has joined #openstack-swift19:15
*** monn has joined #openstack-swift19:15
*** gvernik has joined #openstack-swift19:17
*** ChanServ changes topic to "Hackathon: https://www.eventbrite.com/e/openstack-swift-hack-a-thon-sponsored-by-swiftstack-tickets-15226147818 | Priority Reviews: https://wiki.openstack.org/wiki/Swift/PriorityReviews | Review Dashboard: http://goo.gl/r2mxbe"19:17
*** gyee has joined #openstack-swift19:19
claygis that now?19:21
tdasilvay19:22
acolesclayg: yup :D19:22
*** Bsony has joined #openstack-swift19:24
*** aix has quit IRC19:24
*** gvernik has quit IRC19:25
acolesclayg: still here?19:32
notmynamemeeting notes: http://eavesdrop.openstack.org/meetings/swift/2015/swift.2015-03-04-19.00.html19:32
claygacoles: sure19:32
notmynameoh, note that the review dashboard links in the topic have a "starred" section for priority reviews. I meant to mention that in the meeting19:33
acolesclayg: hi. was saying in meeting - i can spend more time on https://review.openstack.org/#/c/159637 but saw your comment in scrollback about refactoring into per-policy diskfiles19:33
acolesclayg: so.... am i wasting my time? peluse thinks not19:34
acolesclayg: fwiw i see where you are coming from, there's plenty of 'if this policy or that'19:34
acolessome of which i managed to puch out into plicy classes19:35
acolesnotmyname: cool19:35
claygacoles: i'm debating this with myself19:35
claygacoles: there's a lot of code in the ec diskfile that will basically be exactly the same but do this one extra little thing different19:35
acolesclayg: yup19:36
claygacoles: in some respects I think it'd be better to just duplicate it - in others I think we'll find ourselves wanting to make the same changes in two places :'(19:36
claygacoles: either way - I think we should keep hacking on that patch19:36
claygacoles: I'm going to try and contribute by cleaning up the abstraction of the new suffix class thing - based on previous ideas about improving object replciation it seems like we could move that abstraction around to make it easier to 1) do a different thing for ec 2) make it easier to do a different thing for replication19:37
acolesclayg: ok, i have not looked at the suffix class, so no overlap there19:38
acolesclayg: isit worth splitting the patch up into smaller pieces or is it all inter-dependent?19:38
claygacoles: my experience reiewing proxy changes to accomodate ec may be coloring my view - in the proxy the ec path is much different than in the object server path19:38
claygacoles: I think peluse is already trying to split it up19:38
acolesclayg: yeah, don't get me wrong, i sympathise, just wondering what best way to use my effort is19:39
claygacoles: I also think getting something merged onto feature/ec is a higher priority than trying to split up the change for master19:39
acolesclayg: agree19:39
clayger... rather we'll have to rethink how to present the logical changes when we push to master - so for now I'm more interested in just running as fast as we can to "works" on feature/ec19:39
claygif it changes before master it won't make a difference - the master change log will only show the final state19:40
acolesclayg: ok so a plan: tomorrow i will fix get_ondisk_files and hash_cleanup some more, and look for othe rplaces where ec policy behavior can be encapsulated into policy class, and where we can resolve policy and stop passing around the policy_index19:41
*** kota_ has quit IRC19:41
acolesclayg: i'll leave suffix hash stuff alone.19:41
acolesclayg: that sound ok?19:41
claygacoles: sounds great, thanks for the help!19:43
acolesclayg: cool, i love a plan!19:43
acolesjrichli: i reviewed your middleware patch, but as an afterthought, maybe focus on the en/decrypter pieces for now and not worry about trivialkeymaster. Thats easily separable.19:44
jrichliacoles: understood19:46
jrichliacoles: and thanks for reviewing!  I will go over the comments now.19:46
*** ChanServ sets mode: +v mattoliverau19:47
acolesjrichli: np. have a good afternoon, i'm done for today.19:47
jrichlihave a good evening!19:47
*** acoles is now known as acoles_away19:47
mattoliveraunight acoles!19:48
mattoliverauor now acoles_away ^^19:48
*** Bsony has quit IRC19:48
*** EmilienM is now known as EmilienM|afk19:56
*** reed has joined #openstack-swift19:56
*** devlaps has joined #openstack-swift19:58
*** tsg has quit IRC20:10
*** tsg has joined #openstack-swift20:10
*** rdaly2 has joined #openstack-swift20:10
*** tsg has quit IRC20:16
claygpeluse: I don't think that tombstones really have #frag_index parts in their filename?20:23
openstackgerritOpenStack Proposal Bot proposed openstack/swift: Updated from global requirements  https://review.openstack.org/8873620:23
*** logan2 has joined #openstack-swift20:29
*** chlong has joined #openstack-swift20:29
notmynamewell, now it's real. I just sent out a status update to ${not_swift_devs} about having an EC beta in Kilo20:38
notmynamegrrr...looks like the openstack bot resurrected a stale topic message20:39
openstackgerritSamuel Merritt proposed openstack/swift: Fix EC chunking  https://review.openstack.org/16107020:39
*** ChanServ changes topic to "Review Dashboard: http://goo.gl/uRzLBX | Overview Dashboard: http://goo.gl/9EI0Sz | Priority Reviews: https://wiki.openstack.org/wiki/Swift/PriorityReviews | Ideas: https://wiki.openstack.org/wiki/Swift/ideas"20:40
*** dmsimard is now known as dmsimard_away20:53
*** Bsony has joined #openstack-swift20:55
*** silor has quit IRC20:56
*** Bsony has quit IRC20:59
claygnotmyname: is there any progress on py26 as a gate check?  do we run our tests on the feature/ec branch?  assertGreaterEqual :'(21:14
notmynameclayg: there is no py26 check in the official gate. (and there will not be). I've mentioned it to charz to add it to the community cluster testing21:15
notmynameI just saw this am that the community cluster is back to doing tests and reporting. so I think that can be added to it21:15
claygnotmyname: yeah that's what I ment21:16
notmynameso now that the cluster is back online, I'll follow up with him asap21:17
*** tsg has joined #openstack-swift21:18
*** tsg_ has joined #openstack-swift21:20
*** Anticimex has quit IRC21:21
*** sandywalsh has quit IRC21:23
*** tsg has quit IRC21:23
*** sandywalsh has joined #openstack-swift21:24
*** Anticimex has joined #openstack-swift21:27
*** rdaly2 has quit IRC21:29
*** monn has quit IRC21:31
*** EmilienM|afk is now known as EmilienM21:43
*** tsg_ has quit IRC22:02
*** tsg_ has joined #openstack-swift22:03
*** tsg_ has quit IRC22:08
*** annegentle has quit IRC22:10
*** annegentle has joined #openstack-swift22:11
* notmyname goes offline for a while22:12
*** chlong has quit IRC22:12
*** jamielennox|away is now known as jamielennox22:14
*** erlon has quit IRC22:41
*** Anticimex has quit IRC22:49
*** tsg has joined #openstack-swift22:50
*** Anticimex has joined #openstack-swift22:51
claygnotmyname: not sure if you're around - but something is going on with the gate not testing python-swiftclient patches maybe?23:00
*** annegentle has quit IRC23:04
claygnm, turns out if you scroll all the way down to the bottom of http://status.openstack.org/zuul/ the graphs tell the story - bunch of vms got turned off last night at rax and there's some issues getting new ones up23:07
*** jrichli has quit IRC23:10
*** tsg_ has joined #openstack-swift23:17
*** tsg has quit IRC23:19
*** annegentle has joined #openstack-swift23:25
*** annegentle has quit IRC23:32
*** annegentle has joined #openstack-swift23:33
claygnotmyname: you back?  my +3 on https://review.openstack.org/#/c/161070/ didn't work, it's still not merged23:38
claygacoles_away: peluse: cschwede: mattoliverau: zaitcev: redbo: dfg: glange: ^23:41
claygswifterdarrell: ^23:41
*** Nadeem has quit IRC23:41
*** tab____ has quit IRC23:42
*** chlong has joined #openstack-swift23:42
*** km has joined #openstack-swift23:42
*** annegentle has quit IRC23:43
*** annegentle has joined #openstack-swift23:43
*** chlong has quit IRC23:46
*** chlong has joined #openstack-swift23:46
*** annegentle has quit IRC23:48
mattoliverauclayg: will take a loom23:49
mattoliverauLook even23:50
*** Tahmina has joined #openstack-swift23:50
* clayg hugs mattoliverau 23:50
mattoliverauAww23:52
*** EmilienM is now known as EmilienM|afk23:56
*** annegentle has joined #openstack-swift23:56
*** annegentle has quit IRC23:57
*** annegentle has joined #openstack-swift23:57
*** annegentle has quit IRC23:58
*** dmsimard_away is now known as dmsimard23:58
openstackgerritTim Burke proposed openstack/python-swiftclient: Compare each chunk of large objects when uploading  https://review.openstack.org/16104323:58
*** Tahmina has quit IRC23:58
*** dmsimard is now known as dmsimard_away23:58

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