Tuesday, 2021-02-09

*** openstackgerrit has quit IRC00:49
*** openstackstatus has quit IRC01:20
*** openstack has joined #openstack-swift01:22
*** ChanServ sets mode: +o openstack01:22
*** rcernin has quit IRC02:01
*** thiago__ has quit IRC02:21
*** rcernin has joined #openstack-swift02:25
*** rcernin has quit IRC02:30
*** rcernin has joined #openstack-swift03:04
*** abelur has quit IRC03:26
*** abelur has joined #openstack-swift03:28
*** rcernin has quit IRC03:50
*** openstackgerrit has joined #openstack-swift04:03
openstackgerritPete Zaitcev proposed openstack/swift master: Add a read-only role to keystoneauth  https://review.opendev.org/c/openstack/swift/+/77115804:03
*** rcernin has joined #openstack-swift04:28
*** rcernin has quit IRC04:40
openstackgerritMatthew Oliver proposed openstack/swift master: Add a config file option to swift-manage-shard-ranges  https://review.opendev.org/c/openstack/swift/+/77458404:41
*** rcernin has joined #openstack-swift04:41
openstackgerritTim Burke proposed openstack/swift master: Fix s3api unit tests when run at roughly midnight UTC  https://review.opendev.org/c/openstack/swift/+/77458905:28
timburkewell *that* was stupid05:28
zaitcevSo, what does midnight have to do with it?05:30
zaitcevWhat I see is only someone forgot that v2 existed05:30
*** evrardjp has quit IRC05:33
*** evrardjp has joined #openstack-swift05:33
zaitcevOr wait. It's all v405:34
zaitcevI see, one of them supplied by the client (I guess) and other is generated from the server time.05:43
zaitcevStill, it's unclear what midnight has with anything.05:44
openstackgerritMatthew Oliver proposed openstack/swift master: Add shrink candidates to recon dump  https://review.opendev.org/c/openstack/swift/+/77262405:46
openstackgerritMatthew Oliver proposed openstack/swift master: Add shrink candidates to recon dump  https://review.opendev.org/c/openstack/swift/+/77262405:53
*** dsariel has joined #openstack-swift05:53
*** dsariel has quit IRC05:58
*** gyee has quit IRC06:11
openstackgerritMerged openstack/swift master: Add a read-only role to keystoneauth  https://review.opendev.org/c/openstack/swift/+/77115806:34
*** rcernin has quit IRC07:15
*** rcernin has joined #openstack-swift07:32
*** rcernin has quit IRC07:43
*** rpittau|afk is now known as rpittau07:59
*** rcernin has joined #openstack-swift08:10
*** rcernin has quit IRC08:20
*** dsariel has joined #openstack-swift08:35
*** dsariel has quit IRC08:41
*** dsariel has joined #openstack-swift08:41
*** cschwede has joined #openstack-swift09:41
*** ChanServ sets mode: +v cschwede09:41
*** evrardjp has quit IRC11:10
*** coreycb has quit IRC11:10
*** nicolasbock has quit IRC11:11
*** coreycb has joined #openstack-swift11:11
*** nicolasbock has joined #openstack-swift11:11
*** evrardjp has joined #openstack-swift11:12
*** m75abrams has quit IRC11:13
*** m75abrams has joined #openstack-swift11:27
*** coreycb has quit IRC12:08
*** nicolasbock has quit IRC12:09
*** coreycb has joined #openstack-swift12:10
*** nicolasbock has joined #openstack-swift12:12
*** rcernin has joined #openstack-swift12:27
*** rcernin has quit IRC12:31
*** sorrison has quit IRC12:48
*** sorrison has joined #openstack-swift12:49
openstackgerritMerged openstack/swift master: relinker: Pull arg parsing into module  https://review.opendev.org/c/openstack/swift/+/77357114:26
openstackgerritMerged openstack/swift master: relinker: Allow conf files for configuration  https://review.opendev.org/c/openstack/swift/+/77357214:27
*** fingo has quit IRC14:46
*** m75abrams has quit IRC14:47
*** m75abrams has joined #openstack-swift14:48
*** stand has joined #openstack-swift14:51
*** fingo has joined #openstack-swift14:54
zigotimburke: On my preprod cluster, I have proxy, account and containers using uwsgi. This works well. However, when I switch the object server to uwsgi as well, when I upload an object, I get this:15:10
zigohttp://paste.openstack.org/show/802480/15:10
zigoYet another Eventlet woopsy ... Any idea what this could be ?15:10
zigoOn the object server side, it logs a HTTP 499 error.15:10
zigoAs if the swift-proxy was disconnecting...15:11
zigoI'm having a hard time to understand who's at fault (ie: the proxy or the server).15:11
zigoCould it be that uwsgi isn't supporting the chuncks that the swift-proxy is trying to use?15:11
openstackgerritAlistair Coles proposed openstack/swift master: sharder: Ignore already shrinking sequence when compacting  https://review.opendev.org/c/openstack/swift/+/77456015:36
openstackgerritAlistair Coles proposed openstack/swift master: find_compactable_shard_sequences: fix skipped range  https://review.opendev.org/c/openstack/swift/+/77468415:36
*** m75abrams has quit IRC16:04
timburkegood morning16:16
zigoHi !16:18
timburkezaitcev, the problem was entirely in the test -- v4 signatures have the request-signing-day as part of the credential scope, and the test would always use "now" even when it was moving the signing time around a bit to now +/- some time deltas16:18
timburkezigo, looking...16:18
*** m75abrams has joined #openstack-swift16:19
timburkeis that for a replicated or EC policy? is encryption enabled?16:19
zaitcevtimburke: I understand, but why midnight? There's no wrap-around issue that I can see (using the hour of the day without carry into the day - it's all in seconds everywhere).16:20
zigoNo encryption, just normal replicated setup.16:20
zigoIf I switch back the object-server to Eventlet, everything goes back to working (though, still this LogThread issue... which is what I'm trying to solve with uwsgi).16:22
zigoNote that proxy, account and container servers are all already working very well with uwsgi, as much as I can tell.16:23
timburkezaitcev, it's that the date in the credential scope wasn't matching the date in the x-amz-date header -- so you get failures like with https://zuul.opendev.org/t/openstack/build/f407f778665a4bcf81790eb1bfd1c479/log/job-output.txt#9421-9435 where the credential had a date of 2021-02-08 but the header had a date of 2021-02-09 because the test kicked it 10 minutes into the future16:26
*** rcernin has joined #openstack-swift16:27
*** rcernin has quit IRC16:32
timburkezigo, so there are definitely some eventlet-isms in the object server -- look for set_hundred_continue_response_headers and send_hundred_continue_response in particular -- but i wouldn't have expected that to be needed for non-encrypted replicated PUTs16:36
timburke(see also, https://bugs.launchpad.net/swift/+bug/1496636)16:36
openstackLaunchpad bug 1496636 in OpenStack Object Storage (swift) "EC: Chunked transfer/commit protocol is *not* HTTP" [Medium,In progress] - Assigned to Pete Zaitcev (zaitcev)16:36
zigotimburke: That's precisely what I encountered, and double-guessed (ie: that there's an issue with Transfer-Encoding: chunked).16:37
zigotimburke: So, this means object-server is not uwsgi ready, right?16:37
zaitcevzigo: That bug only deals with EC which has commit phase.16:38
zigoWell, in production, we might use EC and object encryption ... :/16:38
timburkecertainly not if you want a full-featured swift. this case (non-encrypted, replicated) should be simple enough that standard 100-continue support is all that's needed (and not the crazy hooks that we got into eventlet)16:39
timburkei wonder if uwsgi doesn't support 100-continue *at all*, though -- the proxy would send the Expect header and wait for the object servers to ack before sending data; meanwhile, the object servers ignore this header they don't understand and keep waiting for the proxy to send data before sending a response16:40
timburke:-/ this sure *looks* like support, though: https://github.com/unbit/uwsgi/blob/ce787a6e3210333a71f5074e5775885428e8b73c/plugins/http/http.c#L1259-L126216:52
timburkemight need to tcpdump the connections to the backend to figure out what going on on-the-wire, see who's not holding up their end of the conversation16:55
*** m75abrams has quit IRC17:00
zigotimburke: I *do* want full feature (ie: EC + object encryption), so I will leave it using Eventlet then.17:04
*** gyee has joined #openstack-swift17:20
*** dsariel has quit IRC17:25
*** dsariel has joined #openstack-swift17:26
*** rpittau is now known as rpittau|afk17:56
*** cschwede has quit IRC19:29
openstackgerritClay Gerrard proposed openstack/swift master: Shink the public shrinking interface  https://review.opendev.org/c/openstack/swift/+/77473120:39
*** timburke_ has joined #openstack-swift21:09
*** ChanServ sets mode: +v timburke_21:09
*** timburke has quit IRC21:11
*** ianychoi__ has joined #openstack-swift21:40
*** ianychoi_ has quit IRC21:43
*** rcernin has joined #openstack-swift21:50
*** gmann is now known as gmann_afk22:13
*** timburke_ has quit IRC22:49
*** timburke__ has joined #openstack-swift22:50
*** ChanServ sets mode: +v timburke__22:50

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!