Wednesday, 2016-10-12

*** nikivi has quit IRC00:01
*** dfflanders has joined #openstack-swift00:02
*** ediardo has quit IRC00:23
*** ediardo has joined #openstack-swift00:23
*** fungi has quit IRC00:26
*** gyee has quit IRC00:29
*** fungi has joined #openstack-swift00:30
*** chsc has quit IRC00:45
*** sudorandom has quit IRC00:47
*** mvk has quit IRC00:50
*** sudorandom has joined #openstack-swift00:53
*** m_kazuhiro has joined #openstack-swift00:55
*** jamielennox is now known as jamielennox|away00:56
*** jamielennox|away is now known as jamielennox01:02
kota_good morning01:06
*** dfflanders has quit IRC01:07
kota_timburke: are you working for fixing pyeclib.get_metadata memory fix? ;-)01:08
kota_s/memory/memory leak/01:08
openstackgerritClay Gerrard proposed openstack/swift: Fix signal handling for daemons with InternalClient  https://review.openstack.org/25934701:13
openstackgerritJamie Lennox proposed openstack/python-swiftclient: Use requests_mock and oslotest  https://review.openstack.org/36029801:15
mattoliveraukota_: morning01:16
kota_mattolivearu: good morning01:20
*** trananhkma has quit IRC01:25
*** kei_yama has quit IRC01:25
*** kei_yama has joined #openstack-swift01:26
zhengyinkota_, mattoliverau: morning01:28
*** trananhkma has joined #openstack-swift01:30
charzkota_ mattoliverau zhengyin morning01:32
zhengyincharz:morning01:33
*** dfflanders has joined #openstack-swift01:37
*** dmorita has joined #openstack-swift01:44
*** zacksh has quit IRC01:50
*** zacksh has joined #openstack-swift01:52
*** Guest50630 has quit IRC02:03
*** zigo has joined #openstack-swift02:05
*** zigo is now known as Guest5942702:06
*** dfflanders has quit IRC02:09
mattoliveraucharz, zhengyin o/02:12
mattoliveraukota_: you might be proud of me, last night I made curry katsu don for dinner :)02:12
*** tqtran has quit IRC02:15
kota_mattoliverau: curry katsu don!? It's my first time to hear such a food :P02:23
kota_zhengyin, charz: morning02:23
mattoliveraukota_: lol, well you should try it some time.. I hear they may make it somewhere in Japan :P02:24
kota_mattoliverau: interesting, sounds good anyway. ah, perhaps it's katsu carry in Japanse word???02:27
kota_mattolivearu: https://goo.gl/TyPjTx02:28
mattoliveraukota_: yeah, that :)02:30
kota_mattolivearu: i like it ;-)02:33
openstackgerritKota Tsuyuzaki proposed openstack/liberasurecode: Fix reconstruct to return an error when memory allocation failed  https://review.openstack.org/38524002:39
*** dmorita has quit IRC02:41
*** dmorita has joined #openstack-swift02:41
kota_acoes: i'm not sure if the patch 385240 addresses admin6's issue but it seems happen to make broken fragment via reconstruct function in liberasurecode02:42
patchbothttps://review.openstack.org/#/c/385240/ - liberasurecode - Fix reconstruct to return an error when memory all...02:42
*** dmorita has quit IRC02:42
kota_To figure out that it would happen, I tried via https://gist.github.com/bloodeagle40234/6af7f2d3e67e9cddabbea7e84c453c9002:47
kota_acoles: the gist is a diff for current liberasurecode master to break the reconstruct function as similer with the case when memory allocation failed.02:48
*** amit213 has quit IRC02:48
*** amit213 has joined #openstack-swift02:49
kota_acoles: after compiling that code, and I tried to call https://gist.github.com/bloodeagle40234/bb18ef695c65785910bf4a846c93e0e902:49
kota_that one is for trying to reconstruct02:49
kota_via pyeclib02:49
kota_acoles: amazingly, the reconstruct call is succesed (i.e. no error/exception occurred at the time) but it returns broken fragments with same length of expected fragments.02:51
kota_acoles: if it occurred, swift cannot detect the broken frag anymore.02:51
kota_acoles: I'll try to ask admin6 he can find some syslog messages to suggest such a memory allocation error from liberasurecode02:52
*** dmorita has joined #openstack-swift03:00
*** dmorita has quit IRC03:04
*** dmorita has joined #openstack-swift03:09
jrichlimattoliverau: yum!  did it turn out well?03:11
mattoliverauyeah, it did actually, I think it'll be a keeper :)03:12
*** dmorita has quit IRC03:14
*** pdardeau has quit IRC03:17
*** jamielennox is now known as jamielennox|away03:20
*** dmorita has joined #openstack-swift03:28
*** dmorita has quit IRC03:32
*** m_kazuhiro has quit IRC03:36
*** jamielennox|away is now known as jamielennox03:38
*** m_kazuhiro_ has joined #openstack-swift03:38
*** dmorita has joined #openstack-swift03:40
*** dmorita has quit IRC03:41
*** dmorita has joined #openstack-swift03:42
*** m_kazuhiro_ has quit IRC03:43
mahatic_good morning03:44
* mahatic_ is back after a local holiday in India03:44
*** m_kazuhiro_ has joined #openstack-swift03:45
*** psachin has joined #openstack-swift03:57
*** Jeffrey4l_ has quit IRC04:05
kota_mahatic: good morning and welcome back04:06
*** chsc has joined #openstack-swift04:07
kota_hmm... my though might be wrong. if something like memory allocation failure happen in the liberasurecode, the reconstracted fragment should consist of 004:07
*** chsc has quit IRC04:09
*** links has joined #openstack-swift04:10
*** jistr has quit IRC04:15
*** jistr has joined #openstack-swift04:16
*** klrmn has quit IRC04:23
*** wasmum has quit IRC04:23
mattoliveraumahatic_: o/04:25
*** Jeffrey4l has joined #openstack-swift04:26
*** wasmum has joined #openstack-swift04:35
mahatic_kota_: mattoliverau : hello04:40
*** dmorita has quit IRC05:03
kota_acole: i found *really* interesting thing when trying rebuilding the corrupted fragment, please ping me if you get online.05:07
*** SkyRocknRoll has joined #openstack-swift05:07
kota_acoles:^^05:07
*** SkyRocknRoll has quit IRC05:07
*** SkyRocknRoll has joined #openstack-swift05:07
*** wasmum has quit IRC05:09
*** Jeffrey4l has quit IRC05:09
*** Jeffrey4l has joined #openstack-swift05:11
*** Jeffrey4l has quit IRC05:12
*** Jeffrey4l has joined #openstack-swift05:13
*** Jeffrey4l has quit IRC05:13
*** Jeffrey4l_ has joined #openstack-swift05:13
*** tanee_away is now known as tanee05:19
*** wasmum has joined #openstack-swift05:19
openstackgerritzheng yin proposed openstack/swift: Simplify container_updater test code  https://review.openstack.org/35585805:21
*** ChubYann has quit IRC05:25
*** dmorita has joined #openstack-swift05:34
*** gabor_antal_ has joined #openstack-swift05:35
*** gabor_antal has quit IRC05:37
*** Jeffrey4l_ has quit IRC05:41
openstackgerritJamie Lennox proposed openstack/python-swiftclient: Use requests_mock and testtools  https://review.openstack.org/36029805:43
openstackgerritTuan Luong-Anh proposed openstack/swift: Remove xrange for run both Python 2 and Python 3  https://review.openstack.org/38528505:46
*** Jeffrey4l_ has joined #openstack-swift05:52
*** links has quit IRC05:54
*** dmorita has quit IRC05:55
*** Jeffrey4l_ has quit IRC05:57
*** Jeffrey4l_ has joined #openstack-swift05:58
*** dmorita has joined #openstack-swift05:58
*** Jeffrey4l_ has quit IRC06:06
*** links has joined #openstack-swift06:10
*** dmorita has quit IRC06:10
*** dmorita has joined #openstack-swift06:11
*** Jeffrey4l_ has joined #openstack-swift06:19
*** dmorita has quit IRC06:21
*** Jeffrey4l_ has quit IRC06:23
*** Jeffrey4l_ has joined #openstack-swift06:23
* mattoliverau is calling it a day, have great night swift land o/06:29
*** m_kazuhiro_ has quit IRC06:35
*** rcernin has joined #openstack-swift06:36
*** dmorita has joined #openstack-swift06:39
*** takashi has joined #openstack-swift06:40
*** qwertyco has joined #openstack-swift06:55
*** tesseract has joined #openstack-swift07:06
*** tesseract is now known as Guest2489007:06
*** ppai has joined #openstack-swift07:09
*** wasmum has quit IRC07:10
*** rledisez has joined #openstack-swift07:13
openstackgerritChristian Schwede proposed openstack/swift: Throttle update_auditor_status calls  https://review.openstack.org/38370707:14
*** wasmum has joined #openstack-swift07:15
*** dmorita has quit IRC07:26
*** acoles_ is now known as acoles07:28
*** acoles is now known as acoles_07:30
*** acoles_ is now known as acoles07:31
acoleskota_: ping07:31
kota_hi acoles07:33
*** Guest59427 is now known as zigo07:33
*** dmorita has joined #openstack-swift07:33
acoleskota_: looks like you made some progress07:34
kota_acoles: yeah, i updated the status on https://bugs.launchpad.net/swift/+bug/163114407:35
openstackLaunchpad bug 1631144 in OpenStack Object Storage (swift) "Swift Erasure Coding : Error when decoding fragment" [High,Confirmed]07:35
kota_acoles: summrizing that, i think reconstructor or ssync protocol seems broken.07:35
kota_acoles: i didn't get how it was broken yet07:36
*** dmorita has quit IRC07:37
acoleskota_: ok I'm reading the bug report updates07:37
kota_acoles: thanks07:37
*** geaaru has joined #openstack-swift07:40
kota_acoles: i think, you can see the actual corrupted fragment archive body sent from admin607:41
kota_too07:41
*** dmorita has joined #openstack-swift07:41
acoleskota_: oh, wow, so the bad frag has some http headers in it :/ ?07:42
kota_acoles: yeah, unbelievable but that's fact07:43
acolesand of course the reconstructor does not send etag so the receiving object server does not know, and writes the etag of whatever it received.07:43
kota_acoles: i wonder if you could double check that.07:43
acoleskota_: double :/ and :(07:43
kota_that's assumption what i am going to tell you since now, though07:44
*** rcernin has quit IRC07:45
kota_if a http client reuse the tcp connection, it could happen the receiver regard the header as the body, maybe?07:45
kota_in my experience, i had gotten the case http sever regards the http body as another request header.07:46
kota_not sure right now.07:46
kota_i think sort of that may occurred in the ssync protocol.07:47
acoleskota_: yes I see the headers in the bad fragment07:47
kota_acoles: ok, that's fact we're now.07:47
*** rcernin has joined #openstack-swift07:49
kota_and i'm thinking of how we could test/reproduce that07:49
kota_acoles: that's current status to me07:50
acoleskota_: the headers do not appear to be for that fragment - the timestamp is different07:50
kota_acoles: what do you mean "that fragment"?07:51
kota_acoles: corrupted one? normal one?07:51
*** bkeller` has quit IRC07:52
*** bkeller^ has joined #openstack-swift07:52
acoleskota_: the bad fragment. well the bad object, the timestamp in the headers is newer than in the fragment .data filename07:53
*** bkeller^ is now known as bkeller`07:53
kota_acoles: ah, good catch07:54
*** dmorita_ has joined #openstack-swift07:54
kota_acoles: i didn't notice that, exactly it looks... older?07:54
*** dmorita has quit IRC07:55
kota_header timestamp < .data timestamp?07:55
kota_let me check both bad frags07:55
*** cshastri has joined #openstack-swift07:55
acolesnewer, header timestamp > filename timestamp07:56
admin6acoles, kota_: Hi08:01
kota_admin6: hi08:01
acoleskota_: one hypothesis - if an error occurs in ssync protocol that causes a frag body to not be sent, then the receiver may interpret the next set of *headers and body* as the missing body08:01
acolesadmin6: hi08:01
acoleskota_: which would mean that the content of the corrupt frag file (a) has headers! and (b) is frag data for a different object!08:02
kota_acoles: ah, that might be true08:02
acoleskota_: I am not sure how we could verify that (b) is true08:03
kota_acoles: to tell more, i noticed one more header in the mid of the corrupted file.08:03
*** dmorita_ has quit IRC08:03
admin6II’m currently checking deeper but so far (on 10 different files tested), the corrupted fragment always is a reconstructed fragment.08:03
*** dmorita has joined #openstack-swift08:04
kota_acoles: could we confirm that to check the path?08:04
acolesadmin6: interesting. thanks.08:07
kota_admin6: thanks for your good info. i think we are going to be close the reason08:10
*** qwertyco has quit IRC08:12
cnfmorning08:36
cnfkota_ i got swift3 working btw08:36
cnfeventually :P08:37
kota_cnf: morning and congrats!08:37
cnfkota_ if you care about the back story, with keystone v3 port 5000 and 35357 are the same08:38
cnfso the rest of my setup was all on port 5000, reverse proxied behind an ssl terminator08:39
cnfof course, swift3 was ahrdcoded to v2, which doesn't work on port 500008:39
cnftimburke submitted a patch to make that configurable08:40
cnfthanks to him for that, and thanks to you for swift3 :P08:40
*** wasmum has quit IRC08:54
*** wasmum has joined #openstack-swift08:58
*** mvk has joined #openstack-swift09:17
*** dmorita has quit IRC09:34
*** mvk has quit IRC09:50
*** silor has joined #openstack-swift09:53
*** silor has quit IRC09:56
*** silor has joined #openstack-swift09:57
*** silor1 has joined #openstack-swift10:12
*** silor has quit IRC10:13
*** silor1 is now known as silor10:13
openstackgerritCharles Hsu proposed openstack/swift: Make Content-Disposition support `inline; filename=...` format.  https://review.openstack.org/38538210:24
*** mvk has joined #openstack-swift10:25
*** takashi has quit IRC10:27
*** ppai_ has joined #openstack-swift10:27
*** dosaboy_ is now known as dosaboy10:28
*** ppai has quit IRC10:29
charzkota_: acoles Just found a minor issue in tempurl ^^10:29
charzif you have time, please help to review it. :)10:29
acolescharz: hi. I'm going to be busy on this https://bugs.launchpad.net/swift/+bug/1631144 today, sorry10:32
openstackLaunchpad bug 1631144 in OpenStack Object Storage (swift) "Swift Erasure Coding : Error when decoding fragment" [Critical,Confirmed]10:32
charzacoles: kk, no rush.10:32
*** SkyRocknRoll has quit IRC10:36
openstackgerritCharles Hsu proposed openstack/swift: Make Content-Disposition support `inline; filename=...` format.  https://review.openstack.org/38538210:36
*** kei_yama has quit IRC10:39
*** dmorita has joined #openstack-swift11:01
*** dmorita has quit IRC11:01
*** dmorita has joined #openstack-swift11:02
*** Praveen__ has joined #openstack-swift11:09
Praveen__Hi clayg, Can you please give me pointers/suggestions for the bug 1632171 !!11:15
openstackbug 1632171 in OpenStack Object Storage (swift) "stub account response output missing policy information" [Low,New] https://launchpad.net/bugs/1632171 - Assigned to Praveen N (praveenn)11:15
acoleskota_: I managed to get a test to reproduce the issue on bug 1631144, need some more time to clean it up11:18
openstackbug 1631144 in OpenStack Object Storage (swift) "Swift Erasure Coding : Error when decoding fragment" [Critical,Confirmed] https://launchpad.net/bugs/163114411:18
openstackgerritKazuhiro MIYAHARA proposed openstack/swift: Remove 'X-Static-Large-Object' from .meta files  https://review.openstack.org/38541211:45
*** ppai_ has quit IRC11:50
*** nottrobin has quit IRC11:58
*** nottrobin has joined #openstack-swift11:59
*** dmorita has quit IRC12:03
*** dmorita has joined #openstack-swift12:19
*** dmorita has quit IRC12:23
*** dmorita has joined #openstack-swift12:27
*** Praveen__ has quit IRC12:29
*** links has quit IRC12:42
*** cshastri has quit IRC12:44
*** klamath has joined #openstack-swift13:05
*** klamath has quit IRC13:05
*** klamath has joined #openstack-swift13:06
*** cdelatte has joined #openstack-swift13:13
*** rcernin has quit IRC13:13
*** rcernin has joined #openstack-swift13:13
*** tongli has joined #openstack-swift13:24
*** dmorita has quit IRC13:32
kota_back to online13:33
kota_thanks acoles for updating  https://launchpad.net/bugs/163114413:34
openstackLaunchpad bug 1631144 in OpenStack Object Storage (swift) "Swift Erasure Coding : Error when decoding fragment" [Critical,Confirmed]13:34
kota_cnf: yeah, it looks patch 384659 which timburke is working?13:36
patchbothttps://review.openstack.org/#/c/384659/ - swift3 - Make s3token work in a Keystone-V3-only world13:36
kota_it seems good stuff13:37
openstackgerritMerged openstack/swift: Update test-requirements.txt  https://review.openstack.org/36527113:38
openstackgerritMerged openstack/swift: [api-ref] Remove temporary block in conf.py  https://review.openstack.org/36653813:39
kota_charz: thanks for pinging me, will look at in this week. probably we got a priority for the bug acoles suggested in the irc log in my opinion.13:39
acoleskota_: np. I may even have a fix, at least a fix for the test I wrote. It's kinda ugly though.13:40
kota_acoles: yeah, if i undestand correctly, we have a bunch of broken parts to fix for the bug, broken ssync protocol, broken reconstruction, etc...13:42
kota_but some of them can be pend as improvement, the most priority thing is to prevent the broken frag anyway.13:43
acoleskota_: exactly, I'm just looking for a pragmatic quick fix first.13:44
kota_acoles: great13:44
acoleskota_: get some sleep !)13:45
kota_acoles: thanks! though I'm back to online, I'm getting feeling i might be tired a bit ;-)13:46
* kota_ is going to be offline again to take a rest for tommorrow meeting.13:46
acoleskota_: see you in 7 hours or so :)13:47
*** dmorita has joined #openstack-swift13:48
*** dmorita has quit IRC13:52
*** pcaruana has joined #openstack-swift13:54
*** tqtran has joined #openstack-swift14:03
*** tqtran has quit IRC14:07
*** kmARC has quit IRC14:09
cnfkota_ that's the one14:15
*** kmARC has joined #openstack-swift14:16
*** dmorita has joined #openstack-swift14:20
*** SkyRocknRoll has joined #openstack-swift14:22
*** SkyRocknRoll has quit IRC14:22
*** SkyRocknRoll has joined #openstack-swift14:22
*** dmorita has quit IRC14:24
*** pcaruana has quit IRC14:27
*** _JZ_ has joined #openstack-swift14:30
*** sgundur has joined #openstack-swift14:34
*** sgundur has quit IRC14:40
*** _JZ_ has quit IRC14:47
*** dmorita has joined #openstack-swift14:51
*** vint_bra has joined #openstack-swift14:53
*** dmorita has quit IRC14:56
*** psachin has quit IRC15:05
*** urth has quit IRC15:08
*** vint_bra has quit IRC15:08
*** urth has joined #openstack-swift15:10
*** vint_bra has joined #openstack-swift15:10
*** klrmn has joined #openstack-swift15:10
*** SkyRocknRoll has quit IRC15:12
*** rcernin has quit IRC15:19
*** joeljwright has joined #openstack-swift15:21
*** ChanServ sets mode: +v joeljwright15:21
*** dmorita has joined #openstack-swift15:23
*** dmorita has quit IRC15:27
*** vint_bra has quit IRC15:28
*** vern has joined #openstack-swift15:38
*** links has joined #openstack-swift15:39
*** pdardeau has joined #openstack-swift15:42
*** sgundur has joined #openstack-swift15:43
*** david-lyle has joined #openstack-swift15:46
*** david-lyle has quit IRC15:52
*** dmorita has joined #openstack-swift15:55
*** dmorita has quit IRC15:59
*** jordanP has joined #openstack-swift16:02
*** david-lyle has joined #openstack-swift16:06
*** klrmn has quit IRC16:19
*** dmorita has joined #openstack-swift16:26
*** dmorita has quit IRC16:31
openstackgerritOpenStack Proposal Bot proposed openstack/swift: Updated from global requirements  https://review.openstack.org/8873616:34
notmynamegood morning16:39
*** links has quit IRC16:45
*** rledisez has quit IRC16:46
*** nikivi has joined #openstack-swift16:47
*** rcernin has joined #openstack-swift16:57
notmynamewow. interesting ML thread on the PTG. seems to be a bit of concern that this will make communication with users harder and limit available travel16:57
*** dmorita has joined #openstack-swift16:58
*** Guest24890 has quit IRC17:00
*** jeblair_ is now known as jeblair17:03
*** dmorita has quit IRC17:05
*** zul has quit IRC17:11
*** jordanP has quit IRC17:12
*** zul has joined #openstack-swift17:12
*** cshastri has joined #openstack-swift17:15
*** klrmn has joined #openstack-swift17:16
*** ChubYann has joined #openstack-swift17:22
claygyeah, but I mean realisticly some orgs were already sturggling to get ops to international summit events - i didn't expect PTG's to make it any better17:22
*** sgundur has quit IRC17:23
claygI think for teams that didn't have a culture of self organizing mid-cycles the reality of traveling 4 times a year is setting in17:23
notmynameFYI http://lists.openstack.org/pipermail/openstack-dev/2016-October/105600.html17:31
*** sgundur has joined #openstack-swift17:34
*** rcernin has quit IRC17:56
*** thebloggu has joined #openstack-swift18:00
*** tqtran has joined #openstack-swift18:04
*** tqtran has quit IRC18:08
*** lcurtis has joined #openstack-swift18:11
*** seventy has joined #openstack-swift18:14
*** geaaru has quit IRC18:23
*** mvk has quit IRC18:28
tdasilvanotmyname: cool email!18:43
*** sgundur has quit IRC18:46
notmynamethanks18:46
ahalei liked yours in the tripleo thread tdasilva18:46
*** tqtran has joined #openstack-swift18:47
tdasilvaahale: yeah, i was trying to understand what they were trying to achieve, it's too bad they are going away from swift, it would be nice to understand better what the issues are18:48
tdasilvaahale: i was just learning today about the differences between the gnocchi ceph driver and the swift driver18:49
ahalei havent dared look that closely at it18:50
tdasilvaahale: so I guess there are certain work loads where it is not very suitable for swift, at least not in the way it is written18:50
tdasilvabut i'd have to play more with it18:50
*** thebloggu has quit IRC18:57
*** sgundur has joined #openstack-swift19:02
*** mvk has joined #openstack-swift19:28
claygwhen is the meeting?19:35
acolesclayg in 1hour 17mins19:43
openstackgerritClay Gerrard proposed openstack/swift: Fix signal handling for daemons with InternalClient  https://review.openstack.org/25934719:44
acolestdasilva: thanks for your comments on patch 355958, I have been busy on the EC corrupt fragments bug today but will respond19:47
patchbothttps://review.openstack.org/#/c/355958/ - swift - EC - eliminate .durable files19:47
*** cshastri has quit IRC19:49
tdasilvaacoles: so i was about to update that comment, i have verified that the issue I found is fixed by patch 37663019:50
patchbothttps://review.openstack.org/#/c/376630/ - swift - EC: reconstruct using non-durable fragments19:50
acolestdasilva: cool. thanks!19:50
acolesI just started seein "liberasurecode_backend_open: dynamic linking error libisal.so: cannot open shared object file: No such file or directory" when running probe tests, what magic do I need to do?19:52
acolessince I upgraded to pyeclib 1.3.119:52
tdasilvaupgrade liberasurecode19:52
acolesk19:53
tdasilvai'm assuming you are running 1.1.119:53
*** vern has quit IRC19:53
*** _JZ_ has joined #openstack-swift19:54
acolesI was on pyeclib 1.2.019:54
tdasilvammmmm :/19:54
tdasilvaoh yeah...sorry...pyeclib19:55
tdasilvaso yeah, you want to upgrade liberasurecode from 1.1.1 to 1.2.019:55
acolessorry yes I have liberasurecode 1.1.0 so I need to upgrade. so pyeclib does not update liberasurecode as a dependency?19:57
* acoles vaguely remembers that changing at soem point19:57
tdasilvaacoles: i don't think so20:02
*** vern has joined #openstack-swift20:02
tdasilvaplus i don't think there are distro builds available for 1.2.0 yet ???? maybe ???20:03
tdasilvai think there's isn't for rh* based systems and remember there not being avaiable for ubuntu either20:03
*** sgundur has quit IRC20:06
*** sgundur has joined #openstack-swift20:11
openstackgerritAlistair Coles proposed openstack/swift: Prevent ssync writing bad fragment data to diskfile  https://review.openstack.org/38560920:12
acoleskota_: ^^20:13
acolestdasilva: k, thanks20:13
*** silor has quit IRC20:14
*** tqtran has quit IRC20:19
*** acoles is now known as acoles_20:20
*** cdelatte has quit IRC20:27
openstackgerritMerged openstack/swift: Simplify container_updater test code  https://review.openstack.org/35585820:30
*** mtreinish has quit IRC20:31
*** mtreinish has joined #openstack-swift20:32
*** sgundur has quit IRC20:39
clayggd lp bug #163283920:42
openstackLaunchpad bug 1632839 in OpenStack Object Storage (swift) "Reconstructor does not send an ETag with a reconstructed fragment" [Undecided,New] https://launchpad.net/bugs/163283920:42
claygis that the root cause of lp bug #163114420:42
openstackLaunchpad bug 1631144 in OpenStack Object Storage (swift) "Swift Erasure Coding : Error when decoding fragment" [Critical,Confirmed] https://launchpad.net/bugs/163114420:42
claygcause I think I sorta remember someone pointing out that rebuilt fragments don't do etag validation (we had to pop the etag from the metadata of ondisk frag to avoid ssync throwing a conflict)20:43
*** dmorita has joined #openstack-swift20:43
clayg... but I probably said something stupid like "that should rarely be an issue"20:43
mattoliveraumorning20:49
*** seventy has quit IRC20:49
*** m_kazuhiro has joined #openstack-swift20:50
claygthe woliver!20:52
mattoliverauyeah its me!20:53
clayg:D20:53
claygmakes me so happy20:53
*** kei_yama has joined #openstack-swift20:56
kota_good morning20:57
notmynameclayg: you know, mattoliverau keeps saying the "au" is for australia, but I think it's really for "gold"20:58
mattoliveraukota_: morning20:58
*** acoles_ is now known as acoles20:58
mattoliveraunotmyname: well of course it is, I'm the golden woliver :P20:58
claygahh... the golden woliver is the rarest of the wiolivers20:59
mattoliverauyeah we are only found on the Australian NSW coast near beaches ;P20:59
kota_mattoliverau: o/20:59
notmynamemeeting time in #openstack-meeting21:00
kota_acoles thanks for the patch, will look at in daytime today21:00
acolesclayg: bug 1632839 isn't the cause of bug 1631144 but it contributes to 1631144 not being detected21:00
openstackbug 1632839 in OpenStack Object Storage (swift) "Reconstructor does not send an ETag with a reconstructed fragment" [Undecided,New] https://launchpad.net/bugs/163283921:00
openstackbug 1631144 in OpenStack Object Storage (swift) "Swift Erasure Coding : Error when decoding fragment" [Critical,Confirmed] https://launchpad.net/bugs/163114421:00
claygacoles: i thought i read somewhere that the fragment data was corrupt/bad despite matching the etag that was written on it?  so... some sort of error in transmission?21:01
*** sgundur has joined #openstack-swift21:02
acolesclayg: yes, bad data is sent with no ETag (cos reconstructor can't send an ETag header for a sync frag job) so receiver just computes md5 of garbage and adds it to metadata21:03
acolesthere is no validation21:03
acolesin this case we don't suspect on the wire corruption, just ssync protocol doing the wrong thing21:03
claygoh jesus - yeah i'm reading the commit message on patch 38560921:03
patchbothttps://review.openstack.org/#/c/385609/ - swift - Prevent ssync writing bad fragment data to diskfile21:03
acolesclayg: it's...a...mess21:04
clayggood21:04
acolesclayg: meeting time, I was late but you're later ;)21:04
claygglad you're all over it then21:04
*** mtreinish has quit IRC21:05
acolesclayg: worse thing is the reporter has seen this for real, more than once :/21:06
*** joeljwright has quit IRC21:06
claygyup but there's some hope repairing the bad frags will allow them to rebuild successfully21:06
clayggo go ec durability!21:07
acolesyes, just they will have to be manually removed cos the auditor thinks they are fine and dandy21:08
*** sgundur1 has joined #openstack-swift21:10
*** joeljwright has joined #openstack-swift21:11
*** ChanServ sets mode: +v joeljwright21:11
*** mtreinish has joined #openstack-swift21:12
*** cebruns_ has quit IRC21:15
*** zigo_ has joined #openstack-swift21:17
*** kota_ has quit IRC21:17
*** kota_ has joined #openstack-swift21:18
*** ChanServ sets mode: +v kota_21:18
kota_did irc halt?21:18
acoleskota_: I didn't work out an nice way to test the read timeout case but the other cases cover same outcome.21:19
*** kmARC has quit IRC21:19
*** wasmum has quit IRC21:19
*** zigo has quit IRC21:19
*** cebruns_ has joined #openstack-swift21:20
claygcan people try and raise a hand in #openstack-meeting?21:20
kota_acoles: ok, maybe i can try to cover that case21:20
*** lcurtis has quit IRC21:20
*** lcurtis has joined #openstack-swift21:21
acoleskota_: one thought I had was to mock the Timeout context manager to raise an exception. not great. but I didn't want to go putting timeouts into unit tests.21:21
kota_acoles: ah, you mean it's overkill for *unit*?21:22
*** philipw has quit IRC21:23
*** ujjain has quit IRC21:23
*** StevenK has quit IRC21:23
*** CrackerJackMack has quit IRC21:23
*** kmARC has joined #openstack-swift21:23
*** ujjain has joined #openstack-swift21:23
*** ujjain has joined #openstack-swift21:23
*** klrmn has quit IRC21:24
acoleskota_: I mean its tricky because if we reduce the timeout times we risk the tests being flakey, and if we leave them high then the test is slow21:24
acoleskota_: i.e. if I mock a response that just sleeps on a read, then how long should it sleep?21:25
kota_acoles: gotcha, if we do that, probably need to patch the timeout value or sort of mock the read method to raise Timeout exception?21:26
*** philipw has joined #openstack-swift21:28
*** StevenK has joined #openstack-swift21:28
*** CrackerJackMack has joined #openstack-swift21:28
*** wasmum has joined #openstack-swift21:28
*** wasmum has quit IRC21:28
*** wasmum has joined #openstack-swift21:28
*** zigo_ is now known as zigo21:28
*** vern has quit IRC21:34
*** Jeffrey4l__ has joined #openstack-swift21:34
*** vern has joined #openstack-swift21:35
*** Jeffrey4l_ has quit IRC21:35
*** mtreinish has quit IRC21:36
*** mtreinish has joined #openstack-swift21:42
*** tongli has quit IRC21:45
jrichlitorgomatic: dont leave us hangin22:01
claygtorgomatic: wait what was the bunch line!?22:01
tdasilvalol22:01
torgomaticyou get drunk22:01
notmynameI was really hoping to get that in the meeting notes :-)22:01
torgomaticbecause √beer * √beer is just beer22:02
jrichli:p22:02
acoleskota_: problem I saw is the timeout is on a wait for the threads in the pile, and I am not sure what happens if a mocked method raises a timeout exception in that thread? I always get confused by that.22:03
claygnotmyname: he was looking up the ascii codes for square root symbols :\22:03
*** m_kazuhiro has quit IRC22:03
notmynameheh22:03
cschwedeclayg: let’s chat on your Thursday morning? i have a few ideas to add tomorrow22:04
kota_acoles: IIRC, we need to catch the exception in the thread and have to wrap or...22:04
kota_acoles: hmm... probably I have to play with code to clarify the feasibility.22:04
acoleskota_: yeah but IDK how you get the Timeout to pop without making the thread sleep22:05
*** mrda has joined #openstack-swift22:05
kota_acoles: k, will try (it might fail though) and let you know if better way found.22:06
kota_acoles: thanks for calling it anyway.22:06
claygacoles: is patch 385609 wip?22:07
patchbothttps://review.openstack.org/#/c/385609/ - swift - Prevent ssync writing bad fragment data to diskfile22:07
claygcschwede: for me tomorrow is thursday - ah yeah so that would be after your work day22:08
acolesclayg: wip == written-in-panic ? ;)22:08
acolesclayg: no but since you ask I am now wondering if you think it should be22:08
claygrofl22:08
*** joeljwright has quit IRC22:08
*** mtreinish has quit IRC22:08
cschwedeclayg: yep, in my evening. i’m available from ~ 11am-2pm your time - just let me know what works for you22:08
claygRebuildingECDiskFileStream grew some initialized state that seems unused?22:09
acolesclayg: it could probably use some more tests, but I think what is there is probably sufficient to get a fix out.22:09
claygcschwede: np, same time as last time except tomorrow wfm22:09
acolesclayg: :/ self.bytes_read ?? I guess that without even looking22:09
acolesclayg: it was a long day, I'll rev it tomorrow22:09
cschwedeclayg: ok, does Friday work better for you?22:10
claygacoles: maybe *I* should rev it while i'm reviewing and if I htink it's good enough I should +2 it!?22:10
claygcschwede: no tomorrow - maybe on Friday again if we need it - it's getting close :\22:10
* kota_ is going breakfast22:11
claygacoles: well before you go could you at lest give me a hint why yout hink it's only a partial fix?22:11
clayg... fi you're still here22:11
cschwedeclayg: i think my brain sleeps already, sorry - when do we meet?22:11
acolesclayg: fine with me22:11
clayg:D22:11
notmynameFYI public transport pass in barca https://www.tmb.cat/en/barcelona/fares-metro-bus/card-transport-barcelona22:12
claygcschwede: 6:00:00 pm UTC | Thursday, October 13, 201622:12
cschwedeclayg: awesome! alright. tty tomorrow!22:13
acolesclayg: I went for partial because ...hmm, well I guess that's me being cautious again. Having stared at it all day there's other ways the reconstructor might sync garbage. e.g. there is no check that frags fetched are consistent with their etags. maybe pyeclib would blow up in that case.22:14
acolesclayg: IDK, on reflection I could live with closes-bug22:14
claygwell I think this guys bug based on kota's examination - all signs point the issue you fixing22:16
jrichlifor tourist stuff, acoles had mentioned this place earlier: http://www.sagradafamilia.org/en/22:16
claygthere may be other issues - but let's just open other bugs (failure to send etag is an obvious belts and braces thing that allowed this to happen)22:17
acolesclayg: OK. well since you'll be editing the commit message to add yourself as co-author youcan also s/Partial/Closes/ :)22:18
claygnice :P22:18
claygg'night22:18
*** mtreinish has joined #openstack-swift22:19
claygand a context switching we will go; and a context ... wait what was I doing?22:19
acolesgood night22:20
claygjoel<tab><tab> :'(22:20
*** sgundur has quit IRC22:23
*** nikivi has quit IRC22:24
*** acoles is now known as acoles_22:25
claygi'm getting liberasurecode_backend_open: dynamic linking error when restarting with swift-init now?22:28
clayg... on saio22:28
claygwell - i should rephrase - i *always* get that in syslog restarting - but it only shows up at the console with pyeclib 1.3.122:33
*** mtreinish has quit IRC22:41
kota_clayg: please check your liberasurecode version?22:46
*** mtreinish has joined #openstack-swift22:47
*** klrmn has joined #openstack-swift22:49
*** lcurtis has quit IRC22:52
*** vern has quit IRC22:55
*** vern has joined #openstack-swift22:57
*** klrmn has quit IRC23:00
claygkota_: it's the one that comes with the distro - 1.1 on xenail23:05
claygis there instructions for installing over distro package versions?23:05
claygis there anywhere to get pre-built binaries or is it essentailly "compile by hand and put the .so some where in your ld path"23:06
claygjust git clone - configure; make; make install ????23:06
*** klrmn has joined #openstack-swift23:15
kota_clayg: clone, bindep, apt, autogen.sh, configure, make, make install23:29
kota_Sorry, now typing from mobile23:29
kota_clayg: That's procedure to install, maybe you can see that in sort of README23:31
kota_Or repo after xenail (forgot the name) looks to include liberasurecode 1.2.123:33
kota_anyway liberasurecode 1.1 + Pyeclib 1.3.1 will show the message, liberasurecode>=1.2.0 is needed23:36
*** klamath has quit IRC23:40
claygyakkety23:41
clayg16.10 - comes out tomorrow23:42
kota_Ah, tomorrow :/23:50
kota_Maybe i saw it for rc staff23:50
*** dmorita_ has joined #openstack-swift23:59

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