Wednesday, 2017-01-25

*** tdasilva has joined #openstack-swift00:03
*** caiobrentano_ has joined #openstack-swift00:11
*** caiobrentano_ has quit IRC00:12
claygso it turns out file systems reuse inode numbers because of course they do00:19
claygis it really so hard to ask has the data in this file changed since I read it earlier?00:20
zaitcevmtime00:38
claygunless you're *really* quick00:38
clayganyone know cindy?  lp bug #165914700:41
openstackLaunchpad bug 1659147 in OpenStack Object Storage (swift) "swift-get-nodes not working in "full" mitaka " [Undecided,New] https://launchpad.net/bugs/165914700:41
patchbotError: No closing quotation00:41
zaitcevYes, you have assume the worst if times match.00:41
claygright - so if you expected times to mostly *not* match and sometimes match that would be great - but we expect times to mostly match - only even when the match - you might still be racing00:42
claygso we took to mocking it in tests00:43
claygthen I thought I'd use inode because it's good enough for os.path.samefile00:43
claygthen I thought I'd use inode + mtime because ... probabilities?00:43
claygthen I thought - i hate computers - so I'm going to do this when I need to know if the content in the file is the same as when I read it earlier00:44
clayg*read the content and see if it's the same as what I read earlier*00:44
claygcrazy right?00:44
claygit's not like this is 5GB of data - these are less than a block size - i could probably md5 instead of deserialize - maybe that's more attractive by some measure00:45
zaitcevyou can compare hashes but yes, you have to read the current state. Unless you want to use locking.00:45
*** vinsh has quit IRC01:06
*** ChubYann has quit IRC01:13
*** chosafine has joined #openstack-swift01:14
*** sams-gleb has joined #openstack-swift01:22
*** ChubYann has joined #openstack-swift01:25
*** sams-gleb has quit IRC01:28
*** tqtran has quit IRC01:37
*** vint_bra has quit IRC01:47
*** clu_ has quit IRC01:57
openstackgerritClay Gerrard proposed openstack/swift: Better optimistic lock in get_hashes  https://review.openstack.org/41978702:02
*** vint_bra has joined #openstack-swift02:03
*** vint_bra has quit IRC02:10
clayg^ tdasilva onovy timburke jrichli mahatic mattoliverau notmyname acoles_ this change is my everything02:14
openstackgerritClay Gerrard proposed openstack/swift: Better optimistic lock in get_hashes  https://review.openstack.org/41978702:23
*** two_tired has joined #openstack-swift02:29
*** winggundamth has joined #openstack-swift02:35
*** JimCheung has quit IRC02:39
*** ppai has joined #openstack-swift03:16
*** sams-gleb has joined #openstack-swift03:25
*** sams-gleb has quit IRC03:30
*** ppai has quit IRC03:33
*** JimCheung has joined #openstack-swift03:35
*** JimCheung has quit IRC03:39
*** chsc has joined #openstack-swift03:41
*** chsc has joined #openstack-swift03:41
*** vinsh has joined #openstack-swift03:42
*** vinsh has quit IRC03:43
*** chsc has quit IRC03:57
*** SkyRocknRoll has joined #openstack-swift04:00
*** kei_yama has joined #openstack-swift04:07
*** abhitechie has joined #openstack-swift04:34
*** ppai has joined #openstack-swift04:41
*** psachin has joined #openstack-swift04:42
*** mkrai_ has joined #openstack-swift04:42
*** chosafine has quit IRC05:02
*** jamielennox is now known as jamielennox|away05:06
*** klrmn has quit IRC05:16
*** sams-gleb has joined #openstack-swift05:28
*** abhitechie has quit IRC05:32
*** sams-gleb has quit IRC05:32
*** tqtran has joined #openstack-swift05:37
*** tqtran has quit IRC05:41
*** abhitechie has joined #openstack-swift05:42
*** dmorita has quit IRC05:48
*** two_tired has quit IRC05:49
*** dmorita has joined #openstack-swift05:49
*** dmorita has quit IRC05:54
*** dmorita has joined #openstack-swift06:36
*** chsc has joined #openstack-swift06:36
*** dmorita has quit IRC06:40
*** chsc has quit IRC06:42
*** ChubYann has quit IRC06:47
*** Jeffrey4l_ has quit IRC06:48
*** Jeffrey4l has joined #openstack-swift06:53
*** abhitechie has quit IRC07:06
*** SkyRocknRoll has quit IRC07:25
*** pcaruana has joined #openstack-swift07:28
*** sams-gleb has joined #openstack-swift07:30
*** tesseract has joined #openstack-swift07:34
*** sams-gleb has quit IRC07:35
*** jlwhite has quit IRC07:40
*** sams-gleb has joined #openstack-swift07:42
*** abhitechie has joined #openstack-swift07:43
*** abhitechie has quit IRC07:43
*** hseipp has joined #openstack-swift07:50
*** mingyu has joined #openstack-swift07:54
*** si1ver has quit IRC08:04
*** rledisez has joined #openstack-swift08:10
*** oshritf_ has joined #openstack-swift08:27
*** mkrai_ has quit IRC08:29
*** Jeffrey4l_ has joined #openstack-swift08:30
*** Jeffrey4l has quit IRC08:34
*** sams-gleb has quit IRC08:38
*** jlwhite has joined #openstack-swift08:39
*** geaaru has joined #openstack-swift08:42
*** si1ver has joined #openstack-swift08:56
*** cbartz has joined #openstack-swift09:03
*** oshritf_ has quit IRC09:07
*** jordanP has joined #openstack-swift09:16
*** oshritf_ has joined #openstack-swift09:16
*** tqtran has joined #openstack-swift09:24
*** geaaru has quit IRC09:28
*** tqtran has quit IRC09:28
*** geaaru has joined #openstack-swift09:30
*** winggundamth_ has joined #openstack-swift09:33
*** winggundamth has quit IRC09:35
*** JimCheung has joined #openstack-swift09:40
*** m_kazuhiro has joined #openstack-swift09:44
*** JimCheung has quit IRC09:45
m_kazuhirogood morning.09:45
openstackgerritKazuhiro MIYAHARA proposed openstack/swift: WIP: Swift Automated Tiering  https://review.openstack.org/28705709:49
*** m_kazuhiro has quit IRC09:52
*** m_kazuhiro has joined #openstack-swift09:53
*** manous has joined #openstack-swift10:05
*** manous has quit IRC10:09
*** openstackgerrit has quit IRC10:17
*** lespaul has quit IRC10:23
*** mvk has quit IRC10:23
*** kei_yama has quit IRC10:24
*** mingyu has quit IRC10:29
*** sams-gleb has joined #openstack-swift10:31
*** rledisez has quit IRC10:42
*** rledisez has joined #openstack-swift10:44
*** sams-gleb has quit IRC10:47
*** sams-gleb has joined #openstack-swift10:48
*** dmorita has joined #openstack-swift10:48
*** jordanP has quit IRC10:51
*** dmorita has quit IRC10:52
*** sams-gleb has quit IRC10:52
*** jordanP has joined #openstack-swift10:54
*** mvk has joined #openstack-swift10:56
*** vatlas has joined #openstack-swift10:57
*** dfflanders has quit IRC10:59
*** jordanP has quit IRC11:04
*** sams-gleb has joined #openstack-swift11:05
*** openstackgerrit has joined #openstack-swift11:10
openstackgerritGábor Antal proposed openstack/swift: Use more specific asserts in test/unit/common/middleware  https://review.openstack.org/34277011:10
*** ganders has joined #openstack-swift11:13
*** m_kazuhiro has quit IRC11:16
*** mkrai has joined #openstack-swift11:22
*** cdelatte has quit IRC11:23
*** mingyu has joined #openstack-swift11:29
*** vatlas has quit IRC11:33
*** mingyu has quit IRC11:34
openstackgerritChristian Schwede proposed openstack/swift: Add support to increase object ring partition power  https://review.openstack.org/33729711:57
*** cdelatte has joined #openstack-swift12:10
*** d0ugal has quit IRC12:10
*** sams-gleb has quit IRC12:14
*** sams-gleb has joined #openstack-swift12:14
*** sams-gleb has quit IRC12:15
*** sams-gleb has joined #openstack-swift12:15
*** d0ugal has joined #openstack-swift12:19
*** catintheroof has joined #openstack-swift12:28
openstackgerritJan Krcmar proposed openstack/python-swiftclient: Add kerberos http authentication  https://review.openstack.org/42458112:43
-openstackstatus- NOTICE: Gerrit is going to be restarted due to slow performance12:47
-openstackstatus- NOTICE: Gerrit has been successfully restarted12:50
*** sams-gle_ has joined #openstack-swift13:15
*** sams-gleb has quit IRC13:15
*** ppai has quit IRC13:22
*** tqtran has joined #openstack-swift13:25
*** tongli has joined #openstack-swift13:27
*** NM has joined #openstack-swift13:28
*** tqtran has quit IRC13:30
*** mingyu has joined #openstack-swift13:31
*** mingyu has quit IRC13:36
*** jordanP has joined #openstack-swift13:39
*** caiobrentano_ has joined #openstack-swift13:43
*** acoles_ is now known as acoles13:44
*** sams-gleb has joined #openstack-swift13:51
*** sams-gle_ has quit IRC13:51
*** psachin has quit IRC14:22
*** winggundamth_ has quit IRC14:31
*** mingyu has joined #openstack-swift14:32
*** mingyu has quit IRC14:37
*** tongli has quit IRC14:42
*** mvk has quit IRC14:55
*** catinthe_ has joined #openstack-swift15:00
*** catintheroof has quit IRC15:02
*** vint_bra has joined #openstack-swift15:06
*** sams-gleb has quit IRC15:09
*** sams-gleb has joined #openstack-swift15:10
*** oshritf_ has quit IRC15:13
*** sams-gleb has quit IRC15:14
*** sams-gleb has joined #openstack-swift15:28
*** vinsh has joined #openstack-swift15:31
*** oshritf has joined #openstack-swift15:37
*** catintheroof has joined #openstack-swift15:43
*** catinthe_ has quit IRC15:46
*** oshritf has quit IRC15:54
*** klrmn has joined #openstack-swift16:12
*** chsc has joined #openstack-swift16:14
*** chsc has joined #openstack-swift16:14
openstackgerritThiago da Silva proposed openstack/swift: remove func-fast-post from tox.ini  https://review.openstack.org/42526416:17
*** _JZ_ has joined #openstack-swift16:19
*** chsc has quit IRC16:20
*** mvk has joined #openstack-swift16:25
*** chosafine has joined #openstack-swift16:25
*** chosafine has quit IRC16:26
*** zul has quit IRC16:28
*** zul has joined #openstack-swift16:29
*** mingyu has joined #openstack-swift16:33
*** mingyu has quit IRC16:38
tdasilvarledisez: hi, was just reading your comments on patch 287057 and I have been thinking that maybe patch 212824 could be useful for the issue you brought up16:47
patchbothttps://review.openstack.org/#/c/287057/ - swift - WIP: Swift Automated Tiering16:47
patchbothttps://review.openstack.org/#/c/212824/ - swift - Let developers/operators add watchers to object audit16:47
openstackgerritChristopher Bartz proposed openstack/python-swiftclient: ISO 8601 timestamps for tempurl  https://review.openstack.org/42337716:51
rlediseztdasilva: hi. i'm not big fan of disk scanning. i mean, i really hate that :)16:55
rlediseztdasilva: i sometime think i could stop the auditor because in the end, there is always opportunities to quarantine a corrupted object (customer access, replication during rebalance)16:55
rlediseztdasilva: i prefer the method to register a task in a container and distributing the load accross object servers16:55
rlediseztdasilva, but it's the same topic, avoiding code duplication16:55
notmynamegood morning16:56
*** chsc has joined #openstack-swift16:56
tdasilvarledisez: I agree that scanning is not good and I think that was torgomatic intention with that patch, to try to limit how much scanning is done and try to do as much as possible in a single scan pass16:58
rlediseznotmyname: hello. i was wondering about the etherpad for PTG, is topic list "open". i mean, can i ask a topic ? (small files)16:59
*** dmorita has joined #openstack-swift16:59
notmynamerledisez: yes! absolutely16:59
tdasilvarledisez: OTOH, there's scanning that reads the data (and take a long time) and there's scanning that don't read the data17:00
*** jordanP has quit IRC17:00
tdasilvarledisez: either way I think what m_kazuhiro is proposing is to register tasks in a container, but to do that, he needs to first scan the files to discover what needs to be moved, no?17:01
rlediseztdasilva: "and there's scanning that don't read the data" => does not mean it's fast, not for us at least, mainly because every directory access means disk access because inode are evicted from cache quickly17:04
*** dmorita has quit IRC17:04
rlediseztdasilva: in our experience, just scanning files on one disk is few days17:04
tdasilvarledisez: without reading data?17:05
rlediseztdasilva: yes ;)17:05
rlediseztdasilva: i'm wondering if it would be ok for some kind of task to only work on new uploads so that registration happens at upload (like object expirer)17:05
rledisezand providing a tool that scan objects once for ops that need to work on past object too17:06
tdasilvarledisez: i was thinking if it would make sense to scan container server?17:06
*** Jeffrey4l_ has quit IRC17:06
tdasilvato build up the list of what needs to be moved17:06
rledisezcould be smart, it's easier to scale, and often on SSD so few impact on latencies. and still, it would be a "run once" job17:07
tdasilvaand by scan, i mean traverse the container list, but17:07
tdasilvaor a db query17:08
notmynamerledisez: oh! I wanted to say that I'm very interested in the small-file optimization work you wrote up on the wiki. it looks very very interesting, and I think your approach might be better than my first idea17:09
notmynametdasilva: rledisez: on the topic of drive scans being slow...17:09
*** joeljwright has joined #openstack-swift17:10
*** ChanServ sets mode: +v joeljwright17:10
notmynametdasilva: rledisez: not sure how much specifics I can share yet (currently in progress writing up somethign that will be public), but I've been working with Intel on their CAS software, and I've seen great results when using it to cache filesystem metadata (eg replication cycle time being 5-6x faster)17:10
rlediseznotmyname: i have a colleague working on that for few weeks now. for now, we are very close of your original idea for one main reason: being compatible with the existing deployment. it would be insane to have to re-format everything in some kind of ZFS or else...17:12
rledisezbut it could be an intermediary step17:13
*** klrmn has quit IRC17:13
notmynamerledisez: interesting. I really like your "put it in a DB" idea (especially after talking with an XFS dev last week)17:13
rledisezoh, yes, that we have it now. very efficient. about 20 bytes per object in memory :)17:14
rledisezit opens many possibilities17:14
tdasilva"that we have it now" ?17:16
tdasilvayou mean in prod?17:16
tdasilvain swift?17:16
rledisezno, not in prod, not yet at least :D17:17
rledisezin the dev version17:17
tdasilvaheh17:17
notmynameyeah, which part of https://wiki.openstack.org/wiki/Swift/ideas/small_files/experimentations do you have?17:17
rledisezit's actually very close to haystack in the idea. we were concerned about the compaction, so we tried to reduce it with the use of fallocate PUNCH HOLE17:18
rledisezso we free space without resizing the "volumes"17:18
rledisezand a key value database to locate data in the volumes/ ohash -> volume,offset17:19
rledisezthe bigger problem for now is eventlet :/17:20
notmynamerledisez: I'd expect to have some pretty tricky problems when trying to manage the gc/compaction of slab files. In fact, it might be nearly a whole filesystem to do that. So that's why the idea of "just store small files in a DB" is interesting. Turns out swift is pretty good at understanding and managing sqlite DBs17:21
notmynamewhat's the eventlet problem?17:21
*** Jeffrey4l has joined #openstack-swift17:25
*** tqtran has joined #openstack-swift17:26
tdasilvanotmyname: cool info on the CAS software, looking forward to your future report17:29
*** Jeffrey4l has quit IRC17:30
rlediseznotmyname: mainly process hanging without obvious reason. sometimes a thread closing the wring file descriptor. using a C module create some weird behavior too17:32
notmynamehmm..interesting17:32
*** joeljwright has quit IRC17:32
rlediseznotmyname: there is a lot of bug reports about that kind of stuff17:32
rledisezmonkey patching is not that magic :D17:32
notmynamerledisez: we're going to rewrite all that in golang, though, right? ;-)17:33
rlediseznotmyname: yeah!17:33
*** tesseract has quit IRC17:33
*** BlackDex has left #openstack-swift17:34
openstackgerritAlistair Coles proposed openstack/swift: Bring docs inline with changes to tox envs  https://review.openstack.org/42531117:34
acolestdasilva: jrichli small follow up to recent tox changes ^^17:35
rlediseznotmyname: very interested too about CAS. we had some contact few weeks ago with intel R&D in poland about that, i think some colleague are about to test it on ceph17:36
jrichliacoles: looks good, thx17:36
acolesjrichli: tdasilva is doing the significant work :)17:37
claygwoooo!17:38
*** JimCheung has joined #openstack-swift17:39
*** dmorita has joined #openstack-swift17:39
tdasilvaacoles: lol, timburke did all the work17:39
acolesclayg: I've been on other stuff today but I'll try to look at your "everything" patch soon, looks like you made a lot of progress17:39
claygmahatic: that stupid comment was like a honey pot - I thought about leaving it out - but I was like "nah, *someone* is going to ask why it's there"17:40
claygacoles: the inode thing didnt' work out :'(17:40
claygbut then sleeping on it I had this idea that maybe if I held an open file handle the whole time the file system couldn't reuse *my* inode - but idk, that's an even bigger refactor17:40
acolesclayg: and yeah sorry I should have squashed patch 421385 and patch 422076 rather than having the to-ing and fro-ing in the tests :/17:41
patchbothttps://review.openstack.org/#/c/421385/ - swift - Add more tests for diskfile consolidate_hashes17:41
patchbothttps://review.openstack.org/#/c/422076/ - swift - Handle unreadable hashes.pkl same way as non-exist...17:41
claygthis could easily be adjusted to just checksum the contents instead of deserialization if we find that's significantly faster17:41
openstackgerritJoel Wright proposed openstack/swift: Add Preamble and Postamble to SLO and SegmentedIterable  https://review.openstack.org/36537117:41
acolestdasilva: true, sorry timburke !17:41
tdasilva:)17:42
claygacoles: that was my conclusion as well - I was happy to see the tests work out (in the good final state) after rebasing on patch 41978717:42
patchbothttps://review.openstack.org/#/c/419787/ - swift - Better optimistic lock in get_hashes17:42
claygacoles: I really did rip off your "handle missing/bad/etc hashes all the same"17:42
acolesclayg: it simply reflected my workflow - one day I wrote some tests while reviewing your patch, next day I wrote a fix, next day I didn't go back and squash them17:43
*** JimCheung has quit IRC17:43
claygbut there may be even more better goodness if we keep working on it - but for now I think patch 419787 *could* be a "stopping" point17:43
patchbothttps://review.openstack.org/#/c/419787/ - swift - Better optimistic lock in get_hashes17:43
claygacoles: k, np - makes total sense - i work like that as well17:44
acolesclayg: yeah, I had a quick scan over your latest. we better land it before you think too much more ;)17:44
*** JimCheung has joined #openstack-swift17:46
*** jcaron has quit IRC17:53
*** rledisez has quit IRC17:56
timburkewhat's this? who's accusing me of doing what work?17:58
*** hseipp has quit IRC17:59
claygtimburke: it was tdasilva - he said it was all your fault (but we all know it's notmyname's fault)18:00
notmynameit normally is18:00
tdasilvayep18:00
timburkePTL-for-life: it's a blessing *and* a curse18:00
claygfor notmyname - a curse - for everyone not notmyname - a blessing18:01
tdasilvayou mean, blessing to us and curse to notmyname, right?18:01
claygtdasilva: jinx!18:01
notmynamelol18:01
notmynamecschwede: FYI https://www.heise.de/newsticker/meldung/Call-for-Papers-Deutsche-OpenStack-Tage-2017-3607542.html?wt_mc=rss.ho.beitrag.atom&utm_source=dlvr.it&utm_medium=twitter18:02
acolescschwede: I understand the photo :)18:04
clayggoogle translate in chrome is getting *scary* good - like star trek good18:04
claygacoles: oh, what is up with that swift-get-nodes bug report?18:06
tdasilvaclayg: is that an extension?18:06
claygtdasilva: my chrome just asked me if i wanted to transalte - when I clicked yes it was all like well spoken nice grammer english explain all about the OpenStack Days in Germany18:07
claygmaybe technical news is easier to translate than some other subjects - just seemed to do a better job than I normally expect form google translate18:07
tdasilvaneat18:08
acolesclayg: IDK, I'm confused by reference to partions (dbs) and dbs in object dirs18:11
acolesclayg: I'll leave another reply18:11
claygacoles: GL!18:11
acolesdoes anyone have a trick to filter bug emails so that ones you care about get your attention? I want to be able to "star" a bug and then have emails about that bug have some flag that lets me filter them18:13
*** jcaron has joined #openstack-swift18:14
*** klrmn has joined #openstack-swift18:18
*** acoles is now known as acoles_18:25
*** chsc has quit IRC18:25
*** cbartz has left #openstack-swift18:27
*** ChubYann has joined #openstack-swift18:28
*** catinthe_ has joined #openstack-swift18:33
*** mingyu has joined #openstack-swift18:35
*** catintheroof has quit IRC18:35
*** mingyu has quit IRC18:39
*** Jeffrey4l has joined #openstack-swift18:41
*** mvk has quit IRC18:42
*** Jeffrey4l has quit IRC18:46
*** pcaruana has quit IRC18:53
*** cebreidian has quit IRC19:05
*** chsc has joined #openstack-swift19:07
*** chsc has joined #openstack-swift19:07
*** Jeffrey4l has joined #openstack-swift19:10
*** cebreidian has joined #openstack-swift19:12
*** Jeffrey4l has quit IRC19:15
*** geaaru has quit IRC19:18
*** caiobrentano_ has quit IRC19:18
*** caiobrentano_ has joined #openstack-swift19:19
clayg#easyfollowupforsomeone19:21
*** tqtran has quit IRC19:26
*** ganders has quit IRC19:33
*** mingyu has joined #openstack-swift19:36
*** mingyu has quit IRC19:40
*** Jeffrey4l has joined #openstack-swift19:58
*** cebreidian has quit IRC19:59
*** Jeffrey4l has quit IRC20:04
*** tqtran has joined #openstack-swift20:05
*** cebreidian has joined #openstack-swift20:08
*** mvk has joined #openstack-swift20:13
clayg^ that's just an OH i thought was funny20:15
claygis it meeting time!?20:15
tdasilva45min20:15
claygaacacahcphaphcap20:23
clayglet's have a *pre* meeting20:23
*** catinthe_ has quit IRC20:27
claygtorgomatic: so I saw lp bug #1554233 and was thinking "gah that's not merged yet!?" - then I saw I had a -1 on patch 289664 - so I was thinking about taking it over?20:31
openstackLaunchpad bug 1554233 in OpenStack Object Storage (swift) "Servers-per-port can consume excessive OS threads" [High,In progress] https://launchpad.net/bugs/1554233 - Assigned to Samuel Merritt (torgomatic)20:31
patchbothttps://review.openstack.org/#/c/289664/ - swift - Make eventlet.tpool's thread count configurable in...20:31
*** lespaul has joined #openstack-swift20:32
*** NM has quit IRC20:35
claygI think patch 326967 is a quick easy win20:36
patchbothttps://review.openstack.org/#/c/326967/ - swift - Rebalance with min_part_seconds_left > 020:36
claygcloses an open bug!20:36
claygtimburke: tdasilva: you gotta keep after patch 371150 - it's a really good one!20:37
patchbothttps://review.openstack.org/#/c/371150/ - swift - Return 404 on a GET if tombstone is newer20:37
*** caiobrentano_ has quit IRC20:37
*** Jeffrey4l has joined #openstack-swift20:45
*** jamielennox|away has quit IRC20:46
tdasilvaclayg: re patch 371150 I hit a snag trying to understand the EC code in the proxy controller20:46
patchbothttps://review.openstack.org/#/c/371150/ - swift - Return 404 on a GET if tombstone is newer20:46
tdasilvaso i need more time to dig in20:46
tdasilvathere's some crazy stuff there with best buckets vs bad buckets and all that20:47
notmynameclayg: yeah https://bugs.launchpad.net/swift/+bug/1629410 is fix-released. I marked it as such20:48
openstackLaunchpad bug 1629410 in OpenStack Object Storage (swift) "update pyeclib dependency to >=1.3.1" [Medium,Fix released] - Assigned to Nicolas Helgeson (nhelgeson)20:48
notmynameclayg: the hummingbird stuff was because the landed patch was included in a merge from master. so it triggered the scripts20:48
claygand do the scripts maybe like un-mark it was fix released?20:52
claygi found another old bug that seemed to be fixed and everyone was tagged there was comment in gerrit calling it reviewed (and the a bunch of other commnets about other swift feature branches) - and it was in progress?20:52
claygif there's an automated tool out there going around re-opening bugs we've already closed i'ma be sorta disappointed20:53
clayglp bug #1537811 (and esspecially all the bug reports it references is still a great read)20:53
openstackLaunchpad bug 1537811 in OpenStack Object Storage (swift) "204 No Content responses have Content-Length specified" [Low,In progress] https://launchpad.net/bugs/1537811 - Assigned to Trevor McCasland (twm2016)20:53
notmynameno, I don't think it reopens things. I think it never closed it because it was tagged as "partial-fix" instead of "closes-bug"20:53
claygesp. the varnish bug report clients be all like "umm... this has regressed" and varnish is like "no your backend is returning invalid - INVALID - respones" but then they were like "ok, fine, I guess some crappy web servers (e.g. apache with mod_php) are just *broken*"20:56
*** jamielennox|away has joined #openstack-swift20:56
*** jamielennox|away is now known as jamielennox20:56
claygdoing the right thing for clients always feels good - doing the right thing for documents on the internet (when it hurts clients) always makes you sound like a monster20:56
*** acoles_ is now known as acoles20:58
claygtimburke: I lost that one bug report for the reconstructor blowing up unless you move sleep20:58
notmynameok, *now* it's time for the swift meeting20:59
notmynamein #openstack-meeting20:59
*** m_kazuhiro has joined #openstack-swift20:59
mattoliveraumonring21:00
claygtimburke: nm, found it lp bug #165317421:02
openstackLaunchpad bug 1653174 in OpenStack Object Storage (swift) "weird traceback/race in manual reconstructor: cannot switch to a different thread " [Undecided,Confirmed] https://launchpad.net/bugs/165317421:02
*** caiobrentano_ has joined #openstack-swift21:12
*** caiobrentano_ has quit IRC21:13
*** JimCheung has quit IRC21:18
timburkefyi on the mod_wsgi thing -- i think that might be the cause of https://bugs.launchpad.net/python-swiftclient/+bug/1586690 and https://bugs.launchpad.net/python-swiftclient/+bug/1621581 ? at any rate, one of those included some logging that saw a Server: Apache header but not Content-Length which seems... bad...21:19
openstackLaunchpad bug 1586690 in python-swiftclient "Uploading empty(0 B) file fails" [Undecided,Incomplete] - Assigned to Uday Swami (swamius)21:19
*** JimCheung has joined #openstack-swift21:19
openstackLaunchpad bug 1621581 in python-swiftclient "swiftclient returns response headers without 'Content-Length' param, thus causing upload object to fail" [Undecided,In progress] - Assigned to Arun Mani (arun-mani)21:19
*** JimCheung has quit IRC21:23
*** vint_bra has quit IRC21:23
*** vint_bra has joined #openstack-swift21:24
*** vint_bra has quit IRC21:24
*** Jeffrey4l_ has joined #openstack-swift21:34
*** Jeffrey4l has quit IRC21:35
*** mingyu has joined #openstack-swift21:37
*** clu_ has joined #openstack-swift21:38
*** jidar has quit IRC21:39
*** jidar has joined #openstack-swift21:40
*** mingyu has quit IRC21:42
*** NM has joined #openstack-swift21:48
*** dmorita has quit IRC21:58
*** dmorita has joined #openstack-swift22:00
*** clu_ has quit IRC22:00
claygtdasilva: did you manage to *land* that refactoring?22:00
*** clu_ has joined #openstack-swift22:01
*** dmorita has quit IRC22:01
*** dmorita has joined #openstack-swift22:01
claygtdasilva: because that'd acctually make me a lot more hopeful about this change22:01
tdasilvaclayg: itś better now: https://github.com/openstack/swift/tree/master/test/functional22:01
mattoliverautdasilva: I was only joking about volenteering you :)22:01
tdasilvamattoliverau: hehe, psachin's got it22:01
mattoliveraulol22:01
jrichliclayg: I am sometimes a little nervous on test refactoring because we dont test tests.  so you have to know the same things are still getting tested for sure22:01
claygtdasilva: can you dig up the series of reviews where you did that and suggest that in the review?22:01
tdasilvaclayg: sure22:02
acoleswhen tdasilva broke up the func tests it really helped review that almost nothing changed other then lines of code moving between files22:02
jrichliyes, that was nice22:02
claygacoles: i don't really have a good tool workflow for that - i've seen you do it in your ide before22:02
claygjrichli: you have a trick for that too!?22:02
acolesclayg: yeah pycharm lets me copy N lines from one branch to clipboard and then compare that with another selection22:03
jrichliclayg: i wish i had tricks22:03
timburkei seem to remember going through two test files more-or-less line-by-line and checking that we still had comparable coverage... for versioned_writes, maybe? or copy? pulling one of those out as middleware, anyway22:03
acolesoh no did I just volunteer to review this monster !?!? ;)22:03
claygacoles: but you don't know how to do that outside of your tool suite?22:03
tdasilvaacoles: hehe22:03
kota_timburk, clayg: could you continue the review for my global ec patch? Maybe i would be back to upstream in the next week22:03
claygkota_: yes yes yes!  i very very much want that in ocata acctually!22:04
acolesclayg: no, but I have never looked for another tool22:04
mattoliverauk, me is going to go Australia day. Hooroo <-- aussie slang for good bye.22:04
claygmattoliverau: ttyl22:04
jrichliHooroo!22:04
timburkekota_: yup, was coming up with a diff and a few comments before the meeting22:04
acolesmattoliverau: ttfn22:04
kota_Thanks a lot!22:04
acolesclayg: tdasilva anyone, the tip being - don't try to fix or cleanup anything unless absolutely necessary, just move the lines of code22:05
tdasilvaclayg: what about using a tool like meld, copy and paste?? it's manual work, but should work...22:05
* kota_ should get well soon. Still on bed though22:05
timburkekota_: i'm still a little nervous that we can get some pathologically bad distributions (both copied of frags 0, 1, and 2 in region 1, both copies of frags 3, 4, and 5 in region 2), but maybe it's better to land it and iterate from there?22:06
acoleskota_: stay there and get well!22:06
claygtdasilva: I've never used meld - i would really like instructions (on the review) that tell me how to see "here's the lines that moved, here's the the lines that then changed"22:07
tdasilvanotmyname: what's the next step with this: http://lists.openstack.org/pipermail/openstack-dev/2017-January/110467.html ?22:07
*** jcaron has left #openstack-swift22:07
claygtimburke: IME it just worked!?  but then kota_ said I just got much lucky :'(22:07
tdasilvaclayg: agreed, i will post a comment for psachin to try and help him out with that22:07
*** JimCheung has joined #openstack-swift22:07
claygtimburke: I think that composite rings help with that somehow eventually?22:08
acolestdasilva: I also like your suggestion to move out one controller at a time. hard to say how big a deal it is without looking at the patch though22:08
claygtimburke: i'm not sure the value of landing it to iterate - in fact - w/o the other stuff maybe we should documnt it as experimental dragons or something?22:08
timburkeclayg: yeah, and i see how it *should* Just Work most of the time, but there's nothing to *prevent* it, which means with lots of device additions/removals, it could certainly happen22:08
tdasilvaacoles: yeah, i'm trying to bring it up now to take a closer look22:09
*** m_kazuhiro has quit IRC22:09
claygtimburke: I don't even know why it "just works" *some* of the time - i went looking because I was so sure it would be wrong - i was very surprised22:09
acolesgood night22:09
claygacoles: thanks!22:09
claygtdasilva: good question on TC golang resolution!22:10
*** acoles is now known as acoles_22:10
*** NM has quit IRC22:19
notmynametdasilva: honestly, I don't know yet22:20
*** sams-gleb has quit IRC22:22
*** sams-gleb has joined #openstack-swift22:23
portantekey notmyname, did you see the pinterest talk at linux.conf.au on debugging website latencies?   https://www.youtube.com/watch?v=bectZn_yNwg22:26
portanteseemed pretty good22:26
openstackgerritClay Gerrard proposed openstack/swift: Do not revert fragments to handoffs  https://review.openstack.org/42544122:27
claygportante: !!!22:27
portantehi clayg22:27
*** sams-gleb has quit IRC22:27
*** sams-gleb has joined #openstack-swift22:28
*** sams-gleb has quit IRC22:29
claygwhy do i know joe - i know he's been around openstack?  infra?  hp?  redhat?  rax?  swift?22:29
claygOMG if the punchline is golang I'm going to be so mad :D22:30
tdasilvalol22:30
tdasilvaclayg: added the comment there, feel free to shoot it down if you don't like what i said22:31
timburkei was watching that just yesterday, seemed pretty cool. i'm curious what the statistical profiling looks like on a proxy server22:31
tdasilvanotmyname: no worries, thanks22:31
timburkeesp. replicated vs. ec, encryption vs. not...22:32
tdasilvagotta run, have a good night everyone22:32
claygyeah!!! MOAR processes!22:36
claygthere was a pretty visualization - it wasn't helpful - pstats FTW22:41
notmynameportante: hey. yeah, it turns out I just watched that this morning. I didn't catch it in person22:50
portantecool22:50
notmynameportante: but mattoliverau saw it in person (and said he wanted to make some flame graphs for swift)22:51
portanteyes, that would be cool22:51
claygtimburke: you want one of those flame graph things!?22:51
portanteat 1.5 speed the talk seemed to be pretty to the point22:51
timburkemattoliverau: yes! do it! but finish sharding first ;-)22:51
timburkeclayg: largely for the diffs22:52
claygawww man and it's picking on deepcopy now22:53
timburkeyup22:53
claygprotobufffffffffffff22:55
claygawwww the punchline is "C" :'(22:55
claygthat's not as funny22:56
claygthis is really slowing down my rebasing :)22:56
claygportante: yeah that was great - thanks22:58
*** jordanP has joined #openstack-swift22:59
zaitcevas in extern "C" { ... } ?22:59
*** jordanP has quit IRC22:59
*** catintheroof has joined #openstack-swift23:00
*** NM has joined #openstack-swift23:03
*** cebreidian has quit IRC23:09
openstackgerritJohn Dickinson proposed openstack/python-swiftclient: 3.3.0 authors/changelog update  https://review.openstack.org/42545423:13
notmynamefor your consideration for this week's client release ^23:13
*** cebreidian has joined #openstack-swift23:18
*** dmorita has quit IRC23:23
timburkelooks like i get to dig into some C code, too? yay! "*** Error in `python': free(): invalid size: 0x0000000001bd1790 *** Aborted" "Segmentation fault"23:24
*** dmorita has joined #openstack-swift23:25
*** sams-gleb has joined #openstack-swift23:29
*** sams-gleb has quit IRC23:34
*** NM has quit IRC23:35
*** mingyu has joined #openstack-swift23:39
*** caiobrentano_ has joined #openstack-swift23:39
*** caiobrentano_ has quit IRC23:39
*** mingyu has quit IRC23:43
*** caiobrentano_ has joined #openstack-swift23:52
*** chsc has quit IRC23:54
*** kei_yama has joined #openstack-swift23:57
*** catintheroof has quit IRC23:58
*** catintheroof has joined #openstack-swift23:59

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