openstackgerrit | Tim Burke proposed openstack/pyeclib master: Try harder to find liberasurecode https://review.openstack.org/636501 | 00:09 |
---|---|---|
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Try harder to find liberasurecode https://review.openstack.org/636501 | 00:16 |
*** mikecmpbll has quit IRC | 00:24 | |
*** openstackgerrit has quit IRC | 00:52 | |
*** gyee has quit IRC | 01:07 | |
*** gyee has joined #openstack-swift | 01:10 | |
zaitcev | It turned out, liberasurecode has its own problem on s390x: | 01:23 |
zaitcev | BUILDSTDERR: 14 - test_metadata_crcs_le: UNKNOWN (idx=0) ... lt-liberasurecode_test: liberasurecode_test.c:1779: test_metadata_crcs_le: Assertion `liberasurecode_get_fragment_metadata(header, &res) == 0' failed. | 01:23 |
*** gyee has quit IRC | 01:47 | |
*** spsurya has joined #openstack-swift | 03:55 | |
*** openstackgerrit has joined #openstack-swift | 05:06 | |
openstackgerrit | Merged openstack/swift master: Leave less garbage in /var/tmp https://review.openstack.org/636262 | 05:06 |
*** psachin has joined #openstack-swift | 05:49 | |
openstackgerrit | Pete Zaitcev proposed openstack/liberasurecode master: Make our alt crc32 more portable https://review.openstack.org/636556 | 06:45 |
openstackgerrit | Pete Zaitcev proposed openstack/liberasurecode master: Remove get_chksum to hush warnings https://review.openstack.org/636558 | 06:48 |
*** gkadam has joined #openstack-swift | 07:00 | |
*** e0ne has joined #openstack-swift | 07:36 | |
*** ccamacho has joined #openstack-swift | 07:48 | |
*** tkajinam has quit IRC | 08:36 | |
*** early has quit IRC | 08:54 | |
*** early has joined #openstack-swift | 08:56 | |
*** early has quit IRC | 09:06 | |
*** e0ne has quit IRC | 09:06 | |
*** e0ne has joined #openstack-swift | 09:07 | |
*** mikecmpbll has joined #openstack-swift | 09:12 | |
*** early has joined #openstack-swift | 09:15 | |
*** hseipp has joined #openstack-swift | 09:21 | |
openstackgerrit | Nguyen Hai proposed openstack/swift master: fix tox python3 overrides https://review.openstack.org/572983 | 09:30 |
openstackgerrit | Kota Tsuyuzaki proposed openstack/swift feature/losf: WIP - Abstract file and directory operations in replicator/reconstructor https://review.openstack.org/636577 | 09:32 |
openstackgerrit | Kota Tsuyuzaki proposed openstack/swift feature/losf: Lots Of Small Files - alternate diskfile https://review.openstack.org/636578 | 09:32 |
kota_ | alecuyer, rledisez: pushed the two changes in alecuyer's repo into feature/losf branch with rebased. | 09:33 |
*** e0ne has quit IRC | 09:34 | |
*** hseipp has quit IRC | 09:40 | |
*** hseipp has joined #openstack-swift | 09:43 | |
rledisez | kota_: thx a lot. starting tomorrow i'll be off for one week. but after that, we need to take some time so I can start working on the feature branch with you | 09:43 |
rledisez | alecuyer: is off this week, he'll be back at work on Monday | 09:43 |
kota_ | rledisez: no worries, I had a week off in the last week ;) then, starting to review the design docs in this week. | 09:44 |
kota_ | I'm thinking that would be nice to have an extra gate job running functests with losf that helps us to know any regressions even if we decided design changes. | 09:45 |
kota_ | ok, we got a lot of failures with sort of tox, pep8, functests in those tests. Let's fix them to stand at the start line. | 10:13 |
*** e0ne has joined #openstack-swift | 10:46 | |
*** e0ne_ has joined #openstack-swift | 10:50 | |
*** e0ne has quit IRC | 10:52 | |
openstackgerrit | Kota Tsuyuzaki proposed openstack/swift feature/losf: WIP - Abstract file and directory operations in replicator/reconstructor https://review.openstack.org/636577 | 11:17 |
kota_ | anyway, fix the failures step by step. just fix pep8 failures. I still had a bunch of unit test failures in my local env. | 11:19 |
openstackgerrit | Alexandra Settle proposed openstack/swift stable/rocky: Fixing target for cross-ref 'get_data_dir' https://review.openstack.org/636595 | 11:21 |
*** asettle has joined #openstack-swift | 11:23 | |
*** ccamacho has quit IRC | 11:39 | |
*** e0ne has joined #openstack-swift | 11:56 | |
*** e0ne has quit IRC | 11:57 | |
*** e0ne_ has quit IRC | 11:57 | |
*** e0ne has joined #openstack-swift | 12:08 | |
*** ccamacho has joined #openstack-swift | 12:09 | |
*** e0ne has quit IRC | 12:12 | |
*** e0ne has joined #openstack-swift | 12:22 | |
*** e0ne has quit IRC | 13:11 | |
*** e0ne has joined #openstack-swift | 13:17 | |
*** e0ne_ has joined #openstack-swift | 13:20 | |
*** e0ne has quit IRC | 13:23 | |
*** e0ne_ has quit IRC | 13:40 | |
*** e0ne has joined #openstack-swift | 13:45 | |
*** e0ne_ has joined #openstack-swift | 13:50 | |
*** e0ne has quit IRC | 13:51 | |
*** hseipp has quit IRC | 13:54 | |
*** e0ne_ has quit IRC | 14:04 | |
*** mikecmpbll has quit IRC | 14:05 | |
*** mikecmpbll has joined #openstack-swift | 14:07 | |
*** e0ne has joined #openstack-swift | 14:29 | |
*** e0ne_ has joined #openstack-swift | 14:35 | |
*** e0ne has quit IRC | 14:35 | |
*** e0ne_ has quit IRC | 14:41 | |
*** e0ne has joined #openstack-swift | 14:50 | |
*** e0ne_ has joined #openstack-swift | 14:55 | |
*** e0ne has quit IRC | 14:55 | |
*** hseipp has joined #openstack-swift | 15:40 | |
*** supamatt has joined #openstack-swift | 15:47 | |
*** supamatt has quit IRC | 15:49 | |
*** supamatt has joined #openstack-swift | 15:49 | |
*** gkadam has quit IRC | 15:53 | |
*** ianychoi has quit IRC | 16:08 | |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Try harder to find liberasurecode https://review.openstack.org/636501 | 16:46 |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Fix gate https://review.openstack.org/636668 | 16:47 |
timburke | :-( still no release that includes https://github.com/pypa/twine/commit/125b772 ? | 16:47 |
*** e0ne has joined #openstack-swift | 16:47 | |
*** e0ne_ has quit IRC | 16:48 | |
*** gyee has joined #openstack-swift | 16:55 | |
*** itlinux has joined #openstack-swift | 17:05 | |
*** itlinux has quit IRC | 17:09 | |
*** itlinux_ has joined #openstack-swift | 17:09 | |
*** itlinux_ has quit IRC | 17:14 | |
openstackgerrit | Tim Burke proposed openstack/swift master: Make get_data/async/tmp_dir explicit https://review.openstack.org/636684 | 17:14 |
openstackgerrit | Tim Burke proposed openstack/swift master: Fixing target for cross-ref 'get_data_dir' https://review.openstack.org/636685 | 17:14 |
*** itlinux has joined #openstack-swift | 17:14 | |
*** psachin has quit IRC | 17:18 | |
*** hseipp has quit IRC | 17:21 | |
*** e0ne has quit IRC | 17:25 | |
*** itlinux has quit IRC | 17:30 | |
*** mikecmpbll has quit IRC | 17:30 | |
*** itlinux has joined #openstack-swift | 17:35 | |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Fix gate https://review.openstack.org/636668 | 17:37 |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Try harder to find liberasurecode https://review.openstack.org/636501 | 17:37 |
*** itlinux_ has joined #openstack-swift | 17:39 | |
*** itlinux has quit IRC | 17:41 | |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Fix gate https://review.openstack.org/636668 | 17:43 |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Try harder to find liberasurecode https://review.openstack.org/636501 | 17:43 |
zaitcev | Everyone runs "tox -e pep8" locally before "git review", right? RIGHT? | 17:45 |
openstackgerrit | Tim Burke proposed openstack/swift master: Make get_data/async/tmp_dir explicit https://review.openstack.org/636684 | 17:48 |
openstackgerrit | Tim Burke proposed openstack/swift master: Fixing target for cross-ref 'get_data_dir' https://review.openstack.org/636685 | 17:48 |
timburke | <_< | 17:48 |
timburke | >_> | 17:48 |
* timburke slinks away sheepily | 17:49 | |
zaitcev | Well, I missed the TMPDIR_BASE so we're even | 17:55 |
openstackgerrit | Merged openstack/pyeclib master: Fix gate https://review.openstack.org/636668 | 17:56 |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Move project-config job definitions in-tree https://review.openstack.org/636698 | 18:00 |
timburke | zaitcev: again, though -- `tox -e pep8` would've caught it | 18:05 |
timburke | the issue i always have is that running tox for a project whose source is hosted on a vagrant shared filesystem *sucks* -- takes several minutes to *setup*, not even run anything... | 18:08 |
*** itlinux_ has quit IRC | 18:08 | |
zaitcev | Hmm. I just have a directory on the laptop, but I guess in the age of iPad everything you need a VM or VPS somewhere to do anything sensible. | 18:10 |
zaitcev | timburke: so, about that py3 "sharder" patch https://review.openstack.org/570320 | 18:16 |
patchbot | patch 570320 - swift - py3: port the container sharder - 14 patch sets | 18:16 |
zaitcev | Matt promised to look it, too, but he's not been seen since then. | 18:16 |
*** e0ne has joined #openstack-swift | 18:22 | |
*** mikecmpbll has joined #openstack-swift | 18:42 | |
*** mikecmpbll has quit IRC | 18:47 | |
openstackgerrit | Merged openstack/swift master: Display crypto data/metadata details in swift-object-info https://review.openstack.org/586903 | 18:53 |
*** vr42 has joined #openstack-swift | 19:41 | |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Move project-config job definitions in-tree https://review.openstack.org/636698 | 19:46 |
*** e0ne has quit IRC | 19:47 | |
timburke | so... are we cool with pyeclib dropping support for libec < 1.4.0? in particular, i want to take advantage of https://github.com/openstack/liberasurecode/commit/09d8bbf in setup.py, instead of trusting the filename of the library... | 20:00 |
timburke | i suppose 1.3.1 at least had p 393595 ... | 20:00 |
patchbot | https://review.openstack.org/#/c/393595/ - liberasurecode - Fix error handling on gf_ivnert_matrix in isa-l ba... (MERGED) - 4 patch sets | 20:00 |
timburke | but you needed 1.4.0 to get p 393263 | 20:01 |
patchbot | https://review.openstack.org/#/c/393263/ - liberasurecode - ISA-L Cauchy support (MERGED) - 8 patch sets | 20:01 |
notmyname | timburke: IMO, gut reaction, yes. supporting >=1.4.0 is fine | 20:02 |
notmyname | that was tagged dec 2016 | 20:03 |
timburke | yeah, we've had a nice long period... | 20:03 |
openstackgerrit | John Dickinson proposed openstack/swift stable/ocata: ocata stable backports https://review.openstack.org/636471 | 20:20 |
*** vr42 has quit IRC | 20:20 | |
openstackgerrit | John Dickinson proposed openstack/swift stable/pike: pike stable backports https://review.openstack.org/636470 | 20:22 |
openstackgerrit | John Dickinson proposed openstack/swift stable/queens: changelog for queens backports https://review.openstack.org/636468 | 20:23 |
openstackgerrit | John Dickinson proposed openstack/swift stable/rocky: rocky stable backports https://review.openstack.org/636469 | 20:24 |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Try harder to find liberasurecode https://review.openstack.org/636501 | 20:27 |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Use liberasurecode_get_version() https://review.openstack.org/636748 | 20:27 |
tdasilva | timburke: for p 636698, wondering if it would make sense to add py36 to the gate as non-voting (instead of experimental) | 20:32 |
patchbot | https://review.openstack.org/#/c/636698/ - pyeclib - Move project-config job definitions in-tree - 2 patch sets | 20:32 |
tdasilva | in fact, maybe all those experimental could be moved | 20:32 |
openstackgerrit | Merged openstack/pyeclib master: Move project-config job definitions in-tree https://review.openstack.org/636698 | 20:36 |
timburke | tdasilva: we should probably have both 35 and 36 in check and gate, and both as voting | 20:39 |
timburke | i wouldn't be opposed to moving all of them though | 20:40 |
tdasilva | timburke: just so that we don't have to keep doing the 'check experimental' piece | 20:40 |
tdasilva | i think they are stable enough | 20:40 |
tdasilva | plust the jobs that run pyeclib against libec master are useful IMO | 20:41 |
timburke | agreed; they pass now, they are unlikely to suddenly *not pass* later -- if they don't, it's probably a legit probelm with the patch | 20:41 |
timburke | notmyname: oh no! http://logs.openstack.org/48/636748/1/check/openstack-tox-py27/a23bfb2/ara-report/result/37b898bb-1321-4a94-a810-3cf420d747de/ says it's testing with 1.1.0 :-( | 20:43 |
notmyname | 1.1.0 is just right out | 20:44 |
timburke | notmyname: well... https://packages.ubuntu.com/xenial/liberasurecode1 | 20:45 |
timburke | bionic's cool -- it's at 1.5.0 | 20:45 |
tdasilva | right | 20:46 |
tdasilva | can we change the job so that it doesn't use xenial? | 20:46 |
timburke | centos is smart and doesn't include our code in the distro repos :P | 20:47 |
timburke | earliest cloud repo seems to be ocata, which is 1.4.0 | 20:47 |
timburke | tdasilva: i could get behind that idea... but i could also fix the heuristic in my change | 20:48 |
timburke | idk that i want to keep people from being able to do dev work on xenial just yet... | 20:50 |
kota_ | morning | 20:58 |
notmyname | just about meeting time in #openstack-meeting | 20:58 |
notmyname | kota_: mattoliverau: timburke: clayg: tdasilva: zaitcev: rledisez: curtesey ping | 20:59 |
clayg | thanks! | 21:00 |
*** itlinux has joined #openstack-swift | 21:04 | |
*** asettle has quit IRC | 21:21 | |
openstackgerrit | Tim Burke proposed openstack/liberasurecode master: Make our alt crc32 more portable https://review.openstack.org/636556 | 21:25 |
clayg | great meeting | 21:39 |
clayg | py3 FTW | 21:39 |
timburke | clayg: https://review.openstack.org/#/c/636501/ was kinda interesting, needing to backport a cpython fix from dec 2016... | 21:41 |
patchbot | patch 636501 - pyeclib - Try harder to find liberasurecode - 6 patch sets | 21:41 |
timburke | (+/-) | 21:41 |
timburke | things like https://github.com/python/cpython/pull/7932 still scare me though -- when are we actually going to be able to run our func tests?? | 21:43 |
clayg | timburke: at least https://bugs.python.org/issue9998 is resolved now right! | 21:44 |
timburke | clayg: and https://bugs.python.org/issue22233 ! | 21:44 |
notmyname | isn't there another cpython bug you were tracking, though? | 21:45 |
timburke | notmyname: i've written https://bugs.python.org/issue33973 and https://bugs.python.org/issue34698 | 21:46 |
clayg | "a py2 project with a very... robust... test suite" awww that's sweet | 21:46 |
timburke | first one i also wrote a patch for it | 21:46 |
clayg | the cpython patch! this one https://github.com/python/cpython/pull/7932 ? That's awesome! | 21:48 |
zaitcev | clayg: check this out, I'm coming to a meetup about gRPC https://www.meetup.com/GDG-Cloud-Austin/events/258700734/ | 21:54 |
zaitcev | You might make it in time if you jump into your car _right now_ | 21:54 |
clayg | zaitcev: that's tomorrow! | 21:54 |
clayg | oh, or tonight | 21:54 |
clayg | can't make it tonight tho :'( | 21:55 |
notmyname | do we have any docs page that describes eventual consistency in one place? | 21:55 |
clayg | let me know how it turns out tho! | 21:55 |
*** itlinux has quit IRC | 21:58 | |
*** rchurch has joined #openstack-swift | 22:00 | |
*** rchurch_ has quit IRC | 22:01 | |
openstackgerrit | Merged openstack/liberasurecode master: Make our alt crc32 more portable https://review.openstack.org/636556 | 22:04 |
zaitcev | We're starting on Thursday May 2 in Denver, right? | 22:17 |
*** openstackgerrit has quit IRC | 22:22 | |
*** openstackgerrit has joined #openstack-swift | 22:27 | |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Use liberasurecode_get_version() https://review.openstack.org/636748 | 22:27 |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Try harder to find liberasurecode https://review.openstack.org/636501 | 22:27 |
timburke | notmyname: thoughts on https://review.openstack.org/#/c/636470/2/releasenotes/notes/2_15_2_release-6996eccabba558b4.yaml@20 ? how do we assure deployers that they won't regress the fix they picked up in 2.15.2 when they upgrade to 2.21.0 if not all the 2.15.2 fixes are on master's changelog? | 22:33 |
patchbot | patch 636470 - swift (stable/pike) - pike stable backports - 2 patch sets | 22:33 |
notmyname | the fact of the matter is that we backported stuff that wasn't in the master changelog | 22:35 |
timburke | so how about we backfill master? | 22:35 |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Use liberasurecode_get_version() https://review.openstack.org/636748 | 22:38 |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Try harder to find liberasurecode https://review.openstack.org/636501 | 22:38 |
openstackgerrit | Tim Burke proposed openstack/pyeclib master: Move experimental jobs to be gating https://review.openstack.org/636771 | 22:41 |
timburke | see, i *knew* i could come up with something that'd still be happy w/1.1.0... | 22:49 |
*** tkajinam has joined #openstack-swift | 23:05 | |
*** mrjk_ has quit IRC | 23:10 | |
notmyname | zaitcev: timburke: updated https://wiki.openstack.org/wiki/Swift/PriorityReviews | 23:31 |
zaitcev | notmyname: thanks, I missed one on that list | 23:37 |
zaitcev | I'm off to grpc meetup | 23:37 |
*** zaitcev has quit IRC | 23:38 | |
timburke | https://review.openstack.org/#/c/611686/ was follow-up from zaitcev's py3-account-server work, but isn't strictly needed for py3 | 23:48 |
patchbot | patch 611686 - swift - Clean up account-reaper a bit - 2 patch sets | 23:48 |
timburke | still, removing a bunch of string concatenation in favor of ''.join(), net-negative lines, having tests that more-closely reflect reality... what's not to love? | 23:49 |
notmyname | I simply searched for patches with "py3" in the topic or subject. please feel free to update the wiki as needed :-) | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!