*** tdasilva has quit IRC | 00:03 | |
*** tdasilva has joined #openstack-swift | 00:03 | |
*** ChanServ sets mode: +v tdasilva | 00:03 | |
*** zaitcev_ has quit IRC | 00:41 | |
*** threestrands has joined #openstack-swift | 00:46 | |
*** gyee has quit IRC | 00:49 | |
*** zaitcev_ has joined #openstack-swift | 00:54 | |
*** ChanServ sets mode: +v zaitcev_ | 00:54 | |
*** tdasilva has quit IRC | 01:04 | |
*** tdasilva has joined #openstack-swift | 01:05 | |
*** ChanServ sets mode: +v tdasilva | 01:05 | |
*** diablo_rojo has joined #openstack-swift | 01:30 | |
*** diablo_rojo has quit IRC | 01:48 | |
*** tdasilva has quit IRC | 02:06 | |
*** tdasilva has joined #openstack-swift | 02:06 | |
*** ChanServ sets mode: +v tdasilva | 02:06 | |
*** henriqueof has joined #openstack-swift | 02:17 | |
*** henriqueof has quit IRC | 02:21 | |
*** zaitcev_ has quit IRC | 02:24 | |
*** zaitcev_ has joined #openstack-swift | 02:36 | |
*** ChanServ sets mode: +v zaitcev_ | 02:36 | |
*** early has quit IRC | 02:37 | |
*** early has joined #openstack-swift | 02:39 | |
*** early has quit IRC | 02:56 | |
*** early has joined #openstack-swift | 02:57 | |
*** tdasilva has quit IRC | 03:07 | |
*** tdasilva has joined #openstack-swift | 03:07 | |
*** ChanServ sets mode: +v tdasilva | 03:07 | |
*** tdasilva has quit IRC | 03:15 | |
*** tdasilva has joined #openstack-swift | 03:16 | |
*** ChanServ sets mode: +v tdasilva | 03:16 | |
*** BjoernT has joined #openstack-swift | 03:17 | |
*** tdasilva has quit IRC | 03:19 | |
*** tdasilva has joined #openstack-swift | 03:20 | |
*** ChanServ sets mode: +v tdasilva | 03:20 | |
*** BjoernT has quit IRC | 03:21 | |
*** BjoernT has joined #openstack-swift | 03:21 | |
*** zaitcev_ is now known as zaitcev | 03:22 | |
zaitcev | You guys won't believe the patch I'm preparing | 03:22 |
---|---|---|
*** gkadam has joined #openstack-swift | 03:31 | |
*** zaitcev has quit IRC | 03:31 | |
*** psachin has joined #openstack-swift | 03:32 | |
*** gkadam has quit IRC | 03:40 | |
*** zaitcev has joined #openstack-swift | 03:44 | |
*** ChanServ sets mode: +v zaitcev | 03:44 | |
*** zaitcev has quit IRC | 03:52 | |
openstackgerrit | Pete Zaitcev proposed openstack/swift master: py3: Fix testUtf8Container (test.functional.tests.TestContainer) https://review.opendev.org/675244 | 04:03 |
*** zaitcev has joined #openstack-swift | 04:06 | |
*** ChanServ sets mode: +v zaitcev | 04:06 | |
*** tdasilva has quit IRC | 04:08 | |
*** tdasilva has joined #openstack-swift | 04:09 | |
*** ChanServ sets mode: +v tdasilva | 04:09 | |
*** zaitcev has quit IRC | 04:11 | |
*** BjoernT has quit IRC | 04:17 | |
*** BjoernT has joined #openstack-swift | 04:26 | |
*** henriqueof has joined #openstack-swift | 04:56 | |
*** spsurya has joined #openstack-swift | 05:02 | |
*** BjoernT has quit IRC | 05:15 | |
*** ccamacho has quit IRC | 06:03 | |
*** tdasilva has quit IRC | 06:11 | |
*** rcernin has quit IRC | 06:18 | |
*** tdasilva has joined #openstack-swift | 06:31 | |
*** ChanServ sets mode: +v tdasilva | 06:31 | |
*** ianychoi has quit IRC | 07:01 | |
*** ianychoi has joined #openstack-swift | 07:01 | |
*** ccamacho has joined #openstack-swift | 07:03 | |
*** baojg has joined #openstack-swift | 07:08 | |
*** ianychoi has quit IRC | 07:24 | |
*** ianychoi has joined #openstack-swift | 07:25 | |
*** tesseract has joined #openstack-swift | 07:31 | |
*** tdasilva has quit IRC | 07:32 | |
*** tdasilva has joined #openstack-swift | 07:32 | |
*** ChanServ sets mode: +v tdasilva | 07:32 | |
*** tdasilva has quit IRC | 08:00 | |
*** tdasilva has joined #openstack-swift | 08:01 | |
*** ChanServ sets mode: +v tdasilva | 08:01 | |
*** takamatsu has quit IRC | 08:06 | |
*** threestrands has quit IRC | 08:19 | |
*** mikecmpbll has joined #openstack-swift | 08:25 | |
*** tdasilva has quit IRC | 08:33 | |
*** tdasilva has joined #openstack-swift | 08:33 | |
*** ChanServ sets mode: +v tdasilva | 08:33 | |
*** e0ne has joined #openstack-swift | 09:04 | |
*** henriqueof has quit IRC | 09:28 | |
*** tdasilva has quit IRC | 09:29 | |
*** tdasilva has joined #openstack-swift | 10:52 | |
*** ChanServ sets mode: +v tdasilva | 10:52 | |
*** baojg has quit IRC | 11:39 | |
*** baojg has joined #openstack-swift | 11:41 | |
*** baojg has quit IRC | 11:41 | |
evrardjp | hey, sorry to bother you with paperwork... Can I use part of the "What is Swift" text here: https://wiki.openstack.org/wiki/Swift to describe Swift mission in governance? | 11:48 |
evrardjp | anyone seeing something against that? | 11:48 |
tdasilva | rledisez: kota_: curious why we have `grpcio>=1.10.0` if we know it will fail starting with 1.3.5? https://github.com/openstack/swift/blob/feature/losf/requirements.txt#L17,L20 | 11:51 |
*** ccamacho has quit IRC | 12:24 | |
*** ccamacho has joined #openstack-swift | 13:23 | |
*** BjoernT has joined #openstack-swift | 13:28 | |
timburke | evrardjp, that sounds great. sorry i hadn't gotten around to submitting that (or even acking that i'd seen the request) | 14:46 |
*** diablo_rojo has joined #openstack-swift | 14:58 | |
evrardjp | timburke: I have proposed something | 15:02 |
evrardjp | https://review.opendev.org/#/c/675307/ | 15:03 |
patchbot | patch 675307 - governance - Add Swift mission into governance - 1 patch set | 15:03 |
evrardjp | sorry I didn't re-ping here, I thought I did. | 15:03 |
*** psachin has quit IRC | 15:15 | |
*** cschwede has joined #openstack-swift | 15:15 | |
*** ChanServ sets mode: +v cschwede | 15:15 | |
*** spsurya has quit IRC | 15:34 | |
*** gyee has joined #openstack-swift | 15:43 | |
openstackgerrit | Tim Burke proposed openstack/swift master: Bring our IPv6 setup more in line with devstack's https://review.opendev.org/675395 | 16:02 |
*** zaitcev has joined #openstack-swift | 16:16 | |
*** ChanServ sets mode: +v zaitcev | 16:16 | |
*** e0ne has quit IRC | 16:16 | |
*** zaitcev_ has joined #openstack-swift | 16:17 | |
*** ChanServ sets mode: +v zaitcev_ | 16:17 | |
*** zaitcev has quit IRC | 16:20 | |
*** mikecmpbll has quit IRC | 16:34 | |
*** diablo_rojo has quit IRC | 16:37 | |
timburke | huh. this stable check's bizarro -- how'd we get a retry? https://logs.opendev.org/periodic-stable/opendev.org/openstack/swift/stable/ocata/openstack-tox-py27/b7f4cf7/job-output.txt.gz#_2019-08-08_06_44_31_128041 | 16:46 |
timburke | like, a 503 would make some sense -- two timeouts popped -- but how'd we get to a 202?? | 16:48 |
*** zaitcev_ has quit IRC | 16:57 | |
*** tdasilva has quit IRC | 16:59 | |
*** BjoernT has quit IRC | 17:02 | |
*** BjoernT has joined #openstack-swift | 17:03 | |
*** BjoernT has quit IRC | 17:03 | |
*** e0ne has joined #openstack-swift | 17:05 | |
*** BjoernT has joined #openstack-swift | 17:06 | |
openstackgerrit | Tim Burke proposed openstack/swift master: py3: Finish porting s3 func tests https://review.opendev.org/675227 | 17:06 |
*** BjoernT has quit IRC | 17:06 | |
*** BjoernT has joined #openstack-swift | 17:07 | |
*** BjoernT has quit IRC | 17:07 | |
timburke | clayg, idk -- "hardlink" is a decent shorthand for the semantic we're going for, but i don't want to conflate the feature *too* much with the filesystem notion of a hardlink -- they're so different! | 17:08 |
*** BjoernT has joined #openstack-swift | 17:08 | |
timburke | naming things is hard | 17:08 |
*** BjoernT has quit IRC | 17:09 | |
clayg | timburke: 100% agree, but also I think it's really wordy/confusing to keep calling them "Symlinks created with a X-Symlink-Target-Etag header" | 17:09 |
*** BjoernT_ has joined #openstack-swift | 17:10 | |
*** zaitcev_ has joined #openstack-swift | 17:10 | |
*** ChanServ sets mode: +v zaitcev_ | 17:10 | |
*** BjoernT_ has quit IRC | 17:10 | |
clayg | Is the suggestion then that we *should* avoid the name hardlink and need to find another noun? do you have a suggestion? Or were you just describing/highlighting the reasons we've avoided it in the past and might want to continue to do so? | 17:11 |
*** BjoernT has joined #openstack-swift | 17:11 | |
*** BjoernT has quit IRC | 17:11 | |
*** BjoernT_ has joined #openstack-swift | 17:12 | |
*** BjoernT_ has quit IRC | 17:12 | |
DHE | validating symlink? | 17:13 |
*** BjoernT has joined #openstack-swift | 17:13 | |
*** BjoernT has quit IRC | 17:14 | |
timburke | clayg, mostly highlighting why we might want to avoid it. i don't actually have a better suggestion, though | 17:14 |
timburke | "symlinks with integrity" -- unlike those other cads :P | 17:14 |
*** BjoernT_ has joined #openstack-swift | 17:14 | |
*** BjoernT_ has quit IRC | 17:14 | |
*** BjoernT has joined #openstack-swift | 17:15 | |
*** BjoernT has quit IRC | 17:16 | |
*** BjoernT has joined #openstack-swift | 17:16 | |
*** BjoernT has quit IRC | 17:17 | |
*** BjoernT_ has joined #openstack-swift | 17:17 | |
*** BjoernT_ has quit IRC | 17:17 | |
*** BjoernT has joined #openstack-swift | 17:18 | |
*** BjoernT has quit IRC | 17:18 | |
*** BjoernT has joined #openstack-swift | 17:19 | |
*** BjoernT has quit IRC | 17:20 | |
*** BjoernT_ has joined #openstack-swift | 17:20 | |
*** BjoernT_ has quit IRC | 17:20 | |
*** BjoernT has joined #openstack-swift | 17:21 | |
*** BjoernT has quit IRC | 17:21 | |
*** BjoernT has joined #openstack-swift | 17:22 | |
*** BjoernT has quit IRC | 17:22 | |
*** BjoernT has joined #openstack-swift | 17:22 | |
*** BjoernT has quit IRC | 17:23 | |
*** BjoernT_ has joined #openstack-swift | 17:23 | |
*** tesseract has quit IRC | 17:27 | |
*** BjoernT_ has quit IRC | 17:28 | |
*** e0ne has quit IRC | 17:35 | |
openstackgerrit | Tim Burke proposed openstack/python-swiftclient stable/rocky: Fix SLO re-upload https://review.opendev.org/673322 | 17:37 |
*** ianychoi has quit IRC | 17:43 | |
*** ianychoi has joined #openstack-swift | 17:47 | |
clayg | literal link? | 17:49 |
timburke | static link? akin to static large object? | 17:50 |
timburke | (there's the overloading with compiler-speak, but at least it's not a filesystem thing) | 17:50 |
*** ianychoi_ has joined #openstack-swift | 17:54 | |
*** ianychoi_ has quit IRC | 17:55 | |
*** ianychoi_ has joined #openstack-swift | 17:55 | |
*** BjoernT has joined #openstack-swift | 17:57 | |
*** ianychoi has quit IRC | 17:58 | |
*** psachin has joined #openstack-swift | 18:08 | |
*** psachin has quit IRC | 18:11 | |
openstackgerrit | Tim Burke proposed openstack/swift stable/stein: container-replicator: Add a timeout for get_shard_ranges https://review.opendev.org/675432 | 18:21 |
openstackgerrit | Tim Burke proposed openstack/swift stable/stein: reconciler: Enqueue right work for shard containers https://review.opendev.org/675433 | 18:22 |
openstackgerrit | Tim Burke proposed openstack/swift stable/stein: Remove invalid dict entries from hashes.pkl https://review.opendev.org/675434 | 18:22 |
openstackgerrit | Tim Burke proposed openstack/swift stable/stein: Fix quarantine when object path is not a directory https://review.opendev.org/675435 | 18:23 |
openstackgerrit | Tim Burke proposed openstack/swift stable/stein: Make GreenAsyncPile not hang https://review.opendev.org/675436 | 18:23 |
openstackgerrit | Tim Burke proposed openstack/swift stable/stein: s3api: Allow clients to upload with UNSIGNED-PAYLOAD https://review.opendev.org/675437 | 18:26 |
openstackgerrit | Tim Burke proposed openstack/swift stable/stein: s3api: Block ETag header from CompleteMultipartUpload requests https://review.opendev.org/675438 | 18:27 |
*** zaitcev_ has quit IRC | 18:28 | |
clayg | i like how static link sounds - i'll probably take to talking about symlinks and statlinks | 18:36 |
*** zaitcev_ has joined #openstack-swift | 18:42 | |
*** ChanServ sets mode: +v zaitcev_ | 18:42 | |
*** BjoernT has quit IRC | 19:15 | |
*** e0ne has joined #openstack-swift | 19:16 | |
*** BjoernT has joined #openstack-swift | 19:16 | |
openstackgerrit | Clay Gerrard proposed openstack/swift master: Consolidate Container-Update-Override headers https://review.opendev.org/675451 | 19:16 |
clayg | mattoliverau: p 675451 - i'd appreciate your thoughts! | 19:17 |
patchbot | https://review.opendev.org/#/c/675451/ - swift - Consolidate Container-Update-Override headers - 1 patch set | 19:17 |
*** BjoernT has quit IRC | 19:20 | |
clayg | The obj server code said the sysmeta takes precedence, but if it’s not tested who knows. 🤷♂️ | 20:11 |
timburke | seems about right. here's the case i worry about, though: https://github.com/openstack/swift/blob/2.22.0/test/unit/obj/test_server.py#L1791-L1795 | 20:21 |
*** diablo_rojo has joined #openstack-swift | 20:24 | |
timburke | testing locally, my fears seem unfounded, though? at any rate, i see the symlink func tests passing on a saio with a defaul EC policy... | 20:27 |
*** diablo_rojo__ has joined #openstack-swift | 20:30 | |
*** diablo_rojo has quit IRC | 20:33 | |
*** e0ne has quit IRC | 20:37 | |
*** zaitcev_ has quit IRC | 20:41 | |
*** diablo_rojo__ is now known as diablo_rojo | 20:52 | |
*** zaitcev_ has joined #openstack-swift | 20:54 | |
*** ChanServ sets mode: +v zaitcev_ | 20:54 | |
*** diablo_rojo has quit IRC | 21:07 | |
*** diablo_rojo has joined #openstack-swift | 21:08 | |
*** diablo_rojo__ has joined #openstack-swift | 21:16 | |
*** diablo_rojo has quit IRC | 21:19 | |
openstackgerrit | Tim Burke proposed openstack/swift master: slo: Add X-Manifest-Etag to responses https://review.opendev.org/675469 | 21:20 |
*** diablo_rojo__ is now known as diablo_rojo | 21:25 | |
*** kei-ichi has quit IRC | 22:03 | |
*** rcernin has joined #openstack-swift | 22:53 | |
*** zaitcev_ is now known as zaitcev | 23:05 | |
zaitcev | AssertionError: Errors in response headers: | 23:05 |
zaitcev | Expected keep-alive but got Keep-Alive for connection | 23:05 |
zaitcev | in test.functional.tests.TestFileUTF8.testGetResponseHeaders | 23:05 |
zaitcev | rings a bell for anyone? | 23:05 |
*** diablo_rojo has quit IRC | 23:05 | |
*** diablo_rojo has joined #openstack-swift | 23:06 | |
openstackgerrit | Merged openstack/swift master: Bring our IPv6 setup more in line with devstack's https://review.opendev.org/675395 | 23:10 |
timburke | zaitcev, py3 actually remembers what the casing was on the wire, where py2 had a .lower() buried in the parsing somewhere (httplib, probably?) | 23:18 |
zaitcev | timburke: this came up with my functests running under py2, so I was just wondering if you already had a solution for it, like with encoding_dance | 23:19 |
timburke | there's a bunch of places in the func tests where i do something like `headers = {h.lower(): v for h, v in resp.getheaders()}` where the code used to just wrap it in a dict(...) | 23:19 |
timburke | i know i did a lot of that in https://github.com/openstack/swift/commit/506279235d12f1590bd928de51d991b7fb425e6e | 23:20 |
zaitcev | Okay. Maybe just one or two places slipped through... I have Apache as the front proxy and it may be camel-casing. | 23:21 |
timburke | oh! it's in the value! | 23:23 |
zaitcev | Finding that encoding_dance was difficult and I spent a few days on it, thanks to my sheer incompetence. First, I looked for errors=, but encoding_dance used positional arguments, so I missed it. | 23:24 |
zaitcev | In the value? Is it one of the headers that list other headers, like CORS | 23:24 |
timburke | take a look at the test: https://github.com/openstack/swift/blob/2.22.0/test/functional/tests.py#L1317 | 23:25 |
zaitcev | Oh. "connection" is actually name, so "Connection: Keep-Alive" | 23:26 |
timburke | yup | 23:26 |
timburke | the curious thing is that we definitely have some case-sensitive checks; look at x-object-meta-fruit a few lines up | 23:27 |
zaitcev | But Banana is cased | 23:28 |
zaitcev | Something is tinkering with the _values_, I can't believe it. | 23:29 |
timburke | right?? weird... | 23:29 |
zaitcev | I'll look into this later. | 23:29 |
zaitcev | Don't worry about this, please. I just asked, in case. | 23:29 |
timburke | assuming that eventlet really is the one sending it (and my understanding is that it *should* be), it seems to be lower: https://github.com/eventlet/eventlet/blob/master/eventlet/wsgi.py#L528 | 23:30 |
zaitcev | go go gadget tcpdump | 23:35 |
zaitcev | Okay, so this is Apache's fault. It's Connection: keep-alive on the loopback but Connection: Keep-Alive on the outside. | 23:49 |
zaitcev | I don't know if this is valid and something we need to work around in functests. | 23:50 |
zaitcev | gtg | 23:50 |
timburke | https://github.com/apache/httpd/blob/a709b2df87f3feaf631569c51d927901f8d2f32e/modules/http/http_protocol.c#L295 i guess? | 23:50 |
timburke | https://tools.ietf.org/html/rfc7230#section-6.1 says "Connection options are case-insensitive." so it sure sounds (to me) like the func tests ought to tolerate either | 23:53 |
zaitcev | I see. | 23:54 |
*** diablo_rojo has quit IRC | 23:57 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!