Tuesday, 2015-03-03

mattoliveraunotmyname: nice!00:01
notmynamethe "starred by any core reviewer", in my mind, is "what are you thinking about and looking at". ie more for me :-)00:02
*** ho has joined #openstack-swift00:03
*** cl__ has joined #openstack-swift00:06
*** annegentle has quit IRC00:14
*** chlong has quit IRC00:15
*** zaitcev has joined #openstack-swift00:20
*** ChanServ sets mode: +v zaitcev00:20
hogood morning guys00:20
mattoliverauho: morning00:22
openstackgerritOpenStack Proposal Bot proposed openstack/python-swiftclient: Updated from global requirements  https://review.openstack.org/8925000:22
homattoliverau: morning!00:22
openstackgerritOpenStack Proposal Bot proposed openstack/swift: Updated from global requirements  https://review.openstack.org/8873600:22
*** tsg has joined #openstack-swift00:50
*** jrichli has joined #openstack-swift00:55
*** ujjain has quit IRC00:57
*** annegentle has joined #openstack-swift01:17
openstackgerritTushar Gohad proposed openstack/swift: EC: Multi-phase commit conversation for PUTs  https://review.openstack.org/15384301:20
tsgpeluse, torgomatic: ^^^^ (got rid of the diskfile changes and also removed the eventlet patch dependency)01:22
*** nellysmitt has joined #openstack-swift01:25
openstackgerritMatthew Oliver proposed openstack/swift: Fixed links in multi-server Swift documentation  https://review.openstack.org/16019601:26
*** annegentle has quit IRC01:29
*** nellysmitt has quit IRC01:30
charzclayg: That's similar one, thanks! I'll add unit tests to the patch.01:40
*** annegentle has joined #openstack-swift01:45
*** dmorita has joined #openstack-swift01:59
*** dmsimard_away is now known as dmsimard02:01
*** zhill has quit IRC02:02
*** annegentle has quit IRC02:03
*** rmcall has joined #openstack-swift02:06
*** omame has quit IRC02:16
claygcharz: awesome!02:20
*** panbalag has quit IRC02:21
*** annegentle has joined #openstack-swift02:30
*** annegentle has quit IRC02:36
*** dmsimard is now known as dmsimard_away02:40
*** tsg has quit IRC02:47
*** MVenesio has joined #openstack-swift03:06
thurloatawsum03:07
*** amandap is now known as amandap_03:24
*** nellysmitt has joined #openstack-swift03:26
openstackgerritClay Gerrard proposed openstack/swift: Support of the Linux socket option SO_REUSEPORT  https://review.openstack.org/13765903:28
openstackgerritClay Gerrard proposed openstack/swift: wip: cleanup SO_REUSEPORT  https://review.openstack.org/16062103:28
*** nellysmitt has quit IRC03:31
*** amandap_ is now known as amandap03:33
*** MVenesio has quit IRC03:47
*** MVenesio has joined #openstack-swift03:47
openstackgerritMerged openstack/swift-specs: Composite Tokens and Service Accounts are done  https://review.openstack.org/15942503:50
*** jrichli has quit IRC03:54
*** bkopilov has quit IRC03:54
claygwhooooo a done spec!04:12
notmyname:-)04:12
claygmattoliverau: why you messing with docs - it's such a bikeshed fiesta04:13
*** tanee has quit IRC04:14
mattoliverauclayg: sigh.. i know.. I was just auditing bugs, found one that wasn't an issue anymore, but there was a problem with linking so thought I'll submit a simple patch so we can close it. #canofworms04:14
openstackgerritClay Gerrard proposed openstack/swift: Fixed links in multi-server Swift documentation  https://review.openstack.org/16019604:15
claygoh no... i get it... I'm totally enthralled now04:16
claygI'm just saying if you were to just try harder to ignore it - we wouldn't be in this mess!04:16
mattoliverauclayg: lol04:17
clayghonestly I think i'm just happy for any excuse to play with my autodoc script on vagrant-swift-all-in-one, plus I didn't like the way the one sentance page rendered with a big grey square?04:18
* mattoliverau wonders if he should point out to clayg that debain install guide might be valid for old releases... but how can we tell :P04:18
mattoliverauclayg: yeah I noticed that too.. however, there was still annoying grey on the old version, I'm going to have to build your patchset docs and look now :P04:18
claygmattoliverau: yeah i'm down - if there's an old link that still works we could totally add it04:18
claygmight be useful if someone wanted to wing it with juno04:19
clayganother great reason to keep the "all the swift docs" links on one page we maintain - do it!04:19
claygsee04:19
claygnext time you'll remember it's better to put your head in the ground04:19
mattoliveraulol04:19
claygportante and zaitcev got on kick fixing up the swift-all-in-one docs one time and we lost *weeks* i tell you!04:20
mattoliveraunoted.. there is no such thing as, "oh it'll be simple change" when it comes to docs :P04:21
claygthat's the learning you only get at school of hard knocks brother04:22
mattoliverauteacher my teacher04:24
*** tanee has joined #openstack-swift04:24
* clayg is just funning around04:25
claygdoc changes are a whole different level of difficultly than your average "add new behavior to distributed storage system running production peta-byte scale clusters in a variety of deployemnts around the globe" change04:27
notmynamelol04:29
claygnotmyname knows what clayg is talking about04:29
notmyname+100004:29
*** MVenesio has quit IRC04:30
openstackgerritMatthew Oliver proposed openstack/swift: Fixed links in multi-server Swift documentation  https://review.openstack.org/16019604:36
mattoliverauclayg: debian link added to havana ^^04:36
claygwhat the crap is this ec multi-phase put:  'Object upload failed: remote and local object md5 did not match, 4f9e5b770657de3d48671d751dac2b7c != b4f01a1d0ba0b46b05c73b2ac909b1df\nremote object has not been removed.'04:38
claygalso what gives with the \n04:38
claygwhat branch am I on04:38
*** sandywalsh_ has quit IRC04:56
*** sandywalsh has joined #openstack-swift04:58
*** tsg has joined #openstack-swift05:02
*** david-lyle_afk has joined #openstack-swift05:06
*** ppai has joined #openstack-swift05:09
homattoliverau: the link for Juno/Suse should be http://docs.openstack.org/juno/install-guide/install/zypper/content/ch_swift.html05:11
homattoliverau: other links work without no problem05:12
hos/without no problem/with no problem/05:14
mattoliverauho: nice spot! new patchset coming05:18
openstackgerritMatthew Oliver proposed openstack/swift: Fixed links in multi-server Swift documentation  https://review.openstack.org/16019605:19
claygdoes wakefield hang out in here?05:20
clayghmph - does Joel?05:20
*** bkopilov has joined #openstack-swift05:20
notmynameclayg: ya, I think I've seen joel's nick. but not seen him speak05:21
notmynamejoeljwright05:22
claygah, yeah doesn't seem to be around atm - thanks05:22
*** zaitcev has quit IRC05:22
mattoliveraunotmyname, clayg do you guys ever sleep?05:22
notmynamemattoliverau: I don't htink clayg does ;-)05:22
claygi'm basically asleep now05:22
mattoliveraulol05:23
notmynamelol05:23
claygidk, why I keep acting like it's a good idea to avoid setting up a seperate vm for ec...05:23
notmynamesome idea that ec is supposed to work with existing swift?05:24
claygit's not even that hard with a seperate clone of vagrant-swift-all-in-one - I think I just have to change the ip in localrc05:24
claygbut like for some reason I keep thinking i should be able to have one vm that does both?  why do I keep thinking that?05:24
notmynameclayg: last time I looked at vSAIO, I thought thee would be a nice use for support of multiple config files05:25
claygoh vSAIO i like that - sounds like something sony built05:25
claygalso I don't think i know what you mean :P05:26
notmynamewell if it reveals anything about how recently I looked at it, I'm 18 commits behind your upstream05:27
*** nellysmitt has joined #openstack-swift05:27
claygho: how you feelin on mattoliverau's last change to multi-node doc links - i'm bout ready to put it to bed!05:27
claygacpahpcahcapahcpahpcahph - you don't even have a opensource license05:28
hoclayg: I checked all links and they worked.05:29
claygho: ditto!05:29
claygspeaking of seperate vSAIO for ec - doug just added that thing to specify a branch - i wonder if I can set it to feature/ec05:30
*** nellysmitt has quit IRC05:31
notmynameso I just looked in your github account for the vsaio. then I realized it's in swiftstack's. then I realized I can just merger stuff there ;-)05:32
* notmyname tries to remember what a github workflow looks like05:33
claygmerger stuff - or murder stuff?05:34
notmynameboth? ;-)05:34
notmynamehttps://github.com/swiftstack/vagrant-swift-all-in-one/pull/1405:35
claygnotmyname: well but *mine* is acctually called localrc05:36
notmynamedone05:37
notmyname(that was easy)05:37
claygdork05:37
notmynameI'm just enjoying pushing code and having it auto-update a pull request ;-)05:38
claygman, i sure screwed up dipping my toe in the abandon ML thread05:39
notmynameclayg: while I do agree with you, I think fifieldt has some good wisdom there05:39
claygi like tom a lot05:39
fifieldtwisdom, me?05:39
fifieldtimpossible05:39
claygoh shit - didn't realize he was listening - i take it back05:40
notmynamelol05:40
fifieldt:D05:40
fifieldtare you feeling ok clayg?05:40
fifieldtI haven't read many replies after yours05:40
fifieldtI hope I didn't offend with mine :(05:40
clayglol, i'm *really* hard to offend05:40
mattoliverauits true, I've been trigin to offend him for months now.05:41
mattoliverau*tryin05:41
claygi was just realizing that the whole notion of "abandoning a change might have a negative connotative signal" is not the realm where I can contribute in a meaningful way05:41
fifieldtpersonally I still think it was worthwhile05:42
fifieldtyou're not wrong in your post :)05:42
claygtotally useful - *i'm* not *helpful* with that problem05:42
notmynamefifieldt: I think it does come down to the fact that an action is taken. but to your point, the phrasing is important. like with the change in working a while back to the -1 text in gerrit05:43
claygthe only reason I jumped in was to say that if we don't abandon them they'll just pile up05:43
notmynameie when abandoning something, it's not just words. a patch by someone is actually abandoned05:43
fifieldtindeed! it is fair to use the abandon button for abandoned things :) and I'm not entirely convinced by the "let's make a new dashboard to fix this" argument either05:43
*** david-lyle_afk has quit IRC05:44
* fifieldt shrugs05:44
fifieldtstorm in a teacup?05:44
fifieldthmm, sorry, I think that is supposed to be "Tempest in a teapot"05:45
claygho: where is everyone making these charts that don't render for me05:45
notmynamefifieldt: actually that's where I'd defer to you. from my american perspective I think it is as you say. from a non-western view? I don' t know. I defer to you05:46
*** SkyRocknRoll has joined #openstack-swift05:46
claygho: OH!  no i get it, 1 & 3 don't exist - delete_objs is always None if you use rsync ;)05:47
fifieldtI can't say I have much more to offer :D05:47
fifieldtargh. you know that time when you open a bunch of reviews in gerrit only to realise you're not logged in? hate that.05:48
mattoliveraunotmyname, clayg: if your still here, are you guys happy if I +A tsg's EC PUT patch, peluse has +2, and in my limitted PUT testing its working. I'm happy to +2 to get it moving and do more testing once EC is more complete. I ask cause its sounds like (from earlier channel posts) clayg might still be looking at it.06:07
mattoliverauif clayg is still iterating on it, I'll just +2 it and leave it to him to +A06:07
claygwell i started to look at it06:07
mattoliverauclayg: before getting sidetracked on vSAIO? :P06:08
* notmyname isn't currently looking at it06:08
claygmattoliverau: I'd so go ahead and land it if you like it - paul also said something about dlo doesn't work?  it's whatever - we'll fix it all eventually06:08
mattoliverauyeah, DLO will be fixed in a later patch, or something, in my testing I was getting wierd GET behaviour, specifically not using the .durable files, but that is nothing to do with this patch, and will obviously be something added once this lands.06:11
mattoliveraunotmyname: noted :)06:11
mattoliverauK, I need to go make a start on dinner so it'll be ready by the time the wife gets home.06:17
*** ppai has quit IRC06:22
claygmattoliverau: idk, when i test it i get md5 mismatch after a round trip so idk06:22
*** gyee has quit IRC06:22
mattoliverauOh really.. Bugger.. I'll have to jump on my box and take another look.. Maybe that should be a test then06:32
*** ppai has joined #openstack-swift06:33
mattoliverauclayg: when I put a file and then pull it out again md5s match... So hmm.06:39
*** nellysmitt has joined #openstack-swift06:42
*** david-lyle_afk has joined #openstack-swift06:44
*** jroll has quit IRC06:49
openstackgerritMerged openstack/swift: Fixed links in multi-server Swift documentation  https://review.openstack.org/16019606:52
*** reed has quit IRC06:58
*** rmcall has quit IRC06:58
tsgmattoliverau: are you using swift client to test ec put/gets?06:59
*** jroll has joined #openstack-swift06:59
*** mrda is now known as mrda-away07:04
*** SkyRocknRoll has quit IRC07:04
*** mmcardle has joined #openstack-swift07:05
*** SkyRocknRoll has joined #openstack-swift07:18
*** SkyRocknRoll has quit IRC07:18
*** SkyRocknRoll has joined #openstack-swift07:18
*** foexle has joined #openstack-swift07:20
*** openstackgerrit has quit IRC07:22
*** openstackgerrit has joined #openstack-swift07:22
mattoliverauYup.. And curl07:29
*** AnxiousGarlic has joined #openstack-swift07:29
*** AnxiousGarlic has left #openstack-swift07:30
*** ppai has quit IRC07:34
openstackgerritMerged openstack/swift: EC: Multi-phase commit conversation for PUTs  https://review.openstack.org/15384307:41
*** ppai has joined #openstack-swift07:48
*** nshaikh has joined #openstack-swift08:06
*** rledisez has joined #openstack-swift08:08
openstackgerritCharles Hsu proposed openstack/python-swiftclient: Add ability to download objects to particular folder.  https://review.openstack.org/16028308:11
*** jistr has joined #openstack-swift08:46
*** acoles_away is now known as acoles08:47
*** geaaru has joined #openstack-swift08:48
*** omame has joined #openstack-swift09:01
*** JoshNang has quit IRC09:07
*** jordanP has joined #openstack-swift09:23
*** ppai has quit IRC09:49
*** foexle has quit IRC09:51
*** ppai has joined #openstack-swift10:03
*** foexle has joined #openstack-swift10:08
*** dmorita has quit IRC10:13
*** ujjain has joined #openstack-swift10:25
*** ujjain has quit IRC10:25
*** ujjain has joined #openstack-swift10:25
*** bkopilov has quit IRC10:54
*** ho has quit IRC10:56
openstackgerritOpenStack Proposal Bot proposed openstack/swift: Updated from global requirements  https://review.openstack.org/8873611:03
*** tellesnobrega has quit IRC11:27
*** km has quit IRC11:28
openstackgerritAlistair Coles proposed openstack/swift: Multiple fragment Archive Index Support  https://review.openstack.org/15963711:29
acolespeluse: ^^11:29
*** tellesnobrega has joined #openstack-swift11:33
*** EmilienM|afk is now known as EmilienM12:05
peluseacoles, cool!  I'll look a bit later this morning.  Have a packed schdule here coming up12:25
*** MVenesio has joined #openstack-swift12:26
acolespeluse: np. i think there are some opportunities to simplify some code but that can wait. important thing now is to validate that I got the ondisk file search 'algorithm' correct.12:28
acolespeluse: i didn't look at any of the suffix_hash stuff in that review, maybe get to that later today12:28
*** mmcardle has quit IRC12:30
*** aix has quit IRC12:44
*** tongli has joined #openstack-swift13:03
*** jroll has quit IRC13:07
*** jroll has joined #openstack-swift13:07
*** Bsony has joined #openstack-swift13:09
*** mmcardle has joined #openstack-swift13:14
*** lpabon has joined #openstack-swift13:17
openstackgerritAlistair Coles proposed openstack/swift-specs: Updates to encryption spec  https://review.openstack.org/15431813:21
*** Bsony has quit IRC13:22
*** aix has joined #openstack-swift13:23
*** Bsony has joined #openstack-swift13:23
*** panbalag has joined #openstack-swift13:28
*** stockpirate has joined #openstack-swift13:31
peluseacoles, OK, on the sufix_hash stuff note the big comment there that could result is big simplication of the class.  Might not even want separate subclasses but I hesitate to mke that change until we get everything fully working on one branch...13:42
acolespeluse: ok, noted thx13:42
peluseacoles, what's not in use at all right now is the 2nd by_fi dict13:42
peluseacoles, and really the only thing left in the reconstructor (functinliaty) is local file cleanup of some corner cases so I doubt we'll need it.  But I really don't want to rip it out and have to put it back in (lots of little changes to test code) so that's why I'm waiting :)13:43
acolespeluse: makes sense!13:43
* peluse off to shuttle kids to school... 13:44
*** aix has quit IRC13:44
*** aix has joined #openstack-swift13:44
*** jbonjean has quit IRC13:46
*** jbonjean has joined #openstack-swift13:46
*** jbonjean has quit IRC13:47
*** jbonjean has joined #openstack-swift13:48
*** jbonjean has quit IRC13:48
*** jbonjean has joined #openstack-swift13:48
*** nshaikh has quit IRC13:49
*** bill_az has joined #openstack-swift14:00
*** mahatic has joined #openstack-swift14:09
*** rdaly2 has joined #openstack-swift14:11
*** dmsimard_away is now known as dmsimard14:17
*** ppai has quit IRC14:20
*** chlong has joined #openstack-swift14:27
*** mahatic has quit IRC14:29
*** anayag has joined #openstack-swift14:33
*** tongli has quit IRC14:36
anayagHi I am doing some experiments with swift. My cluster has 5 storage nodes and 1 proxy. I performed 90% writes and 10% reads to the cluster using ssbench having Read Quorum 1 Write Quorum 5 and Read Quorum 5 and Write Quorum 1BUt I could not observe any significant change at all14:37
anayagcould anybody explain me this behaviour?14:37
openstackgerritAlistair Coles proposed openstack/swift-specs: Updates to encryption spec  https://review.openstack.org/15431814:40
*** mahatic has joined #openstack-swift14:42
*** MVenesio_ has joined #openstack-swift14:44
*** bkopilov has joined #openstack-swift14:45
*** MVenesio has quit IRC14:47
*** MVenesio has joined #openstack-swift15:05
*** MVenesio_ has quit IRC15:08
openstackgerritAlistair Coles proposed openstack/swift: Fix tempauth acl checks when simplejson has no speedups  https://review.openstack.org/15953015:13
*** jasondotstar has joined #openstack-swift15:16
*** tsg has quit IRC15:17
openstackgerritCharles Hsu proposed openstack/python-swiftclient: Add ability to download objects to particular folder.  https://review.openstack.org/16028315:19
*** rdaly2 has quit IRC15:26
*** rdaly2 has joined #openstack-swift15:33
*** jrichli has joined #openstack-swift15:41
*** annegentle has joined #openstack-swift15:42
peluseacoles, there?15:56
acolespeluse: here15:56
pelusehey great job the GOF/HCL stuff!15:56
pelusestill reviewing the test code then need to rebase it again since multiphase landed w/some changes, then I'll add it as a dependecy to the reconstructor and test from there....15:57
acolespeluse: np. i rebased it this morning so I think its based on the multiphase commit. in fact, i used diskfile.write_durable_timestamp!15:58
*** tsg has joined #openstack-swift15:59
*** rdaly2 has quit IRC15:59
*** rdaly2 has joined #openstack-swift16:00
peluseacoles, ok cool.  I thought i saw something that was outdated but may have been mistaken.  really like test GOF tests too, looking through those now16:01
*** lpabon has quit IRC16:01
*** rmcall has joined #openstack-swift16:02
*** tsg_ has joined #openstack-swift16:02
*** lpabon has joined #openstack-swift16:03
*** ekarlso has quit IRC16:03
*** tsg has quit IRC16:05
*** annegentle has quit IRC16:06
*** annegentle has joined #openstack-swift16:06
*** ekarlso has joined #openstack-swift16:07
*** nexusz99 has joined #openstack-swift16:08
*** homegrown_ has joined #openstack-swift16:10
*** JoshNang has joined #openstack-swift16:14
peluseclayg, mattoliverau FYI I didn't have any md5 mismstaches testing the multiphase either.... just the DLO thing16:21
notmynamegood morning16:21
pelusespeaking of which...16:21
pelusetdasilva, if you're around you're now free to knock out the DB udpate thing on feature/EC now that both PUT and GET have landed16:21
pelusemorning notmyname16:21
tdasilvapeluse: but did acoles change make it to feature/EC ?16:22
notmynameinteresting http://www.hgst.com/press-room/press-releases/HGST-Announces-Definitive-Agreement-to-Acquire-Amplidata16:23
tdasilvapeluse: this one: https://review.openstack.org/#/c/158821/16:23
pelusetdasilva, one sec16:23
peluseso I didn't merge master on Fri because I wanted multiphase to land w/ more complications so no.... will do that now16:25
pelusew/o more compications that is :)16:26
tdasilvapeluse: got it! thanks, will send the patch once that lands16:27
*** bill_az has quit IRC16:32
pelusetdasilva, ha, only one tiny doc conflict.  will run a few tests and throw it in the hopper....16:33
tdasilvapeluse: cool...almost done with obj. versioning, so timing is good :)16:33
*** otoolee has quit IRC16:36
*** Bsony has quit IRC16:36
*** david-lyle_afk has quit IRC16:37
*** reed has joined #openstack-swift16:38
openstackgerritDenis Cavalcante proposed openstack/swift: Add storage policy support for sorting method  https://review.openstack.org/16087716:39
*** otoolee has joined #openstack-swift16:44
openstackgerritThiago da Silva proposed openstack/swift: versioned writes middleware  https://review.openstack.org/13434716:44
*** jistr has quit IRC16:50
tdasilvapeluse: stepping out during lunch and will be back later...16:51
*** Sanchit has joined #openstack-swift16:52
pelusetdasilva, cool.  working through some failures that are probably in my env but may not have it submitted til after a meeting I have scoming up here in 45 min or so.... stay tuned16:52
*** annegent_ has joined #openstack-swift16:54
*** mahatic has quit IRC16:54
SanchitHi, I am using Swift+Ceilometer Icehouse version. I want to know if the publishing calls from swift to rabbitmq/ceilometer are asyncronus or not.16:55
notmynameSanchit: better to ask the ceilometer devs16:55
notmynameSanchit: I know it used to be. and now it might not be. but I don't know when that may have changed16:56
SanchitOkay notmyname . I'll  ask it in the other channel. anyways, any input would be better.16:57
*** SkyRocknRoll has quit IRC16:57
*** annegentle has quit IRC16:58
*** gyee has joined #openstack-swift16:58
openstackgerritpaul luse proposed openstack/swift: Merge master to feature/ec  https://review.openstack.org/16089016:59
*** nellysmitt has quit IRC17:03
*** Bsony has joined #openstack-swift17:14
*** chlong has quit IRC17:17
*** rledisez has quit IRC17:20
SanchitJust in case if anybody using swift with ceilometer has any idea regarding this query of mine:17:26
SanchitI am using Ceilometer Icehouse version. I want to know the difference between normal rpc calls and rpc cast. Also, are the publishing calls from swift to rabbitmq/ceilometer asynchronus call or else?17:27
*** mmcardle has quit IRC17:27
*** jordanP has quit IRC17:39
*** anayag has quit IRC17:40
*** tellesnobrega has quit IRC17:43
*** JoshNang has left #openstack-swift17:43
*** Anticimex has quit IRC17:54
*** Anticimex has joined #openstack-swift17:56
*** 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"17:56
*** geaaru has quit IRC18:02
*** tellesnobrega has joined #openstack-swift18:04
*** acoles is now known as acoles_away18:07
*** Bsony has quit IRC18:10
*** Bsony has joined #openstack-swift18:10
*** tsg_ has quit IRC18:14
*** erlon has joined #openstack-swift18:17
*** nexusz99 has quit IRC18:26
*** tsg has joined #openstack-swift18:28
*** tsg has quit IRC18:39
*** zhill has joined #openstack-swift18:45
*** tsg has joined #openstack-swift18:50
*** tsg_ has joined #openstack-swift18:52
*** nexusz99 has joined #openstack-swift18:53
*** tsg has quit IRC18:55
*** nexusz99 has quit IRC18:57
claygpeluse: yes, it's very strange - when I get the object with my old swift-ec-get script the resulting md5 matches the uploaded object - but not the etag?18:58
claygpeluse: md5 matched on small objects18:59
*** rdaly2 has quit IRC18:59
claygit seems like PUT is giving me back one of the FA's ETags - it's not consistent19:04
claygpeluse: can you make sure you're running the latest version of the swift client tool (with md5 checking on upload) and try uploading an object that's a few mb19:05
*** zaitcev has joined #openstack-swift19:10
*** ChanServ sets mode: +v zaitcev19:10
*** aix has quit IRC19:11
redbowhy's sync_method a required config for handoff deletes to happen now?  https://github.com/openstack/swift/blob/master/swift/obj/replicator.py#L28019:12
*** Nadeem has joined #openstack-swift19:18
*** Bsony has quit IRC19:21
*** Bsony has joined #openstack-swift19:24
*** Bsony has quit IRC19:25
*** Bsony has joined #openstack-swift19:25
*** Nadeem has quit IRC19:26
claygredbo: because no one has merged https://review.openstack.org/#/c/159683/19:30
claygredbo: sorry about that!19:31
redboha, okay.  We just ran into it in testing by accident.19:35
claygyeah it totally fucking sucks, thank god for probetests, i feel like a sucker for letting it slip19:35
claygredbo: it's those hack-a-thons, people fly in from japan and shit and I get all antsy to merge stuff thinking we can just clean it up later19:36
claygwell turns out now is better than never, but never is often better than *right* now19:37
*** bkopilov has quit IRC19:42
*** cebruns has quit IRC19:49
claygtdasilva: thanks for the new versioned writes middleware patchset - first glance looks pretty good - i'll play around with it!19:53
claygredbo: I didn't use self.sync_method == self.rsync (or whatever) because sometimes tests want to set replicator.sync_method == _fake_sync_method19:54
tdasilvaclayg: thanks! yeah, still a bit more work to do, but wanted to get something out for review sooner rather than later19:54
claygtdasilva: good call19:54
claygi'm trying to grok around why the signatures for self.contaienr_request and self.object_request are so different19:55
peluseclayg, just got back from, well, out goofing off.  Yes I will give it another try for sure.  I've been using a 3MB random binary and doing a cmp of the original with what I get back so checking integrity bu not explcitly the calc'd md5 so I guess I was talking apples to semi-apples19:55
claygI was also hoping you'd pull in the doc change to keep the version write landing page - I keep thinking it will come in handy as an anchor poing when we get around to https://review.openstack.org/#/c/155421/ (the undelete shadow copy version manifest pointer object thing019:56
claygpeluse: so digging into the proxy i'm not seeing it's do anything to graph the X-Object-Sysmeta-Ec-Etag instead of just the Etag in best_responses when the not policy.stores_object_verbaitem19:58
clayg... so i'm not surprised that PUT is giving me back one of the FA's etags - but I am surprised no one else noticed it19:58
peluseclayg, which ver of swift client are you using (just so I know I have the same exact thing)19:59
claygmaster19:59
peluseclayg, and hmmmm on your comment.  I thought I saw that code in one of a zillion recent reviews19:59
clayg925c01ebfb or so19:59
claygmaybe *I'm* on the wrong patch!?20:00
clayghrmm... looks right 619ba2f403520:00
*** hemanthm has joined #openstack-swift20:03
notmynameredbo: zaitcev: thanks for landing that patch20:03
zaitcevwell if I paid attention at hackathon, it would not be needed in the first place20:04
hemanthmHey, swift folks! Can I get a review on this glance MP please? https://review.openstack.org/#/c/159223/220:05
* clayg and zaitcev take turns punching each other in the face20:05
*** tacticus has quit IRC20:06
*** tsg_ has quit IRC20:06
clayghemanthm: multipart-manifest=delete seems like the right thing to do for SLO - but I don't think there's anything like that for DLO20:07
clayghemanthm: in the tests I'm not seeing anything that shows that the DLO segments are deleted, it'd take a container list and a bunch of delete calls20:08
clayghemanthm: oh, code seems to do it though, maybe i missed something in the test...20:09
hemanthmclayg: Glance does that already for DLOs20:09
hemanthmclayg: the tests I added in that MP are just related to SLO deletion20:11
clayghemanthm: oic, the second test isn't for a dlo it's just for a whole objects20:11
clayghemanthm: looks good to me20:11
tdasilvaclayg: re: version doc. The reason for removing that was because I copied the content to versioned_writes.py, so I didn't want to have duplicated content.20:12
hemanthmclayg: cool, thanks!20:13
peluseclayg, hells bells you're right.  I'll track it down (and the unit test escape) and take care of it (unless you've already done so?)20:13
claygtdasilva: yeah in my follow up patch i did something that just inlined it20:13
*** tsg_ has joined #openstack-swift20:13
hemanthmclayg: while we are talking about DLO deletion, do you there is a better way of deleting DLOs?20:13
claygpeluse: well sorta - i mean it's leaky because that method wouldn't otherwise know about an ec specific header like X-Object-Sysmeta-Ec-Etag20:13
claygpeluse: but torgomatic is going to extract all that shit to the ECObjectController so it won't matter20:14
clayghemanthm: no20:14
clayghemanthm: well... for this class of large object slo's are just better20:14
hemanthmclayg: would you recommend bulk delete?20:14
peluseclayg, OK, I'll leave it be then.  w/o the md5 check (just using curl) the data is still in tact at least so we've got that going for us (get matchs what was put)20:14
clayghemanthm: yeah, ok, fair enough - you still have to do the container listing, and make the manifest, and at the end of the day the proxy is making the same requests the client would20:15
peluseclayg, I'll throw somthing on trello though20:15
clayghemanthm: also currently the bulk delete doesn't do anything concurrently - so if you're managing the connections on your end you can spin up a dozen works and make the requests a little faster than you'd get out of bulk delete20:16
hemanthmclayg: yeah, we are looking at using SLOs, but since SLOs have an upper limit on segments, we gotta create SLO of SLOs I think20:16
claygalso if you're w/i close proximity to the swift endpoint some of the connection overhead is minimized20:16
clayghemanthm: honestly it's not worth nearly the effort of getting everything on slo's20:16
tdasilvaclayg: ha! I had made that from your follow up patch and just noticed it didn't make it in the last patchset, sorry...20:17
clayghemanthm: rly?  you could make your segments bigger20:17
tdasilva*made that change20:17
hemanthmclayg: its configurable, if some operator uses a small segment size, its very easy to go beyond 1000 segments20:18
*** silor has joined #openstack-swift20:18
*** tsg_ has quit IRC20:19
claygman...20:19
*** zhill has quit IRC20:20
claygI think 100 MB is reasonable segment size, with default 1000 segment limit that's like what - 100 GB?  what a good size for an image?20:20
hemanthmclayg: I have seen images over 1TB :)20:21
claygminimum segment size 1GB - done20:21
clayg:)20:21
hemanthmhehe20:21
claygoh crap... you said *over* 1TB :\20:22
claygstupid data20:22
hemanthmclayg: would you mind leaving a comment/+1 on the review, glance cores will be happy that it has swift's blessings20:24
claygoh yeah shit - i meant to do that - blame peluse and tdasilva20:24
tdasilvalol20:24
tdasilvaclayg is blaming everyone today, japan, peluse, tdasilva :P20:25
tdasilvahehe20:25
hemanthmhaha20:25
*** gyee has quit IRC20:28
*** silor has quit IRC20:28
*** zhill has joined #openstack-swift20:32
pelusehell I'll take the blame20:35
pelusewhat was it for again?20:35
hemanthmclayg: thanks for the review :)20:36
*** mrda-away is now known as mrda20:44
claygpeluse: jesus man - the container listing also has some random FA's hash?20:48
claygyeah all three replicas of the container have a different has for the object20:48
*** aserw1 has joined #openstack-swift20:48
*** zhill has quit IRC20:49
aserw1.20:49
*** foexle has quit IRC20:50
aserw1Hello to all. Can anyone point me to good delegetion explanation with code samples? I am just learning with Lynda.com and do not understand their explanation. Thank you20:52
aserw1I did some google, but mostly i get c# samples.20:53
notmynameaserw1: are you asking about the Swift programming language?20:54
*** nexusz99 has joined #openstack-swift20:54
notmyname(that's not what we do here)20:54
aserw1yes. sorry, i thougt from the channel name that this channel is for swift users20:55
notmynameaserw1: no, this is openstack swift, a distributed object storage system. http://docs.openstack.org/developer/swift/20:56
*** zhill has joined #openstack-swift20:57
*** zhill has quit IRC20:57
aserw1sorry again) it seems i got confused by the channel name20:57
notmynameaserw1: no worries. I think there might be a #swift-lang or #swiftlang or something around here somewhere20:58
glangeaserw1: try #taylorswift20:59
*** nexusz99 has quit IRC20:59
notmynameaserw1: also, ignore glange ;-)20:59
glange:)20:59
aserw1I will:)20:59
*** aserw1 has left #openstack-swift21:01
claygglange: +121:02
glangeshe knows a lot about computer security, it wasn't that ridiculous a suggestion:  https://twitter.com/swiftonsecurity21:03
portanteglange: wow, #taylorswift exists!21:04
glangeportante: you sound surprised :)21:04
portanteand regarding security, she just says, "shake it off"21:04
glangeshe's big, really big21:04
glangehaha21:04
claygthese jokes mean nothing to me21:06
openstackgerritClay Gerrard proposed openstack/swift: Sync local regions first, and always shuffle nodes  https://review.openstack.org/15924121:10
openstackgerritClay Gerrard proposed openstack/swift: Cleanup object-replicator handoff/rebalance partition syncing  https://review.openstack.org/15892721:10
*** Bsony has quit IRC21:12
mattoliverauMorning21:13
pelusemornin'21:18
openstackgerritpaul luse proposed openstack/swift: Minor cleanup in proxy obj controller  https://review.openstack.org/16098521:24
openstackgerritMerged openstack/swift: Merge master to feature/ec  https://review.openstack.org/16089021:25
*** zhill has joined #openstack-swift21:29
openstackgerritMerged openstack/swift: Fix object replicator partition cleanup  https://review.openstack.org/15968321:31
pelusetdasilva, OK, feature/ec has been updated so has the container override stuff on it now21:33
tdasilvapeluse: got it, rebasing my patch now21:34
tdasilvathx!21:34
pelusehey, my pleasure :)21:34
notmynamewow. there's a lot of hate for eventlet in the openstack world21:40
zaitcevand the alternative is... Apache?21:40
claygzaitcev: probably tulip21:41
zaitcevI didn't think tulip included the webserver like the one that comes with eventlet21:41
claygoh, is the dog specifically on eventlet.wsgi's server then or just implicity co-operative yeilding?21:42
notmynameclayg: mostly the latter21:43
notmynameand #openstack-meeting right now is debating the proposed spec to force everyone to use asyncio21:44
*** EmilienM has quit IRC21:44
*** EmilienM has joined #openstack-swift21:44
*** gyee has joined #openstack-swift21:44
notmynameand since that would "just" be an oslo change, it's pretty much trivial. /s21:45
claygnot to be confused with asyncore!21:45
clayg*I* was under the impression asyncio is a real chore without yeild from21:46
notmynameah. monkeypatching is bad, therefore rewrite everything away from eventlet21:46
claygit's true monkeypatching is bad21:47
notmynametherefore rewrite in golang! ;-)21:47
claygwell you might be onto something there21:47
claygoic, Trollius has yield From(...)21:48
claygyeah Trollius looks good, we could rewrite eventlet.wsgi's http server and add http2 support while we're at it...21:52
claygnotmyname: put it on the docket for Liberty21:53
notmynameclayg: why not kilo? we've still got time, right? ;-)21:53
claygnope - ec21:53
notmynameI know. "just use a different concurrency library" sounds like a pretty big change21:54
notmyname(my only response to seeing openstack governance in action today is laughter)21:54
*** erlon has quit IRC22:01
*** MVenesio has quit IRC22:08
*** lpabon has quit IRC22:12
*** gyee has quit IRC22:13
*** devlaps has joined #openstack-swift22:13
zaitcevclayg: Did you look closely into HTTP/2? Can you confirm my reading that they provide an equivalent of 100-Continue in an officially blessed way (via frames and flow control)?22:13
*** devlaps has quit IRC22:13
*** Bsony has joined #openstack-swift22:19
*** Bsony has quit IRC22:23
openstackgerritpaul luse proposed openstack/swift: Erasure Code Reconstructor  https://review.openstack.org/13187222:23
openstackgerritpaul luse proposed openstack/swift: Multiple fragment Archive Index Support  https://review.openstack.org/15963722:23
*** rdaly2 has joined #openstack-swift22:24
peluse^ unless I totally jacked that up these are now carved apart with the recon being needed by mutiple FI support22:25
*** gyee has joined #openstack-swift22:33
openstackgerritpaul luse proposed openstack/swift: Multiple fragment Archive Index Support  https://review.openstack.org/15963722:35
*** rmcall has quit IRC22:36
*** os1 has joined #openstack-swift22:36
os1Hi22:36
os1Is it possible to make it so that xprofile only collects data at specified times?22:37
notmynameos1: no22:37
os1Because if it's always running, then I imagine that would generate significant overhead.22:37
*** jrichli has quit IRC22:39
*** rdaly2 has quit IRC22:41
os1notmyname : Is there anything that can be done about the overhead? How significant is it?22:43
*** nexusz99 has joined #openstack-swift22:44
notmynameos1: it's designed to be used as-needed and more for relative profile tests (as opposed to always-on debug info)22:44
notmynameos1: I wouldn't use it much outside of controlled areas22:44
*** nexusz99 has quit IRC22:48
*** annegent_ has quit IRC22:49
*** openstackgerrit has quit IRC22:51
*** openstackgerrit has joined #openstack-swift22:52
*** km has joined #openstack-swift23:07
*** chlong has joined #openstack-swift23:31
openstackgerritThiago da Silva proposed openstack/swift: WIP: Use Container override header to update etag and size  https://review.openstack.org/16103023:32
*** hemanthm has left #openstack-swift23:32
*** os1 has left #openstack-swift23:37
*** dmorita has joined #openstack-swift23:45
*** zhill has quit IRC23:47
*** annegentle has joined #openstack-swift23:49
*** dmsimard is now known as dmsimard_away23:51
claygtdasilva: that change will make the PUT return the whole object md5's to the proxy on PUT?23:52
tdasilvaclayg: yeah23:53
claygtdasilva: I sorta think http would rather we return the etag of the object that was just uploaded... I mean didn't the proxy *send* us the computed etag (mainly for the container update)23:53
tdasilvaclayg: oh, are you referring to that last line I added in obj/server.py: etag = update_headers['x-etag']23:55

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