Wednesday, 2016-12-07

*** asettle has joined #openstack-swift00:02
notmynamebackports approved. the existing +1s from other cores there was very helpful00:06
notmyname(I hope that the order I went through them doesn't cause merge issues)00:06
*** dmorita has quit IRC00:06
*** asettle has quit IRC00:07
*** dmorita has joined #openstack-swift00:07
claygnotmyname: hello!00:15
claygnotmyname: did i read your tweet right?  SLC (!?)00:15
notmynamesalt lake city is cold!00:15
notmynameit's currently 23! Fahrenheit!00:15
notmynamelow tonight is 1000:15
notmynameten00:15
claygscoff - global "warming"00:15
notmynameI saw an article that claims it's colder now that it was in june! (although mattoliverau wouldn't agree with that statement)00:16
mattoliverauwow, yeah, not that cold here. esp when compared to June. Seeing as june is winter :P00:18
notmyname:-)00:19
mattoliverausigh, ssbench was killed over night due to OOM killer. Time to make a cloud server with a little more ram to be the master. Also, I had to hack ssbench to allow me to send container headers to put (turn on sharding).. so will add some tests and push that up at some stage.00:20
notmynameoh, cool00:21
*** dmorita has quit IRC00:21
notmynameyeah, that would be good00:21
notmynamewere you able to use the same mechanisms used for sending the policy header?00:21
*** dmorita has joined #openstack-swift00:22
mattoliverauin a way, the policy is sent in as a param. But now you can specify container_put_headers in your scenario files.00:22
mattoliverauI might clean it up a little. its more just to get ssbench to do my bidding :P00:23
*** catintheroof has quit IRC00:23
*** catintheroof has joined #openstack-swift00:24
*** dmorita has quit IRC00:26
*** dmorita has joined #openstack-swift00:28
*** catintheroof has quit IRC00:28
*** _JZ_ has quit IRC00:32
*** manous has joined #openstack-swift00:46
claygkota_: patch 393263 is still looking good!!!00:46
patchbothttps://review.openstack.org/#/c/393263/ - liberasurecode - ISA-L Cauchy support00:46
claygcschwede: friendly ping to add patch 346475 to loop back to patch on your list cc: torgomatic timburke anyone else interested in rings00:49
patchbothttps://review.openstack.org/#/c/346475/ - swift - For any part, only one replica can move in a rebal...00:49
*** tovin07 has joined #openstack-swift00:52
claygyay tsg and kevin are both making comments on the ISA-L change (!?)00:57
*** arch-nemesis has quit IRC00:58
kota_good morning01:01
mattoliveraukota_: morning01:01
kota_clayg: thanks for reviewing and tsg and kmgreen2 make small comments, i will update soon to land quickly01:02
*** asettle has joined #openstack-swift01:03
*** Jeffrey4l has quit IRC01:06
*** Jeffrey4l has joined #openstack-swift01:06
*** jamielennox is now known as jamielennox|away01:07
*** asettle has quit IRC01:07
*** Jeffrey4l has quit IRC01:08
openstackgerritKota Tsuyuzaki proposed openstack/liberasurecode: ISA-L Cauchy support  https://review.openstack.org/39326301:09
*** Jeffrey4l has joined #openstack-swift01:09
kota_mattoliverau: morning01:09
kota_clayg: done01:09
kota_hope that can land soon01:09
*** vinsh has joined #openstack-swift01:19
*** Jeffrey4l has quit IRC01:20
*** Jeffrey4l has joined #openstack-swift01:21
*** jamielennox|away is now known as jamielennox01:21
*** david-lyle has joined #openstack-swift01:33
kota_clayg, timburke: it look like your turn to land patch 393263 ;-)01:37
patchbothttps://review.openstack.org/#/c/393263/ - liberasurecode - ISA-L Cauchy support01:37
claygkota_: it's too bad maybe that isa-l is not tested in the gate?  not sure...01:37
*** Jeffrey4l has quit IRC01:39
kota_clayg: exactly, we cannot test for now01:39
*** Jeffrey4l has joined #openstack-swift01:39
kota_clayg: i think if we will have sort of test_setup.sh supported by infra team, I'd like to make a script to setup isa-l from source01:39
kota_clayg: or if you know how to run some scripts with sudo before testing in the gate, please let me know, that's a way I'm looking for.01:41
kota_Or could we have custom test script for c lang project?01:42
*** mingyu has joined #openstack-swift01:43
kota_it might be a time to look at project-config - Or asking tdasilva01:43
kota_clayg: note that, at least, I ran test ISA-L Cauchy patch with the environ isal installed and all test passed as green. So I'm sure it's nice.01:44
timburkei guess we go edit https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/liberasurecode.yaml ?01:45
kota_timburke: nice, it looks like we can add isa-l installation there.01:47
*** mingyu has quit IRC01:49
*** mingyu has joined #openstack-swift01:51
*** StraubTW has joined #openstack-swift01:52
openstackgerritThiago da Silva proposed openstack/swift: adding unit tests for if-none-match fix  https://review.openstack.org/40104401:52
kota_clayg: to consider you want to use ISA-L Cauchy asap, it may be better that the setting for isa-l testing land after ISA-L Cauchy land. Otherwise, probably dependency chain will stand up against to us to block ISA-L likey 'older version doesn't support the ISA-L cauchy and we cannot test that so -1 (from jenkins)'01:54
kota_complicated dependency chain between c - python problem :/01:55
kota_ah, could we add building script liberasurecode from source as well as pyeclib?01:56
*** StraubTW has quit IRC01:56
kota_um... probably it's no because we set *python-job* for pyeclib so no hooking point to install liberasurecode from source different from liberasurecode :/01:57
claygkota_: I think i said to john that I'd like one pyeclib job that tests from packaged distro liberasure.so and other that tests on built from source (maybe?)01:58
claygkota_: another option/idea is on one of those also go ahead and check out master isa-l (maybe?)01:58
claygkota_: none of this is a blocker for landing isa-l-rs-cauchy (because I do not want to wait!)01:58
timburkethe split pyeclib jobs might be doable via tox envs. is there a way for us to tell the pyeclib compile about some user-provided libs so we don't have to go install liberasurecode before dropping sudo?02:00
kota_tibmurke: I don't hink we can go install c library via tox w/o sudo02:02
kota_timburke: IIRC, it's the resaon why we split the repo pyeclib/liberasurecode02:02
*** asettle has joined #openstack-swift02:04
kota_timburke: say cannot but it may not impossible but it should be too tricky to install c lib via tox02:04
kota_timburke: Or great idea for the way is always welcome02:05
*** Jeffrey4l has quit IRC02:06
*** Jeffrey4l has joined #openstack-swift02:07
*** asettle has quit IRC02:08
*** sanchitmalhotra has quit IRC02:12
claygkota_: ok, down to patch 393276 now!02:28
patchbothttps://review.openstack.org/#/c/393276/ - pyeclib - ISA-L Cauchy support02:28
kota_clayg: nice!02:29
*** StraubTW has joined #openstack-swift02:32
openstackgerritMerged openstack/liberasurecode: ISA-L Cauchy support  https://review.openstack.org/39326302:34
kota_yey!02:34
*** hogepodge has quit IRC02:38
*** m_kazuhiro has joined #openstack-swift02:45
claygfor someone running liberasurecode *not* built from source (using distro packages)02:47
claygit might be interesting to try out patch 39327602:47
patchbothttps://review.openstack.org/#/c/393276/ - pyeclib - ISA-L Cauchy support02:47
claygit's just a little change to pyeclib - but you could totally clone it and pip install it and try running your swifts - everything should work - but it might be a good introduction to "all this crazy pyeclib/liberasurecode/isa-l stuff" if you were looking to dip a toe in!02:48
*** asettle has joined #openstack-swift03:04
*** mingyu has quit IRC03:07
*** asettle has quit IRC03:09
*** links has joined #openstack-swift03:22
*** Jeffrey4l has quit IRC03:23
*** kei_yama has quit IRC03:23
*** Jeffrey4l has joined #openstack-swift03:24
openstackgerritTim Burke proposed openstack/pyeclib: Add tox environment to test against liberasurecode master  https://review.openstack.org/40782903:25
kota_timburke: oh, is that a way?03:28
*** winggundamth_ has joined #openstack-swift03:31
claygkota_: he's explaining it to me03:32
openstackgerritKota Tsuyuzaki proposed openstack/pyeclib: Make hard-coded dependency for liberasurecode>=1.3.1  https://review.openstack.org/39599803:32
kota_trying to rebase my had-corded dependency patch to that.03:32
claygit seems to let you set a env var and that plus a script will do a user space install that "works" because of crazy shit in setup.py that tells the extension about some library and include paths?03:32
claygi don't understand how c works03:32
kota_maybe?03:35
*** m_kazuhiro has quit IRC03:36
kota_not sure for now, the lib looks like to be installed to user space but how we could notify the lib path for dynamic linking03:36
*** dmorita has quit IRC03:37
jrichlinotmyname: I will be in Chicago next week where it is expected that lows remain in the teens.  Same week in Austin, lows don't go below 45 next week.03:37
claygkota_: basically timburke says you can bring up a fresh vm with all the stuff in bindep (but *not* liberasurecode) and when you tox -e liberasurecode-master it works03:37
kota_clayg: hmmm03:38
*** Jeffrey4l has quit IRC03:38
claygin theory you'd get a .tox/liberasurecode-master than when you import pyeclib you can sorta get the gist of it using this LIBERASURECODE_DIR path03:38
claygi starred it03:38
*** Jeffrey4l has joined #openstack-swift03:38
claygthere's a bunch of moving parts I don't really know about - timburke is like a sponge tho - he knows everything03:39
kota_or run sort of `ldconfig $LIBERASURECODE_DIR` to find the shared object lib?03:39
kota_ok, we need to avoid liberasurecode from bindep to use liberasurecode from source03:42
*** dmorita has joined #openstack-swift03:42
*** dmorita has quit IRC03:43
*** dmorita has joined #openstack-swift03:44
claygkota_: open question for me is how the contortions in the setup.py and virtualenv handle preferences when different versions are installed in different places03:47
claygI don't even know currently how it handles the case where I have system packages installed and then also installed from source with sudo!03:48
*** dmorita has quit IRC03:48
claygbe nice to bounce the whole idea of someone in infra that may have some experience doing this elsewhere ... or not03:48
claygI also don't know what tests we should run and how they should vote even if tim's change works - but it'd be good to know that it works sorta seperate from that03:49
kota_AFAIK, that depends on search path order03:50
kota_clayg: so if the user space is prior to system path like /usr/lib (or /usr/include for header), it could be ok to setup different version into different places.03:51
kota_but order depended work could make another ploblem...03:52
kota_idk, how order, vurtialenv refers the c libs but i think virtualenv doesn't take care of the system-wide setting in outside of python03:54
kota_FWIW, looking at the result of http://logs.openstack.org/98/395998/5/check/gate-pyeclib-python27-ubuntu-xenial/de7474f/console.html#_2016-12-07_03_35_13_98212603:55
kota_it looks like pyeclib still refers /usr/local/lib/liberasurecode.so (installed by package repo) in current status.03:56
kota_probably, we need something, (dropping liberasurecode from bindep? or ldconf to the source installed dir? not sure w/o trying)03:57
kota_to enable pyeclib to refer user installed liberasurecode.03:57
*** StraubTW has quit IRC03:57
*** tqtran has quit IRC03:58
*** asettle has joined #openstack-swift04:05
*** mingyu has joined #openstack-swift04:08
kota_oops, I was missing the tox environ is set as a different environ :/04:09
kota_so the hard-coded dependency should be failed04:09
kota_let me check testenv:liberasurecode-master04:09
*** asettle has quit IRC04:10
*** Jeffrey4l has quit IRC04:11
*** Jeffrey4l has joined #openstack-swift04:11
*** psachin has joined #openstack-swift04:12
*** mingyu has quit IRC04:12
*** links has quit IRC04:20
*** dmorita has joined #openstack-swift04:25
*** Jeffrey4l has quit IRC04:29
*** Jeffrey4l has joined #openstack-swift04:30
*** dmorita has quit IRC04:30
*** Jeffrey4l has quit IRC04:30
*** Jeffrey4l has joined #openstack-swift04:31
*** psachin has quit IRC04:32
kota_timburke: nice, all tests for patch 395998 passed with liberasurecode-master env so that way can enable to use the master.04:32
patchbothttps://review.openstack.org/#/c/395998/ - pyeclib - Make hard-coded dependency for liberasurecode>=1.3.104:32
kota_er, it looks like runtime_library_dirs is a magic word04:36
kota_https://docs.python.org/2.7/distutils/apiref.html#distutils.core.Extension04:38
kota_nice!!!!!!!!!04:38
kota_runtime_library_dirslist of directories to search for C/C++ libraries at run time (for shared extensions, this is when the extension is loaded)04:39
*** psachin has joined #openstack-swift04:49
*** links has joined #openstack-swift04:57
*** tqtran has joined #openstack-swift04:59
*** tqtran has quit IRC05:00
*** ppai has joined #openstack-swift05:05
*** asettle has joined #openstack-swift05:06
*** asettle has quit IRC05:11
*** robcresswell has quit IRC05:14
*** onovy has quit IRC05:14
*** AndyWojo has quit IRC05:14
*** mgagne has quit IRC05:15
*** nottrobin has quit IRC05:15
*** DuncanT_ has joined #openstack-swift05:15
*** samueldmq_ has joined #openstack-swift05:15
*** wasmum has quit IRC05:15
*** CrackerJackMack has quit IRC05:15
*** DuncanT has quit IRC05:15
*** DuncanT_ is now known as DuncanT05:16
*** zacksh has quit IRC05:16
*** jamielennox has quit IRC05:16
*** samueldmq has quit IRC05:16
*** samueldmq_ is now known as samueldmq05:17
*** robcresswell has joined #openstack-swift05:17
*** zacksh has joined #openstack-swift05:18
*** AndyWojo has joined #openstack-swift05:19
*** onovy has joined #openstack-swift05:20
*** wasmum has joined #openstack-swift05:20
*** CrackerJackMack has joined #openstack-swift05:20
*** dmorita has joined #openstack-swift05:22
*** nottrobin has joined #openstack-swift05:23
*** dmorita has quit IRC05:26
*** jamielennox has joined #openstack-swift05:31
*** Jeffrey4l has quit IRC05:35
*** Jeffrey4l has joined #openstack-swift05:35
*** dmorita has joined #openstack-swift06:24
*** SkyRocknRoll has joined #openstack-swift06:25
*** dmorita has quit IRC06:28
*** SkyRocknRoll has quit IRC06:30
*** acorwin_ has quit IRC06:32
*** acorwin has joined #openstack-swift06:34
*** ChubYann has quit IRC06:42
*** sams-gleb has joined #openstack-swift06:42
*** SkyRocknRoll has joined #openstack-swift06:43
timburkekota_: yeah, but you still need library_dirs to build it in the first place. you need the combo06:44
timburkei ought to look at using sys.prefix, though06:44
*** silor has joined #openstack-swift06:51
*** mingyu has joined #openstack-swift06:54
*** silor1 has joined #openstack-swift06:56
*** silor has quit IRC06:57
*** silor1 is now known as silor06:57
*** sams-gleb has quit IRC06:59
*** sams-gleb has joined #openstack-swift06:59
*** sams-gleb has quit IRC07:03
*** winggundamth_ has quit IRC07:05
openstackgerritTim Burke proposed openstack/pyeclib: Add tox environment to test against liberasurecode master  https://review.openstack.org/40782907:07
*** asettle has joined #openstack-swift07:08
*** jamielennox is now known as jamielennox|away07:11
kota_timburke: perhaps, we could build isa-l from source too with same way for testing at gate?07:12
*** asettle has quit IRC07:12
timburkekota_: yeah, i'd expect we can. hadn't looked into it yet07:12
*** sileht has quit IRC07:13
kota_timburke: amazing work, absolutely worthful07:13
timburkei wonder if just cloning/installing isa-l first, and using the same --prefix for both might be enough?07:14
kota_timburke: I think so because you have worked on sys.prefix07:14
kota_timburke: isa-l installation is really similar way with libersaurecode, autogen, configure, make, make install07:15
timburke(part of it was i realized that we're going to be fighting these same battles repeatedly, so i ought to start looking for some solutions now)07:15
*** manous has quit IRC07:20
*** klrmn has quit IRC07:21
*** hseipp has joined #openstack-swift07:31
*** dmorita has joined #openstack-swift07:34
*** rcernin has joined #openstack-swift07:34
*** manous has joined #openstack-swift07:37
*** dmorita has quit IRC07:38
*** pcaruana has joined #openstack-swift07:41
*** mvk has quit IRC07:51
*** Raymii has quit IRC07:53
*** Raymii has joined #openstack-swift07:59
*** zackmdavis has quit IRC08:00
*** zackmdavis has joined #openstack-swift08:00
*** tdasilva has quit IRC08:08
*** asettle has joined #openstack-swift08:08
*** asettle has quit IRC08:13
*** manous has quit IRC08:16
*** Jeffrey4l has quit IRC08:17
*** Jeffrey4l has joined #openstack-swift08:17
*** zul has quit IRC08:18
*** mvk has joined #openstack-swift08:26
*** sileht has joined #openstack-swift08:26
*** amoralej|off is now known as amoralej08:29
*** mingyu has quit IRC08:31
*** zul has joined #openstack-swift08:32
*** tdasilva has joined #openstack-swift08:43
*** dmorita has joined #openstack-swift08:46
*** sorrison_laptop has joined #openstack-swift08:51
*** dmorita has quit IRC08:51
*** daemontool_ has quit IRC08:59
*** sorrison_laptop is now known as sorrison_09:00
*** Worry has joined #openstack-swift09:02
*** asettle has joined #openstack-swift09:09
*** asettle has quit IRC09:14
*** cbartz has joined #openstack-swift09:20
*** oshritf has joined #openstack-swift09:20
*** jordanP has joined #openstack-swift09:22
*** oshritf has quit IRC09:23
*** oshritf has joined #openstack-swift09:27
*** oshritf has quit IRC09:37
*** NeverWorry has joined #openstack-swift09:42
*** Worry has quit IRC09:44
*** Worry has joined #openstack-swift09:44
*** Worry has quit IRC09:45
*** Worry has joined #openstack-swift09:45
*** NeverWorry has quit IRC09:46
*** asettle has joined #openstack-swift09:52
*** acoles_ is now known as acoles09:53
*** mvk has quit IRC09:54
*** Worry has quit IRC09:54
*** Worry has joined #openstack-swift09:55
*** mvk has joined #openstack-swift09:55
*** tqtran has joined #openstack-swift10:01
*** Worry has quit IRC10:02
*** tqtran has quit IRC10:05
*** abalfour has quit IRC10:09
*** abalfour has joined #openstack-swift10:09
*** sorrison_ has quit IRC10:09
*** DuncanT has quit IRC10:33
*** DuncanT has joined #openstack-swift10:33
*** david-lyle_ has joined #openstack-swift10:35
*** david-lyle has quit IRC10:37
*** mgagne has joined #openstack-swift10:47
*** mgagne is now known as Guest261510:47
*** npf has joined #openstack-swift10:49
*** delewis has quit IRC10:56
*** delewis has joined #openstack-swift10:58
*** EmilienM has left #openstack-swift11:20
*** SkyRocknRoll has quit IRC11:29
*** openstackgerrit has quit IRC11:32
*** cdelatte has joined #openstack-swift11:34
*** ppai has quit IRC11:37
*** arch-nemesis has joined #openstack-swift11:42
*** SkyRocknRoll has joined #openstack-swift11:44
*** arch-nemesis has quit IRC11:49
*** ppai has joined #openstack-swift11:53
*** catintheroof has joined #openstack-swift11:55
*** arch-nemesis has joined #openstack-swift12:02
*** tqtran has joined #openstack-swift12:02
*** dmorita has joined #openstack-swift12:03
*** tqtran has quit IRC12:07
*** dmorita has quit IRC12:07
*** vint_bra has joined #openstack-swift12:24
*** catintheroof has quit IRC12:32
*** catintheroof has joined #openstack-swift12:32
*** catintheroof has quit IRC12:37
*** asettle has quit IRC12:37
*** asettle has joined #openstack-swift12:38
*** catintheroof has joined #openstack-swift12:54
*** amoralej is now known as amoralej|lunch13:08
*** catinthe_ has joined #openstack-swift13:14
*** catintheroof has quit IRC13:16
*** tuan_luong has joined #openstack-swift13:36
*** rcernin has quit IRC13:49
*** SkyRocknRoll has quit IRC13:50
*** rcernin has joined #openstack-swift13:52
*** openstackgerrit has joined #openstack-swift13:52
openstackgerritMerged openstack/pyeclib: ISA-L Cauchy support  https://review.openstack.org/39327613:52
*** Guest2615 is now known as mgagne13:54
*** mgagne has quit IRC13:54
*** mgagne has joined #openstack-swift13:54
*** klamath has joined #openstack-swift13:54
*** bkopilov has quit IRC13:59
*** links has quit IRC13:59
*** bkopilov has joined #openstack-swift13:59
*** catintheroof has joined #openstack-swift14:02
*** manous has joined #openstack-swift14:02
*** tqtran has joined #openstack-swift14:04
*** catinthe_ has quit IRC14:04
*** amoralej|lunch is now known as amoralej14:07
*** tqtran has quit IRC14:09
*** hseipp has quit IRC14:12
*** SkyRocknRoll has joined #openstack-swift14:14
*** SkyRocknRoll has quit IRC14:29
*** Jeffrey4l has quit IRC14:30
*** Jeffrey4l has joined #openstack-swift14:31
*** tuan_luong has quit IRC14:38
*** SkyRocknRoll has joined #openstack-swift14:42
*** StraubTW has joined #openstack-swift14:44
*** tongli has joined #openstack-swift14:53
*** links has joined #openstack-swift15:03
*** chsc has joined #openstack-swift15:08
*** sams-gleb has joined #openstack-swift15:12
*** ppai has quit IRC15:23
*** daemontool has joined #openstack-swift15:23
*** catintheroof has quit IRC15:26
*** catintheroof has joined #openstack-swift15:26
*** Jeffrey4l has quit IRC15:27
*** Jeffrey4l has joined #openstack-swift15:27
*** hseipp has joined #openstack-swift15:28
*** catintheroof has quit IRC15:31
*** dmorita has joined #openstack-swift15:38
*** dmorita has quit IRC15:43
*** isotope has joined #openstack-swift15:45
*** klrmn has joined #openstack-swift15:49
*** hogepodge has joined #openstack-swift15:50
*** manous has quit IRC15:52
*** isotope has quit IRC15:55
*** catintheroof has joined #openstack-swift15:57
*** chlong has joined #openstack-swift15:58
*** tongli has quit IRC16:03
*** links has quit IRC16:13
*** SkyRocknRoll has quit IRC16:14
*** hseipp has quit IRC16:14
openstackgerritSachin Patil proposed openstack/swift: Fix swift-get-nodes arg parsing for missing ring  https://review.openstack.org/40601216:17
*** pcaruana has quit IRC16:18
*** rcernin has quit IRC16:19
*** psachin has quit IRC16:26
*** mvk has quit IRC16:31
*** _JZ_ has joined #openstack-swift16:33
*** xionchen_ has joined #openstack-swift16:34
*** MooingLemur has quit IRC16:47
*** chlong has quit IRC16:56
*** asettle has quit IRC17:07
*** rcernin has joined #openstack-swift17:11
*** klrmn has quit IRC17:16
*** dmorita has joined #openstack-swift17:21
*** mvk has joined #openstack-swift17:24
*** MooingLemur has joined #openstack-swift17:29
*** chlong has joined #openstack-swift17:30
*** dmorita has quit IRC17:32
*** dmorita has joined #openstack-swift17:32
*** calebb has quit IRC17:34
*** daemontool has quit IRC17:35
*** dmorita has quit IRC17:37
claygzomg zomg zomg did isa-l-rs-cauchy land!  ?!!! wooooo!17:45
claygtdasilva: notmyname: kota: RE-LEASE RE-LEASE RE-LEASE17:46
claygI think last time we did a liberasure/pyeclib release I lied and said something stupid like "next time you guys should teach me how to do it" - i'm such a liar17:47
tdasilvalol17:48
tdasilvaclayg: we want to release both right?17:48
tdasilvait will be coincidentally 1.3.2 of each17:48
claygzomg zomg zomg - i want them so much - i could hug you!17:49
clayglast time with pyeclib i remeber it took two tries for some reason?  so... i don't if it would be useful to look at the history of those releases and double check all the p's and q's - i have it on good authority that software is hard17:50
patchbothard17:50
timburkei feel like https://review.openstack.org/#/c/405926/ might be nice to get in17:50
patchbotpatch 405926 - liberasurecode - Add get_version functionality to liberasurecode lib17:50
claygtimburke: that's the ticket!17:50
claygtimburke: how do i sanity check it - do you have a comment in your review like "when I fobrinate the wizbang - boom"17:51
timburkelike, what's the point of using a dynamically loaded library if you can't actually know which version you're loading?17:51
claygi feel like it would maybe be something like build something and then look at some attirbute on a module somewhere before and after?17:52
zaitcevI did not know "rescure" was a word17:52
timburkei could whip something up... hold on...17:52
zaitcevwait, it's a typo17:52
claygoh... hrmm... this is just the liberasurecode side - so it'd be a little 'test.c' thingy - is there a pyeclib side that goes with it!?17:52
claygcause I will totally review end-to-end17:53
timburkezaitcev: or is it ... "what supertoast would do"? what?? http://www.urbandictionary.com/define.php?term=rescure17:53
tdasilvaclayg, timburke: while you look at patch 405926 i will start loading release procedures into my brain again and get ready to release when you guys say go17:54
patchbothttps://review.openstack.org/#/c/405926/ - liberasurecode - Add get_version functionality to liberasurecode lib17:54
claygtdasilva: i don't think it matters that much if the released pyeclib isn't using it17:54
claygmaybe we should *just* have liberasurecode - which is obstentially a c project that provides .so's but that *also* happens to have just a *tad* bith of python in it to provide python bindings and may some day have bindings for another compiled language - can't think off the top of my head which language that might be...17:55
clayg*ostensibly - i can't even *say* that word - much less spell it17:56
timburkeclayg: https://review.openstack.org/#/c/405929/ iirc17:56
patchbotpatch 405929 - pyeclib - WIP: Change the version reference17:56
claygtimburke: that's the ticket!17:56
* clayg votes timburke for google in the next election for "keeper of all worlds knowledge"17:56
*** dmorita has joined #openstack-swift17:57
claygtimburke: except patch 405929 can't merge because it tests a new function on liberasurecode master17:58
patchbothttps://review.openstack.org/#/c/405929/ - pyeclib - WIP: Change the version reference17:58
claygthe gate unittest failure is legit :\17:58
claygtimburke: so we're back to your make tox install liberasurecode from source .. and also probably update the gate .. and also probably break everyone's dev environments if we try to *use* the new pyeclib function from swift17:59
claygtimburke: I don't think we *should* try to get this in right now?  it doesn't fix any critical production issues for me and it seems... hard?18:00
*** calebb has joined #openstack-swift18:00
claygtimburke: also instead of compiling from source I'm going for drop pyeclib and move python bindings for liberasurecode into the liberasurecode project and change swift to depend on python-liberasurecode instead of pyeclib18:00
timburkeyeah, and the more i think about it, the more i realize that we probably need to add a #define GET_VERSION_AVAILABLE or something on the liberasure side so we actually know whether that functions is defined. maybe better to hold off18:00
tdasilvatimburke, clayg: at some point i was thinking about creating a new gate job for pyeclib where we would build liberasurecode from source, is that what you did with your tox change?18:00
clayg... until someone tells me that's a bad idea18:01
claygand i'm not worrying about it today18:01
timburketdasilva: yup. playing with getting isa-l installed, too18:01
tdasilvaah ok18:01
tdasilvacool!18:01
claygtdasilva: he created the tox job - not the gate job - but that should make the gate job easier (!?)18:01
*** david-lyle_ is now known as david-lyle18:02
claygtimburke: tdasilva: no whey - get on board with this fork pyeclib into a subdirectory of liberasure code train - it's going to be the bomb!18:02
tdasilvapyeclib is just a python binding of liberasurecode, right?18:03
tdasilvawhy not?18:03
claygyeah it makes sense to me - and also when we have golang bindings for liberasurecode - as if we're going to be able to keep *three* repos stright?18:03
claygadd a thing to liberasurecode c - then expose it in python and golang bindings - all three unittests pass - merge it, build it, test it, ship it, $$$18:04
tdasilvaclayg: if we ever wanted to make use of another ec library like https://memoscale.com/18:04
tdasilvathat would be plugged in to liberasurecode or pyeclib?18:04
tdasilvaactually NTT also has a ec library right?18:05
timburkeshss18:05
tdasilvayeah, that18:05
claygfor us - and the way we deal with ec - i'm guessing they'd plug into liberasurecode - and then we could expose it in the python or golang bindings at the same time18:06
claygall in one repo18:06
tdasilvaclayg: sounds like a good idea to me18:06
claygwe couldn't consume it directly from swift - even if they have python and golang bindings - unless we made a shim to match the liberasurecode/pyeclib interface18:06
claygbut yeah - just like isa-l or ssh - it'd be easy to have liberasurecode be the shim that wraps up the other ec library to a common interface and then exposed in all the language bindings that liberasurecode supports18:07
claygI think someone thought if liberasurecode grew lots of bindings for different languages - it would be easier if they were seperate repos - but I think ... i'm pretty sure ... based on experience and hindsight - this is not easy18:08
claygit's possible the other way may *also* be not easy18:08
clayggrass is greener kind of problem18:08
*** cbartz has quit IRC18:09
claygtdasilva: but for now - let's cut the release - and maybe we can discuss more with kota and think about making a move in Jan?18:10
*** cnf has quit IRC18:10
claygtdasilva: timburke and me think maybe the version thing might need a few more revs18:10
*** calebb has quit IRC18:11
*** klrmn has joined #openstack-swift18:11
*** cnf has joined #openstack-swift18:11
tdasilvaclayg: ok, sounds good18:12
*** isotope has joined #openstack-swift18:18
*** asettle has joined #openstack-swift18:21
*** amoralej is now known as amoralej|off18:25
*** rickflare has quit IRC18:26
*** rickflare has joined #openstack-swift18:31
tdasilvaclayg, timburke: libec adding support for isa-l cauchy, makes it a 1.3.*2* release or a 1.*4*.0 release?18:32
tdasilvathoughts?18:32
timburkei'm partial towards 1.4.018:34
*** chlong has quit IRC18:38
openstackgerritThiago da Silva proposed openstack/liberasurecode: 1.4.0 release  https://review.openstack.org/40825718:40
tdasilvatimburke: me too :)18:40
tdasilvaalthough i changed my mind after creating branch name18:40
*** mcarden has quit IRC18:40
tdasilvaand forgot to rename it18:40
*** mcarden has joined #openstack-swift18:41
*** clu_ has joined #openstack-swift18:45
openstackgerritTim Burke proposed openstack/pyeclib: Add tox environment to test against liberasurecode master  https://review.openstack.org/40782918:48
openstackgerritTim Burke proposed openstack/pyeclib: Install isa-l from source when testing liberasure-master  https://review.openstack.org/40825918:48
openstackgerritThiago da Silva proposed openstack/pyeclib: renaming README.md to README.rst  https://review.openstack.org/40826018:50
tdasilvatimburke: i'd like to get that last patch 408260 in before release18:51
patchbothttps://review.openstack.org/#/c/408260/ - pyeclib - renaming README.md to README.rst18:51
openstackgerritTim Burke proposed openstack/pyeclib: Add tests for the availability of individual backends  https://review.openstack.org/40826118:51
tdasilvapypi doesn't support markdown, so nothing gets displayed here https://pypi.python.org/pypi/PyECLib/1.3.118:51
timburkelooking now. was it really already rst? no conversion necessary?18:52
*** jordanP has quit IRC18:52
tdasilvatimburke: i only quickly checked with an atom preview package i have and it looked ok18:55
*** krypto has joined #openstack-swift18:55
*** chlong has joined #openstack-swift18:57
tdasilvatimburke: http://rst.ninjs.org/?n=33b8954acf3b589b07ac528062b6b490&theme=basic18:57
kryptohello all i have one vm as swift server with 1 Tb disk in the environment and planning to add 5 new physical servers each with 6x1 TBdisks and then slowly remove the vm from the environment.Is this possible ie a cluster with un even number of disks and then removing node from that cluste18:59
*** dmorita has quit IRC19:04
*** dmorita has joined #openstack-swift19:05
tdasilvakrypto: initially that plan looks ok IMO, but in your one vm with one disk do you have only a single replica ring?19:14
kryptotdasilva if i may i ask ,how can i check the rings .. honetsly its deployed by somebody else19:18
tdasilvakrypto: do you have access to your vm? like ssh access?19:19
timburkeclayg: tdasilva: kota_: oh man, metaclasses are great! look at that: http://logs.openstack.org/61/408261/1/check/gate-pyeclib-python27-ubuntu-xenial/ac635f4/console.html#_2016-12-07_18_53_37_979203 beautiful! somebody should go merge https://review.openstack.org/#/c/408261/ like right now ;-)19:19
patchbotpatch 408261 - pyeclib - Add tests for the availability of individual backends19:19
kryptotdasilva yes i do19:19
*** silor has quit IRC19:20
*** asettle has quit IRC19:21
*** tqtran has joined #openstack-swift19:26
tdasilvakrypto: the rings are in /etc/swift, you can use the swift-ring-builder tool to check the rings19:28
tdasilvakrypto: it's probably a good idea to checkout the docs here: http://docs.openstack.org/developer/swift/overview_ring.html http://docs.openstack.org/developer/swift/admin_guide.html19:28
tdasilvathere's a ton of detail about the rings19:29
kryptothanks i will check those19:31
*** AbyssOne has quit IRC19:33
tdasilvatimburke: thanks for the review on the README, what i'm going to do is to abandon that change and make all the changes in one "release" commit19:33
tdasilvathat will contain the changelog and version updates19:33
tdasilvathat ok with you?19:33
timburketdasilva: sure19:34
tdasilvatimburke: ok, thanks, sorry for the noise19:34
*** acoles is now known as acoles_19:35
*** AbyssOne has joined #openstack-swift19:36
*** chlong has quit IRC19:36
*** npf has quit IRC19:37
*** npf has joined #openstack-swift19:37
tdasilvatimburke: https://review.openstack.org/#/c/408260/1/README.rst@40 do you mean liberasurecode_rs_cauchy or isa_l_rs_cauchy ?19:41
patchbotpatch 408260 - pyeclib - renaming README.md to README.rst19:41
*** ChubYann has joined #openstack-swift19:41
timburketdasilva: bah, i meant isa_l_rs_cauchy -- you're right19:42
*** StraubTW has quit IRC19:45
*** StraubTW has joined #openstack-swift19:46
*** npf has quit IRC19:52
*** calebb has joined #openstack-swift19:58
*** chlong has joined #openstack-swift20:00
openstackgerritThiago da Silva proposed openstack/pyeclib: release 1.4.0  https://review.openstack.org/40828020:02
timburketdasilva: on footnotes, i mean things that link to the bit at the bottom, like in http://docutils.sourceforge.net/docs/user/rst/quickref.html#footnotes20:02
tdasilvatimburke: oh yeah, i understand now20:04
*** xionchen_ has quit IRC20:04
tdasilvatimburke: i think that whole README needs a bit of reformatting, it looks quite different from a typical README one finds on github. could we leave that for after this release?20:05
timburketdasilva: sure. i think as long as we add the new feature, i'm happy20:05
tdasilva:)20:06
tdasilvakota_, clayg, timburke, notmyname : patch  408257 and patch 408280 are ready for review, we should be able to tag the release after these two merge20:09
patchbothttps://review.openstack.org/#/c/408257/ - liberasurecode - 1.4.0 release20:09
patchbothttps://review.openstack.org/#/c/408280/ - pyeclib - release 1.4.020:09
*** dmorita has quit IRC20:14
*** dmorita has joined #openstack-swift20:16
openstackgerritTim Burke proposed openstack/pyeclib: Install jerasure from source when testing liberasure-master  https://review.openstack.org/40828620:16
timburkethat's the stuff. https://review.openstack.org/#/c/408286/ (and the two parent commits in its chain) + https://review.openstack.org/#/c/408261/ = win20:18
patchbotpatch 408286 - pyeclib - Install jerasure from source when testing liberasu...20:18
patchbotpatch 408261 - pyeclib - Add tests for the availability of individual backends20:18
timburkekota_: if only i knew how to install shss :-(20:18
*** dmorita has quit IRC20:18
*** dmorita has joined #openstack-swift20:20
*** isotope has quit IRC20:23
*** asettle has joined #openstack-swift20:25
*** isotope has joined #openstack-swift20:26
timburketdasilva: do we want to fix one more memory leak ahead of the pyeclib release? https://review.openstack.org/#/c/407302/20:29
patchbotpatch 407302 - pyeclib - Fix checksum memory leak20:29
*** asettle has quit IRC20:30
*** arch-nemesis has quit IRC20:30
*** diogogmt has joined #openstack-swift20:43
*** m_kazuhiro has joined #openstack-swift20:47
*** dmorita has quit IRC20:52
*** dmorita has joined #openstack-swift20:52
*** isotope has quit IRC20:52
*** tqtran is now known as tqtran-afk20:59
tdasilvaoh, no meeting today, right?21:03
mathiasbtdasilva: that's what I just remembered as well21:03
*** hseipp has joined #openstack-swift21:04
dmoritaActually, I checked last week's log, then notmyname said no meeting today.21:08
*** asettle has joined #openstack-swift21:09
*** asettle has quit IRC21:09
*** asettle has joined #openstack-swift21:10
*** vint_bra has quit IRC21:10
*** asettle has quit IRC21:12
*** asettle has joined #openstack-swift21:12
*** manous has joined #openstack-swift21:13
tdasilvatimburke: it seems like a good idea to hold up for that memory leak fix. what does clayg think?21:18
timburketdasilva: we'll find out :-) i think he's on his way in currently21:18
timburkeon the plus side, it's a fairly small memory leak; we were having trouble detecting it with the 1000x iterations21:19
*** m_kazuhiro has quit IRC21:20
*** a1|away has quit IRC21:21
MooingLemurHas there been any movement on the ability to change storage policies of existing containers (background migration)?21:21
tdasilvaMooingLemur: i think that's something that dmorita was working on....but i'm not sure about the current status21:23
dmoritaMooingLemur: yeah, I am working on it. I am now rewriting patch to work this feature with fastPost. Concept is done. But some unit tests do not pass. Therefore, I am now checking some regressions.21:25
dmoritaMooingLemur: I think I can update my working patch by next week.21:26
MooingLemurThat's great news :)  Thanks for all your efforts.  I have some... mistakes... to correct :)  I have a ring with too high a part power I'd like to migrate away from.21:27
*** sams-gleb has quit IRC21:32
*** a1|away has joined #openstack-swift21:32
*** sams-gleb has joined #openstack-swift21:33
*** Jeffrey4l has quit IRC21:35
*** Jeffrey4l has joined #openstack-swift21:35
*** catintheroof has quit IRC21:37
*** sams-gleb has quit IRC21:37
claygtimburke: where is it - let me at it121:38
*** catintheroof has joined #openstack-swift21:38
timburkeclayg: https://review.openstack.org/#/c/407302/21:38
patchbotpatch 407302 - pyeclib - Fix checksum memory leak21:38
tdasilvatimburke: fwiw i had to increate to 4M to see the failure21:39
clayglol @ loop 400K21:39
patchbothttps://review.openstack.org/#/c/400/21:39
claygwtf patchbot - you're drunk21:39
timburkei think it's the p. like p 40730221:39
patchbothttps://review.openstack.org/#/c/407302/ - pyeclib - Fix checksum memory leak21:39
timburkeyeah...21:39
tdasilvatimburke: well...4M was the first line i tried after the failure didn't happen at 400k21:40
tdasilvas/first line/first number21:40
timburketdasilva: hrm. we can bump it higher... but if we start getting test times like swift's, maybe we're better off with the subtle memory leaks...21:41
claygrofl21:42
tdasilvahehehe21:42
*** catintheroof has quit IRC21:42
tdasilvaactually, error that i see is with something else, but I did notice that looping 400K times without the actual fix did not show any errors for me21:43
claygtdasilva: you have to `sudo pip install -e .` when you change the c code to make with the rebuild or something21:45
claygit def pops for me @ 400K21:45
clayg... but not at 1K - today 2-3K seems to the ticket21:46
tdasilvaclayg: i was running `sudo python setup.py develop` ? is that the same?21:46
timburkeclayg: but you and i are probably using rather similar vsaio environments. i wonder what kota_ typically uses...21:47
claygtdasilva: they *are* the same - unless your setup.py uses pbr - then your command is a good way to traceback while newish pip will mostly work21:47
claygtdasilva: so you should replace `pip install -e .` in your brain everytime you normally type `python setup.py develop`21:48
*** abalfour has left #openstack-swift21:48
clayg`pip install .` is good for `python setup.py install`21:48
claygif you need `pyton setup.py sdist` you're screwed - because no one cares about distro packagers anymore21:48
*** jamielennox|away is now known as jamielennox21:49
claygok, that's going in21:49
tdasilvaclayg: ok, thanks for the heads up21:50
timburkeclayg: did you try it for each of test/, test/test_pyeclib_api.py, test/test_pyeclib_api.py:TestPyECLibDriver, and test/test_pyeclib_api.py:TestPyECLibDriver.test_get_metadata_memory_usage ? the test ought to fail regardless of scope; i saw some numbers that would fail for all of tests/ but not test/test_pyeclib_api.py:TestPyECLibDriver, say21:50
openstackgerritMerged openstack/pyeclib: Fix checksum memory leak  https://review.openstack.org/40730221:51
claygtimburke: i was mostly focused on just test/test_pyeclib_api.py:TestPyECLibDriver.test_get_metadata_memory_usage to ensure it was stable in isolation - i'm happy enough that it's not going to be horribly problematic - "definately better; probably good enough"21:53
*** hseipp has quit IRC21:57
mattoliverauMorning21:57
tdasilvamattoliverau: o/21:59
*** asettle has quit IRC22:10
*** cnf has quit IRC22:10
timburkeclayg: the 40M iterations thing -- was it a MemoryError on loop_range = range(40000000)?22:11
claygtimburke: ain't nobody got time for that!  i didn't test it22:12
*** cnf has joined #openstack-swift22:12
*** tqtran-afk is now known as tqtran22:13
timburkeoh, it was just speculation; got it22:13
*** clu_ has quit IRC22:13
claygah, yeah - sorry - i see now that could be read either way - although if it failed for me a 40M i probably (heh, there it is again) would have said "this failed for me when I tried it at 40M, but it's NBD IMHO"22:15
*** clu_ has joined #openstack-swift22:15
claygok timburke are you done fixing all the bugs now?  can tdasilva make a tarball for me!?  :D22:16
* clayg imagines timburke saying "well *acctually* there is this one patch that fixes this other critical bug that's been open for months..."22:16
timburkeclayg: i think? the other stuff i've got out there is mainly test infrastructure22:16
pdardeautimburke: at first i read that ^ as 'manly' test infrastructure. now i gotta know - is it manly? ;-)22:18
timburkeclayg: 400M seems to check out fine, so i'm calling it good?22:19
claygtimburke: definately better - probably good enough22:21
timburkepdardeau: well, the chain ending in https://review.openstack.org/#/c/408286/ is a little ugly & brutish, so maybe? i liked writing https://review.openstack.org/#/c/408261/ more22:21
patchbotpatch 408286 - pyeclib - Install jerasure from source when testing liberasu...22:21
patchbotpatch 408261 - pyeclib - Add tests for the availability of individual backends22:21
clayginstall jerasure from source!  like pirates!?22:22
timburkeclayg: idk? there seems to be a public git server willing to send me files? *i'm* not doing any distributing, at any rate22:23
clayghehhehe22:25
timburkeclayg: http://jerasure.org/jerasure/jerasure/blob/master/License.txt looks like 3-clause BSD22:25
timburkeas does http://jerasure.org/jerasure/gf-complete/blob/master/License.txt22:29
timburkeooooh, *patent* issues. yeah, i got nothing there. seems like the isa-l stuff might be of more interest anyway? w/e, it's the last in the chain, we can drop it22:32
*** dmorita has quit IRC22:33
*** dmorita has joined #openstack-swift22:33
claygtimburke: umm... IANAL - so I don't really speak on those things - i have no idea about any of it really - i just know that I don't use jerasure and don't pay much attention to it22:36
*** thurloat has quit IRC22:39
claygtimburke: umm... IANAL - so I don't really speak on those things - i have no idea about any of it really - i just know that I don't use jerasure and don't pay much attention to it22:45
clayggah!22:45
*** rcernin has quit IRC22:48
*** thurloat has joined #openstack-swift22:50
*** cnf has quit IRC23:01
*** cnf has joined #openstack-swift23:04
*** StraubTW has quit IRC23:06
*** chlong has quit IRC23:06
*** dmorita has quit IRC23:07
*** dmorita has joined #openstack-swift23:08
*** asettle has joined #openstack-swift23:11
*** asettle has quit IRC23:16
*** jamielennox is now known as jamielennox|away23:27
*** jamielennox|away is now known as jamielennox23:28
claygtdasilva: there's probalby some code that has to merge before a tag show up on at https://github.com/openstack/liberasurecode/releases right?23:36
*** klamath has quit IRC23:53
timburkeoh, clayg, here's that demo code for get_version -- http://paste.openstack.org/show/591738/23:56
timburkei installed liberasurecode to some junk user dir like tmp, then checked that it worked correctly with something like `gcc test.c -L tmp/lib -I tmp/include -l erasurecode -l dl -Xlinker -rpath -Xlinker $(pwd)/tmp/lib && ./a.out`23:57
timburkeafter that, i played around with switching between before/after kota_'s change or changing the numbers in erasurecode_version.h then running `make && make install && ./a.out`, and even blew away all of tmp/lib and running it again23:57
timburkei stand by my +223:57

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